[PATCH v3 5/5] config/arm: add AMD CDX

[Ferruh Yigit]

On 4/21/2023 3:54 PM, Nipun Gupta wrote:
> Adding support for AMD CDX devices
> 
> Signed-off-by: Nipun Gupta <nipun.gupta at amd.com>
> ---
>  config/arm/arm64_cdx_linux_gcc | 17 +++++++++++++++++
>  config/arm/meson.build         | 14 ++++++++++++++
>  2 files changed, 31 insertions(+)
>  create mode 100644 config/arm/arm64_cdx_linux_gcc
> 
> diff --git a/config/arm/arm64_cdx_linux_gcc b/config/arm/arm64_cdx_linux_gcc
> new file mode 100644
> index 0000000000..8e6d619dae
> --- /dev/null
> +++ b/config/arm/arm64_cdx_linux_gcc
> @@ -0,0 +1,17 @@
> +[binaries]
> +c = ['ccache', 'aarch64-linux-gnu-gcc']
> +cpp = ['ccache', 'aarch64-linux-gnu-g++']
> +ar = 'aarch64-linux-gnu-ar'
> +as = 'aarch64-linux-gnu-as'
> +strip = 'aarch64-linux-gnu-strip'
> +pkgconfig = 'aarch64-linux-gnu-pkg-config'
> +pcap-config = ''
> +
> +[host_machine]
> +system = 'linux'
> +cpu_family = 'aarch64'
> +cpu = 'armv8-a'
> +endian = 'little'
> +
> +[properties]
> +platform = 'cdx'
> diff --git a/config/arm/meson.build b/config/arm/meson.build
> index 5213434ca4..39b8929534 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -305,6 +305,18 @@ soc_bluefield = {
>      'numa': false
>  }
>  
> +soc_cdx = {
> +    'description': 'AMD CDX',
> +    'implementer': '0x41',
> +    'part_number': '0xd42',
> +    'flags': [
> +        ['RTE_MACHINE', '"cdx"'],
> +        ['RTE_MAX_LCORE', 16],
> +        ['RTE_MAX_NUMA_NODES', 1]
> +    ],
> +    'numa': false
> +}

Hi Nipun,

Why we need a new arm platform/config? Is it because of above flags?
If it can work with default values, I think we can drop this patch.