From mboxrd@z Thu Jan  1 00:00:00 1970
From: Soeren Moch <smoch@web.de>
Date: Fri, 29 May 2015 20:02:11 +0200
Subject: [U-Boot] [PATCH 03/10] imx6: move generic imx6 options to
	mx6_common.h
In-Reply-To: <556462D1.2080907@web.de>
References: <555F7302.1060609@web.de> <55645B7E.3000107@denx.de>
 <556462D1.2080907@web.de>
Message-ID: <5568A9A3.8020805@web.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Stefano,
>> As far as I can see, there are only two boards : gw_ventana and tbs2910.
>> CONFIG_DISPLAY_BOARDINFO_LATE enables show_board_info(), and this is not
>> weak, printing twice if we have both CONFIG_DISPLAY_BOARDINFO_LATE and
>> CONFIG_DISPLAY_BOARDINFO. I would suggest that we add
>>     #undef CONFIG_DISPLAY_BOARDINFO
>>     #define CONFIG_DISPLAY_BOARDINFO_LATE
>>
>> to gw_ventana.h and tbs2910.h after applying this series.
>
>
> It is unfortunate that there is no CONFIG_DISPLAY_CPUINFO_LATE.
> The real solution would be to print both board and cpu info
> after enabling the HDMI frame buffer. BOARDINFO_LATE alone
> is not that important. So I agree to apply this patch series
> without another fix.
>
> Maybe we should add
> #ifndef CONFIG_DISPLAY_BOARDINFO_LATE
> #define CONFIG_DISPLAY_BOARDINFO
> #endif
> to mx6_common.h later. There are already other conditional defines
> in there, and that could save the #undef in individual board files.

I will add a PRE_CONSOLE_BUFFER to tbs2910.h , so there is no
need anymore to add CONFIG_DISPLAY_BOARDINFO_LATE.

I'm fine with the merged consolidation series, no further action required
on this topic from my side.

Regards,
Soeren