[U-Boot] [PATCH 1/2] 83xx: Replace CONFIG_MPC83XX with CONFIG_MPC83xx

Wolfgang Denk wd at denx.de
Sun May 24 22:13:27 CEST 2009


Dear Peter,

In message <1243192952.703.7.camel at ptyser-laptop> you wrote:
>
> > CodingStyle Ch.12 suggests that it's the other fsl arches that are in
> > violation here, not 83XX:

Agreed...

> Either way is fine with me, as long as things are consistent.  It looked
> like most Freescale CPUs in U-Boot as well as Linux used the lowercase
> convention for CONFIG_<cpu type> so I thought:
> a. Most likely someone made a conscious decision to use lowercase
> b. It'd be much easier to change just 83xx than all other arches:)
> 
> If others would prefer standardizing on uppercase, I can submit new
> patches for 85xx, 5xxx, 86xx, etc.  This would be still be inconsistent
> with Linux though, which is a bit annoying.

It seems Linux uses 8?xx with very few exceptions (CONFIG_SND_VIA82XX*
and CONFIG_EDAC_MPC85XX), so let's do the same here.

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's certainly  convenient  the  way  the  crime  (or  condition)  of
stupidity   carries   with   it  its  own  punishment,  automatically
admisistered without remorse, pity, or prejudice. :-)
         -- Tom Christiansen in <559seq$ag1$1 at csnews.cs.colorado.edu>


More information about the U-Boot mailing list