On Jul 7, 2009, at 4:36 PM, Wolfgang Denk wrote:

Dear Kumar Gala,

In message 9FC82254-6AA7-4DAE-9780-283C66D5D79D@kernel.crashing.org you wrote:

...

Can anyone explain why CONFIG_CMD_NFS is part of include/ config_cmd_default.h. There are a few others in there that I'd question but this is one that I'm scratching my head on right now.

Because NFS support is pretty small in terms of code size and pretty useful in terms of download speed (usually much faster than TFTP, and less restricted, too).

its about 7k of size in my MPC8572DS config

text data bss dec hex filename 396624 50716 44181 491521 78001 u-boot 392120 50536 41957 484613 76505 u-boot (w/o CMD_NFS)

Seems like the 2k buff for the path is overkill.

static char nfs_path_buff[2048];

In other words: because I want it :-)

Ahh, one of those :)

- k