From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick DELAUNAY Date: Wed, 1 Jul 2020 13:01:43 +0000 Subject: [PATCH] video: extend stdout video console work-around for 'vga' In-Reply-To: <20200619155550.05b81533@crub> References: <5b441050-d103-68e0-cae1-7abb84855275@web.de> <20200523151120.4273-1-agust@denx.de> <20200619155550.05b81533@crub> Message-ID: <5beee82b3f0945818070ab28be19c111@SFHDAG6NODE3.st.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Anatolij > From: U-Boot On Behalf Of Anatolij Gustschin > Sent: vendredi 19 juin 2020 15:56 > > On Sat, 23 May 2020 17:11:20 +0200 > Anatolij Gustschin agust at denx.de wrote: > > > cfb_console driver use 'vga' console name and we still have board > > environments defining this name. Re-use existing DM_VIDEO work- around > > for console name to support 'vga' name in stdout environment. > > > > Signed-off-by: Anatolij Gustschin > > --- > > arch/arm/mach-tegra/Kconfig | 1 - > > common/console.c | 7 ++++--- > > drivers/video/Kconfig | 16 +++++++++------- > > 3 files changed, 13 insertions(+), 11 deletions(-) > > Applied to u-boot-video/next, thanks! This patch breaks the stm32mp15 boot in console_init_r.on the next branch. The board have with DM_VIDEO activated. In previous version the workaround was not activated for the board: CONFIG_VIDCONSOLE_AS_LCD=n (default) PS: the config is activated in few board but still as bolean but defined as sting in Kconfig configs/peach-pit_defconfig:76:CONFIG_VIDCONSOLE_AS_LCD=y configs/snow_defconfig:87:CONFIG_VIDCONSOLE_AS_LCD=y configs/peach-pi_defconfig:77:CONFIG_VIDCONSOLE_AS_LCD=y configs/spring_defconfig:88:CONFIG_VIDCONSOLE_AS_LCD=y Now it is a string, the CONFIG is always defined = value is "lcd" or "vga" or "" And the code in console.c under this compilation flags cause the crash. I push a patch [1] to fix up it this update: VIDCONSOLE_AS_LCD => Boolean used in defconfig as previously VIDCONSOLE_AS_NAME => name the the string to used.... NEW [1] "video: restore CONFIG_VIDCONSOLE_AS_LCD as Boolean" Regards Patrick