[U-Boot] [PATCH 3/6] P1021: add P1021MDS board support

[Wolfgang Denk]

Dear Scott Wood,

In message <20110131140801.33609642 at udp111988uds.am.freescale.net> you wrote:
>
> > Please rather omit the setting instead of using fillers that are of no
> > practical value.
> 
> Well, they do make it easier for a user to quickly see what the names
> are that U-Boot expects for such commonly used things, rather than
> having to scan the manual.

I doubt both the "quicly see" part (in such a long list of settings)
and the "rather than having to scan the manual" part.

> > > +	"ramdiskfile=your.ramdisk.u-boot\0"				\
> > 
> > Ditto. [BTW: why "....ramdisk.u-boot"? U-Boot does not use ramdisks.
> > The ramdisk is only used for some OS, so that should probably be
> > "...ramdisk.linux" instead?]
> 
> We often use the ".u-boot" suffix on ramdisks that have been wrapped
> with a uImage header.

That would be a "uRamdisk" then (similar to uImage).

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
It usually takes more than three weeks to prepare  a  good  impromptu
speech.                                                  - Mark Twain