Dear Liu Hui-R64343,
In message 3850A844E6A3854C827AC5C0BEC7B60A2B0596@zch01exm23.fsl.freescale.net you wrote:
I suggest we do the same. Please add a "mx5" resp. "MX5" to the new identifier names.
Yes, agree.
Thanks.
Please also keep in mind that CONFIG_* options should be documented in the README.
Do you mean all the specific CONFIG_* options in include/configs/boardname.h should be documented? I found lots of that is not doing what you called for? :(
Yes, we have not always been that strict, but we try to improve.
BTW, which definition is better?
CONFIG_SYS_MX5_HCLK_FREQ 24000000 CONFIG_MX5_HCLK_FREQ 24000000
According to the REAME, I prefer to the former, what do you think.
Indeed CONFIG_SYS_* seems more appropriate here. I would probably call this "CONFIG_SYS_MX5_HCLK" ("CLK" and "FREQ" seem a bit redundant here).
Best regards,
Wolfgang Denk