[v2,20/27] video: Allow use without CONFIG_CMDLINE

Message ID	20231007231255.277623-21-sjg@chromium.org
State	Changes Requested
Delegated to:	Tom Rini
Headers	show Return-Path: <u-boot-bounces@lists.denx.de> From: Simon Glass <sjg@chromium.org> To: U-Boot Mailing List <u-boot@lists.denx.de> Cc: Tom Rini <trini@konsulko.com>, Simon Glass <sjg@chromium.org>, Anatolij Gustschin <agust@denx.de> Subject: [PATCH v2 20/27] video: Allow use without CONFIG_CMDLINE Date: Sat, 7 Oct 2023 17:12:39 -0600 Message-ID: <20231007231255.277623-21-sjg@chromium.org> In-Reply-To: <20231007231255.277623-1-sjg@chromium.org> References: <20231007231255.277623-1-sjg@chromium.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: list Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" <u-boot-bounces@lists.denx.de>
Series	Tidy up use of CONFIG_CMDLINE \| expand [v2,00/27] Tidy up use of CONFIG_CMDLINE [v2,01/27] command.h: Add a U_BOOT_LONGHELP macro [v2,02/27] cmd: Convert existing long help messages to the new macro [v2,03/27] buildman: Use oldconfig when adjusting the config [v2,04/27] bootstd: Correct dependencies on CMDLINE [v2,05/27] autoboot: Correct dependencies on CMDLINE [v2,06/27] cmd: Add a few more dependencies on CMDLINE [v2,07/27] test: Make UNIT_TEST depend on CMDLINE [v2,08/27] sifive: Change #ifdef for nop [v2,09/27] fastboot: Declare a dependency on CMDLINE [v2,10/27] cli: Always build cli_getch [v2,11/27] cmd: Use an #ifdef around run_commandf() [v2,12/27] Move bootmenu_conv_key() into its own file [v2,13/27] pxe: Depend on CMDLINE [v2,14/27] env: Split out non-command code into a new file [v2,15/27] console: Move SYS_PBSIZE into common/ [v2,16/27] bootm: Allow building when cleanup functions are missing [v2,17/27] fdt: Move working_fdt into fdt_support [v2,18/27] net: Depend on CONFIG_CMDLINE [v2,19/27] log: Allow use without CONFIG_CMDLINE [v2,20/27] video: Allow use without CONFIG_CMDLINE [v2,21/27] video: Dont require the font command [v2,22/27] efi: Depend on CMDLINE for efi_loader [v2,23/27] cmd: Make all commands depend on CMDLINE [v2,24/27] sandbox: Avoid requiring cmdline [v2,25/27] arm: x86: Drop discarding of command linker-lists [v2,26/27] mmc: env: Unify the U_BOOT_ENV_LOCATION conditions [v2,27/27] sandbox: Add a test for disabling CONFIG_CMDLINE

Message ID

20231007231255.277623-21-sjg@chromium.org

State

Changes Requested

Delegated to:

Tom Rini

Headers

From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Tom Rini <trini@konsulko.com>, Simon Glass <sjg@chromium.org>,
 Anatolij Gustschin <agust@denx.de>
Subject: [PATCH v2 20/27] video: Allow use without CONFIG_CMDLINE
Date: Sat,  7 Oct 2023 17:12:39 -0600
Message-ID: <20231007231255.277623-21-sjg@chromium.org>
In-Reply-To: <20231007231255.277623-1-sjg@chromium.org>
References: <20231007231255.277623-1-sjg@chromium.org>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: list
Errors-To: u-boot-bounces@lists.denx.de
Sender: "U-Boot" <u-boot-bounces@lists.denx.de>

Series

Tidy up use of CONFIG_CMDLINE | expand

Commit Message

Simon Glass Oct. 7, 2023, 11:12 p.m. UTC

Provide a fallback for when CONFIG_SYS_CBSIZE is not provided, so that
the console can still be used.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

(no changes since v1)

 drivers/video/console_truetype.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/video/console_truetype.c b/drivers/video/console_truetype.c
index 0f9bb49e44f7..8186b5b49f0b 100644
--- a/drivers/video/console_truetype.c
+++ b/drivers/video/console_truetype.c
@@ -124,7 +124,11 @@  struct pos_info {
  * Allow one for each character on the command line plus one for each newline.
  * This is just an estimate, but it should not be exceeded.
  */
+#ifdef CONFIG_SYS_CBSIZE
 #define POS_HISTORY_SIZE	(CONFIG_SYS_CBSIZE * 11 / 10)
+#else
+#define POS_HISTORY_SIZE	(250 * 11 / 10)
+#endif
 
 /**
  * struct console_tt_metrics - Information about a font / size combination

[v2,20/27] video: Allow use without CONFIG_CMDLINE

Commit Message

Patch