Hi Marek,
From: Marek Vasut marex@denx.de Sent: jeudi 9 janvier 2020 15:27
On 1/9/20 2:50 PM, Patrick DELAUNAY wrote:
Hi Marek,
Hi,
From: Marek Vasut marex@denx.de Sent: mercredi 18 décembre 2019 07:58 To: u-boot@lists.denx.de Cc: Marek Vasut marex@denx.de; Patrick DELAUNAY patrick.delaunay@st.com; Patrice CHOTARD patrice.chotard@st.com 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 marex@denx.de Cc: Patrick Delaunay patrick.delaunay@st.com Cc: Patrice Chotard patrice.chotard@st.com
Reviewed-by: Patrick Delaunay patrick.delaunay@st.com
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_XXXX
(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