u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: Simon Glass <sjg@chromium.org>
To: U-Boot Mailing List <u-boot@lists.denx.de>
Cc: Mark Kettenis <kettenis@openbsd.org>,
	huang lin <hl@rock-chips.com>,
	Jeffy Chen <jeffy.chen@rock-chips.com>,
	Simon Glass <sjg@chromium.org>,
	Kever Yang <kever.yang@rock-chips.com>,
	Philipp Tomsich <philipp.tomsich@theobroma-systems.com>,
	Tom Rini <trini@konsulko.com>,
	Vagrant Cascadian <vagrant@debian.org>
Subject: [PATCH v7 05/11] bootstd: Show a message sometimes if no bootflows are found
Date: Sun,  2 Apr 2023 14:02:27 +1200	[thread overview]
Message-ID: <20230402140231.v7.5.I53dbbf7d5e7585e4d1678117165a46de1d8d7eac@changeid> (raw)
In-Reply-To: <20230402140231.v7.1.Id5a4b4ee7b653591dfb25e75204c17215521164b@changeid>

Enable some messages that might provide hints, but only for
CMD_BOOTFLOW_FULL since otherwise the -l flag is not available.

Signed-off-by: Simon Glass <sjg@chromium.org>
Suggested-by: Tom Rini <trini@konsulko.com>
---

(no changes since v6)

Changes in v6:
- Add new patch to show a message sometimes if no bootflows are found

 cmd/bootflow.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/cmd/bootflow.c b/cmd/bootflow.c
index 42f6e14a4370..aa06999e3db3 100644
--- a/cmd/bootflow.c
+++ b/cmd/bootflow.c
@@ -181,6 +181,9 @@ static int do_bootflow_scan(struct cmd_tbl *cmdtp, int flag, int argc,
 	if (list)
 		show_footer(i, num_valid);
 
+	if (IS_ENABLED(CONFIG_CMD_BOOTFLOW_FULL) && !num_valid && !list)
+		printf("No bootflows found; try again with -l\n");
+
 	return 0;
 }
 
-- 
2.40.0.348.gf938b09366-goog


  parent reply	other threads:[~2023-04-02  2:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-02  2:02 [PATCH v7 01/11] bootstd: Tweak bootflow logic for device tree Simon Glass
2023-04-02  2:02 ` [PATCH v7 02/11] virtio: Ensure PCI is set up first Simon Glass
2023-04-02  2:02 ` [PATCH v7 03/11] bootstd: Support booting EFI where multiple options exist Simon Glass
2023-04-02  2:02 ` [PATCH v7 04/11] bootstd: Report missing labels only when asked Simon Glass
2023-04-02  2:02 ` Simon Glass [this message]
2023-04-02  2:02 ` [PATCH v7 06/11] rockchip: Move to standard boot Simon Glass
2023-04-02  2:02 ` [PATCH v7 07/11] rockchip: Use the same boot_targets for all boards Simon Glass
2023-04-02  2:02 ` [PATCH v7 08/11] bootstd: Disable BOOTSTD for boards with custom commands Simon Glass
2023-04-02  2:02 ` [PATCH v7 09/11] xilinx: Disable CONFIG_BOOTSTD_DEFAULTS for some xilinx boards Simon Glass
2023-04-02  2:02 ` [PATCH v7 10/11] atmel: Disable SPL_EFI_PARTITION for sama5d27_som1_ek_qspiflash Simon Glass
2023-04-02  2:02 ` [PATCH v7 11/11] bootstd: Enable BOOTSTD_DEFAULTS by default Simon Glass

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=20230402140231.v7.5.I53dbbf7d5e7585e4d1678117165a46de1d8d7eac@changeid \
    --to=sjg@chromium.org \
    --cc=hl@rock-chips.com \
    --cc=jeffy.chen@rock-chips.com \
    --cc=kettenis@openbsd.org \
    --cc=kever.yang@rock-chips.com \
    --cc=philipp.tomsich@theobroma-systems.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vagrant@debian.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).