* [RFT][PATCH v1 0/2] Fix CONFIG_SYS_UBOOT_START default value
@ 2019-12-06 14:01 Patrice Chotard
2019-12-06 14:01 ` [PATCH v1 1/2] Makefile: " Patrice Chotard
2019-12-06 14:01 ` [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files Patrice Chotard
0 siblings, 2 replies; 7+ messages in thread
From: Patrice Chotard @ 2019-12-06 14:01 UTC (permalink / raw)
To: u-boot
This series fixes a regression introduced by
d3e97b53c1f2 ("spl: fix entry_point equal to load_addr")
which breaks some board boot (at least STM32MP15 in basic configuration)
If CONFIG_SYS_UBOOT_START is not set, it's set to CONFIG_SYS_TEXT_BASE
by default.
This allows also to clean include/configs board files in which
CONFIG_SYS_UBOOT_START is already set to CONFIG_SYS_TEXT_BASE.
Patrice Chotard (2):
Makefile: Fix CONFIG_SYS_UBOOT_START default value
treewide: Remove CONFIG_SYS_UBOOT_START from configs board files
Makefile | 2 +-
include/configs/gardena-smart-gateway-at91sam.h | 1 -
include/configs/ls1046a_common.h | 1 -
include/configs/mccmon6.h | 1 -
include/configs/microblaze-generic.h | 2 --
include/configs/mt7623.h | 1 -
include/configs/mt7629.h | 1 -
include/configs/mt8518.h | 1 -
include/configs/omap3_cairo.h | 1 -
include/configs/pumpkin.h | 1 -
include/configs/tegra-common.h | 1 -
include/configs/x600.h | 1 -
include/configs/xilinx_zynqmp_r5.h | 2 --
include/configs/zynq-common.h | 2 --
14 files changed, 1 insertion(+), 17 deletions(-)
--
2.17.1
^ permalink raw reply [flat|nested] 7+ messages in thread* [PATCH v1 1/2] Makefile: Fix CONFIG_SYS_UBOOT_START default value 2019-12-06 14:01 [RFT][PATCH v1 0/2] Fix CONFIG_SYS_UBOOT_START default value Patrice Chotard @ 2019-12-06 14:01 ` Patrice Chotard 2019-12-30 1:21 ` Simon Glass 2020-01-08 20:12 ` Tom Rini 2019-12-06 14:01 ` [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files Patrice Chotard 1 sibling, 2 replies; 7+ messages in thread From: Patrice Chotard @ 2019-12-06 14:01 UTC (permalink / raw) To: u-boot This patches restores boot on boards which rely on CONFIG_SYS_UBOOT_START equal to CONFIG_SYS_TEXT_BASE when using SPL Fixes: d3e97b53c1f2 ("spl: fix entry_point equal to load_addr") Signed-off-by: Patrice Chotard <patrice.chotard@st.com> --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index 0ee0c70102..d679459305 100644 --- a/Makefile +++ b/Makefile @@ -1265,7 +1265,7 @@ u-boot.ldr.hex u-boot.ldr.srec: u-boot.ldr FORCE # from the SPL U-Boot version. # ifndef CONFIG_SYS_UBOOT_START -CONFIG_SYS_UBOOT_START := 0 +CONFIG_SYS_UBOOT_START := $(CONFIG_SYS_TEXT_BASE) endif # Boards with more complex image requirements can provide an .its source file -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v1 1/2] Makefile: Fix CONFIG_SYS_UBOOT_START default value 2019-12-06 14:01 ` [PATCH v1 1/2] Makefile: " Patrice Chotard @ 2019-12-30 1:21 ` Simon Glass 2020-01-08 20:12 ` Tom Rini 1 sibling, 0 replies; 7+ messages in thread From: Simon Glass @ 2019-12-30 1:21 UTC (permalink / raw) To: u-boot On Fri, 6 Dec 2019 at 07:01, Patrice Chotard <patrice.chotard@st.com> wrote: > > This patches restores boot on boards which rely on > CONFIG_SYS_UBOOT_START equal to CONFIG_SYS_TEXT_BASE when using SPL > > Fixes: d3e97b53c1f2 ("spl: fix entry_point equal to load_addr") > > Signed-off-by: Patrice Chotard <patrice.chotard@st.com> > --- > > Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Simon Glass <sjg@chromium.org> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v1 1/2] Makefile: Fix CONFIG_SYS_UBOOT_START default value 2019-12-06 14:01 ` [PATCH v1 1/2] Makefile: " Patrice Chotard 2019-12-30 1:21 ` Simon Glass @ 2020-01-08 20:12 ` Tom Rini 1 sibling, 0 replies; 7+ messages in thread From: Tom Rini @ 2020-01-08 20:12 UTC (permalink / raw) To: u-boot On Fri, Dec 06, 2019 at 03:01:48PM +0100, Patrice Chotard wrote: > This patches restores boot on boards which rely on > CONFIG_SYS_UBOOT_START equal to CONFIG_SYS_TEXT_BASE when using SPL > > Fixes: d3e97b53c1f2 ("spl: fix entry_point equal to load_addr") > > Signed-off-by: Patrice Chotard <patrice.chotard@st.com> > Reviewed-by: Simon Glass <sjg@chromium.org> Applied to u-boot/master, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200108/ecba36af/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files 2019-12-06 14:01 [RFT][PATCH v1 0/2] Fix CONFIG_SYS_UBOOT_START default value Patrice Chotard 2019-12-06 14:01 ` [PATCH v1 1/2] Makefile: " Patrice Chotard @ 2019-12-06 14:01 ` Patrice Chotard 2019-12-08 14:24 ` Lukasz Majewski 2020-01-08 20:13 ` Tom Rini 1 sibling, 2 replies; 7+ messages in thread From: Patrice Chotard @ 2019-12-06 14:01 UTC (permalink / raw) To: u-boot As previous CONFIG_SYS_UBOOT_START is now set by default to CONFIG_SYS_TEXT_BASE when not defined, CONFIG_SYS_UBOOT_START can be removed from include/configs board files. Signed-off-by: Patrice Chotard <patrice.chotard@st.com> --- include/configs/gardena-smart-gateway-at91sam.h | 1 - include/configs/ls1046a_common.h | 1 - include/configs/mccmon6.h | 1 - include/configs/microblaze-generic.h | 2 -- include/configs/mt7623.h | 1 - include/configs/mt7629.h | 1 - include/configs/mt8518.h | 1 - include/configs/omap3_cairo.h | 1 - include/configs/pumpkin.h | 1 - include/configs/tegra-common.h | 1 - include/configs/x600.h | 1 - include/configs/xilinx_zynqmp_r5.h | 2 -- include/configs/zynq-common.h | 2 -- 13 files changed, 16 deletions(-) diff --git a/include/configs/gardena-smart-gateway-at91sam.h b/include/configs/gardena-smart-gateway-at91sam.h index 482e4714b1..f5ee65cb8a 100644 --- a/include/configs/gardena-smart-gateway-at91sam.h +++ b/include/configs/gardena-smart-gateway-at91sam.h @@ -64,7 +64,6 @@ #define CONFIG_SPL_NAND_RAW_ONLY #define CONFIG_SYS_NAND_U_BOOT_OFFS 0x40000 #define CONFIG_SYS_NAND_U_BOOT_SIZE 0xa0000 -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_NAND_U_BOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_NAND_U_BOOT_DST CONFIG_SYS_TEXT_BASE diff --git a/include/configs/ls1046a_common.h b/include/configs/ls1046a_common.h index cc8f4c0210..6543cfd868 100644 --- a/include/configs/ls1046a_common.h +++ b/include/configs/ls1046a_common.h @@ -98,7 +98,6 @@ CONFIG_SPL_BSS_MAX_SIZE) #define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 #define CONFIG_SYS_MONITOR_LEN 0x100000 -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #endif /* NAND SPL */ diff --git a/include/configs/mccmon6.h b/include/configs/mccmon6.h index 045a9f7bdf..0aee1e1cf6 100644 --- a/include/configs/mccmon6.h +++ b/include/configs/mccmon6.h @@ -12,7 +12,6 @@ #define CONFIG_SPL_LIBCOMMON_SUPPORT #include "imx6_spl.h" -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_UBOOT_BASE (CONFIG_SYS_FLASH_BASE + 0x80000) #define CONFIG_SYS_SPL_ARGS_ADDR 0x18000000 diff --git a/include/configs/microblaze-generic.h b/include/configs/microblaze-generic.h index 385b30c99b..8ca0e83c78 100644 --- a/include/configs/microblaze-generic.h +++ b/include/configs/microblaze-generic.h @@ -173,8 +173,6 @@ /* Just for sure that there is a space for stack */ #define CONFIG_SPL_STACK_SIZE 0x100 -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE - #define CONFIG_SPL_MAX_FOOTPRINT (CONFIG_SYS_INIT_RAM_SIZE - \ CONFIG_SYS_INIT_RAM_ADDR - \ CONFIG_SYS_MALLOC_F_LEN - \ diff --git a/include/configs/mt7623.h b/include/configs/mt7623.h index e5182aeea8..faab0913fc 100644 --- a/include/configs/mt7623.h +++ b/include/configs/mt7623.h @@ -31,7 +31,6 @@ #define CONFIG_ENV_OVERWRITE /* Preloader -> Uboot */ -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE + SZ_2M - \ GENERATED_GBL_DATA_SIZE) diff --git a/include/configs/mt7629.h b/include/configs/mt7629.h index 4aef894c6e..6a6c2f2414 100644 --- a/include/configs/mt7629.h +++ b/include/configs/mt7629.h @@ -40,7 +40,6 @@ #define CONFIG_SYS_UBOOT_BASE (CONFIG_SPI_ADDR + CONFIG_SPL_PAD_TO) /* SPL -> Uboot */ -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE + SZ_2M - \ GENERATED_GBL_DATA_SIZE) diff --git a/include/configs/mt8518.h b/include/configs/mt8518.h index a7fe83a605..514722be99 100644 --- a/include/configs/mt8518.h +++ b/include/configs/mt8518.h @@ -29,7 +29,6 @@ #define CONFIG_SYS_BOOTM_LEN SZ_64M /* Uboot definition */ -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE + \ SZ_2M - \ GENERATED_GBL_DATA_SIZE) diff --git a/include/configs/omap3_cairo.h b/include/configs/omap3_cairo.h index 1b1a56d7cd..c76c81ddd5 100644 --- a/include/configs/omap3_cairo.h +++ b/include/configs/omap3_cairo.h @@ -26,7 +26,6 @@ * other needs. We use this rather than the inherited defines from * ti_armv7_common.h for backwards compatibility. */ -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SPL_BSS_START_ADDR 0x80000000 #define CONFIG_SPL_BSS_MAX_SIZE (512 << 10) /* 512 KB */ #define CONFIG_SYS_SPL_MALLOC_START 0x80208000 diff --git a/include/configs/pumpkin.h b/include/configs/pumpkin.h index 35e28be950..9c52cae41d 100644 --- a/include/configs/pumpkin.h +++ b/include/configs/pumpkin.h @@ -23,7 +23,6 @@ #define CONFIG_SYS_NS16550_COM1 0x11005000 #define CONFIG_SYS_NS16550_CLK 26000000 -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE + SZ_2M - \ GENERATED_GBL_DATA_SIZE) diff --git a/include/configs/tegra-common.h b/include/configs/tegra-common.h index b4da1f8428..f2cdd9c019 100644 --- a/include/configs/tegra-common.h +++ b/include/configs/tegra-common.h @@ -61,7 +61,6 @@ #define PHYS_SDRAM_1 NV_PA_SDRC_CS0 #define PHYS_SDRAM_1_SIZE 0x20000000 /* 512M */ -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 #define CONFIG_SYS_BOOTMAPSZ (256 << 20) /* 256M */ diff --git a/include/configs/x600.h b/include/configs/x600.h index 63092b24a5..8b6caae7be 100644 --- a/include/configs/x600.h +++ b/include/configs/x600.h @@ -27,7 +27,6 @@ #define CONFIG_SYS_SPL_LEN CONFIG_SPL_PAD_TO #define CONFIG_SYS_UBOOT_BASE (CONFIG_SYS_FLASH_BASE + \ CONFIG_SYS_SPL_LEN) -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_MONITOR_BASE CONFIG_SYS_FLASH_BASE #define CONFIG_SYS_MONITOR_LEN 0x60000 diff --git a/include/configs/xilinx_zynqmp_r5.h b/include/configs/xilinx_zynqmp_r5.h index 38d952d0c0..155d7fe883 100644 --- a/include/configs/xilinx_zynqmp_r5.h +++ b/include/configs/xilinx_zynqmp_r5.h @@ -35,8 +35,6 @@ /* Extend size of kernel image for uncompression */ #define CONFIG_SYS_BOOTM_LEN (60 * 1024 * 1024) -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE - #define CONFIG_SKIP_LOWLEVEL_INIT /* 0x0 - 0x40 is used for placing exception vectors */ diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h index 274cc19112..189ca81bbe 100644 --- a/include/configs/zynq-common.h +++ b/include/configs/zynq-common.h @@ -274,6 +274,4 @@ #define CONFIG_SPL_LOAD_FIT_ADDRESS 0x10000000 -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE - #endif /* __CONFIG_ZYNQ_COMMON_H */ -- 2.17.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files 2019-12-06 14:01 ` [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files Patrice Chotard @ 2019-12-08 14:24 ` Lukasz Majewski 2020-01-08 20:13 ` Tom Rini 1 sibling, 0 replies; 7+ messages in thread From: Lukasz Majewski @ 2019-12-08 14:24 UTC (permalink / raw) To: u-boot On Fri, 6 Dec 2019 15:01:49 +0100 Patrice Chotard <patrice.chotard@st.com> wrote: > As previous CONFIG_SYS_UBOOT_START is now set by default to > CONFIG_SYS_TEXT_BASE when not defined, CONFIG_SYS_UBOOT_START > can be removed from include/configs board files. > > Signed-off-by: Patrice Chotard <patrice.chotard@st.com> > > --- > > include/configs/gardena-smart-gateway-at91sam.h | 1 - > include/configs/ls1046a_common.h | 1 - > include/configs/mccmon6.h | 1 - > include/configs/microblaze-generic.h | 2 -- > include/configs/mt7623.h | 1 - > include/configs/mt7629.h | 1 - > include/configs/mt8518.h | 1 - > include/configs/omap3_cairo.h | 1 - > include/configs/pumpkin.h | 1 - > include/configs/tegra-common.h | 1 - > include/configs/x600.h | 1 - > include/configs/xilinx_zynqmp_r5.h | 2 -- > include/configs/zynq-common.h | 2 -- > 13 files changed, 16 deletions(-) > > diff --git a/include/configs/gardena-smart-gateway-at91sam.h > b/include/configs/gardena-smart-gateway-at91sam.h index > 482e4714b1..f5ee65cb8a 100644 --- > a/include/configs/gardena-smart-gateway-at91sam.h +++ > b/include/configs/gardena-smart-gateway-at91sam.h @@ -64,7 +64,6 @@ > #define CONFIG_SPL_NAND_RAW_ONLY > #define CONFIG_SYS_NAND_U_BOOT_OFFS 0x40000 > #define CONFIG_SYS_NAND_U_BOOT_SIZE 0xa0000 > -#define CONFIG_SYS_UBOOT_START > CONFIG_SYS_TEXT_BASE #define > CONFIG_SYS_NAND_U_BOOT_START CONFIG_SYS_TEXT_BASE #define > CONFIG_SYS_NAND_U_BOOT_DST CONFIG_SYS_TEXT_BASE > diff --git a/include/configs/ls1046a_common.h > b/include/configs/ls1046a_common.h index cc8f4c0210..6543cfd868 100644 > --- a/include/configs/ls1046a_common.h > +++ b/include/configs/ls1046a_common.h > @@ -98,7 +98,6 @@ > CONFIG_SPL_BSS_MAX_SIZE) > #define CONFIG_SYS_SPL_MALLOC_SIZE 0x100000 > #define CONFIG_SYS_MONITOR_LEN 0x100000 > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #endif > > /* NAND SPL */ > diff --git a/include/configs/mccmon6.h b/include/configs/mccmon6.h > index 045a9f7bdf..0aee1e1cf6 100644 > --- a/include/configs/mccmon6.h > +++ b/include/configs/mccmon6.h > @@ -12,7 +12,6 @@ > #define CONFIG_SPL_LIBCOMMON_SUPPORT > #include "imx6_spl.h" > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SYS_UBOOT_BASE (CONFIG_SYS_FLASH_BASE + 0x80000) > #define CONFIG_SYS_SPL_ARGS_ADDR 0x18000000 > > diff --git a/include/configs/microblaze-generic.h > b/include/configs/microblaze-generic.h index 385b30c99b..8ca0e83c78 > 100644 --- a/include/configs/microblaze-generic.h > +++ b/include/configs/microblaze-generic.h > @@ -173,8 +173,6 @@ > /* Just for sure that there is a space for stack */ > #define CONFIG_SPL_STACK_SIZE 0x100 > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > - > #define CONFIG_SPL_MAX_FOOTPRINT (CONFIG_SYS_INIT_RAM_SIZE - \ > CONFIG_SYS_INIT_RAM_ADDR - \ > CONFIG_SYS_MALLOC_F_LEN - \ > diff --git a/include/configs/mt7623.h b/include/configs/mt7623.h > index e5182aeea8..faab0913fc 100644 > --- a/include/configs/mt7623.h > +++ b/include/configs/mt7623.h > @@ -31,7 +31,6 @@ > #define CONFIG_ENV_OVERWRITE > > /* Preloader -> Uboot */ > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE > + SZ_2M - \ GENERATED_GBL_DATA_SIZE) > > diff --git a/include/configs/mt7629.h b/include/configs/mt7629.h > index 4aef894c6e..6a6c2f2414 100644 > --- a/include/configs/mt7629.h > +++ b/include/configs/mt7629.h > @@ -40,7 +40,6 @@ > #define CONFIG_SYS_UBOOT_BASE (CONFIG_SPI_ADDR + > CONFIG_SPL_PAD_TO) > /* SPL -> Uboot */ > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE > + SZ_2M - \ GENERATED_GBL_DATA_SIZE) > > diff --git a/include/configs/mt8518.h b/include/configs/mt8518.h > index a7fe83a605..514722be99 100644 > --- a/include/configs/mt8518.h > +++ b/include/configs/mt8518.h > @@ -29,7 +29,6 @@ > #define CONFIG_SYS_BOOTM_LEN SZ_64M > > /* Uboot definition */ > -#define CONFIG_SYS_UBOOT_START > CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_INIT_SP_ADDR > (CONFIG_SYS_TEXT_BASE + \ SZ_2M - \ > GENERATED_GBL_DATA_SIZE) > diff --git a/include/configs/omap3_cairo.h > b/include/configs/omap3_cairo.h index 1b1a56d7cd..c76c81ddd5 100644 > --- a/include/configs/omap3_cairo.h > +++ b/include/configs/omap3_cairo.h > @@ -26,7 +26,6 @@ > * other needs. We use this rather than the inherited defines from > * ti_armv7_common.h for backwards compatibility. > */ > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SPL_BSS_START_ADDR 0x80000000 > #define CONFIG_SPL_BSS_MAX_SIZE (512 << 10) /* > 512 KB */ #define CONFIG_SYS_SPL_MALLOC_START 0x80208000 > diff --git a/include/configs/pumpkin.h b/include/configs/pumpkin.h > index 35e28be950..9c52cae41d 100644 > --- a/include/configs/pumpkin.h > +++ b/include/configs/pumpkin.h > @@ -23,7 +23,6 @@ > #define CONFIG_SYS_NS16550_COM1 0x11005000 > #define CONFIG_SYS_NS16550_CLK 26000000 > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SYS_INIT_SP_ADDR (CONFIG_SYS_TEXT_BASE > + SZ_2M - \ GENERATED_GBL_DATA_SIZE) > > diff --git a/include/configs/tegra-common.h > b/include/configs/tegra-common.h index b4da1f8428..f2cdd9c019 100644 > --- a/include/configs/tegra-common.h > +++ b/include/configs/tegra-common.h > @@ -61,7 +61,6 @@ > #define PHYS_SDRAM_1 NV_PA_SDRC_CS0 > #define PHYS_SDRAM_1_SIZE 0x20000000 /* 512M */ > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > #define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 > > #define CONFIG_SYS_BOOTMAPSZ (256 << 20) /* 256M */ > diff --git a/include/configs/x600.h b/include/configs/x600.h > index 63092b24a5..8b6caae7be 100644 > --- a/include/configs/x600.h > +++ b/include/configs/x600.h > @@ -27,7 +27,6 @@ > #define CONFIG_SYS_SPL_LEN CONFIG_SPL_PAD_TO > #define CONFIG_SYS_UBOOT_BASE > (CONFIG_SYS_FLASH_BASE + \ CONFIG_SYS_SPL_LEN) > -#define CONFIG_SYS_UBOOT_START > CONFIG_SYS_TEXT_BASE #define CONFIG_SYS_MONITOR_BASE > CONFIG_SYS_FLASH_BASE #define CONFIG_SYS_MONITOR_LEN > 0x60000 > diff --git a/include/configs/xilinx_zynqmp_r5.h > b/include/configs/xilinx_zynqmp_r5.h index 38d952d0c0..155d7fe883 > 100644 --- a/include/configs/xilinx_zynqmp_r5.h > +++ b/include/configs/xilinx_zynqmp_r5.h > @@ -35,8 +35,6 @@ > /* Extend size of kernel image for uncompression */ > #define CONFIG_SYS_BOOTM_LEN (60 * 1024 * 1024) > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > - > #define CONFIG_SKIP_LOWLEVEL_INIT > > /* 0x0 - 0x40 is used for placing exception vectors */ > diff --git a/include/configs/zynq-common.h > b/include/configs/zynq-common.h index 274cc19112..189ca81bbe 100644 > --- a/include/configs/zynq-common.h > +++ b/include/configs/zynq-common.h > @@ -274,6 +274,4 @@ > > #define CONFIG_SPL_LOAD_FIT_ADDRESS 0x10000000 > > -#define CONFIG_SYS_UBOOT_START CONFIG_SYS_TEXT_BASE > - > #endif /* __CONFIG_ZYNQ_COMMON_H */ Acked-by: Lukasz Majewski <lukma@denx.de> Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lukma at denx.de -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 488 bytes Desc: OpenPGP digital signature URL: <https://lists.denx.de/pipermail/u-boot/attachments/20191208/af449129/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files 2019-12-06 14:01 ` [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files Patrice Chotard 2019-12-08 14:24 ` Lukasz Majewski @ 2020-01-08 20:13 ` Tom Rini 1 sibling, 0 replies; 7+ messages in thread From: Tom Rini @ 2020-01-08 20:13 UTC (permalink / raw) To: u-boot On Fri, Dec 06, 2019 at 03:01:49PM +0100, Patrice Chotard wrote: > As previous CONFIG_SYS_UBOOT_START is now set by default to > CONFIG_SYS_TEXT_BASE when not defined, CONFIG_SYS_UBOOT_START > can be removed from include/configs board files. > > Signed-off-by: Patrice Chotard <patrice.chotard@st.com> > Acked-by: Lukasz Majewski <lukma@denx.de> Applied to u-boot/master, thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: not available URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200108/38232a0b/attachment.sig> ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-01-08 20:13 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-12-06 14:01 [RFT][PATCH v1 0/2] Fix CONFIG_SYS_UBOOT_START default value Patrice Chotard 2019-12-06 14:01 ` [PATCH v1 1/2] Makefile: " Patrice Chotard 2019-12-30 1:21 ` Simon Glass 2020-01-08 20:12 ` Tom Rini 2019-12-06 14:01 ` [PATCH v1 2/2] treewide: Remove CONFIG_SYS_UBOOT_START from configs board files Patrice Chotard 2019-12-08 14:24 ` Lukasz Majewski 2020-01-08 20:13 ` Tom Rini
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox