From: Soeren Moch <smoch@web.de>
To: u-boot@lists.denx.de
Subject: [PATCH] tbs2910: migrate to DM_VIDEO
Date: Sat, 23 May 2020 09:55:48 +0200 [thread overview]
Message-ID: <5b441050-d103-68e0-cae1-7abb84855275@web.de> (raw)
In-Reply-To: <20200522232417.4202-1-agust@denx.de>
On 23.05.20 01:24, Anatolij Gustschin wrote:
> Migration to DM_VIDEO driver is long overdue, configure it in
> board config files. To enable the display set stdout like:
>
> setenv stdout serial,vidconsole
And this exactly is the problem.
There are some users of this board without serial console. They have no
chance to set this after upgrading u-boot.
What is the reason behind this incompatible change? In other words, why
vga cannot be an alias for vidconsole, if this is the preferred name
nowadays?
Regards,
Soeren
>
> Signed-off-by: Anatolij Gustschin <agust@denx.de>
> ---
> This is only build tested.
>
> configs/tbs2910_defconfig | 7 ++++++-
> include/configs/tbs2910.h | 2 --
> 2 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/configs/tbs2910_defconfig b/configs/tbs2910_defconfig
> index 2ff0e160f7..b40641c17b 100644
> --- a/configs/tbs2910_defconfig
> +++ b/configs/tbs2910_defconfig
> @@ -1,6 +1,7 @@
> CONFIG_ARM=y
> CONFIG_ARCH_MX6=y
> CONFIG_SYS_TEXT_BASE=0x17800000
> +CONFIG_SYS_MALLOC_F_LEN=0x4000
> CONFIG_ENV_SIZE=0x2000
> CONFIG_ENV_OFFSET=0x60000
> CONFIG_TARGET_TBS2910=y
> @@ -84,7 +85,11 @@ CONFIG_USB_GADGET_PRODUCT_NUM=0xa4a5
> CONFIG_CI_UDC=y
> CONFIG_USB_GADGET_DOWNLOAD=y
> CONFIG_I2C_EDID=y
> +CONFIG_DM_VIDEO=y
> +# CONFIG_VIDEO_BPP8 is not set
> +# CONFIG_VIDEO_BPP32 is not set
> +# CONFIG_VIDEO_ANSI is not set
> CONFIG_VIDEO_IPUV3=y
> -CONFIG_VIDEO=y
> +CONFIG_SYS_WHITE_ON_BLACK=y
> CONFIG_OF_LIBFDT_ASSUME_MASK=0xff
> # CONFIG_EFI_LOADER is not set
> diff --git a/include/configs/tbs2910.h b/include/configs/tbs2910.h
> index 7376b91f55..8ab3fcfe3e 100644
> --- a/include/configs/tbs2910.h
> +++ b/include/configs/tbs2910.h
> @@ -37,11 +37,9 @@
> #define CONFIG_MXC_UART_BASE UART1_BASE /* select UART1/UART2 */
>
> /* Framebuffer */
> -#ifdef CONFIG_VIDEO
> #define CONFIG_VIDEO_BMP_RLE8
> #define CONFIG_IMX_HDMI
> #define CONFIG_IMX_VIDEO_SKIP
> -#endif
>
> /* PCI */
> #ifdef CONFIG_CMD_PCI
next prev parent reply other threads:[~2020-05-23 7:55 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-22 23:24 [PATCH] tbs2910: migrate to DM_VIDEO Anatolij Gustschin
2020-05-23 7:55 ` Soeren Moch [this message]
2020-05-23 15:11 ` [PATCH] video: extend stdout video console work-around for 'vga' Anatolij Gustschin
2020-05-24 15:21 ` Soeren Moch
2020-05-25 2:14 ` Simon Glass
2020-05-26 18:08 ` Tom Rini
2020-06-19 13:55 ` Anatolij Gustschin
2020-07-01 13:01 ` Patrick DELAUNAY
2020-05-23 15:17 ` [PATCH] tbs2910: migrate to DM_VIDEO Anatolij Gustschin
2020-05-23 15:31 ` Soeren Moch
2020-05-24 15:46 ` Soeren Moch
2020-05-26 20:13 ` Anatolij Gustschin
2020-05-28 7:54 ` Soeren Moch
2020-05-28 8:07 ` Anatolij Gustschin
2020-05-30 16:01 ` Soeren Moch
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5b441050-d103-68e0-cae1-7abb84855275@web.de \
--to=smoch@web.de \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.