[U-Boot] the weirdness with cmd_df.c

Albert ARIBAUD albert.u.boot at aribaud.net
Mon Feb 4 17:00:25 CET 2013


Hi Robert,

On Mon, 04 Feb 2013 10:54:05 -0500, "Robert P. J. Day"
<rpjday at crashcourse.ca> wrote:

>    i mentioned this this weekend as well, can someone clarify this?
> the source file cmd_df.c looks weird:
> 
> U_BOOT_CMD(
>          sf,     2,      1,      do_serial_flash,
>          "Serial flash sub-system",
>          "probe [bus:]cs         - init flash device on given SPI bus and CS")
> 
> so the source file "cmd_df.c" allegedly defines a command called "sf",  
> which invokes "do_serial_flash", which isn't even defined in that  
> source file -- what's defined in that file is "do_df".  i don't see  
> "do_serial_flash" defined anywhere in the u-boot source tree.  am i  
> just being blind?  i see no reference to that source file in Makefile,  
> so i'm guessing it's historical cruft that could be deleted?

do_serial_flash() indeed does not appear anywhere. As for deleting, you
should check which makefile builds cmd_df.c, under which condition,
and if this condition is met in any U-Boot MAKEALL target. If not, just
submit a patch to remove cmd_df.c from the baseline.

BTW: IMO, you should just submit a patch instead of asking. If the patch
is ok, it'll be taken, and you didn't have anything to ask; if the
patch is not OK, the one who NAKs it will tell you why, and you'll get
the answer without asking. :)

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list