[U-Boot] [PATCH 2/2] configs: ti: armv7_common: Enable DISPLAY_{CPU/BOARD}INFO

[Lokesh Vutla]

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 <lokeshvutla at 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