[PATCH] spec/bsps: Deduplicate objxilinxsupport
Gedare Bloom
gedare at rtems.org
Mon Feb 6 22:09:49 UTC 2023
ok. I'm not sure, maybe we need a note about this in
https://docs.rtems.org/branches/master/eng/build-system.html
On Mon, Feb 6, 2023 at 2:02 PM Kinsey Moore <kinsey.moore at oarcorp.com> wrote:
>
> The objxilinxsupport build object was accidentally included twice in
> some of the ZynqMP BSPs by two different drivers that required it. This
> commit manually deduplicates the inclusions by moving that inclusion to
> the BSP. Duplication of object inclusions is considered a bug and can
> cause race conditions in the build system.
> ---
> spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml | 2 ++
> spec/build/bsps/objnandpsu.yml | 4 +---
> spec/build/bsps/objqspipsu.yml | 2 --
> 3 files changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> index f0c3a13ffd..a00490a826 100644
> --- a/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> @@ -52,6 +52,8 @@ links:
> uid: ../../objdevspixil
> - role: build-dependency
> uid: ../../objmem
> +- role: build-dependency
> + uid: ../../objxilinxsupport
> - role: build-dependency
> uid: ../../optcachedata
> - role: build-dependency
> diff --git a/spec/build/bsps/objnandpsu.yml b/spec/build/bsps/objnandpsu.yml
> index a0ff1b0b9e..253c598e8c 100644
> --- a/spec/build/bsps/objnandpsu.yml
> +++ b/spec/build/bsps/objnandpsu.yml
> @@ -17,9 +17,7 @@ install:
> - bsps/include/dev/nand/xnandpsu.h
> - bsps/include/dev/nand/xnandpsu_hw.h
> - bsps/include/dev/nand/xnandpsu_onfi.h
> -links:
> -- role: build-dependency
> - uid: objxilinxsupport
> +links: []
> source:
> - bsps/shared/dev/nand/xnandpsu_bbm.c
> - bsps/shared/dev/nand/xnandpsu.c
> diff --git a/spec/build/bsps/objqspipsu.yml b/spec/build/bsps/objqspipsu.yml
> index 5f8679c83c..205172146e 100644
> --- a/spec/build/bsps/objqspipsu.yml
> +++ b/spec/build/bsps/objqspipsu.yml
> @@ -19,8 +19,6 @@ install:
> - bsps/include/dev/spi/xqspipsu-flash-helper.h
> - bsps/include/dev/spi/xqspipsu.h
> links:
> -- role: build-dependency
> - uid: objxilinxsupport
> - role: build-dependency
> uid: optxpssysctrlbaseaddress
> source:
> --
> 2.30.2
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
More information about the devel
mailing list