Re: [PATCH v2] config/arm: add Marvell Odyssey

[Ruifeng Wang <Ruifeng.Wang@arm.com>]

On 2024/3/5 7:13 PM, Anoob Joseph wrote:
> Add meson build configuration for Marvell Odyssey platform with 64-bit
> ARM Neoverse V2 cores.
> 
> Signed-off-by: Anoob Joseph <anoobj@marvell.com>
> ---
> 
> Depends-on: series-31141 ("config/arm: add Neoverse V2 part number")
> 
> Changes in v2:
> - Renamed config file
> 
>   config/arm/arm64_odyssey_linux_gcc | 17 +++++++++++++++++
>   config/arm/meson.build             | 15 +++++++++++++++
>   2 files changed, 32 insertions(+)
>   create mode 100644 config/arm/arm64_odyssey_linux_gcc
> 
> diff --git a/config/arm/arm64_odyssey_linux_gcc b/config/arm/arm64_odyssey_linux_gcc
> new file mode 100644
> index 0000000000..69b5cd42d8
> --- /dev/null
> +++ b/config/arm/arm64_odyssey_linux_gcc
> @@ -0,0 +1,17 @@
> +[binaries]
> +c = ['ccache', 'aarch64-marvell-linux-gnu-gcc']
> +cpp = ['ccache', 'aarch64-marvell-linux-gnu-g++']
> +ar = 'aarch64-marvell-linux-gnu-gcc-ar'
> +strip = 'aarch64-marvell-linux-gnu-strip'
> +pkgconfig = 'aarch64-linux-gnu-pkg-config'
> +
> +[host_machine]
> +system = 'linux'
> +cpu_family = 'aarch64'
> +cpu = 'armv9-a'
> +endian = 'little'
> +
> +[properties]
> +platform = 'odyssey'
> +
> +[built-in options]
> diff --git a/config/arm/meson.build b/config/arm/meson.build
> index 3886d0e2dc..94159efaa4 100644
> --- a/config/arm/meson.build
> +++ b/config/arm/meson.build
> @@ -500,6 +500,20 @@ soc_n2 = {
>       'numa': false
>   }
>   
> +soc_odyssey = {
> +    'description' : 'Marvell Odyssey',
> +    'implementer' : '0x41',
> +    'flags': [
> +        ['RTE_MAX_LCORE', 80],
> +        ['RTE_MAX_NUMA_NODES', 1],
> +        ['RTE_MEMPOOL_ALIGN', 128],
> +    ],
> +    'part_number': '0xd4f',
> +    'extra_march_features': ['crypto'],
> +    'numa': false,
> +    'sve_acle': false
> +}
> +
>   soc_cn9k = {
>       'description': 'Marvell OCTEON 9',
>       'implementer': '0x43',
> @@ -617,6 +631,7 @@ socs = {
>       'kunpeng930': soc_kunpeng930,
>       'n1sdp': soc_n1sdp,
>       'n2': soc_n2,
> +    'odyssey' : soc_odyssey,

The SoC string list above also needs update. It is for documentation.
With the change:
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>

>       'stingray': soc_stingray,
>       'thunderx2': soc_thunderx2,
>       'thunderxt88': soc_thunderxt88,