Hello Ilya,
On 05/19/2013 12:09 PM, Ilya Bakulin wrote:
Hi list,
I use U-Boot for starting FreeBSD/arm on Globalscale DreamPlug.
On this platform FreeBSD uses "ubldr" second-stage bootloader, which is
an U-Boot API consumer and uses U-Boot API to access block devices, network,
etc.
Dreamplug has several block devices accessible: internal SD card, SD card
reader,
and any number of USB-attached mass storage devices.
But when I boot ubldr, I can see only one block device.
I have traced down the problem and it seems to be in U-Boot API. When doing
block device enumeration, the "more" flag is not set properly on first access.
This patch fixes the problem for me. After applying the patch, FreeBSD ubldr
is able to see and access all block devices that U-Boot self knows.
diff --git a/api/api_storage.c b/api/api_storage.c
index c535712..1147e79 100644
--- a/api/api_storage.c
+++ b/api/api_storage.c
@@ -129,6 +129,9 @@ static int dev_stor_get(int type, int first, int *more,
struct device_info *di)
else
found = 1;
+ if (specs[type].max_dev > 1)
+ *more = 1;
+
} else {
for (i = 0; i < specs[type].max_dev; i++)
if (di->cookie == (void *)get_dev(specs[type].name, i))
{
I would personally prefer to set i = 0 in the "first" block and move the
/* provide
hint if.. */ out of the else and into the if (found) block. So there
is only one
place for checking the next device, but haven't tested that, just
looking at the
code. Since *more is already set to 0 initially any early problems which
return 0
will already have more = 0. When a device is found, as an exit routine it is
checked if more devices are present, and might set it to 1.
But more important, read http://www.denx.de/wiki/U-Boot/Patches.
This patch misses a signed-off-by (and please make the commit message
reflect
why the api is broken and optionally the cover letter how you found it,
that is a bit
more to the point in my mind).
Regards,
Jeroen
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
