* [Buildroot] [PATCH v7 2/5] boot/xilinx-embeddedsw: only allow apps for device family
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
@ 2025-09-01 8:02 ` Neal Frager via buildroot
2025-09-01 17:00 ` Luca Ceresoli via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 3/5] boot/xilinx-prebuilt: only allow binaries " Neal Frager via buildroot
` (4 subsequent siblings)
5 siblings, 1 reply; 10+ messages in thread
From: Neal Frager via buildroot @ 2025-09-01 8:02 UTC (permalink / raw)
To: buildroot
Cc: ibai.erkiaga-elorza, luca.ceresoli, yann.morin, brandon.maier,
ju.o, Neal Frager, thomas.petazzoni, romain.naour, michal.simek,
romain.naour
Add an architecture cpu dependency to each application to make sure that users
can only build applications which are applicable to their target device
family.
The versal_plm and versal_psmfw applications are specific to versal devices
which are based on BR2_cortex_a72.
The zynqmp_pmufw application is specific to zynqmp devices which are based on
BR2_cortex_a53.
Signed-off-by: Neal Frager <neal.frager@amd.com>
---
V1->V2:
- Replaced new family variant config option with an architecture cpu
dependency, so no new configs are needed.
- Updated patch title and commit message accordingly.
V2->V3:
- Changed package dependency to only appear if the cpu is BR2_cortex_a53
or BR2_cortex_a72. This way, the xilinx-embeddedsw package will not
appear with zero application options if another BR2_aarch64 cpu is
selected.
V3->V4:
- Added BR2_TARGET_XILINX_EMBEDDEDSW_ARCH_SUPPORTS, so that all arch
dependencies are in one place
V4->V5:
- Replace leading space copy paste error with tabs
V5->V6:
- Take advantage of BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS from
boot/Config.in to remove dependencies on BR2_aarch64.
V6->V7:
- aarch64 dependency removed in first patch of series
- improved commit message
---
boot/xilinx-embeddedsw/Config.in | 3 +++
1 file changed, 3 insertions(+)
diff --git a/boot/xilinx-embeddedsw/Config.in b/boot/xilinx-embeddedsw/Config.in
index 5f115726e1..3826f2e18e 100644
--- a/boot/xilinx-embeddedsw/Config.in
+++ b/boot/xilinx-embeddedsw/Config.in
@@ -28,6 +28,7 @@ config BR2_TARGET_XILINX_EMBEDDEDSW_VERSION
config BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PLM
bool "versal plm"
+ depends on BR2_cortex_a72
help
Build versal plm application from Xilinx/embeddedsw repo.
If selected, the xilinx-prebuilt package will not install
@@ -38,6 +39,7 @@ config BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PLM
config BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PSMFW
bool "versal psmfw"
+ depends on BR2_cortex_a72
help
Build versal psmfw application from Xilinx/embeddedsw repo.
If selected, the xilinx-prebuilt package will not install
@@ -48,6 +50,7 @@ config BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PSMFW
config BR2_TARGET_XILINX_EMBEDDEDSW_ZYNQMP_PMUFW
bool "zynqmp pmufw"
+ depends on BR2_cortex_a53
help
Build zynqmp pmufw application from Xilinx/embeddedsw repo.
If selected, the xilinx-prebuilt package will not install
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH v7 2/5] boot/xilinx-embeddedsw: only allow apps for device family
2025-09-01 8:02 ` [Buildroot] [PATCH v7 2/5] boot/xilinx-embeddedsw: only allow apps for device family Neal Frager via buildroot
@ 2025-09-01 17:00 ` Luca Ceresoli via buildroot
0 siblings, 0 replies; 10+ messages in thread
From: Luca Ceresoli via buildroot @ 2025-09-01 17:00 UTC (permalink / raw)
To: Neal Frager
Cc: yann.morin, ibai.erkiaga-elorza, buildroot, brandon.maier, ju.o,
thomas.petazzoni, romain.naour, michal.simek, romain.naour
On Mon, 1 Sep 2025 09:02:46 +0100
Neal Frager <neal.frager@amd.com> wrote:
> Add an architecture cpu dependency to each application to make sure that users
> can only build applications which are applicable to their target device
> family.
>
> The versal_plm and versal_psmfw applications are specific to versal devices
> which are based on BR2_cortex_a72.
>
> The zynqmp_pmufw application is specific to zynqmp devices which are based on
> BR2_cortex_a53.
>
> Signed-off-by: Neal Frager <neal.frager@amd.com>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH v7 3/5] boot/xilinx-prebuilt: only allow binaries for device family
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 2/5] boot/xilinx-embeddedsw: only allow apps for device family Neal Frager via buildroot
@ 2025-09-01 8:02 ` Neal Frager via buildroot
2025-09-01 17:00 ` Luca Ceresoli via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 4/5] configs/versal_*: make savedefconfig cleaning Neal Frager via buildroot
` (3 subsequent siblings)
5 siblings, 1 reply; 10+ messages in thread
From: Neal Frager via buildroot @ 2025-09-01 8:02 UTC (permalink / raw)
To: buildroot
Cc: ibai.erkiaga-elorza, luca.ceresoli, yann.morin, brandon.maier,
ju.o, Neal Frager, thomas.petazzoni, romain.naour, michal.simek,
romain.naour
Add an architecture cpu dependency to each family to make sure that users can
only install prebuilt firmware which is applicable to their target device
family.
The versal family is based on BR2_cortex_a72.
The kria and zynqmp families are based on BR2_cortex_a53.
Signed-off-by: Neal Frager <neal.frager@amd.com>
---
V6:
- new patch added to series
V6->V7:
- aarch64 dependency removed in first patch of series
- improved commit message
---
boot/xilinx-prebuilt/Config.in | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/boot/xilinx-prebuilt/Config.in b/boot/xilinx-prebuilt/Config.in
index a37bdba540..5ebfb83c25 100644
--- a/boot/xilinx-prebuilt/Config.in
+++ b/boot/xilinx-prebuilt/Config.in
@@ -25,16 +25,20 @@ config BR2_TARGET_XILINX_PREBUILT_VERSION
choice
bool "family variant"
default BR2_TARGET_XILINX_PREBUILT_VERSAL if BR2_PACKAGE_VERSAL_FIRMWARE # legacy
+ default BR2_TARGET_XILINX_PREBUILT_VERSAL if BR2_cortex_a72
default BR2_TARGET_XILINX_PREBUILT_ZYNQMP
config BR2_TARGET_XILINX_PREBUILT_ZYNQMP
bool "zynqmp"
+ depends on BR2_cortex_a53
config BR2_TARGET_XILINX_PREBUILT_KRIA
bool "kria"
+ depends on BR2_cortex_a53
config BR2_TARGET_XILINX_PREBUILT_VERSAL
bool "versal"
+ depends on BR2_cortex_a72
endchoice
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH v7 3/5] boot/xilinx-prebuilt: only allow binaries for device family
2025-09-01 8:02 ` [Buildroot] [PATCH v7 3/5] boot/xilinx-prebuilt: only allow binaries " Neal Frager via buildroot
@ 2025-09-01 17:00 ` Luca Ceresoli via buildroot
0 siblings, 0 replies; 10+ messages in thread
From: Luca Ceresoli via buildroot @ 2025-09-01 17:00 UTC (permalink / raw)
To: Neal Frager
Cc: yann.morin, ibai.erkiaga-elorza, buildroot, brandon.maier, ju.o,
thomas.petazzoni, romain.naour, michal.simek, romain.naour
On Mon, 1 Sep 2025 09:02:47 +0100
Neal Frager <neal.frager@amd.com> wrote:
> Add an architecture cpu dependency to each family to make sure that users can
> only install prebuilt firmware which is applicable to their target device
> family.
>
> The versal family is based on BR2_cortex_a72.
>
> The kria and zynqmp families are based on BR2_cortex_a53.
>
> Signed-off-by: Neal Frager <neal.frager@amd.com>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread
* [Buildroot] [PATCH v7 4/5] configs/versal_*: make savedefconfig cleaning
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 2/5] boot/xilinx-embeddedsw: only allow apps for device family Neal Frager via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 3/5] boot/xilinx-prebuilt: only allow binaries " Neal Frager via buildroot
@ 2025-09-01 8:02 ` Neal Frager via buildroot
2025-09-01 8:02 ` [Buildroot] [PATCH v7 5/5] boot/uboot: use BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
` (2 subsequent siblings)
5 siblings, 0 replies; 10+ messages in thread
From: Neal Frager via buildroot @ 2025-09-01 8:02 UTC (permalink / raw)
To: buildroot
Cc: ibai.erkiaga-elorza, luca.ceresoli, yann.morin, brandon.maier,
ju.o, Neal Frager, thomas.petazzoni, romain.naour, michal.simek,
romain.naour
With the new cpu dependencies included in the xilinx-prebuilt package,
BR2_TARGET_XILINX_PREBUILT_VERSAL is now default when BR2_cortex_a72 is
selected. This means that BR2_TARGET_XILINX_PREBUILT_VERSAL is not needed
in the versal_*_defconfig files anymore.
Signed-off-by: Neal Frager <neal.frager@amd.com>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
---
V6:
- new patch added to series
V6->V7:
- no changes
---
configs/versal_vck190_defconfig | 1 -
configs/versal_vek280_defconfig | 1 -
configs/versal_vpk180_defconfig | 1 -
3 files changed, 3 deletions(-)
diff --git a/configs/versal_vck190_defconfig b/configs/versal_vck190_defconfig
index c9d99876d5..7a08501547 100644
--- a/configs/versal_vck190_defconfig
+++ b/configs/versal_vck190_defconfig
@@ -43,7 +43,6 @@ BR2_TARGET_XILINX_EMBEDDEDSW=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PLM=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PSMFW=y
BR2_TARGET_XILINX_PREBUILT=y
-BR2_TARGET_XILINX_PREBUILT_VERSAL=y
BR2_PACKAGE_HOST_BOOTGEN=y
BR2_PACKAGE_HOST_DOSFSTOOLS=y
BR2_PACKAGE_HOST_GENIMAGE=y
diff --git a/configs/versal_vek280_defconfig b/configs/versal_vek280_defconfig
index 4aa25f8d67..16283de2e5 100644
--- a/configs/versal_vek280_defconfig
+++ b/configs/versal_vek280_defconfig
@@ -43,7 +43,6 @@ BR2_TARGET_XILINX_EMBEDDEDSW=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PLM=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PSMFW=y
BR2_TARGET_XILINX_PREBUILT=y
-BR2_TARGET_XILINX_PREBUILT_VERSAL=y
BR2_TARGET_XILINX_PREBUILT_BOARD="vek280"
BR2_PACKAGE_HOST_BOOTGEN=y
BR2_PACKAGE_HOST_DOSFSTOOLS=y
diff --git a/configs/versal_vpk180_defconfig b/configs/versal_vpk180_defconfig
index 500298c2f4..3efe9dd875 100644
--- a/configs/versal_vpk180_defconfig
+++ b/configs/versal_vpk180_defconfig
@@ -43,7 +43,6 @@ BR2_TARGET_XILINX_EMBEDDEDSW=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PLM=y
BR2_TARGET_XILINX_EMBEDDEDSW_VERSAL_PSMFW=y
BR2_TARGET_XILINX_PREBUILT=y
-BR2_TARGET_XILINX_PREBUILT_VERSAL=y
BR2_TARGET_XILINX_PREBUILT_BOARD="vpk180"
BR2_PACKAGE_HOST_BOOTGEN=y
BR2_PACKAGE_HOST_DOSFSTOOLS=y
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* [Buildroot] [PATCH v7 5/5] boot/uboot: use BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
` (2 preceding siblings ...)
2025-09-01 8:02 ` [Buildroot] [PATCH v7 4/5] configs/versal_*: make savedefconfig cleaning Neal Frager via buildroot
@ 2025-09-01 8:02 ` Neal Frager via buildroot
2025-09-01 17:00 ` Luca Ceresoli via buildroot
2025-09-01 17:00 ` [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Luca Ceresoli via buildroot
2025-09-26 17:24 ` Julien Olivain via buildroot
5 siblings, 1 reply; 10+ messages in thread
From: Neal Frager via buildroot @ 2025-09-01 8:02 UTC (permalink / raw)
To: buildroot
Cc: ibai.erkiaga-elorza, luca.ceresoli, yann.morin, brandon.maier,
ju.o, Neal Frager, thomas.petazzoni, romain.naour, michal.simek,
romain.naour
Migrate the BR2_TARGET_UBOOT_ZYNQMP dependency to the new
BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS symbol. Also, add a cpu dependency
on BR2_cortex_a53 since the zynqmp platform is based on the ARM Cortex-A53
and the BR2_TARGET_UBOOT_ZYNQMP should not be available for other CPUs with
aarch64 architecture.
Signed-off-by: Neal Frager <neal.frager@amd.com>
---
V6:
- new patch added to series
V6->V7:
- improved commit message
---
boot/uboot/Config.in | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/boot/uboot/Config.in b/boot/uboot/Config.in
index 52e6050c81..0b27874e24 100644
--- a/boot/uboot/Config.in
+++ b/boot/uboot/Config.in
@@ -575,7 +575,8 @@ config BR2_TARGET_UBOOT_INITIAL_ENV
config BR2_TARGET_UBOOT_ZYNQMP
bool "Boot on the Xilinx ZynqMP SoCs"
- depends on BR2_aarch64
+ depends on BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS
+ depends on BR2_cortex_a53
help
Enable options specific to the Xilinx ZynqMP family of SoCs.
--
2.25.1
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH v7 5/5] boot/uboot: use BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS
2025-09-01 8:02 ` [Buildroot] [PATCH v7 5/5] boot/uboot: use BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
@ 2025-09-01 17:00 ` Luca Ceresoli via buildroot
0 siblings, 0 replies; 10+ messages in thread
From: Luca Ceresoli via buildroot @ 2025-09-01 17:00 UTC (permalink / raw)
To: Neal Frager
Cc: yann.morin, ibai.erkiaga-elorza, buildroot, brandon.maier, ju.o,
thomas.petazzoni, romain.naour, michal.simek, romain.naour
On Mon, 1 Sep 2025 09:02:49 +0100
Neal Frager <neal.frager@amd.com> wrote:
> Migrate the BR2_TARGET_UBOOT_ZYNQMP dependency to the new
> BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS symbol. Also, add a cpu dependency
> on BR2_cortex_a53 since the zynqmp platform is based on the ARM Cortex-A53
> and the BR2_TARGET_UBOOT_ZYNQMP should not be available for other CPUs with
> aarch64 architecture.
>
> Signed-off-by: Neal Frager <neal.frager@amd.com>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
` (3 preceding siblings ...)
2025-09-01 8:02 ` [Buildroot] [PATCH v7 5/5] boot/uboot: use BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
@ 2025-09-01 17:00 ` Luca Ceresoli via buildroot
2025-09-26 17:24 ` Julien Olivain via buildroot
5 siblings, 0 replies; 10+ messages in thread
From: Luca Ceresoli via buildroot @ 2025-09-01 17:00 UTC (permalink / raw)
To: Neal Frager
Cc: yann.morin, ibai.erkiaga-elorza, buildroot, brandon.maier, ju.o,
thomas.petazzoni, romain.naour, michal.simek, romain.naour
Hello Neal,
On Mon, 1 Sep 2025 09:02:45 +0100
Neal Frager <neal.frager@amd.com> wrote:
> Currently, the xilinx-embeddedsw and xilinx-prebuilt packages are appearing
> for any aarch64 processor. As all Xilinx processors that need these packages
> are ARM Cortex-A53 or ARM Cortex-A72, this means that these packages are
> often appearing as options for non-Xilinx platforms.
>
> To reduce this occurrence, this patch creates a new symbol
> BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS to limit the appearance of the
> xilinx-embeddedsw and xilinx-prebuilt boot packages to only platforms based
> on the BR2_cortex_a53 or BR2_cortex_a72 CPU and aarch64 architecture.
>
> Signed-off-by: Neal Frager <neal.frager@amd.com>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS
2025-09-01 8:02 [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Neal Frager via buildroot
` (4 preceding siblings ...)
2025-09-01 17:00 ` [Buildroot] [PATCH v7 1/5] boot: add BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS Luca Ceresoli via buildroot
@ 2025-09-26 17:24 ` Julien Olivain via buildroot
5 siblings, 0 replies; 10+ messages in thread
From: Julien Olivain via buildroot @ 2025-09-26 17:24 UTC (permalink / raw)
To: Neal Frager
Cc: buildroot, ibai.erkiaga-elorza, luca.ceresoli, yann.morin,
brandon.maier, thomas.petazzoni, romain.naour, michal.simek,
romain.naour
On 01/09/2025 10:02, Neal Frager via buildroot wrote:
> Currently, the xilinx-embeddedsw and xilinx-prebuilt packages are
> appearing
> for any aarch64 processor. As all Xilinx processors that need these
> packages
> are ARM Cortex-A53 or ARM Cortex-A72, this means that these packages
> are
> often appearing as options for non-Xilinx platforms.
>
> To reduce this occurrence, this patch creates a new symbol
> BR2_TARGET_XILINX_FIRMWARE_ARCH_SUPPORTS to limit the appearance of the
> xilinx-embeddedsw and xilinx-prebuilt boot packages to only platforms
> based
> on the BR2_cortex_a53 or BR2_cortex_a72 CPU and aarch64 architecture.
>
> Signed-off-by: Neal Frager <neal.frager@amd.com>
Series applied to master, thanks.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 10+ messages in thread