* [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb @ 2022-08-17 9:59 Pali Rohár 2022-08-17 10:05 ` Stefan Roese 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-08-17 9:59 UTC (permalink / raw) To: Stefan Roese, Simon Glass, Tom Rini; +Cc: u-boot File name with pattern u-boot-spl* is used on all places except in kwb image for binary with SPL-only code. Combined binary with both SPL and proper U-Boot in other places has file name pattern u-boot-with-spl*. Make it consistent also for kwb image and rename u-boot-spl.kwb to u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. Signed-off-by: Pali Rohár <pali@kernel.org> --- This is just RFC patch, please let me know what do you think. --- Kconfig | 2 +- Makefile | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/Kconfig b/Kconfig index 991b260182e8..5c64ca843eed 100644 --- a/Kconfig +++ b/Kconfig @@ -455,7 +455,7 @@ config BUILD_TARGET string "Build target special images" default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL default "u-boot-elf.srec" if RCAR_GEN3 default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ ARCH_SUNXI || RISCV || ARCH_ZYNQMP) diff --git a/Makefile b/Makefile index 1a66f69a4b14..b1fbdbe7d726 100644 --- a/Makefile +++ b/Makefile @@ -1432,7 +1432,7 @@ KWD_CONFIG_FILE = $(shell \ MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) \ $(if $(KEYDIR),-k $(KEYDIR)) @@ -1477,7 +1477,7 @@ u-boot.itb: u-boot-nodtb.bin \ $(BOARD_SIZE_CHECK) endif -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE $(call if_changed,mkimage) u-boot.sha1: u-boot.bin -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-08-17 9:59 [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb Pali Rohár @ 2022-08-17 10:05 ` Stefan Roese 2022-08-17 21:00 ` Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Stefan Roese @ 2022-08-17 10:05 UTC (permalink / raw) To: Pali Rohár, Simon Glass, Tom Rini; +Cc: u-boot On 17.08.22 11:59, Pali Rohár wrote: > File name with pattern u-boot-spl* is used on all places except in kwb > image for binary with SPL-only code. Combined binary with both SPL and > proper U-Boot in other places has file name pattern u-boot-with-spl*. > > Make it consistent also for kwb image and rename u-boot-spl.kwb to > u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. > > Signed-off-by: Pali Rohár <pali@kernel.org> > --- > This is just RFC patch, please let me know what do you think. Frankly, I wondered a few days ago if this image only includes the SPL image. So this change makes total sense to me. It might break some out-of-tree building scripts, but I think we should go forward this way: Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > Kconfig | 2 +- > Makefile | 4 ++-- > 2 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/Kconfig b/Kconfig > index 991b260182e8..5c64ca843eed 100644 > --- a/Kconfig > +++ b/Kconfig > @@ -455,7 +455,7 @@ config BUILD_TARGET > string "Build target special images" > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 > - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL > + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL > default "u-boot-elf.srec" if RCAR_GEN3 > default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ > ARCH_SUNXI || RISCV || ARCH_ZYNQMP) > diff --git a/Makefile b/Makefile > index 1a66f69a4b14..b1fbdbe7d726 100644 > --- a/Makefile > +++ b/Makefile > @@ -1432,7 +1432,7 @@ KWD_CONFIG_FILE = $(shell \ > MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) > > -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ > +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) \ > $(if $(KEYDIR),-k $(KEYDIR)) > > @@ -1477,7 +1477,7 @@ u-boot.itb: u-boot-nodtb.bin \ > $(BOARD_SIZE_CHECK) > endif > > -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > $(call if_changed,mkimage) > > u-boot.sha1: u-boot.bin Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-08-17 10:05 ` Stefan Roese @ 2022-08-17 21:00 ` Pali Rohár 2022-09-07 9:24 ` Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-08-17 21:00 UTC (permalink / raw) To: Stefan Roese; +Cc: Simon Glass, Tom Rini, u-boot On Wednesday 17 August 2022 12:05:20 Stefan Roese wrote: > On 17.08.22 11:59, Pali Rohár wrote: > > File name with pattern u-boot-spl* is used on all places except in kwb > > image for binary with SPL-only code. Combined binary with both SPL and > > proper U-Boot in other places has file name pattern u-boot-with-spl*. > > > > Make it consistent also for kwb image and rename u-boot-spl.kwb to > > u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. > > > > Signed-off-by: Pali Rohár <pali@kernel.org> > > --- > > This is just RFC patch, please let me know what do you think. > > Frankly, I wondered a few days ago if this image only includes the > SPL image. So this change makes total sense to me. It might break > some out-of-tree building scripts, but I think we should go forward > this way: > > Reviewed-by: Stefan Roese <sr@denx.de> Ok! If you like it, feel free to take it. First time I was also confused and having consistent naming could help other people too... > Thanks, > Stefan > > > --- > > Kconfig | 2 +- > > Makefile | 4 ++-- > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/Kconfig b/Kconfig > > index 991b260182e8..5c64ca843eed 100644 > > --- a/Kconfig > > +++ b/Kconfig > > @@ -455,7 +455,7 @@ config BUILD_TARGET > > string "Build target special images" > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 > > - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL > > + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL > > default "u-boot-elf.srec" if RCAR_GEN3 > > default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ > > ARCH_SUNXI || RISCV || ARCH_ZYNQMP) > > diff --git a/Makefile b/Makefile > > index 1a66f69a4b14..b1fbdbe7d726 100644 > > --- a/Makefile > > +++ b/Makefile > > @@ -1432,7 +1432,7 @@ KWD_CONFIG_FILE = $(shell \ > > MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) > > -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) \ > > $(if $(KEYDIR),-k $(KEYDIR)) > > @@ -1477,7 +1477,7 @@ u-boot.itb: u-boot-nodtb.bin \ > > $(BOARD_SIZE_CHECK) > > endif > > -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > $(call if_changed,mkimage) > > u-boot.sha1: u-boot.bin > > Viele Grüße, > Stefan Roese > > -- > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-08-17 21:00 ` Pali Rohár @ 2022-09-07 9:24 ` Pali Rohár 2022-11-01 22:39 ` Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-09-07 9:24 UTC (permalink / raw) To: Stefan Roese; +Cc: Simon Glass, Tom Rini, u-boot On Wednesday 17 August 2022 23:00:43 Pali Rohár wrote: > On Wednesday 17 August 2022 12:05:20 Stefan Roese wrote: > > On 17.08.22 11:59, Pali Rohár wrote: > > > File name with pattern u-boot-spl* is used on all places except in kwb > > > image for binary with SPL-only code. Combined binary with both SPL and > > > proper U-Boot in other places has file name pattern u-boot-with-spl*. > > > > > > Make it consistent also for kwb image and rename u-boot-spl.kwb to > > > u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. > > > > > > Signed-off-by: Pali Rohár <pali@kernel.org> > > > --- > > > This is just RFC patch, please let me know what do you think. > > > > Frankly, I wondered a few days ago if this image only includes the > > SPL image. So this change makes total sense to me. It might break > > some out-of-tree building scripts, but I think we should go forward > > this way: > > > > Reviewed-by: Stefan Roese <sr@denx.de> > > Ok! If you like it, feel free to take it. First time I was also confused > and having consistent naming could help other people too... Btw, there is similar "rename" patch for u-boot-with-dtb.bin target: https://patchwork.ozlabs.org/project/uboot/patch/20220801154220.20068-2-pali@kernel.org/ > > Thanks, > > Stefan > > > > > --- > > > Kconfig | 2 +- > > > Makefile | 4 ++-- > > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > > > diff --git a/Kconfig b/Kconfig > > > index 991b260182e8..5c64ca843eed 100644 > > > --- a/Kconfig > > > +++ b/Kconfig > > > @@ -455,7 +455,7 @@ config BUILD_TARGET > > > string "Build target special images" > > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 > > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 > > > - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL > > > + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL > > > default "u-boot-elf.srec" if RCAR_GEN3 > > > default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ > > > ARCH_SUNXI || RISCV || ARCH_ZYNQMP) > > > diff --git a/Makefile b/Makefile > > > index 1a66f69a4b14..b1fbdbe7d726 100644 > > > --- a/Makefile > > > +++ b/Makefile > > > @@ -1432,7 +1432,7 @@ KWD_CONFIG_FILE = $(shell \ > > > MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ > > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) > > > -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > > +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) \ > > > $(if $(KEYDIR),-k $(KEYDIR)) > > > @@ -1477,7 +1477,7 @@ u-boot.itb: u-boot-nodtb.bin \ > > > $(BOARD_SIZE_CHECK) > > > endif > > > -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > > +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > > $(call if_changed,mkimage) > > > u-boot.sha1: u-boot.bin > > > > Viele Grüße, > > Stefan Roese > > > > -- > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-09-07 9:24 ` Pali Rohár @ 2022-11-01 22:39 ` Pali Rohár 2022-11-02 8:55 ` Stefan Roese 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-11-01 22:39 UTC (permalink / raw) To: Stefan Roese; +Cc: Simon Glass, Tom Rini, u-boot On Wednesday 07 September 2022 11:24:58 Pali Rohár wrote: > On Wednesday 17 August 2022 23:00:43 Pali Rohár wrote: > > On Wednesday 17 August 2022 12:05:20 Stefan Roese wrote: > > > On 17.08.22 11:59, Pali Rohár wrote: > > > > File name with pattern u-boot-spl* is used on all places except in kwb > > > > image for binary with SPL-only code. Combined binary with both SPL and > > > > proper U-Boot in other places has file name pattern u-boot-with-spl*. > > > > > > > > Make it consistent also for kwb image and rename u-boot-spl.kwb to > > > > u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. > > > > > > > > Signed-off-by: Pali Rohár <pali@kernel.org> > > > > --- > > > > This is just RFC patch, please let me know what do you think. > > > > > > Frankly, I wondered a few days ago if this image only includes the > > > SPL image. So this change makes total sense to me. It might break > > > some out-of-tree building scripts, but I think we should go forward > > > this way: > > > > > > Reviewed-by: Stefan Roese <sr@denx.de> > > > > Ok! If you like it, feel free to take it. First time I was also confused > > and having consistent naming could help other people too... > > Btw, there is similar "rename" patch for u-boot-with-dtb.bin target: > https://patchwork.ozlabs.org/project/uboot/patch/20220801154220.20068-2-pali@kernel.org/ Rename for u-boot-with-dtb.bin was accepted. > > > Thanks, > > > Stefan > > > > > > > --- > > > > Kconfig | 2 +- > > > > Makefile | 4 ++-- > > > > 2 files changed, 3 insertions(+), 3 deletions(-) > > > > > > > > diff --git a/Kconfig b/Kconfig > > > > index 991b260182e8..5c64ca843eed 100644 > > > > --- a/Kconfig > > > > +++ b/Kconfig > > > > @@ -455,7 +455,7 @@ config BUILD_TARGET > > > > string "Build target special images" > > > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 > > > > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 > > > > - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL > > > > + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL > > > > default "u-boot-elf.srec" if RCAR_GEN3 > > > > default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ > > > > ARCH_SUNXI || RISCV || ARCH_ZYNQMP) > > > > diff --git a/Makefile b/Makefile > > > > index 1a66f69a4b14..b1fbdbe7d726 100644 > > > > --- a/Makefile > > > > +++ b/Makefile > > > > @@ -1432,7 +1432,7 @@ KWD_CONFIG_FILE = $(shell \ > > > > MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ > > > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) > > > > -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > > > +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ > > > > -T kwbimage -a $(CONFIG_SYS_TEXT_BASE) -e $(CONFIG_SYS_TEXT_BASE) \ > > > > $(if $(KEYDIR),-k $(KEYDIR)) > > > > @@ -1477,7 +1477,7 @@ u-boot.itb: u-boot-nodtb.bin \ > > > > $(BOARD_SIZE_CHECK) > > > > endif > > > > -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > > > +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > > > > $(call if_changed,mkimage) > > > > u-boot.sha1: u-boot.bin > > > > > > Viele Grüße, > > > Stefan Roese > > > > > > -- > > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > > Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-11-01 22:39 ` Pali Rohár @ 2022-11-02 8:55 ` Stefan Roese 2022-11-02 17:51 ` [PATCH] " Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Stefan Roese @ 2022-11-02 8:55 UTC (permalink / raw) To: Pali Rohár; +Cc: Simon Glass, Tom Rini, u-boot Hi Pali, On 01.11.22 23:39, Pali Rohár wrote: > On Wednesday 07 September 2022 11:24:58 Pali Rohár wrote: >> On Wednesday 17 August 2022 23:00:43 Pali Rohár wrote: >>> On Wednesday 17 August 2022 12:05:20 Stefan Roese wrote: >>>> On 17.08.22 11:59, Pali Rohár wrote: >>>>> File name with pattern u-boot-spl* is used on all places except in kwb >>>>> image for binary with SPL-only code. Combined binary with both SPL and >>>>> proper U-Boot in other places has file name pattern u-boot-with-spl*. >>>>> >>>>> Make it consistent also for kwb image and rename u-boot-spl.kwb to >>>>> u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. >>>>> >>>>> Signed-off-by: Pali Rohár <pali@kernel.org> >>>>> --- >>>>> This is just RFC patch, please let me know what do you think. >>>> >>>> Frankly, I wondered a few days ago if this image only includes the >>>> SPL image. So this change makes total sense to me. It might break >>>> some out-of-tree building scripts, but I think we should go forward >>>> this way: >>>> >>>> Reviewed-by: Stefan Roese <sr@denx.de> >>> >>> Ok! If you like it, feel free to take it. First time I was also confused >>> and having consistent naming could help other people too... >> >> Btw, there is similar "rename" patch for u-boot-with-dtb.bin target: >> https://patchwork.ozlabs.org/project/uboot/patch/20220801154220.20068-2-pali@kernel.org/ > > Rename for u-boot-with-dtb.bin was accepted. Thanks for the notice. I'm all for adding this kwb change as well. Could you please send a non-RFC patch for this which also renames all the "u-boot-spl.kwb" occurrences in the tree? Thanks, Stefan ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-11-02 8:55 ` Stefan Roese @ 2022-11-02 17:51 ` Pali Rohár 2022-11-06 21:21 ` Pali Rohár 0 siblings, 1 reply; 10+ messages in thread From: Pali Rohár @ 2022-11-02 17:51 UTC (permalink / raw) To: Stefan Roese; +Cc: u-boot File name with pattern u-boot-spl* is used on all places except in kwb image for binary with SPL-only code. Combined binary with both SPL and proper U-Boot in other places has file name pattern u-boot-with-spl*. Make it consistent also for kwb image and rename u-boot-spl.kwb to u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. Also update documentation about file name changes. Signed-off-by: Pali Rohár <pali@kernel.org> Reviewed-by: Stefan Roese <sr@denx.de> --- Kconfig | 2 +- Makefile | 4 ++-- board/Marvell/db-88f6820-gp/README | 6 +++--- board/kobol/helios4/README | 6 +++--- board/solidrun/clearfog/README | 8 ++++---- doc/README.armada-secureboot | 4 ++-- doc/kwboot.1 | 12 ++++++------ include/configs/ds414.h | 2 +- 8 files changed, 22 insertions(+), 22 deletions(-) diff --git a/Kconfig b/Kconfig index ef94f88adc28..43d4a48ac808 100644 --- a/Kconfig +++ b/Kconfig @@ -456,7 +456,7 @@ config BUILD_TARGET string "Build target special images" default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL default "u-boot-elf.srec" if RCAR_GEN3 default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ ARCH_SUNXI || RISCV || ARCH_ZYNQMP) diff --git a/Makefile b/Makefile index d5c2c9700921..eb0cc1ac060f 100644 --- a/Makefile +++ b/Makefile @@ -1422,7 +1422,7 @@ KWD_CONFIG_FILE = $(shell \ MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) \ $(if $(KEYDIR),-k $(KEYDIR)) @@ -1463,7 +1463,7 @@ u-boot.itb: u-boot-nodtb.bin \ $(BOARD_SIZE_CHECK) endif -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE $(call if_changed,mkimage) $(BOARD_SIZE_CHECK) diff --git a/board/Marvell/db-88f6820-gp/README b/board/Marvell/db-88f6820-gp/README index 9bea5b35cb40..b4db6ed62577 100644 --- a/board/Marvell/db-88f6820-gp/README +++ b/board/Marvell/db-88f6820-gp/README @@ -2,17 +2,17 @@ Update from original Marvell U-Boot to mainline U-Boot: ------------------------------------------------------- The resulting image including the SPL binary with the -full DDR setup is "u-boot-spl.kwb". +full DDR setup is "u-boot-with-spl.kwb". To update the SPI NOR flash, please use the following command: -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ sf update 2000000 0 60000 Note that the original Marvell U-Boot seems to have problems with the "sf update" command. This does not work reliable. So here this command should be used: -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ sf erase 0 60000;sf write 2000000 0 60000 diff --git a/board/kobol/helios4/README b/board/kobol/helios4/README index 749f04827824..d615cdd2648b 100644 --- a/board/kobol/helios4/README +++ b/board/kobol/helios4/README @@ -7,12 +7,12 @@ $ make helios4_defconfig $ make The resulting image including the SPL binary with the -full DDR setup is "u-boot-spl.kwb". +full DDR setup is "u-boot-with-spl.kwb". Now all you need to do is copy this image on a SD card. For example with this command: -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 Please use the correct device node for your setup instead of "/dev/sdX" here! @@ -38,7 +38,7 @@ Set the SW1 DIP switches to UART boot (see above). Run the following command to initiate U-Boot download: - ./tools/kwboot -p -b u-boot-spl.kwb /dev/ttyUSBX + ./tools/kwboot -p -b u-boot-with-spl.kwb /dev/ttyUSBX Use the correct UART device node for /dev/ttyUSBX. diff --git a/board/solidrun/clearfog/README b/board/solidrun/clearfog/README index 9375be84957a..ed4a712c5aa2 100644 --- a/board/solidrun/clearfog/README +++ b/board/solidrun/clearfog/README @@ -7,12 +7,12 @@ $ make clearfog_defconfig $ make The resulting image including the SPL binary with the -full DDR setup is "u-boot-spl.kwb". +full DDR setup is "u-boot-with-spl.kwb". Now all you need to do is copy this image on a SD card. For example with this command: -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 Please use the correct device node for your setup instead of "/dev/sdX" here! @@ -29,7 +29,7 @@ command: Install U-Boot on eMMC boot partition from Linux running on Clearfog: echo 0 > /sys/block/mmcblk0boot0/force_ro - dd if=u-boot-spl.kwb of=/dev/mmcblk0boot0 + dd if=u-boot-with-spl.kwb of=/dev/mmcblk0boot0 Note that the SD card is not accessible when the Clearfog SOM has eMMC. Consider initial boot from UART (see below). @@ -66,7 +66,7 @@ Set the SW1 DIP switches to UART boot (see above). Run the following command to initiate U-Boot download: - ./tools/kwboot -b u-boot-spl.kwb /dev/ttyUSBX + ./tools/kwboot -b u-boot-with-spl.kwb /dev/ttyUSBX Use the correct UART device node for /dev/ttyUSBX. diff --git a/doc/README.armada-secureboot b/doc/README.armada-secureboot index 157cb5a231a9..4ec89d5c061f 100644 --- a/doc/README.armada-secureboot +++ b/doc/README.armada-secureboot @@ -257,13 +257,13 @@ ARM architecture The creation of the boot image is done via the usual invocation of make (with a suitably set CROSS_COMPILE environment variable, of course). The resulting boot -image u-boot-spl.kwb can then be tested, if so desired. The hdrparser from [5] +image u-boot-with-spl.kwb can then be tested, if so desired. The hdrparser from [5] can be used for this purpose. To build the tool, invoke make in the 'tools/marvell/doimage_mv' directory of [5], which builds a stand-alone hdrparser executable. A test can be conducted by calling hdrparser with the produced boot image and the following (mandatory) parameters: -./hdrparser -k 0 -t u-boot-spl.kwb +./hdrparser -k 0 -t u-boot-with-spl.kwb Here we assume that the CSK index is 0 and the boot image file resides in the same directory (adapt accordingly if needed). The tool should report that all diff --git a/doc/kwboot.1 b/doc/kwboot.1 index 0863d197fd49..a528fbbe8c38 100644 --- a/doc/kwboot.1 +++ b/doc/kwboot.1 @@ -149,25 +149,25 @@ Tested values for \fIbaudrate\fP for Armada 38x include: 115200, .SH "EXAMPLES" -Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-spl.kwb\fP +Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP at 115200 Bd and run terminal program: .IP -.B kwboot -b u-boot-spl.kwb -t /dev/ttyUSB0 +.B kwboot -b u-boot-with-spl.kwb -t /dev/ttyUSB0 .PP Instruct BootROM to enter boot Xmodem boot mode, send header of -\fIu-boot-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct +\fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct BootROM to change baudrate to 5200000 Bd, send data part of the kwbimage file via Xmodem at high speed and finally run terminal program: .IP -.B kwboot -b u-boot-spl.kwb -B 5200000 -t /dev/ttyUSB0 +.B kwboot -b u-boot-with-spl.kwb -B 5200000 -t /dev/ttyUSB0 .PP -Only send \fIu-boot-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP +Only send \fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP at 115200 Bd: .IP -.B kwboot -D u-boot-spl.kwb /dev/ttyUSB0 +.B kwboot -D u-boot-with-spl.kwb /dev/ttyUSB0 .PP Instruct BootROM to enter console debug mode and run terminal program on diff --git a/include/configs/ds414.h b/include/configs/ds414.h index f8273a92f11a..76d1713fdc68 100644 --- a/include/configs/ds414.h +++ b/include/configs/ds414.h @@ -50,7 +50,7 @@ "ethmtu=1500\0eth1mtu=1500\0" \ "update_uboot=sf probe; dhcp; " \ "mw.b ${loadaddr} 0x0 0xd0000; " \ - "tftpboot ${loadaddr} u-boot-spl.kwb; " \ + "tftpboot ${loadaddr} u-boot-with-spl.kwb; " \ "sf update ${loadaddr} 0x0 0xd0000\0" -- 2.20.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-11-02 17:51 ` [PATCH] " Pali Rohár @ 2022-11-06 21:21 ` Pali Rohár 2022-11-07 6:25 ` Stefan Roese 2022-11-07 8:24 ` Stefan Roese 0 siblings, 2 replies; 10+ messages in thread From: Pali Rohár @ 2022-11-06 21:21 UTC (permalink / raw) To: Stefan Roese; +Cc: u-boot Stefan, it is OK now? On Wednesday 02 November 2022 18:51:28 Pali Rohár wrote: > File name with pattern u-boot-spl* is used on all places except in kwb > image for binary with SPL-only code. Combined binary with both SPL and > proper U-Boot in other places has file name pattern u-boot-with-spl*. > > Make it consistent also for kwb image and rename u-boot-spl.kwb to > u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. > > Also update documentation about file name changes. > > Signed-off-by: Pali Rohár <pali@kernel.org> > Reviewed-by: Stefan Roese <sr@denx.de> > --- > Kconfig | 2 +- > Makefile | 4 ++-- > board/Marvell/db-88f6820-gp/README | 6 +++--- > board/kobol/helios4/README | 6 +++--- > board/solidrun/clearfog/README | 8 ++++---- > doc/README.armada-secureboot | 4 ++-- > doc/kwboot.1 | 12 ++++++------ > include/configs/ds414.h | 2 +- > 8 files changed, 22 insertions(+), 22 deletions(-) > > diff --git a/Kconfig b/Kconfig > index ef94f88adc28..43d4a48ac808 100644 > --- a/Kconfig > +++ b/Kconfig > @@ -456,7 +456,7 @@ config BUILD_TARGET > string "Build target special images" > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 > default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 > - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL > + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL > default "u-boot-elf.srec" if RCAR_GEN3 > default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ > ARCH_SUNXI || RISCV || ARCH_ZYNQMP) > diff --git a/Makefile b/Makefile > index d5c2c9700921..eb0cc1ac060f 100644 > --- a/Makefile > +++ b/Makefile > @@ -1422,7 +1422,7 @@ KWD_CONFIG_FILE = $(shell \ > MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ > -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) > > -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ > +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ > -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) \ > $(if $(KEYDIR),-k $(KEYDIR)) > > @@ -1463,7 +1463,7 @@ u-boot.itb: u-boot-nodtb.bin \ > $(BOARD_SIZE_CHECK) > endif > > -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE > $(call if_changed,mkimage) > $(BOARD_SIZE_CHECK) > > diff --git a/board/Marvell/db-88f6820-gp/README b/board/Marvell/db-88f6820-gp/README > index 9bea5b35cb40..b4db6ed62577 100644 > --- a/board/Marvell/db-88f6820-gp/README > +++ b/board/Marvell/db-88f6820-gp/README > @@ -2,17 +2,17 @@ Update from original Marvell U-Boot to mainline U-Boot: > ------------------------------------------------------- > > The resulting image including the SPL binary with the > -full DDR setup is "u-boot-spl.kwb". > +full DDR setup is "u-boot-with-spl.kwb". > > To update the SPI NOR flash, please use the following > command: > > -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ > +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ > sf update 2000000 0 60000 > > Note that the original Marvell U-Boot seems to have > problems with the "sf update" command. This does not > work reliable. So here this command should be used: > > -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ > +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ > sf erase 0 60000;sf write 2000000 0 60000 > diff --git a/board/kobol/helios4/README b/board/kobol/helios4/README > index 749f04827824..d615cdd2648b 100644 > --- a/board/kobol/helios4/README > +++ b/board/kobol/helios4/README > @@ -7,12 +7,12 @@ $ make helios4_defconfig > $ make > > The resulting image including the SPL binary with the > -full DDR setup is "u-boot-spl.kwb". > +full DDR setup is "u-boot-with-spl.kwb". > > Now all you need to do is copy this image on a SD card. > For example with this command: > > -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 > +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 > > Please use the correct device node for your setup instead > of "/dev/sdX" here! > @@ -38,7 +38,7 @@ Set the SW1 DIP switches to UART boot (see above). > > Run the following command to initiate U-Boot download: > > - ./tools/kwboot -p -b u-boot-spl.kwb /dev/ttyUSBX > + ./tools/kwboot -p -b u-boot-with-spl.kwb /dev/ttyUSBX > > Use the correct UART device node for /dev/ttyUSBX. > > diff --git a/board/solidrun/clearfog/README b/board/solidrun/clearfog/README > index 9375be84957a..ed4a712c5aa2 100644 > --- a/board/solidrun/clearfog/README > +++ b/board/solidrun/clearfog/README > @@ -7,12 +7,12 @@ $ make clearfog_defconfig > $ make > > The resulting image including the SPL binary with the > -full DDR setup is "u-boot-spl.kwb". > +full DDR setup is "u-boot-with-spl.kwb". > > Now all you need to do is copy this image on a SD card. > For example with this command: > > -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 > +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 > > Please use the correct device node for your setup instead > of "/dev/sdX" here! > @@ -29,7 +29,7 @@ command: > Install U-Boot on eMMC boot partition from Linux running on Clearfog: > > echo 0 > /sys/block/mmcblk0boot0/force_ro > - dd if=u-boot-spl.kwb of=/dev/mmcblk0boot0 > + dd if=u-boot-with-spl.kwb of=/dev/mmcblk0boot0 > > Note that the SD card is not accessible when the Clearfog SOM has eMMC. > Consider initial boot from UART (see below). > @@ -66,7 +66,7 @@ Set the SW1 DIP switches to UART boot (see above). > > Run the following command to initiate U-Boot download: > > - ./tools/kwboot -b u-boot-spl.kwb /dev/ttyUSBX > + ./tools/kwboot -b u-boot-with-spl.kwb /dev/ttyUSBX > > Use the correct UART device node for /dev/ttyUSBX. > > diff --git a/doc/README.armada-secureboot b/doc/README.armada-secureboot > index 157cb5a231a9..4ec89d5c061f 100644 > --- a/doc/README.armada-secureboot > +++ b/doc/README.armada-secureboot > @@ -257,13 +257,13 @@ ARM architecture > > The creation of the boot image is done via the usual invocation of make (with a > suitably set CROSS_COMPILE environment variable, of course). The resulting boot > -image u-boot-spl.kwb can then be tested, if so desired. The hdrparser from [5] > +image u-boot-with-spl.kwb can then be tested, if so desired. The hdrparser from [5] > can be used for this purpose. To build the tool, invoke make in the > 'tools/marvell/doimage_mv' directory of [5], which builds a stand-alone > hdrparser executable. A test can be conducted by calling hdrparser with the > produced boot image and the following (mandatory) parameters: > > -./hdrparser -k 0 -t u-boot-spl.kwb > +./hdrparser -k 0 -t u-boot-with-spl.kwb > > Here we assume that the CSK index is 0 and the boot image file resides in the > same directory (adapt accordingly if needed). The tool should report that all > diff --git a/doc/kwboot.1 b/doc/kwboot.1 > index 0863d197fd49..a528fbbe8c38 100644 > --- a/doc/kwboot.1 > +++ b/doc/kwboot.1 > @@ -149,25 +149,25 @@ Tested values for \fIbaudrate\fP for Armada 38x include: 115200, > > .SH "EXAMPLES" > > -Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-spl.kwb\fP > +Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-with-spl.kwb\fP > kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP at 115200 Bd and run terminal > program: > .IP > -.B kwboot -b u-boot-spl.kwb -t /dev/ttyUSB0 > +.B kwboot -b u-boot-with-spl.kwb -t /dev/ttyUSB0 > > .PP > Instruct BootROM to enter boot Xmodem boot mode, send header of > -\fIu-boot-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct > +\fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct > BootROM to change baudrate to 5200000 Bd, send data part of the kwbimage > file via Xmodem at high speed and finally run terminal program: > .IP > -.B kwboot -b u-boot-spl.kwb -B 5200000 -t /dev/ttyUSB0 > +.B kwboot -b u-boot-with-spl.kwb -B 5200000 -t /dev/ttyUSB0 > > .PP > -Only send \fIu-boot-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP > +Only send \fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP > at 115200 Bd: > .IP > -.B kwboot -D u-boot-spl.kwb /dev/ttyUSB0 > +.B kwboot -D u-boot-with-spl.kwb /dev/ttyUSB0 > > .PP > Instruct BootROM to enter console debug mode and run terminal program on > diff --git a/include/configs/ds414.h b/include/configs/ds414.h > index f8273a92f11a..76d1713fdc68 100644 > --- a/include/configs/ds414.h > +++ b/include/configs/ds414.h > @@ -50,7 +50,7 @@ > "ethmtu=1500\0eth1mtu=1500\0" \ > "update_uboot=sf probe; dhcp; " \ > "mw.b ${loadaddr} 0x0 0xd0000; " \ > - "tftpboot ${loadaddr} u-boot-spl.kwb; " \ > + "tftpboot ${loadaddr} u-boot-with-spl.kwb; " \ > "sf update ${loadaddr} 0x0 0xd0000\0" > > > -- > 2.20.1 > ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-11-06 21:21 ` Pali Rohár @ 2022-11-07 6:25 ` Stefan Roese 2022-11-07 8:24 ` Stefan Roese 1 sibling, 0 replies; 10+ messages in thread From: Stefan Roese @ 2022-11-07 6:25 UTC (permalink / raw) To: Pali Rohár; +Cc: u-boot Hi Pali, On 06.11.22 22:21, Pali Rohár wrote: > Stefan, it is OK now? Yes, I think so. I'm just now running a CI world build and will push this shortly if no problems show up: Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > On Wednesday 02 November 2022 18:51:28 Pali Rohár wrote: >> File name with pattern u-boot-spl* is used on all places except in kwb >> image for binary with SPL-only code. Combined binary with both SPL and >> proper U-Boot in other places has file name pattern u-boot-with-spl*. >> >> Make it consistent also for kwb image and rename u-boot-spl.kwb to >> u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. >> >> Also update documentation about file name changes. >> >> Signed-off-by: Pali Rohár <pali@kernel.org> >> Reviewed-by: Stefan Roese <sr@denx.de> >> --- >> Kconfig | 2 +- >> Makefile | 4 ++-- >> board/Marvell/db-88f6820-gp/README | 6 +++--- >> board/kobol/helios4/README | 6 +++--- >> board/solidrun/clearfog/README | 8 ++++---- >> doc/README.armada-secureboot | 4 ++-- >> doc/kwboot.1 | 12 ++++++------ >> include/configs/ds414.h | 2 +- >> 8 files changed, 22 insertions(+), 22 deletions(-) >> >> diff --git a/Kconfig b/Kconfig >> index ef94f88adc28..43d4a48ac808 100644 >> --- a/Kconfig >> +++ b/Kconfig >> @@ -456,7 +456,7 @@ config BUILD_TARGET >> string "Build target special images" >> default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 >> default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 >> - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL >> + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL >> default "u-boot-elf.srec" if RCAR_GEN3 >> default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ >> ARCH_SUNXI || RISCV || ARCH_ZYNQMP) >> diff --git a/Makefile b/Makefile >> index d5c2c9700921..eb0cc1ac060f 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1422,7 +1422,7 @@ KWD_CONFIG_FILE = $(shell \ >> MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ >> -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) >> >> -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ >> +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ >> -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) \ >> $(if $(KEYDIR),-k $(KEYDIR)) >> >> @@ -1463,7 +1463,7 @@ u-boot.itb: u-boot-nodtb.bin \ >> $(BOARD_SIZE_CHECK) >> endif >> >> -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE >> +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE >> $(call if_changed,mkimage) >> $(BOARD_SIZE_CHECK) >> >> diff --git a/board/Marvell/db-88f6820-gp/README b/board/Marvell/db-88f6820-gp/README >> index 9bea5b35cb40..b4db6ed62577 100644 >> --- a/board/Marvell/db-88f6820-gp/README >> +++ b/board/Marvell/db-88f6820-gp/README >> @@ -2,17 +2,17 @@ Update from original Marvell U-Boot to mainline U-Boot: >> ------------------------------------------------------- >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> To update the SPI NOR flash, please use the following >> command: >> >> -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ >> +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ >> sf update 2000000 0 60000 >> >> Note that the original Marvell U-Boot seems to have >> problems with the "sf update" command. This does not >> work reliable. So here this command should be used: >> >> -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ >> +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ >> sf erase 0 60000;sf write 2000000 0 60000 >> diff --git a/board/kobol/helios4/README b/board/kobol/helios4/README >> index 749f04827824..d615cdd2648b 100644 >> --- a/board/kobol/helios4/README >> +++ b/board/kobol/helios4/README >> @@ -7,12 +7,12 @@ $ make helios4_defconfig >> $ make >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> Now all you need to do is copy this image on a SD card. >> For example with this command: >> >> -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 >> +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 >> >> Please use the correct device node for your setup instead >> of "/dev/sdX" here! >> @@ -38,7 +38,7 @@ Set the SW1 DIP switches to UART boot (see above). >> >> Run the following command to initiate U-Boot download: >> >> - ./tools/kwboot -p -b u-boot-spl.kwb /dev/ttyUSBX >> + ./tools/kwboot -p -b u-boot-with-spl.kwb /dev/ttyUSBX >> >> Use the correct UART device node for /dev/ttyUSBX. >> >> diff --git a/board/solidrun/clearfog/README b/board/solidrun/clearfog/README >> index 9375be84957a..ed4a712c5aa2 100644 >> --- a/board/solidrun/clearfog/README >> +++ b/board/solidrun/clearfog/README >> @@ -7,12 +7,12 @@ $ make clearfog_defconfig >> $ make >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> Now all you need to do is copy this image on a SD card. >> For example with this command: >> >> -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 >> +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 >> >> Please use the correct device node for your setup instead >> of "/dev/sdX" here! >> @@ -29,7 +29,7 @@ command: >> Install U-Boot on eMMC boot partition from Linux running on Clearfog: >> >> echo 0 > /sys/block/mmcblk0boot0/force_ro >> - dd if=u-boot-spl.kwb of=/dev/mmcblk0boot0 >> + dd if=u-boot-with-spl.kwb of=/dev/mmcblk0boot0 >> >> Note that the SD card is not accessible when the Clearfog SOM has eMMC. >> Consider initial boot from UART (see below). >> @@ -66,7 +66,7 @@ Set the SW1 DIP switches to UART boot (see above). >> >> Run the following command to initiate U-Boot download: >> >> - ./tools/kwboot -b u-boot-spl.kwb /dev/ttyUSBX >> + ./tools/kwboot -b u-boot-with-spl.kwb /dev/ttyUSBX >> >> Use the correct UART device node for /dev/ttyUSBX. >> >> diff --git a/doc/README.armada-secureboot b/doc/README.armada-secureboot >> index 157cb5a231a9..4ec89d5c061f 100644 >> --- a/doc/README.armada-secureboot >> +++ b/doc/README.armada-secureboot >> @@ -257,13 +257,13 @@ ARM architecture >> >> The creation of the boot image is done via the usual invocation of make (with a >> suitably set CROSS_COMPILE environment variable, of course). The resulting boot >> -image u-boot-spl.kwb can then be tested, if so desired. The hdrparser from [5] >> +image u-boot-with-spl.kwb can then be tested, if so desired. The hdrparser from [5] >> can be used for this purpose. To build the tool, invoke make in the >> 'tools/marvell/doimage_mv' directory of [5], which builds a stand-alone >> hdrparser executable. A test can be conducted by calling hdrparser with the >> produced boot image and the following (mandatory) parameters: >> >> -./hdrparser -k 0 -t u-boot-spl.kwb >> +./hdrparser -k 0 -t u-boot-with-spl.kwb >> >> Here we assume that the CSK index is 0 and the boot image file resides in the >> same directory (adapt accordingly if needed). The tool should report that all >> diff --git a/doc/kwboot.1 b/doc/kwboot.1 >> index 0863d197fd49..a528fbbe8c38 100644 >> --- a/doc/kwboot.1 >> +++ b/doc/kwboot.1 >> @@ -149,25 +149,25 @@ Tested values for \fIbaudrate\fP for Armada 38x include: 115200, >> >> .SH "EXAMPLES" >> >> -Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-spl.kwb\fP >> +Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-with-spl.kwb\fP >> kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP at 115200 Bd and run terminal >> program: >> .IP >> -.B kwboot -b u-boot-spl.kwb -t /dev/ttyUSB0 >> +.B kwboot -b u-boot-with-spl.kwb -t /dev/ttyUSB0 >> >> .PP >> Instruct BootROM to enter boot Xmodem boot mode, send header of >> -\fIu-boot-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct >> +\fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct >> BootROM to change baudrate to 5200000 Bd, send data part of the kwbimage >> file via Xmodem at high speed and finally run terminal program: >> .IP >> -.B kwboot -b u-boot-spl.kwb -B 5200000 -t /dev/ttyUSB0 >> +.B kwboot -b u-boot-with-spl.kwb -B 5200000 -t /dev/ttyUSB0 >> >> .PP >> -Only send \fIu-boot-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP >> +Only send \fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP >> at 115200 Bd: >> .IP >> -.B kwboot -D u-boot-spl.kwb /dev/ttyUSB0 >> +.B kwboot -D u-boot-with-spl.kwb /dev/ttyUSB0 >> >> .PP >> Instruct BootROM to enter console debug mode and run terminal program on >> diff --git a/include/configs/ds414.h b/include/configs/ds414.h >> index f8273a92f11a..76d1713fdc68 100644 >> --- a/include/configs/ds414.h >> +++ b/include/configs/ds414.h >> @@ -50,7 +50,7 @@ >> "ethmtu=1500\0eth1mtu=1500\0" \ >> "update_uboot=sf probe; dhcp; " \ >> "mw.b ${loadaddr} 0x0 0xd0000; " \ >> - "tftpboot ${loadaddr} u-boot-spl.kwb; " \ >> + "tftpboot ${loadaddr} u-boot-with-spl.kwb; " \ >> "sf update ${loadaddr} 0x0 0xd0000\0" >> >> >> -- >> 2.20.1 >> Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb 2022-11-06 21:21 ` Pali Rohár 2022-11-07 6:25 ` Stefan Roese @ 2022-11-07 8:24 ` Stefan Roese 1 sibling, 0 replies; 10+ messages in thread From: Stefan Roese @ 2022-11-07 8:24 UTC (permalink / raw) To: Pali Rohár; +Cc: u-boot On 06.11.22 22:21, Pali Rohár wrote: > Stefan, it is OK now? Applied to u-boot-marvell/master Thanks, Stefan > On Wednesday 02 November 2022 18:51:28 Pali Rohár wrote: >> File name with pattern u-boot-spl* is used on all places except in kwb >> image for binary with SPL-only code. Combined binary with both SPL and >> proper U-Boot in other places has file name pattern u-boot-with-spl*. >> >> Make it consistent also for kwb image and rename u-boot-spl.kwb to >> u-boot-with-spl.kwb as this image contains both SPL and proper U-Boot code. >> >> Also update documentation about file name changes. >> >> Signed-off-by: Pali Rohár <pali@kernel.org> >> Reviewed-by: Stefan Roese <sr@denx.de> >> --- >> Kconfig | 2 +- >> Makefile | 4 ++-- >> board/Marvell/db-88f6820-gp/README | 6 +++--- >> board/kobol/helios4/README | 6 +++--- >> board/solidrun/clearfog/README | 8 ++++---- >> doc/README.armada-secureboot | 4 ++-- >> doc/kwboot.1 | 12 ++++++------ >> include/configs/ds414.h | 2 +- >> 8 files changed, 22 insertions(+), 22 deletions(-) >> >> diff --git a/Kconfig b/Kconfig >> index ef94f88adc28..43d4a48ac808 100644 >> --- a/Kconfig >> +++ b/Kconfig >> @@ -456,7 +456,7 @@ config BUILD_TARGET >> string "Build target special images" >> default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_ARRIA10 >> default "u-boot-with-spl.sfp" if TARGET_SOCFPGA_GEN5 >> - default "u-boot-spl.kwb" if ARCH_MVEBU && SPL >> + default "u-boot-with-spl.kwb" if ARCH_MVEBU && SPL >> default "u-boot-elf.srec" if RCAR_GEN3 >> default "u-boot.itb" if !BINMAN && SPL_LOAD_FIT && (ARCH_ROCKCHIP || \ >> ARCH_SUNXI || RISCV || ARCH_ZYNQMP) >> diff --git a/Makefile b/Makefile >> index d5c2c9700921..eb0cc1ac060f 100644 >> --- a/Makefile >> +++ b/Makefile >> @@ -1422,7 +1422,7 @@ KWD_CONFIG_FILE = $(shell \ >> MKIMAGEFLAGS_u-boot.kwb = -n $(KWD_CONFIG_FILE) \ >> -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) >> >> -MKIMAGEFLAGS_u-boot-spl.kwb = -n $(KWD_CONFIG_FILE) \ >> +MKIMAGEFLAGS_u-boot-with-spl.kwb = -n $(KWD_CONFIG_FILE) \ >> -T kwbimage -a $(CONFIG_TEXT_BASE) -e $(CONFIG_TEXT_BASE) \ >> $(if $(KEYDIR),-k $(KEYDIR)) >> >> @@ -1463,7 +1463,7 @@ u-boot.itb: u-boot-nodtb.bin \ >> $(BOARD_SIZE_CHECK) >> endif >> >> -u-boot-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE >> +u-boot-with-spl.kwb: u-boot.bin spl/u-boot-spl.bin FORCE >> $(call if_changed,mkimage) >> $(BOARD_SIZE_CHECK) >> >> diff --git a/board/Marvell/db-88f6820-gp/README b/board/Marvell/db-88f6820-gp/README >> index 9bea5b35cb40..b4db6ed62577 100644 >> --- a/board/Marvell/db-88f6820-gp/README >> +++ b/board/Marvell/db-88f6820-gp/README >> @@ -2,17 +2,17 @@ Update from original Marvell U-Boot to mainline U-Boot: >> ------------------------------------------------------- >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> To update the SPI NOR flash, please use the following >> command: >> >> -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ >> +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ >> sf update 2000000 0 60000 >> >> Note that the original Marvell U-Boot seems to have >> problems with the "sf update" command. This does not >> work reliable. So here this command should be used: >> >> -=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-spl.kwb;\ >> +=> sf probe;tftpboot 2000000 db-88f6820-gp/u-boot-with-spl.kwb;\ >> sf erase 0 60000;sf write 2000000 0 60000 >> diff --git a/board/kobol/helios4/README b/board/kobol/helios4/README >> index 749f04827824..d615cdd2648b 100644 >> --- a/board/kobol/helios4/README >> +++ b/board/kobol/helios4/README >> @@ -7,12 +7,12 @@ $ make helios4_defconfig >> $ make >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> Now all you need to do is copy this image on a SD card. >> For example with this command: >> >> -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 >> +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 >> >> Please use the correct device node for your setup instead >> of "/dev/sdX" here! >> @@ -38,7 +38,7 @@ Set the SW1 DIP switches to UART boot (see above). >> >> Run the following command to initiate U-Boot download: >> >> - ./tools/kwboot -p -b u-boot-spl.kwb /dev/ttyUSBX >> + ./tools/kwboot -p -b u-boot-with-spl.kwb /dev/ttyUSBX >> >> Use the correct UART device node for /dev/ttyUSBX. >> >> diff --git a/board/solidrun/clearfog/README b/board/solidrun/clearfog/README >> index 9375be84957a..ed4a712c5aa2 100644 >> --- a/board/solidrun/clearfog/README >> +++ b/board/solidrun/clearfog/README >> @@ -7,12 +7,12 @@ $ make clearfog_defconfig >> $ make >> >> The resulting image including the SPL binary with the >> -full DDR setup is "u-boot-spl.kwb". >> +full DDR setup is "u-boot-with-spl.kwb". >> >> Now all you need to do is copy this image on a SD card. >> For example with this command: >> >> -$ sudo dd if=u-boot-spl.kwb of=/dev/sdX bs=512 seek=1 >> +$ sudo dd if=u-boot-with-spl.kwb of=/dev/sdX bs=512 seek=1 >> >> Please use the correct device node for your setup instead >> of "/dev/sdX" here! >> @@ -29,7 +29,7 @@ command: >> Install U-Boot on eMMC boot partition from Linux running on Clearfog: >> >> echo 0 > /sys/block/mmcblk0boot0/force_ro >> - dd if=u-boot-spl.kwb of=/dev/mmcblk0boot0 >> + dd if=u-boot-with-spl.kwb of=/dev/mmcblk0boot0 >> >> Note that the SD card is not accessible when the Clearfog SOM has eMMC. >> Consider initial boot from UART (see below). >> @@ -66,7 +66,7 @@ Set the SW1 DIP switches to UART boot (see above). >> >> Run the following command to initiate U-Boot download: >> >> - ./tools/kwboot -b u-boot-spl.kwb /dev/ttyUSBX >> + ./tools/kwboot -b u-boot-with-spl.kwb /dev/ttyUSBX >> >> Use the correct UART device node for /dev/ttyUSBX. >> >> diff --git a/doc/README.armada-secureboot b/doc/README.armada-secureboot >> index 157cb5a231a9..4ec89d5c061f 100644 >> --- a/doc/README.armada-secureboot >> +++ b/doc/README.armada-secureboot >> @@ -257,13 +257,13 @@ ARM architecture >> >> The creation of the boot image is done via the usual invocation of make (with a >> suitably set CROSS_COMPILE environment variable, of course). The resulting boot >> -image u-boot-spl.kwb can then be tested, if so desired. The hdrparser from [5] >> +image u-boot-with-spl.kwb can then be tested, if so desired. The hdrparser from [5] >> can be used for this purpose. To build the tool, invoke make in the >> 'tools/marvell/doimage_mv' directory of [5], which builds a stand-alone >> hdrparser executable. A test can be conducted by calling hdrparser with the >> produced boot image and the following (mandatory) parameters: >> >> -./hdrparser -k 0 -t u-boot-spl.kwb >> +./hdrparser -k 0 -t u-boot-with-spl.kwb >> >> Here we assume that the CSK index is 0 and the boot image file resides in the >> same directory (adapt accordingly if needed). The tool should report that all >> diff --git a/doc/kwboot.1 b/doc/kwboot.1 >> index 0863d197fd49..a528fbbe8c38 100644 >> --- a/doc/kwboot.1 >> +++ b/doc/kwboot.1 >> @@ -149,25 +149,25 @@ Tested values for \fIbaudrate\fP for Armada 38x include: 115200, >> >> .SH "EXAMPLES" >> >> -Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-spl.kwb\fP >> +Instruct BootROM to enter boot Xmodem boot mode, send \fIu-boot-with-spl.kwb\fP >> kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP at 115200 Bd and run terminal >> program: >> .IP >> -.B kwboot -b u-boot-spl.kwb -t /dev/ttyUSB0 >> +.B kwboot -b u-boot-with-spl.kwb -t /dev/ttyUSB0 >> >> .PP >> Instruct BootROM to enter boot Xmodem boot mode, send header of >> -\fIu-boot-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct >> +\fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem at 115200 Bd, then instruct >> BootROM to change baudrate to 5200000 Bd, send data part of the kwbimage >> file via Xmodem at high speed and finally run terminal program: >> .IP >> -.B kwboot -b u-boot-spl.kwb -B 5200000 -t /dev/ttyUSB0 >> +.B kwboot -b u-boot-with-spl.kwb -B 5200000 -t /dev/ttyUSB0 >> >> .PP >> -Only send \fIu-boot-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP >> +Only send \fIu-boot-with-spl.kwb\fP kwbimage file via Xmodem on \fI/dev/ttyUSB0\fP >> at 115200 Bd: >> .IP >> -.B kwboot -D u-boot-spl.kwb /dev/ttyUSB0 >> +.B kwboot -D u-boot-with-spl.kwb /dev/ttyUSB0 >> >> .PP >> Instruct BootROM to enter console debug mode and run terminal program on >> diff --git a/include/configs/ds414.h b/include/configs/ds414.h >> index f8273a92f11a..76d1713fdc68 100644 >> --- a/include/configs/ds414.h >> +++ b/include/configs/ds414.h >> @@ -50,7 +50,7 @@ >> "ethmtu=1500\0eth1mtu=1500\0" \ >> "update_uboot=sf probe; dhcp; " \ >> "mw.b ${loadaddr} 0x0 0xd0000; " \ >> - "tftpboot ${loadaddr} u-boot-spl.kwb; " \ >> + "tftpboot ${loadaddr} u-boot-with-spl.kwb; " \ >> "sf update ${loadaddr} 0x0 0xd0000\0" >> >> >> -- >> 2.20.1 >> Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-11-07 8:24 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-08-17 9:59 [PATCH RFC] Makefile: Rename u-boot-spl.kwb to u-boot-with-spl.kwb Pali Rohár 2022-08-17 10:05 ` Stefan Roese 2022-08-17 21:00 ` Pali Rohár 2022-09-07 9:24 ` Pali Rohár 2022-11-01 22:39 ` Pali Rohár 2022-11-02 8:55 ` Stefan Roese 2022-11-02 17:51 ` [PATCH] " Pali Rohár 2022-11-06 21:21 ` Pali Rohár 2022-11-07 6:25 ` Stefan Roese 2022-11-07 8:24 ` Stefan Roese
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox