Re: [PATCH v6 02/90] ASoC: Intel: avs: Include CPUID header at file scope
Cezary Rojewski <[email protected]> Fri, 27 Mar 2026 10:25:15 +0100
| Newsgroups | dev.linux.lists.x86-cpuid,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On 2026-03-27 3:15 AM, Ahmed S. Darwish wrote:
> Commit
>
> cbe37a4d2b3c ("ASoC: Intel: avs: Configure basefw on TGL-based platforms")
>
> includes the main CPUID header from within a C function. This works by
> luck and forbids valid refactorings inside that header.
>
> Include the CPUID header at file scope instead.
>
> Remove the COMPILE_TEST build flag so that the CONFIG_X86 conditionals can
> be removed. The driver gets enough compilation testing already on x86.
>
> For clarity, refactor the CPUID(0x15) code into its own function without
> changing any of the driver's logic.
>
> Suggested-by: Borislav Petkov <[email protected]> # CONFIG_X86 removal
> Signed-off-by: Ahmed S. Darwish <[email protected]>
> Fixes: cbe37a4d2b3c ("ASoC: Intel: avs: Configure basefw on TGL-based platforms")
> References: 47a1886a610a ("ASoC: Intel: avs: Enable AVS driver only on x86 platforms")
> Link: https://lore.kernel.org/r/20251016105514.GCaPDPEu016XyDocfY@fat_crate.local
> Link: https://lore.kernel.org/r/[email protected]
Thank you for sorting this out, looks good.
Acked-by: Cezary Rojewski <[email protected]>