Re: [PATCH] stm32mp1: configs: Make all boot devices in distro boot conditional
On 1/9/20 5:56 PM, Patrick DELAUNAY wrote: > Hi Marek, > >> From: Marek Vasut >> Sent: jeudi 9 janvier 2020 15:27 >> >> On 1/9/20 2:50 PM, Patrick DELAUNAY wrote: >>> Hi Marek, >> >> Hi, >> From: Marek Vasut Sent: mercredi 18 décembre 2019 07:58 To: u-boot@lists.denx.de Cc: Marek Vasut ; Patrick DELAUNAY ; Patrice CHOTARD Subject: [PATCH] stm32mp1: configs: Make all boot devices in distro boot conditional Importance: High Not all systems have all the boot devices enabled, e.g. not all systems have MTD devices and thus do not enable UBI. Make all the boot devices in the distro bootcmd conditional to avoid failures. Signed-off-by: Marek Vasut Cc: Patrick Delaunay Cc: Patrice Chotard >>> >>> Reviewed-by: Patrick Delaunay >>> >>> With a minor remark below. >>> --- include/configs/stm32mp1.h | 34 -- 1 file changed, 28 insertions(+), 6 deletions(-) diff --git a/include/configs/stm32mp1.h b/include/configs/stm32mp1.h index d42a7860be..dfc397c63c 100644 --- a/include/configs/stm32mp1.h +++ b/include/configs/stm32mp1.h @@ -98,12 +98,34 @@ #if !defined(CONFIG_SPL_BUILD) -#define BOOT_TARGET_DEVICES(func) \ - func(MMC, mmc, 1) \ - func(UBIFS, ubifs, 0) \ - func(MMC, mmc, 0) \ - func(MMC, mmc, 2) \ - func(PXE, pxe, na) +#ifdef CONFIG_CMD_MMC +#define BOOT_TARGET_DEVICE_MMC0(func) func(MMC, mmc, 0) +#define BOOT_TARGET_DEVICE_MMC1(func) func(MMC, mmc, 1) +#define BOOT_TARGET_DEVICE_MMC2(func) func(MMC, mmc, 2) +#else +#define BOOT_TARGET_DEVICE_MMC0(func) #define +BOOT_TARGET_DEVICE_MMC1(func) #define >> BOOT_TARGET_DEVICE_MMC2(func) +#endif + +#ifdef CONFIG_NET +#define BOOT_TARGET_DEVICE_PXE(func) func(PXE, pxe, na) +#else +#define BOOT_TARGET_DEVICE_PXE(func) #endif + +#ifdef CONFIG_CMD_UBIFS +#define BOOT_TARGET_DEVICE_UBIFS(func)func(UBIFS, ubifs, 0) +#else +#define BOOT_TARGET_DEVICE_UBIFS(func) #endif + +#define BOOT_TARGET_DEVICES(func) \ + BOOT_TARGET_DEVICE_MMC1(func) \ + BOOT_TARGET_DEVICE_UBIFS(func) \ + BOOT_TARGET_DEVICE_MMC0(func) \ + BOOT_TARGET_DEVICE_MMC2(func) \ + BOOT_TARGET_DEVICE_PXE(func) >>> >>> For other board, the same defines are named >> BOOT_TARGET_DEVICES_ >>> (with a S at the end of DEVICES) >>> >>> ./include/configs/sunxi-common.h:404:#define BOOT_TARGET_DEVICES(func) >> \ >>> ./include/configs/sunxi-common.h:406: >> BOOT_TARGET_DEVICES_MMC(func) \ >>> ./include/configs/sunxi-common.h:407: >> BOOT_TARGET_DEVICES_SCSI(func) \ >>> ./include/configs/sunxi-common.h:408: >> BOOT_TARGET_DEVICES_USB(func) \ >>> ./include/configs/sunxi-common.h:409: >> BOOT_TARGET_DEVICES_PXE(func) \ >>> ./include/configs/sunxi-common.h:410: >> BOOT_TARGET_DEVICES_DHCP(func) >>> >>> Except for ./include/configs/uniphier.h >>> >>> Any reason to change the name for stm32mp1 ? >>> >>> Othe exception for rockchip >>> >>> ./include/configs/rockchip-common.h:41:#define >> BOOT_TARGET_DEVICES(func) \ >>> ./include/configs/rockchip-common.h:42: BOOT_TARGET_MMC(func) \ >>> ./include/configs/rockchip-common.h:43: BOOT_TARGET_USB(func) \ >>> ./include/configs/rockchip-common.h:44: BOOT_TARGET_PXE(func) \ >>> ./include/configs/rockchip-common.h:45: BOOT_TARGET_DHCP(func) >>> >>> If I can choose I prefer the rockchip name (with shorter define name) >> >> We can rename the other boards' macros in a separate patch too, so it's the >> same. > > Yes > > For stm32mp1, you send V2 patch with BOOT_TARGET_XXX name > or you plan to align other board to BOOT_TARGET_DEVICES_XXX ? Done
RE: [PATCH] stm32mp1: configs: Make all boot devices in distro boot conditional
Hi Marek, > From: Marek Vasut > Sent: jeudi 9 janvier 2020 15:27 > > On 1/9/20 2:50 PM, Patrick DELAUNAY wrote: > > Hi Marek, > > Hi, > > >> From: Marek Vasut > >> Sent: mercredi 18 décembre 2019 07:58 > >> To: u-boot@lists.denx.de > >> Cc: Marek Vasut ; Patrick DELAUNAY > >> ; Patrice CHOTARD > >> Subject: [PATCH] stm32mp1: configs: Make all boot devices in distro > >> boot conditional > >> Importance: High > >> > >> Not all systems have all the boot devices enabled, e.g. not all > >> systems have MTD devices and thus do not enable UBI. Make all the > >> boot devices in the distro bootcmd conditional to avoid failures. > >> > >> Signed-off-by: Marek Vasut > >> Cc: Patrick Delaunay > >> Cc: Patrice Chotard > > > > Reviewed-by: Patrick Delaunay > > > > With a minor remark below. > > > >> --- > >> include/configs/stm32mp1.h | 34 -- > >> 1 file changed, 28 insertions(+), 6 deletions(-) > >> > >> diff --git a/include/configs/stm32mp1.h b/include/configs/stm32mp1.h > >> index d42a7860be..dfc397c63c 100644 > >> --- a/include/configs/stm32mp1.h > >> +++ b/include/configs/stm32mp1.h > >> @@ -98,12 +98,34 @@ > >> > >> #if !defined(CONFIG_SPL_BUILD) > >> > >> -#define BOOT_TARGET_DEVICES(func) \ > >> - func(MMC, mmc, 1) \ > >> - func(UBIFS, ubifs, 0) \ > >> - func(MMC, mmc, 0) \ > >> - func(MMC, mmc, 2) \ > >> - func(PXE, pxe, na) > >> +#ifdef CONFIG_CMD_MMC > >> +#define BOOT_TARGET_DEVICE_MMC0(func) func(MMC, mmc, 0) > >> +#define BOOT_TARGET_DEVICE_MMC1(func) func(MMC, mmc, 1) > >> +#define BOOT_TARGET_DEVICE_MMC2(func) func(MMC, mmc, 2) > >> +#else > >> +#define BOOT_TARGET_DEVICE_MMC0(func) #define > >> +BOOT_TARGET_DEVICE_MMC1(func) #define > BOOT_TARGET_DEVICE_MMC2(func) > >> +#endif > >> + > >> +#ifdef CONFIG_NET > >> +#define BOOT_TARGET_DEVICE_PXE(func) func(PXE, pxe, na) > >> +#else > >> +#define BOOT_TARGET_DEVICE_PXE(func) #endif > >> + > >> +#ifdef CONFIG_CMD_UBIFS > >> +#define BOOT_TARGET_DEVICE_UBIFS(func)func(UBIFS, ubifs, 0) > >> +#else > >> +#define BOOT_TARGET_DEVICE_UBIFS(func) #endif > >> + > >> +#define BOOT_TARGET_DEVICES(func) \ > >> + BOOT_TARGET_DEVICE_MMC1(func) \ > >> + BOOT_TARGET_DEVICE_UBIFS(func) \ > >> + BOOT_TARGET_DEVICE_MMC0(func) \ > >> + BOOT_TARGET_DEVICE_MMC2(func) \ > >> + BOOT_TARGET_DEVICE_PXE(func) > > > > For other board, the same defines are named > BOOT_TARGET_DEVICES_ > > (with a S at the end of DEVICES) > > > > ./include/configs/sunxi-common.h:404:#define BOOT_TARGET_DEVICES(func) > \ > > ./include/configs/sunxi-common.h:406: > BOOT_TARGET_DEVICES_MMC(func) \ > > ./include/configs/sunxi-common.h:407: > BOOT_TARGET_DEVICES_SCSI(func) \ > > ./include/configs/sunxi-common.h:408: > BOOT_TARGET_DEVICES_USB(func) \ > > ./include/configs/sunxi-common.h:409: > BOOT_TARGET_DEVICES_PXE(func) \ > > ./include/configs/sunxi-common.h:410: > BOOT_TARGET_DEVICES_DHCP(func) > > > > Except for ./include/configs/uniphier.h > > > > Any reason to change the name for stm32mp1 ? > > > > Othe exception for rockchip > > > > ./include/configs/rockchip-common.h:41:#define > BOOT_TARGET_DEVICES(func) \ > > ./include/configs/rockchip-common.h:42: BOOT_TARGET_MMC(func) \ > > ./include/configs/rockchip-common.h:43: BOOT_TARGET_USB(func) \ > > ./include/configs/rockchip-common.h:44: BOOT_TARGET_PXE(func) \ > > ./include/configs/rockchip-common.h:45: BOOT_TARGET_DHCP(func) > > > > If I can choose I prefer the rockchip name (with shorter define name) > > We can rename the other boards' macros in a separate patch too, so it's the > same. Yes For stm32mp1, you send V2 patch with BOOT_TARGET_XXX name or you plan to align other board to BOOT_TARGET_DEVICES_XXX ? Thanks Patrick
Re: [PATCH] stm32mp1: configs: Make all boot devices in distro boot conditional
On 1/9/20 2:50 PM, Patrick DELAUNAY wrote: > Hi Marek, Hi, >> From: Marek Vasut >> Sent: mercredi 18 décembre 2019 07:58 >> To: u-boot@lists.denx.de >> Cc: Marek Vasut ; Patrick DELAUNAY >> ; Patrice CHOTARD >> Subject: [PATCH] stm32mp1: configs: Make all boot devices in distro boot >> conditional >> Importance: High >> >> Not all systems have all the boot devices enabled, e.g. not all systems have >> MTD >> devices and thus do not enable UBI. Make all the boot devices in the distro >> bootcmd conditional to avoid failures. >> >> Signed-off-by: Marek Vasut >> Cc: Patrick Delaunay >> Cc: Patrice Chotard > > Reviewed-by: Patrick Delaunay > > With a minor remark below. > >> --- >> include/configs/stm32mp1.h | 34 -- >> 1 file changed, 28 insertions(+), 6 deletions(-) >> >> diff --git a/include/configs/stm32mp1.h b/include/configs/stm32mp1.h index >> d42a7860be..dfc397c63c 100644 >> --- a/include/configs/stm32mp1.h >> +++ b/include/configs/stm32mp1.h >> @@ -98,12 +98,34 @@ >> >> #if !defined(CONFIG_SPL_BUILD) >> >> -#define BOOT_TARGET_DEVICES(func) \ >> -func(MMC, mmc, 1) \ >> -func(UBIFS, ubifs, 0) \ >> -func(MMC, mmc, 0) \ >> -func(MMC, mmc, 2) \ >> -func(PXE, pxe, na) >> +#ifdef CONFIG_CMD_MMC >> +#define BOOT_TARGET_DEVICE_MMC0(func) func(MMC, mmc, 0) >> +#define BOOT_TARGET_DEVICE_MMC1(func) func(MMC, mmc, 1) >> +#define BOOT_TARGET_DEVICE_MMC2(func) func(MMC, mmc, 2) >> +#else >> +#define BOOT_TARGET_DEVICE_MMC0(func) >> +#define BOOT_TARGET_DEVICE_MMC1(func) >> +#define BOOT_TARGET_DEVICE_MMC2(func) >> +#endif >> + >> +#ifdef CONFIG_NET >> +#define BOOT_TARGET_DEVICE_PXE(func)func(PXE, pxe, na) >> +#else >> +#define BOOT_TARGET_DEVICE_PXE(func) >> +#endif >> + >> +#ifdef CONFIG_CMD_UBIFS >> +#define BOOT_TARGET_DEVICE_UBIFS(func) func(UBIFS, ubifs, 0) >> +#else >> +#define BOOT_TARGET_DEVICE_UBIFS(func) >> +#endif >> + >> +#define BOOT_TARGET_DEVICES(func) \ >> +BOOT_TARGET_DEVICE_MMC1(func) \ >> +BOOT_TARGET_DEVICE_UBIFS(func) \ >> +BOOT_TARGET_DEVICE_MMC0(func) \ >> +BOOT_TARGET_DEVICE_MMC2(func) \ >> +BOOT_TARGET_DEVICE_PXE(func) > > For other board, the same defines are named BOOT_TARGET_DEVICES_ (with a > S at the end of DEVICES) > > ./include/configs/sunxi-common.h:404:#define BOOT_TARGET_DEVICES(func) \ > ./include/configs/sunxi-common.h:406: BOOT_TARGET_DEVICES_MMC(func) \ > ./include/configs/sunxi-common.h:407: BOOT_TARGET_DEVICES_SCSI(func) \ > ./include/configs/sunxi-common.h:408: BOOT_TARGET_DEVICES_USB(func) \ > ./include/configs/sunxi-common.h:409: BOOT_TARGET_DEVICES_PXE(func) \ > ./include/configs/sunxi-common.h:410: BOOT_TARGET_DEVICES_DHCP(func) > > Except for ./include/configs/uniphier.h > > Any reason to change the name for stm32mp1 ? > > Othe exception for rockchip > > ./include/configs/rockchip-common.h:41:#define BOOT_TARGET_DEVICES(func) \ > ./include/configs/rockchip-common.h:42: BOOT_TARGET_MMC(func) \ > ./include/configs/rockchip-common.h:43: BOOT_TARGET_USB(func) \ > ./include/configs/rockchip-common.h:44: BOOT_TARGET_PXE(func) \ > ./include/configs/rockchip-common.h:45: BOOT_TARGET_DHCP(func) > > If I can choose I prefer the rockchip name (with shorter define name) We can rename the other boards' macros in a separate patch too, so it's the same.
RE: [PATCH] stm32mp1: configs: Make all boot devices in distro boot conditional
Hi Marek, > From: Marek Vasut > Sent: mercredi 18 décembre 2019 07:58 > To: u-boot@lists.denx.de > Cc: Marek Vasut ; Patrick DELAUNAY > ; Patrice CHOTARD > Subject: [PATCH] stm32mp1: configs: Make all boot devices in distro boot > conditional > Importance: High > > Not all systems have all the boot devices enabled, e.g. not all systems have > MTD > devices and thus do not enable UBI. Make all the boot devices in the distro > bootcmd conditional to avoid failures. > > Signed-off-by: Marek Vasut > Cc: Patrick Delaunay > Cc: Patrice Chotard Reviewed-by: Patrick Delaunay With a minor remark below. > --- > include/configs/stm32mp1.h | 34 -- > 1 file changed, 28 insertions(+), 6 deletions(-) > > diff --git a/include/configs/stm32mp1.h b/include/configs/stm32mp1.h index > d42a7860be..dfc397c63c 100644 > --- a/include/configs/stm32mp1.h > +++ b/include/configs/stm32mp1.h > @@ -98,12 +98,34 @@ > > #if !defined(CONFIG_SPL_BUILD) > > -#define BOOT_TARGET_DEVICES(func) \ > - func(MMC, mmc, 1) \ > - func(UBIFS, ubifs, 0) \ > - func(MMC, mmc, 0) \ > - func(MMC, mmc, 2) \ > - func(PXE, pxe, na) > +#ifdef CONFIG_CMD_MMC > +#define BOOT_TARGET_DEVICE_MMC0(func)func(MMC, mmc, 0) > +#define BOOT_TARGET_DEVICE_MMC1(func)func(MMC, mmc, 1) > +#define BOOT_TARGET_DEVICE_MMC2(func)func(MMC, mmc, 2) > +#else > +#define BOOT_TARGET_DEVICE_MMC0(func) > +#define BOOT_TARGET_DEVICE_MMC1(func) > +#define BOOT_TARGET_DEVICE_MMC2(func) > +#endif > + > +#ifdef CONFIG_NET > +#define BOOT_TARGET_DEVICE_PXE(func) func(PXE, pxe, na) > +#else > +#define BOOT_TARGET_DEVICE_PXE(func) > +#endif > + > +#ifdef CONFIG_CMD_UBIFS > +#define BOOT_TARGET_DEVICE_UBIFS(func) func(UBIFS, ubifs, 0) > +#else > +#define BOOT_TARGET_DEVICE_UBIFS(func) > +#endif > + > +#define BOOT_TARGET_DEVICES(func)\ > + BOOT_TARGET_DEVICE_MMC1(func) \ > + BOOT_TARGET_DEVICE_UBIFS(func) \ > + BOOT_TARGET_DEVICE_MMC0(func) \ > + BOOT_TARGET_DEVICE_MMC2(func) \ > + BOOT_TARGET_DEVICE_PXE(func) For other board, the same defines are named BOOT_TARGET_DEVICES_ (with a S at the end of DEVICES) ./include/configs/sunxi-common.h:404:#define BOOT_TARGET_DEVICES(func) \ ./include/configs/sunxi-common.h:406: BOOT_TARGET_DEVICES_MMC(func) \ ./include/configs/sunxi-common.h:407: BOOT_TARGET_DEVICES_SCSI(func) \ ./include/configs/sunxi-common.h:408: BOOT_TARGET_DEVICES_USB(func) \ ./include/configs/sunxi-common.h:409: BOOT_TARGET_DEVICES_PXE(func) \ ./include/configs/sunxi-common.h:410: BOOT_TARGET_DEVICES_DHCP(func) Except for ./include/configs/uniphier.h Any reason to change the name for stm32mp1 ? Othe exception for rockchip ./include/configs/rockchip-common.h:41:#define BOOT_TARGET_DEVICES(func) \ ./include/configs/rockchip-common.h:42: BOOT_TARGET_MMC(func) \ ./include/configs/rockchip-common.h:43: BOOT_TARGET_USB(func) \ ./include/configs/rockchip-common.h:44: BOOT_TARGET_PXE(func) \ ./include/configs/rockchip-common.h:45: BOOT_TARGET_DHCP(func) If I can choose I prefer the rockchip name (with shorter define name) > /* > * bootcmd for stm32mp1: > -- > 2.24.1 Regards Patrick