[U-Boot] [PATCH] Add suport for the graphic controller included in the JADE SOC

Wolfgang Denk wd at denx.de
Sun Jul 5 01:30:17 CEST 2009


Dear Anatolij & Matthias,

in message <4A4C7D73.1080502 at denx.de> you wrote:
>
> >>> +		if(0 == bpp){
> >> if (bpp == 0) {
> > 
> > The spaces are OK but why turn around the comparison? I always use 

Because "0 == bpp" is ugly.

> > this style to prevent from unwanted assignments. Well, GCC warns 
> > about assignments in ifs and as u-boot is GCC only I can do that.
> 
> it is a matter of personal preference. "if (0 == bpp) {" is OK for
> me, too.

NAK for me. Matthias, please do as Anatolij suggested.

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
If a packet hits a pocket on a socket on a port,
And the bus is interrupted as a very last resort,
And the address of the memory makes your floppy disk abort,
Then the socket packet pocket has an error to report! - Ken Burchill?


More information about the U-Boot mailing list