[U-Boot] [PATCH] PPC4xx: PLB4 Arbiter and Memory Queue Optimizations
Wolfgang Denk
wd at denx.de
Mon Aug 11 17:51:59 CEST 2008
Dear Prodyut Hazarika,
In message <0CA0A16855646F4FA96D25A158E299D604D290EC at SDCEXCHANGE01.ad.amcc.com> you wrote:
>
> > I don't particularly like this extra "{" level without need. Please
> remove it.
>
> The { is kept to avoid warnings. Without the "{", I would need to define
> the local variable near
> The beginning of the function. That would cause warning (Unused
Yes, that's where variable definitions belong.
> variable) for non 440EX/GT families.
> Please advice whether you want this warning to come up, if I remove the
No, warnings are not allowed. You can #ifdef the declaration, can't
you?
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
When you die, the first thing you lose is your life. The next thing
is the illusions. - Terry Pratchett, _Pyramids_
More information about the U-Boot
mailing list