diff mbox series

ASoC: amd: Fix build dependency for SND_SOC_AMD_YC_MACH

Message ID 20250110152324.20529-2-aherrmann@suse.de (mailing list archive)
State New
Headers show
Series ASoC: amd: Fix build dependency for SND_SOC_AMD_YC_MACH | expand

Commit Message

Andreas Herrmann Jan. 10, 2025, 3:22 p.m. UTC
Commit 4095cf872084 ("ASoC: amd: yc: Fix for enabling DMIC on acp6x
via _DSD entry") introduced usage of acpi_evaluate_integer in
acp6x-mach.c.

A kernel config which has

  # CONFIG_ACPI is not set
  CONFIG_SND_SOC_AMD_YC_MACH=y

causes following compile error

  sound/soc/amd/yc/acp6x-mach.c: In function ‘acp6x_probe’:
  sound/soc/amd/yc/acp6x-mach.c:580:15: error: implicit declaration of function ‘acpi_evaluate_integer’; did you mean ‘acpi_evaluate_object’? [-Wimplicit-function-declaration]
    580 |         ret = acpi_evaluate_integer(handle, "_WOV", NULL, &dmic_status);
        |               ^~~~~~~~~~~~~~~~~~~~~
        |               acpi_evaluate_object

Thus ACPI is a new build dependency for SND_SOC_AMD_YC_MACH.

Fixes: 4095cf872084 ("ASoC: amd: yc: Fix for enabling DMIC on acp6x via _DSD entry")
Signed-off-by: Andreas Herrmann <aherrmann@suse.de>
Cc: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
---
 sound/soc/amd/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mark Brown Jan. 10, 2025, 3:29 p.m. UTC | #1
On Fri, Jan 10, 2025 at 04:22:39PM +0100, Andreas Herrmann wrote:
> Commit 4095cf872084 ("ASoC: amd: yc: Fix for enabling DMIC on acp6x
> via _DSD entry") introduced usage of acpi_evaluate_integer in
> acp6x-mach.c.

Someone already fixed this.
diff mbox series

Patch

diff --git a/sound/soc/amd/Kconfig b/sound/soc/amd/Kconfig
index c7590d4989bb..803521178279 100644
--- a/sound/soc/amd/Kconfig
+++ b/sound/soc/amd/Kconfig
@@ -105,7 +105,7 @@  config SND_SOC_AMD_ACP6x
 config SND_SOC_AMD_YC_MACH
 	tristate "AMD YC support for DMIC"
 	select SND_SOC_DMIC
-	depends on SND_SOC_AMD_ACP6x
+	depends on SND_SOC_AMD_ACP6x && ACPI
 	help
 	  This option enables machine driver for Yellow Carp platform
 	  using dmic. ACP IP has PDM Decoder block with DMA controller.