[PATCH] bsps/zynqmp: Allow any or all CGEMs to be enabled

[Chris Johns]

Hi,

I am just catching up and missed this one until I saw the patch was pushed. I am
sorry but I am confused by this patch.

Could someone please explain this reason for this addition to the BSP and
approach being taken? Is there something in the BSP that requires this
information be provided here?

Chris

On 9/6/21 6:18 am, Kinsey Moore wrote:
> Provide the options necessary to enable any combination of CGEM ethernet
> interfaces in LibBSD. The default is still CGEM3, so this should
> continue to operate as expected on typical Zynq Ultrascale+ MPSoC
> development hardware.
> ---
>  spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml    |  8 ++++++++
>  .../bsps/aarch64/xilinx-zynqmp/optcgem0.yml      | 16 ++++++++++++++++
>  .../bsps/aarch64/xilinx-zynqmp/optcgem1.yml      | 16 ++++++++++++++++
>  .../bsps/aarch64/xilinx-zynqmp/optcgem2.yml      | 16 ++++++++++++++++
>  .../bsps/aarch64/xilinx-zynqmp/optcgem3.yml      | 16 ++++++++++++++++
>  5 files changed, 72 insertions(+)
>  create mode 100644 spec/build/bsps/aarch64/xilinx-zynqmp/optcgem0.yml
>  create mode 100644 spec/build/bsps/aarch64/xilinx-zynqmp/optcgem1.yml
>  create mode 100644 spec/build/bsps/aarch64/xilinx-zynqmp/optcgem2.yml
>  create mode 100644 spec/build/bsps/aarch64/xilinx-zynqmp/optcgem3.yml
> 
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> index 16e2b8a7e9..1b6b756912 100644
> --- a/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/grp.yml
> @@ -27,6 +27,14 @@ links:
>    uid: optramori
>  - role: build-dependency
>    uid: optclkuart
> +- role: build-dependency
> +  uid: optcgem0
> +- role: build-dependency
> +  uid: optcgem1
> +- role: build-dependency
> +  uid: optcgem2
> +- role: build-dependency
> +  uid: optcgem3
>  - role: build-dependency
>    uid: ../../optconminor
>  - role: build-dependency
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem0.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem0.yml
> new file mode 100644
> index 0000000000..fc878fda60
> --- /dev/null
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem0.yml
> @@ -0,0 +1,16 @@
> +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> +actions:
> +- get-boolean: null
> +- env-enable: null
> +- define-condition: null
> +build-type: option
> +copyrights:
> +- Copyright (C) 2021 On-Line Applications Research
> +default: false
> +default-by-variant: []
> +description: |
> +  Enable support for CGEM0
> +enabled-by: true
> +links: []
> +name: BSP_XILINX_ZYNQMP_USE_CGEM0
> +type: build
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem1.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem1.yml
> new file mode 100644
> index 0000000000..6d5096bbde
> --- /dev/null
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem1.yml
> @@ -0,0 +1,16 @@
> +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> +actions:
> +- get-boolean: null
> +- env-enable: null
> +- define-condition: null
> +build-type: option
> +copyrights:
> +- Copyright (C) 2021 On-Line Applications Research
> +default: false
> +default-by-variant: []
> +description: |
> +  Enable support for CGEM1
> +enabled-by: true
> +links: []
> +name: BSP_XILINX_ZYNQMP_USE_CGEM1
> +type: build
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem2.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem2.yml
> new file mode 100644
> index 0000000000..a8aca3ebbd
> --- /dev/null
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem2.yml
> @@ -0,0 +1,16 @@
> +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> +actions:
> +- get-boolean: null
> +- env-enable: null
> +- define-condition: null
> +build-type: option
> +copyrights:
> +- Copyright (C) 2021 On-Line Applications Research
> +default: false
> +default-by-variant: []
> +description: |
> +  Enable support for CGEM2
> +enabled-by: true
> +links: []
> +name: BSP_XILINX_ZYNQMP_USE_CGEM2
> +type: build
> diff --git a/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem3.yml b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem3.yml
> new file mode 100644
> index 0000000000..8275ad3440
> --- /dev/null
> +++ b/spec/build/bsps/aarch64/xilinx-zynqmp/optcgem3.yml
> @@ -0,0 +1,16 @@
> +SPDX-License-Identifier: CC-BY-SA-4.0 OR BSD-2-Clause
> +actions:
> +- get-boolean: null
> +- env-enable: null
> +- define-condition: null
> +build-type: option
> +copyrights:
> +- Copyright (C) 2021 On-Line Applications Research
> +default: true
> +default-by-variant: []
> +description: |
> +  Enable support for CGEM3
> +enabled-by: true
> +links: []
> +name: BSP_XILINX_ZYNQMP_USE_CGEM3
> +type: build
>