diff mbox series

ASoC: Intel: avs: Enable AVS driver only on x86 platforms

Message ID 20220422133411.169007-1-amadeuszx.slawinski@linux.intel.com (mailing list archive)
State Superseded
Headers show
Series ASoC: Intel: avs: Enable AVS driver only on x86 platforms | expand

Commit Message

Amadeusz Sławiński April 22, 2022, 1:34 p.m. UTC
Only supported platform for AVS are x86 machines, so there is no reason
for it to build on other architectures.

Fixes: 274d79e51875 ("ASoC: Intel: avs: Configure modules according to their type")
Reported-by: kernel test robot <lkp@intel.com>
Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com>
Signed-off-by: Amadeusz Sławiński <amadeuszx.slawinski@linux.intel.com>
---
 sound/soc/intel/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Brown April 22, 2022, 1:54 p.m. UTC | #1
On Fri, Apr 22, 2022 at 03:34:11PM +0200, Amadeusz Sławiński wrote:
> Only supported platform for AVS are x86 machines, so there is no reason
> for it to build on other architectures.

>  config SND_SOC_INTEL_AVS
>  	tristate "Intel AVS driver"
> -	depends on PCI && ACPI
> +	depends on X86 && PCI && ACPI

If it actually builds please add an || COMPILE_TEST in there so people
doing framework updates have to cover fewer build configurations for
testing (this might make sense for the PCI and ACPI bits as well if
those will compile out).
diff mbox series

Patch

diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index c364ddf22267..5dc075673b97 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -211,7 +211,7 @@  config SND_SOC_INTEL_KEEMBAY
 
 config SND_SOC_INTEL_AVS
 	tristate "Intel AVS driver"
-	depends on PCI && ACPI
+	depends on X86 && PCI && ACPI
 	depends on COMMON_CLK
 	select SND_SOC_ACPI
 	select SND_SOC_TOPOLOGY