On Monday 29 August 2016 01:46 AM, Tom Rini wrote:
> On Sat, Aug 27, 2016 at 05:23:55PM +0530, Lokesh Vutla wrote:
>
>> Enable CONFIG_DISPLAY_{CPU/BOARD}INFO on all TI platforms.
>> Also removing the duplicated configs.
>>
>> Signed-off-by: Lokesh Vutla <lokeshvu...@ti.com>
>> ---
>> include/configs/ti_armv7_common.h | 4 ++++
>> include/configs/ti_armv7_keystone2.h | 1 -
>> include/configs/ti_omap4_common.h | 2 --
>> include/configs/ti_omap5_common.h | 3 ---
>> 4 files changed, 4 insertions(+), 6 deletions(-)
>
> This is the wrong direction. Please add an entry to common/Kconfig (a
> "hidden" one) and select it from board/ti/*/Kconfig. And this will need
Sure, will update the patch to add a common Kconfig entry. But I am not
sure about the statement "select it from board/ti/*/Kconfig". You meant
it should be selected by every config TARGET_XXX_EVM or something else?
> some cleaning up of some other boards/ Kconfig files that introduced it
> locally and instead should select this new option. Thanks!
okay, will do it.
Thanks and regards,
Lokesh
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
