[U-Boot] [PATCH] Rename CONFIG_SYS_MDDRC memory constants for all mpc512x boards

Wolfgang Denk wd at denx.de
Thu Sep 10 20:48:34 CEST 2009


Dear Martha M Stan,

In message <12525956202930-git-send-email-mmarx at silicontkx.com> you wrote:
> From: Martha Marx <mmarx at silicontkx.com>

May I ask what the purpose of such a rename is? I mean, which problem
are you tryin to fix, and in which way is the new code supposed to be
better than the old one?

Actually I think the new names are much worse. What's the purpose of
omitting the "CONFIG_SYS_" part?

...
> -		out_be32(&im->mddrc.ddr_command, CONFIG_SYS_MICRON_NOP);
> +		out_be32(&im->mddrc.ddr_command, DDR_NOP);

For example this rename - using "DDR_NOP" for a configurable parameter
in a global file, and having to #define this in the board config files
makes no sense to me.

We have standard rule about naming of configration options. This here
is clearly a case where "CONFIG_SYS_" applies.

All the other renames fall in the same category.


Unless you have a very good reason for this rename I tend to NAK
this.

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
Drawing on my fine command of language, I said nothing.


More information about the U-Boot mailing list