> -----Original Message----- > From: Juraj Linkeš <[email protected]> > Sent: Tuesday, August 17, 2021 6:57 PM > To: [email protected]; [email protected]; > [email protected]; Honnappa Nagarahalli > <[email protected]>; Ruifeng Wang > <[email protected]>; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected] > Cc: [email protected]; Juraj Linkeš <[email protected]> > Subject: [PATCH v4 2/2] config/arm: make n2 'crypto' an optional feature > > Not all Neoverse-N2 cpus must support the crypto feature/extension which > makes it an optional feature. Only enable the feature for SoCs which support > it. > > Signed-off-by: Juraj Linkeš <[email protected]> > --- > v4: rebase > --- > config/arm/meson.build | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/config/arm/meson.build b/config/arm/meson.build index > c11efa1583..c9bcd089df 100644 > --- a/config/arm/meson.build > +++ b/config/arm/meson.build > @@ -76,7 +76,7 @@ part_number_config_arm = { > }, > '0xd49': { > 'march': 'armv8.5-a', > - 'march_features': ['crypto', 'sve2'], > + 'march_features': ['sve2'], > 'flags': [ > ['RTE_MACHINE', '"neoverse-n2"'], > ['RTE_ARM_FEATURE_ATOMICS', true], @@ -278,6 +278,7 @@ > soc_cn10k = { > ['RTE_MAX_NUMA_NODES', 1] > ], > 'part_number': '0xd49', > + 'extra_march_features': ['crypto'], > 'numa': false > } > > -- > 2.20.1
Yes, 'crypto' is an optional extension. N2 has this extension enabled, and there could be variants without crypto extension. Acked-by: Ruifeng Wang <[email protected]>
