[U-Boot] [PATCH v2 11/11] New board support: Nokia RX-51 aka N900
Wolfgang Denk
wd at denx.de
Sun Apr 29 14:49:46 CEST 2012
Dear Pali =?ISO-8859-1?Q?Roh=E1r?=,
In message <2499233.2JzaJFJkCD at pali> you wrote:
>
> > Please stop sending base64 encoded patches! Send plain text
> > only!
>
> I used git format-patch and git send-email. Is there easy way how
> to tell this to git?
Yes: do not use characters that cause problems.
> > While it is a good idea to provide documentation, this is the
> > wrong place for it. Here in the commit message is about the
> > last place where most of the users will search for it. Please
> > provide a README for this board - either in the board
> > directory, or in doc/
>
> Is doc/README.nokia_rx51 correct place?
It ist what many other boards used, so it would be fine.
Personally, I don;t like this ever growing list of board specific
documentation in doc; my personal preference would be something like
board/nokia/rx51/README instead... Feel free to chose.
> > Comment appears to be misplaced.
>
> Ok, comment will be removed.
Rather fix it.
> > > +/* Maemo kernel 2.6.28 will crash if u-boot enable usb tty
> > > */
> > > +/*
> > > +#define CONFIG_USB_TTY
> > > +*/
> >
> > Please remove dead code.
>
> usb tty is usefull for debuging u-boot. because this device does
> not have easy access to serial console, usb tty is only option
> how to connect u-boot terminal to pc. but default (and still only
> one working on this device) kernel crash when u-boot enable usb
> tty. I want to have usb tty option in u-boot and belive that one
> day will be some upstream linux kernel with fixed usb drivers
> working on nokia n900.
As is, it is dead code and should be removed (not to mention that this
is another case of incorrect multi-line comment style).
Please feel free to add such a description to the documentation.
> > Incorrect multiline comment / please remove dead code.
>
> How to write correct comment?
See
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=Documentation/CodingStyle#l446
> Is there way how to decrease u-boot binary size?
Omit features ...
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
To the systems programmer, users and applications serve only to
provide a test load.
More information about the U-Boot
mailing list