[PATCH 1/3] bootstage: Update SPL support Kconfig description

[PATCH 1/3] bootstage: Update SPL support Kconfig description

[Marek Vasut]

Add "in SPL" suffix to the Kconfig description, to make it differ
from the "proper" U-Boot entry.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
 common/Kconfig.boot | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/Kconfig.boot b/common/Kconfig.boot
index a8d4be23a97..d3a12be2281 100644
--- a/common/Kconfig.boot
+++ b/common/Kconfig.boot
@@ -704,7 +704,7 @@ config SHOW_BOOT_PROGRESS
 	  151	common/cmd_nand.c	FIT image format OK
 
 config SPL_SHOW_BOOT_PROGRESS
-	bool "Show boot progress in a board-specific manner"
+	bool "Show boot progress in a board-specific manner in SPL"
 	depends on SPL
 	help
 	  Defining this option allows to add some board-specific code (calling
-- 
2.33.0

[PATCH 2/3] bootstage: Fix up remaining bootstage config misuse

[Marek Vasut]

All these macros really guard show_boot_progress() function, which
is used only if BOOT_PROGRESS is enabled in Kconfig. Fix up the
macro guards.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
 board/Seagate/dockstar/dockstar.c     | 2 +-
 board/Seagate/goflexhome/goflexhome.c | 2 +-
 board/bosch/shc/board.c               | 2 +-
 board/buffalo/lsxl/lsxl.c             | 2 +-
 board/k+p/kp_imx53/kp_imx53.c         | 2 +-
 board/st/stv0991/stv0991.c            | 2 +-
 include/bootstage.h                   | 2 +-
 7 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/board/Seagate/dockstar/dockstar.c b/board/Seagate/dockstar/dockstar.c
index fb691931580..819fc685565 100644
--- a/board/Seagate/dockstar/dockstar.c
+++ b/board/Seagate/dockstar/dockstar.c
@@ -140,7 +140,7 @@ void reset_phy(void)
 }
 #endif /* CONFIG_RESET_PHY_R */
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 #define GREEN_LED	(1 << 14)
 #define ORANGE_LED	(1 << 15)
 #define BOTH_LEDS	(GREEN_LED | ORANGE_LED)
diff --git a/board/Seagate/goflexhome/goflexhome.c b/board/Seagate/goflexhome/goflexhome.c
index 52be64fb8c2..e99335d5339 100644
--- a/board/Seagate/goflexhome/goflexhome.c
+++ b/board/Seagate/goflexhome/goflexhome.c
@@ -175,7 +175,7 @@ void reset_phy(void)
 }
 #endif /* CONFIG_RESET_PHY_R */
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 #define GREEN_LED	(1 << 14)
 #define ORANGE_LED	(1 << 15)
 #define BOTH_LEDS	(GREEN_LED | ORANGE_LED)
diff --git a/board/bosch/shc/board.c b/board/bosch/shc/board.c
index a7a9775fdf4..33edf412871 100644
--- a/board/bosch/shc/board.c
+++ b/board/bosch/shc/board.c
@@ -480,7 +480,7 @@ int board_eth_init(struct bd_info *bis)
 }
 #endif
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 static void bosch_check_reset_pin(void)
 {
 	if (readl(GPIO1_BASE + OMAP_GPIO_IRQSTATUS_SET_0) & RESET_MASK) {
diff --git a/board/buffalo/lsxl/lsxl.c b/board/buffalo/lsxl/lsxl.c
index 738b6bc25ce..3544a779968 100644
--- a/board/buffalo/lsxl/lsxl.c
+++ b/board/buffalo/lsxl/lsxl.c
@@ -271,7 +271,7 @@ int misc_init_r(void)
 }
 #endif
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 void show_boot_progress(int progress)
 {
 	if (progress > 0)
diff --git a/board/k+p/kp_imx53/kp_imx53.c b/board/k+p/kp_imx53/kp_imx53.c
index 7c3a695cb25..81f1191633e 100644
--- a/board/k+p/kp_imx53/kp_imx53.c
+++ b/board/k+p/kp_imx53/kp_imx53.c
@@ -155,7 +155,7 @@ int board_late_init(void)
 	return ret;
 }
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 #define GPIO_DR 0x0
 #define GPIO_GDIR 0x4
 #define GPIO_ALT1 0x1
diff --git a/board/st/stv0991/stv0991.c b/board/st/stv0991/stv0991.c
index 57ca9f659c1..7f1b31b654e 100644
--- a/board/st/stv0991/stv0991.c
+++ b/board/st/stv0991/stv0991.c
@@ -37,7 +37,7 @@ U_BOOT_DRVINFO(stv09911_serials) = {
 };
 #endif
 
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 void show_boot_progress(int progress)
 {
 	printf("%i\n", progress);
diff --git a/include/bootstage.h b/include/bootstage.h
index 8d1989ac0e5..63d338c932e 100644
--- a/include/bootstage.h
+++ b/include/bootstage.h
@@ -233,7 +233,7 @@ void show_boot_progress(int val);
 #endif
 
 #if !defined(USE_HOSTCC)
-#if CONFIG_IS_ENABLED(BOOTSTAGE)
+#if CONFIG_IS_ENABLED(SHOW_BOOT_PROGRESS)
 #define ENABLE_BOOTSTAGE
 #endif
 #endif
-- 
2.33.0

Re: [PATCH 2/3] bootstage: Fix up remaining bootstage config misuse

[Simon Glass]

On Wed, 3 Nov 2021 at 08:20, Marek Vasut <marex@denx.de> wrote:
>
> All these macros really guard show_boot_progress() function, which
> is used only if BOOT_PROGRESS is enabled in Kconfig. Fix up the
> macro guards.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>  board/Seagate/dockstar/dockstar.c     | 2 +-
>  board/Seagate/goflexhome/goflexhome.c | 2 +-
>  board/bosch/shc/board.c               | 2 +-
>  board/buffalo/lsxl/lsxl.c             | 2 +-
>  board/k+p/kp_imx53/kp_imx53.c         | 2 +-
>  board/st/stv0991/stv0991.c            | 2 +-
>  include/bootstage.h                   | 2 +-
>  7 files changed, 7 insertions(+), 7 deletions(-)
>
Reviewed-by: Simon Glass <sjg@chromium.org>

Re: [PATCH 2/3] bootstage: Fix up remaining bootstage config misuse

[Simon Glass]

Hi Marek,

On Thu, 4 Nov 2021 at 20:02, Simon Glass <sjg@chromium.org> wrote:
>
> On Wed, 3 Nov 2021 at 08:20, Marek Vasut <marex@denx.de> wrote:
> >
> > All these macros really guard show_boot_progress() function, which
> > is used only if BOOT_PROGRESS is enabled in Kconfig. Fix up the
> > macro guards.
> >
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Simon Glass <sjg@chromium.org>
> > ---
> >  board/Seagate/dockstar/dockstar.c     | 2 +-
> >  board/Seagate/goflexhome/goflexhome.c | 2 +-
> >  board/bosch/shc/board.c               | 2 +-
> >  board/buffalo/lsxl/lsxl.c             | 2 +-
> >  board/k+p/kp_imx53/kp_imx53.c         | 2 +-
> >  board/st/stv0991/stv0991.c            | 2 +-
> >  include/bootstage.h                   | 2 +-
> >  7 files changed, 7 insertions(+), 7 deletions(-)
> >
> Reviewed-by: Simon Glass <sjg@chromium.org>

This patch and the next one fail to build (make qcheck). Can you
please take a look?

Regards,
Simon

[PATCH 3/3] bootstage: Enable SPL bootstage in sandbox_spl config

[Marek Vasut]

Enable SPL bootstage support in sandbox_spl config to detect build issues in CI.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Simon Glass <sjg@chromium.org>
---
 configs/sandbox_spl_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/configs/sandbox_spl_defconfig b/configs/sandbox_spl_defconfig
index 1e3ccbb98bf..e3f47476c8c 100644
--- a/configs/sandbox_spl_defconfig
+++ b/configs/sandbox_spl_defconfig
@@ -23,6 +23,7 @@ CONFIG_FIT_VERBOSE=y
 CONFIG_SPL_LOAD_FIT=y
 # CONFIG_USE_SPL_FIT_GENERATOR is not set
 CONFIG_BOOTSTAGE=y
+CONFIG_SPL_BOOTSTAGE=y
 CONFIG_BOOTSTAGE_REPORT=y
 CONFIG_BOOTSTAGE_FDT=y
 CONFIG_BOOTSTAGE_STASH=y
-- 
2.33.0

Re: [PATCH 3/3] bootstage: Enable SPL bootstage in sandbox_spl config

[Simon Glass]

On Wed, 3 Nov 2021 at 08:20, Marek Vasut <marex@denx.de> wrote:
>
> Enable SPL bootstage support in sandbox_spl config to detect build issues in CI.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>  configs/sandbox_spl_defconfig | 1 +
>  1 file changed, 1 insertion(+)
>

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

Re: [PATCH 3/3] bootstage: Enable SPL bootstage in sandbox_spl config

[Marek Vasut]

On 11/5/21 03:02, Simon Glass wrote:
> On Wed, 3 Nov 2021 at 08:20, Marek Vasut <marex@denx.de> wrote:
>>
>> Enable SPL bootstage support in sandbox_spl config to detect build issues in CI.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> Cc: Simon Glass <sjg@chromium.org>
>> ---
>>   configs/sandbox_spl_defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
> 
> Reviewed-by: Simon Glass <sjg@chromium.org>

It seems neither of these patches have been applied.

Re: [PATCH 1/3] bootstage: Update SPL support Kconfig description

[Simon Glass]

On Wed, 3 Nov 2021 at 08:20, Marek Vasut <marex@denx.de> wrote:
>
> Add "in SPL" suffix to the Kconfig description, to make it differ
> from the "proper" U-Boot entry.
>
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Simon Glass <sjg@chromium.org>
> ---
>  common/Kconfig.boot | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

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