diff mbox series

[18/45] sandbox: Drop message about writing sandbox state

Message ID	20220925150248.2524421-19-sjg@chromium.org
State	Superseded
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>, Heinrich Schuchardt <xypron.glpk@gmx.de> Subject: [PATCH 18/45] sandbox: Drop message about writing sandbox state Date: Sun, 25 Sep 2022 09:02:21 -0600 Message-Id: <20220925150248.2524421-19-sjg@chromium.org> In-Reply-To: <20220925150248.2524421-1-sjg@chromium.org> References: <20220925150248.2524421-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	vbe: Implement the full firmware flow \| expand [00/45] vbe: Implement the full firmware flow [01/45] Rename CONFIG_SYS_TEXT_BASE to CONFIG_TEXT_BASE [02/45] disk: Drop debug messages in part_efi [03/45] bloblist: Drop debugging [04/45] rsa: Avoid warning in padding_pss_verify() [05/45] spl: Use binman suffix allow symbols of any SPL etype [06/45] spl: Split up the board_init_r() function [07/45] spl: Refactor controls for console output [08/45] spl: Add a separate silence option for SPL [09/45] CI: Install pyelftools for builds [10/45] binman: Allow obtaining a symbol value [11/45] binman: Split out looking up a symbol into a function [12/45] binman: Handle writing ELF symbols in the Entry class [13/45] binman: Support writing symbols into ELF files [14/45] dm: blk: Add udevice functions [15/45] dm: usb: Update the test to cover reading and writing [16/45] dm: blk: mmc: Tidy up some Makefile rules for SPL [17/45] dm: mmc: Allow sandbox emulator to build without writes [18/45] sandbox: Drop message about writing sandbox state [19/45] sandbox: Generalise SPL booting [20/45] sandbox: Add a way to specify the sandbox executable [21/45] bootstd: Add a way to set up a bootflow [22/45] image: Move comment for fit_conf_find_compat() [23/45] test: Report skippped tests [24/45] test: Update tests to use the skip feature [25/45] test: Support tests which can only be run manually [26/45] image: Add the concept of a phase to FIT [27/45] image: Allow loading a FIT config for a particular phase [28/45] image: Correct strncpy() warning with image_set_name() [29/45] vbe: Rename vbe_fixup to vbe_request [30/45] vbe: Use a warning for a failed requests [31/45] spl: Allow multiple loaders of the same type [32/45] sandbox: Support obtaining the next phase from an image [33/45] vbe: Support selecting operations by SPL phase [34/45] vbe: Support reading the next SPL phase via VBE [35/45] vbe: Move OS implementation into a separate file [36/45] vbe: Drop the U-Boot prefix from the version [37/45] vbe: Add Kconfig options for VPL [38/45] vbe: Add info about the VBE device to the fwupd node [39/45] sandbox: Add a binman image for VPL [40/45] vbe: Correct pylint warnings in test_vbe [41/45] vbe: Use a manual test [42/45] vbe: Record which phases loaded using VBE [43/45] vbe: Add docs and a test for the VBE command [44/45] vbe: Add a subcommand to show the VBE state [45/45] vbe: Add a test for the VBE flow into U-Boot proper

Commit Message

Simon Glass Sept. 25, 2022, 3:02 p.m. UTC

This happens every time sandbox moves to the next phase so is not very
interesting. Display the message only when debugging.

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

 arch/sandbox/cpu/state.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Heinrich Schuchardt Sept. 26, 2022, 6:31 a.m. UTC | #1

On 9/25/22 17:02, Simon Glass wrote:
> This happens every time sandbox moves to the next phase so is not very
> interesting. Display the message only when debugging.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>

Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

> ---
>
>   arch/sandbox/cpu/state.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/sandbox/cpu/state.c b/arch/sandbox/cpu/state.c
> index fcc4a337e59..a681e472ab6 100644
> --- a/arch/sandbox/cpu/state.c
> +++ b/arch/sandbox/cpu/state.c
> @@ -470,7 +470,7 @@ int state_uninit(void)
>   	int err;
>
>   	if (state->write_ram_buf || state->write_state)
> -		log_info("Writing sandbox state\n");
> +		log_debug("Writing sandbox state\n");
>   	state = &main_state;
>
>   	/* Finish the bloblist, so that it is correct before writing memory */

diff mbox series

Patch

diff --git a/arch/sandbox/cpu/state.c b/arch/sandbox/cpu/state.c
index fcc4a337e59..a681e472ab6 100644
--- a/arch/sandbox/cpu/state.c
+++ b/arch/sandbox/cpu/state.c
@@ -470,7 +470,7 @@  int state_uninit(void)
 	int err;
 
 	if (state->write_ram_buf || state->write_state)
-		log_info("Writing sandbox state\n");
+		log_debug("Writing sandbox state\n");
 	state = &main_state;
 
 	/* Finish the bloblist, so that it is correct before writing memory */