[U-Boot] ad-hoc config error

[york sun]

On 09/20/2016 03:30 PM, Tom Rini wrote:
> On Tue, Sep 20, 2016 at 09:40:00PM +0000, york sun wrote:
>> On 09/20/2016 02:36 PM, Tom Rini wrote:
>>> On Tue, Sep 20, 2016 at 09:22:09PM +0000, york sun wrote:
>>>
>>>> Tom and Simon,
>>>>
>>>> After commit 371244cb19f9804711dd66e4281ff7979915fd2e, all merges with
>>>> new macros defined will have the compiling error. How shall we fix it?
>>>> Some macros can be added to Kconfig. But some are for local use, better
>>>> than magic numbers. Adding them to the white-list doesn't sound right.
>>>> What's your suggestion?
>>>
>>> Things that don't belong in Kconfig don't belong in the CONFIG namespace
>>> either, probably.  For example, the cache line stuff is in Kconfig and
>>> select'ed but for another example, various magic numbers used in the TI
>>> secure boot stuff (which can vary from SoC to SoC) are just not in the
>>> CONFIG namespace now.
>>>
>>
>> For those can be put in Kconfig, I can convert.
>> Can you point me examples to use macros for magic numbers?
>> What about the white listed macros? Are we supposed to leave them there,
>> or slowly convert them to other name space?
>
> Things on the whitelist should be converted, either to Kconfig or moved
> out of the namespace.  Can you give me an example of something you
> aren't sure how to convert?
>
For example,

CONFIG_SYS_DDR_MODE_1_1000
CONFIG_SYS_DDR_MODE_1_1200
CONFIG_SYS_DDR_MODE_1_1333
CONFIG_SYS_DDR_MODE_1_667
CONFIG_SYS_DDR_MODE_1_800
CONFIG_SYS_DDR_MODE_1_900

Those are DDR parameters defined per board if used. What will be proper 
names to convert to?

York