Re: [PATCH v8 1/5] config/arm: avoid mcpu and march conflicts

[Thomas Monjalon <thomas@monjalon.net>]

14/03/2024 12:38, pbhagavatula@marvell.com:
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
> 
> The compiler options march and mtune are a subset
> of mcpu and will lead to conflicts if improper march
> is chosen for a given mcpu.
> To avoid conflicts, discard part number march when
> mcpu is available and is supported by the compiler.
> 
> Example:
> 	march = armv9-a
> 	mcpu = neoverse-n2
> 
> 	mcpu supported, march supported
> 	machine_args = ['-mcpu=neoverse-n2']
> 
> 	mcpu supported, march not supported
> 	machine_args = ['-mcpu=neoverse-n2']
> 
> 	mcpu not supported, march supported
> 	machine_args = ['-march=armv9-a']
> 
> 	mcpu not supported, march not supported
> 	machine_args = ['-march=armv8.6-a']
> 
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
> Reviewed-by: Juraj Linkeš <juraj.linkes@pantheon.tech>
> ---
> v2 Changes:
> - Cleanup march inconsistencies. (Juraj Linkes)
> - Unify fallback march selection. (Juraj Linkes)
> - Tag along ARM WFE patch.
> v3 Changes:
> - Fix missing 'fallback_march' key check.
> v4 Changes:
> - Discard march when mcpu is supported.
> v5 Changes:
> - Consolidate mcpu and march checks. (Juraj Linkes)
> - Fix unintentionally skipping fallback march (Juraj Linkes)
> v6 Changes:
> - Remove compiler support check when march is forced. (Juraj Linkes)
> - Simplify fallback march configuration.
> v7 Changes:
> - Rebase on master.
> v8 Changes:
> - Split patches to be more explicit about changes.

Applied, thanks for the split.