* [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
@ 2017-11-08 13:03 ` Arnd Bergmann
0 siblings, 0 replies; 15+ messages in thread
From: Arnd Bergmann @ 2017-11-08 13:03 UTC (permalink / raw)
To: Mark Brown
Cc: Arnd Bergmann, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Vinod Koul, Pierre-Louis Bossart, Andy Shevchenko,
Harsha Priya N, Daniel Drake, Naveen M, alsa-devel, linux-kernel
As pointed out by Pierre-Louis Bossart, the depndency I added
was broader than necessary, only Baytrail and Haswell/Broadwell
actually need it, the others don't.
At the same time, we have individual entries for the codecs
that all have the 'select' statement but now don't need it
any more.
Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
sound/soc/intel/Kconfig | 3 ++-
sound/soc/intel/boards/Kconfig | 5 -----
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index e18118209b75..bb8be10b8437 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
config SND_SOC_INTEL_SST_TOPLEVEL
tristate "Intel ASoC SST drivers"
depends on X86 || COMPILE_TEST
- depends on DMADEVICES
select SND_SOC_INTEL_MACH
select SND_SOC_INTEL_COMMON
config SND_SOC_INTEL_HASWELL
tristate "Intel ASoC SST driver for Haswell/Broadwell"
depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
config SND_SOC_INTEL_BAYTRAIL
tristate "Intel ASoC SST driver for Baytrail (legacy)"
depends on SND_SOC_INTEL_SST_TOPLEVEL
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 449bc8baaa60..5063f15b4ca4 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
help
@@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
help
@@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
help
@@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_MAX98090
@@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_RT5640
--
2.9.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
@ 2017-11-08 13:03 ` Arnd Bergmann
0 siblings, 0 replies; 15+ messages in thread
From: Arnd Bergmann @ 2017-11-08 13:03 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, Andy Shevchenko, Arnd Bergmann, Liam Girdwood,
Vinod Koul, linux-kernel, Pierre-Louis Bossart, Takashi Iwai,
Harsha Priya N, Naveen M, Daniel Drake
As pointed out by Pierre-Louis Bossart, the depndency I added
was broader than necessary, only Baytrail and Haswell/Broadwell
actually need it, the others don't.
At the same time, we have individual entries for the codecs
that all have the 'select' statement but now don't need it
any more.
Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
sound/soc/intel/Kconfig | 3 ++-
sound/soc/intel/boards/Kconfig | 5 -----
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index e18118209b75..bb8be10b8437 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
config SND_SOC_INTEL_SST_TOPLEVEL
tristate "Intel ASoC SST drivers"
depends on X86 || COMPILE_TEST
- depends on DMADEVICES
select SND_SOC_INTEL_MACH
select SND_SOC_INTEL_COMMON
config SND_SOC_INTEL_HASWELL
tristate "Intel ASoC SST driver for Haswell/Broadwell"
depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
config SND_SOC_INTEL_BAYTRAIL
tristate "Intel ASoC SST driver for Baytrail (legacy)"
depends on SND_SOC_INTEL_SST_TOPLEVEL
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 449bc8baaa60..5063f15b4ca4 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
help
@@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
help
@@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
help
@@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_MAX98090
@@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_RT5640
--
2.9.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
2017-11-08 13:03 ` Arnd Bergmann
@ 2017-11-08 13:03 ` Arnd Bergmann
-1 siblings, 0 replies; 15+ messages in thread
From: Arnd Bergmann @ 2017-11-08 13:03 UTC (permalink / raw)
To: Mark Brown
Cc: Arnd Bergmann, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Pierre-Louis Bossart, Vinod Koul, Andy Shevchenko,
Harsha Priya N, Naveen M, alsa-devel, linux-kernel
I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
and SND_SOC_INTEL_MACH=y:
ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
The problem here is that the sound/soc/intel/common/ directory
is then entered only for building modules, but the sst-acpi.o
never gets built since it depends on a built-in Kconfig symbol.
That configuration obviously makes no sense since all options
below SND_SOC_INTEL_MACH also depend on something else that
in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
solves the build error. I notice we can also consolidate the
'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
simplify it further and make sure the configuration stays sane.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
sound/soc/intel/boards/Kconfig | 26 +++++---------------------
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 5063f15b4ca4..6f754708a48c 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -1,10 +1,12 @@
config SND_SOC_INTEL_MACH
tristate "Intel Audio machine drivers"
+ depends on SND_SOC_INTEL_SST_TOPLEVEL
select SND_SOC_ACPI_INTEL_MATCH if ACPI
+if SND_SOC_INTEL_MACH
+
config SND_MFLD_MACHINE
tristate "SOC Machine Audio driver for Intel Medfield MID platform"
- depends on SND_SOC_INTEL_MACH
depends on INTEL_SCU_IPC
select SND_SOC_SN95031
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
@@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
@@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
@@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5640
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5651_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5651
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5670
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5645
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_MAX98090
select SND_SOC_TS3A227E
@@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_DA7213
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_ES8316
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
depends on SND_SST_ATOM_HIFI2_PLATFORM
select SND_SST_IPC_ACPI
@@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
config SND_SOC_INTEL_SKL_RT286_MACH
tristate "ASoC Audio driver for SKL with RT286 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT286
@@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_DA7219
@@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
config SND_SOC_INTEL_BXT_RT298_MACH
tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT298
@@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && SPI
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
create an alsa sound card for RT5663 + RT5514 + MAX98927.
Say Y if you have such a device.
If unsure select "N".
+
+endif
--
2.9.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
@ 2017-11-08 13:03 ` Arnd Bergmann
0 siblings, 0 replies; 15+ messages in thread
From: Arnd Bergmann @ 2017-11-08 13:03 UTC (permalink / raw)
To: Mark Brown
Cc: alsa-devel, Andy Shevchenko, Arnd Bergmann, Liam Girdwood,
Vinod Koul, linux-kernel, Pierre-Louis Bossart, Takashi Iwai,
Harsha Priya N, Naveen M
I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
and SND_SOC_INTEL_MACH=y:
ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
The problem here is that the sound/soc/intel/common/ directory
is then entered only for building modules, but the sst-acpi.o
never gets built since it depends on a built-in Kconfig symbol.
That configuration obviously makes no sense since all options
below SND_SOC_INTEL_MACH also depend on something else that
in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
solves the build error. I notice we can also consolidate the
'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
simplify it further and make sure the configuration stays sane.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
sound/soc/intel/boards/Kconfig | 26 +++++---------------------
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 5063f15b4ca4..6f754708a48c 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -1,10 +1,12 @@
config SND_SOC_INTEL_MACH
tristate "Intel Audio machine drivers"
+ depends on SND_SOC_INTEL_SST_TOPLEVEL
select SND_SOC_ACPI_INTEL_MATCH if ACPI
+if SND_SOC_INTEL_MACH
+
config SND_MFLD_MACHINE
tristate "SOC Machine Audio driver for Intel Medfield MID platform"
- depends on SND_SOC_INTEL_MACH
depends on INTEL_SCU_IPC
select SND_SOC_SN95031
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
@@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
@@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
@@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5640
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5651_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5651
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5670
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5645
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_MAX98090
select SND_SOC_TS3A227E
@@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_DA7213
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_ES8316
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
depends on SND_SST_ATOM_HIFI2_PLATFORM
select SND_SST_IPC_ACPI
@@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
config SND_SOC_INTEL_SKL_RT286_MACH
tristate "ASoC Audio driver for SKL with RT286 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT286
@@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_DA7219
@@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
config SND_SOC_INTEL_BXT_RT298_MACH
tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT298
@@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && SPI
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
create an alsa sound card for RT5663 + RT5514 + MAX98927.
Say Y if you have such a device.
If unsure select "N".
+
+endif
--
2.9.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
2017-11-08 13:03 ` Arnd Bergmann
@ 2017-11-08 14:09 ` Vinod Koul
-1 siblings, 0 replies; 15+ messages in thread
From: Vinod Koul @ 2017-11-08 14:09 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Mark Brown, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Pierre-Louis Bossart, Andy Shevchenko, Harsha Priya N,
Daniel Drake, Naveen M, alsa-devel, linux-kernel
On Wed, Nov 08, 2017 at 02:03:19PM +0100, Arnd Bergmann wrote:
> As pointed out by Pierre-Louis Bossart, the depndency I added
/s/depndency/dependency
> was broader than necessary, only Baytrail and Haswell/Broadwell
> actually need it, the others don't.
>
> At the same time, we have individual entries for the codecs
> that all have the 'select' statement but now don't need it
> any more.
>
> Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> sound/soc/intel/Kconfig | 3 ++-
> sound/soc/intel/boards/Kconfig | 5 -----
> 2 files changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
> index e18118209b75..bb8be10b8437 100644
> --- a/sound/soc/intel/Kconfig
> +++ b/sound/soc/intel/Kconfig
> @@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
> config SND_SOC_INTEL_SST_TOPLEVEL
> tristate "Intel ASoC SST drivers"
> depends on X86 || COMPILE_TEST
> - depends on DMADEVICES
> select SND_SOC_INTEL_MACH
> select SND_SOC_INTEL_COMMON
>
> config SND_SOC_INTEL_HASWELL
> tristate "Intel ASoC SST driver for Haswell/Broadwell"
> depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
> + depends on DMADEVICES
> select SND_SOC_INTEL_SST
> select SND_SOC_INTEL_SST_FIRMWARE
>
> config SND_SOC_INTEL_BAYTRAIL
> tristate "Intel ASoC SST driver for Baytrail (legacy)"
> depends on SND_SOC_INTEL_SST_TOPLEVEL
> + depends on DMADEVICES
> select SND_SOC_INTEL_SST
> select SND_SOC_INTEL_SST_FIRMWARE
>
> diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
> index 449bc8baaa60..5063f15b4ca4 100644
> --- a/sound/soc/intel/boards/Kconfig
> +++ b/sound/soc/intel/boards/Kconfig
> @@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> - depends on DMADEVICES
Per Liam this seems to be required for Haswell
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5640
> help
> @@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
> tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && GPIOLIB && I2C
> - depends on DMADEVICES
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5677
> help
> @@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
> tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> - depends on DMADEVICES
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT286
> help
> @@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
> tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> - depends on DMADEVICES
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> select SND_SOC_MAX98090
> @@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> - depends on DMADEVICES
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> select SND_SOC_RT5640
> --
> 2.9.0
>
--
~Vinod
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
@ 2017-11-08 14:09 ` Vinod Koul
0 siblings, 0 replies; 15+ messages in thread
From: Vinod Koul @ 2017-11-08 14:09 UTC (permalink / raw)
To: Arnd Bergmann
Cc: alsa-devel, Liam Girdwood, Harsha Priya N, linux-kernel,
Pierre-Louis Bossart, Takashi Iwai, Mark Brown, Naveen M,
Andy Shevchenko, Daniel Drake
On Wed, Nov 08, 2017 at 02:03:19PM +0100, Arnd Bergmann wrote:
> As pointed out by Pierre-Louis Bossart, the depndency I added
/s/depndency/dependency
> was broader than necessary, only Baytrail and Haswell/Broadwell
> actually need it, the others don't.
>
> At the same time, we have individual entries for the codecs
> that all have the 'select' statement but now don't need it
> any more.
>
> Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> sound/soc/intel/Kconfig | 3 ++-
> sound/soc/intel/boards/Kconfig | 5 -----
> 2 files changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
> index e18118209b75..bb8be10b8437 100644
> --- a/sound/soc/intel/Kconfig
> +++ b/sound/soc/intel/Kconfig
> @@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
> config SND_SOC_INTEL_SST_TOPLEVEL
> tristate "Intel ASoC SST drivers"
> depends on X86 || COMPILE_TEST
> - depends on DMADEVICES
> select SND_SOC_INTEL_MACH
> select SND_SOC_INTEL_COMMON
>
> config SND_SOC_INTEL_HASWELL
> tristate "Intel ASoC SST driver for Haswell/Broadwell"
> depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
> + depends on DMADEVICES
> select SND_SOC_INTEL_SST
> select SND_SOC_INTEL_SST_FIRMWARE
>
> config SND_SOC_INTEL_BAYTRAIL
> tristate "Intel ASoC SST driver for Baytrail (legacy)"
> depends on SND_SOC_INTEL_SST_TOPLEVEL
> + depends on DMADEVICES
> select SND_SOC_INTEL_SST
> select SND_SOC_INTEL_SST_FIRMWARE
>
> diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
> index 449bc8baaa60..5063f15b4ca4 100644
> --- a/sound/soc/intel/boards/Kconfig
> +++ b/sound/soc/intel/boards/Kconfig
> @@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> - depends on DMADEVICES
Per Liam this seems to be required for Haswell
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5640
> help
> @@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
> tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && GPIOLIB && I2C
> - depends on DMADEVICES
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5677
> help
> @@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
> tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> - depends on DMADEVICES
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT286
> help
> @@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
> tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> - depends on DMADEVICES
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> select SND_SOC_MAX98090
> @@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
> depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> - depends on DMADEVICES
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> select SND_SOC_RT5640
> --
> 2.9.0
>
--
~Vinod
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
2017-11-08 14:09 ` Vinod Koul
(?)
@ 2017-11-08 14:10 ` Arnd Bergmann
2017-11-08 14:24 ` Vinod Koul
-1 siblings, 1 reply; 15+ messages in thread
From: Arnd Bergmann @ 2017-11-08 14:10 UTC (permalink / raw)
To: Vinod Koul
Cc: Mark Brown, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Pierre-Louis Bossart, Andy Shevchenko, Harsha Priya N,
Daniel Drake, Naveen M, alsa-devel, Linux Kernel Mailing List
On Wed, Nov 8, 2017 at 3:09 PM, Vinod Koul <vinod.koul@intel.com> wrote:
> On Wed, Nov 08, 2017 at 02:03:19PM +0100, Arnd Bergmann wrote:
>> As pointed out by Pierre-Louis Bossart, the depndency I added
>
> /s/depndency/dependency
Mark, can you fix that up when applying, or should I resend?
>> diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
>> index e18118209b75..bb8be10b8437 100644
>> --- a/sound/soc/intel/Kconfig
>> +++ b/sound/soc/intel/Kconfig
>> @@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
>> config SND_SOC_INTEL_SST_TOPLEVEL
>> tristate "Intel ASoC SST drivers"
>> depends on X86 || COMPILE_TEST
>> - depends on DMADEVICES
>> select SND_SOC_INTEL_MACH
>> select SND_SOC_INTEL_COMMON
>>
>> config SND_SOC_INTEL_HASWELL
>> tristate "Intel ASoC SST driver for Haswell/Broadwell"
>> depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
>> + depends on DMADEVICES
>> select SND_SOC_INTEL_SST
>> select SND_SOC_INTEL_SST_FIRMWARE
>>
>> --- a/sound/soc/intel/boards/Kconfig
>> +++ b/sound/soc/intel/boards/Kconfig
>> @@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
>> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
>> depends on SND_SOC_INTEL_MACH
>> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
>> - depends on DMADEVICES
>
> Per Liam this seems to be required for Haswell
>
>
>> depends on SND_SOC_INTEL_HASWELL
Correct, I just moved it to a different place above, we now get it through
the SND_SOC_INTEL_HASWELL dependency.
Arnd
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
2017-11-08 13:03 ` Arnd Bergmann
@ 2017-11-08 14:11 ` Vinod Koul
-1 siblings, 0 replies; 15+ messages in thread
From: Vinod Koul @ 2017-11-08 14:11 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Mark Brown, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Pierre-Louis Bossart, Andy Shevchenko, Harsha Priya N, Naveen M,
alsa-devel, linux-kernel
On Wed, Nov 08, 2017 at 02:03:20PM +0100, Arnd Bergmann wrote:
> I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
> and SND_SOC_INTEL_MACH=y:
>
> ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
>
> The problem here is that the sound/soc/intel/common/ directory
> is then entered only for building modules, but the sst-acpi.o
> never gets built since it depends on a built-in Kconfig symbol.
>
> That configuration obviously makes no sense since all options
> below SND_SOC_INTEL_MACH also depend on something else that
> in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
>
> Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
> solves the build error. I notice we can also consolidate the
> 'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
> simplify it further and make sure the configuration stays sane.
This makes sense to me.
Acked-By: Vinod Koul <vinod.koul@intel.com>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> sound/soc/intel/boards/Kconfig | 26 +++++---------------------
> 1 file changed, 5 insertions(+), 21 deletions(-)
>
> diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
> index 5063f15b4ca4..6f754708a48c 100644
> --- a/sound/soc/intel/boards/Kconfig
> +++ b/sound/soc/intel/boards/Kconfig
> @@ -1,10 +1,12 @@
> config SND_SOC_INTEL_MACH
> tristate "Intel Audio machine drivers"
> + depends on SND_SOC_INTEL_SST_TOPLEVEL
> select SND_SOC_ACPI_INTEL_MATCH if ACPI
>
> +if SND_SOC_INTEL_MACH
> +
> config SND_MFLD_MACHINE
> tristate "SOC Machine Audio driver for Intel Medfield MID platform"
> - depends on SND_SOC_INTEL_MACH
> depends on INTEL_SCU_IPC
> select SND_SOC_SN95031
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
>
> config SND_SOC_INTEL_HASWELL_MACH
> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5640
> @@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
>
> config SND_SOC_INTEL_BDW_RT5677_MACH
> tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && GPIOLIB && I2C
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5677
> @@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
>
> config SND_SOC_INTEL_BROADWELL_MACH
> tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT286
> @@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
>
> config SND_SOC_INTEL_BYT_MAX98090_MACH
> tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
>
> config SND_SOC_INTEL_BYT_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5640
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5651_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5651
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5670
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5645
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
>
> config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_MAX98090
> select SND_SOC_TS3A227E
> @@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
>
> config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_DA7213
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
>
> config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_ES8316
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
>
> config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> select SND_SST_IPC_ACPI
> @@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
>
> config SND_SOC_INTEL_SKL_RT286_MACH
> tristate "ASoC Audio driver for SKL with RT286 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT286
> @@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
> tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_DA7219
> @@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_RT298_MACH
> tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT298
> @@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && SPI
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> create an alsa sound card for RT5663 + RT5514 + MAX98927.
> Say Y if you have such a device.
> If unsure select "N".
> +
> +endif
> --
> 2.9.0
>
--
~Vinod
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
@ 2017-11-08 14:11 ` Vinod Koul
0 siblings, 0 replies; 15+ messages in thread
From: Vinod Koul @ 2017-11-08 14:11 UTC (permalink / raw)
To: Arnd Bergmann
Cc: alsa-devel, Liam Girdwood, Harsha Priya N, linux-kernel,
Pierre-Louis Bossart, Takashi Iwai, Mark Brown, Naveen M,
Andy Shevchenko
On Wed, Nov 08, 2017 at 02:03:20PM +0100, Arnd Bergmann wrote:
> I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
> and SND_SOC_INTEL_MACH=y:
>
> ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
>
> The problem here is that the sound/soc/intel/common/ directory
> is then entered only for building modules, but the sst-acpi.o
> never gets built since it depends on a built-in Kconfig symbol.
>
> That configuration obviously makes no sense since all options
> below SND_SOC_INTEL_MACH also depend on something else that
> in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
>
> Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
> solves the build error. I notice we can also consolidate the
> 'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
> simplify it further and make sure the configuration stays sane.
This makes sense to me.
Acked-By: Vinod Koul <vinod.koul@intel.com>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> sound/soc/intel/boards/Kconfig | 26 +++++---------------------
> 1 file changed, 5 insertions(+), 21 deletions(-)
>
> diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
> index 5063f15b4ca4..6f754708a48c 100644
> --- a/sound/soc/intel/boards/Kconfig
> +++ b/sound/soc/intel/boards/Kconfig
> @@ -1,10 +1,12 @@
> config SND_SOC_INTEL_MACH
> tristate "Intel Audio machine drivers"
> + depends on SND_SOC_INTEL_SST_TOPLEVEL
> select SND_SOC_ACPI_INTEL_MATCH if ACPI
>
> +if SND_SOC_INTEL_MACH
> +
> config SND_MFLD_MACHINE
> tristate "SOC Machine Audio driver for Intel Medfield MID platform"
> - depends on SND_SOC_INTEL_MACH
> depends on INTEL_SCU_IPC
> select SND_SOC_SN95031
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
>
> config SND_SOC_INTEL_HASWELL_MACH
> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5640
> @@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
>
> config SND_SOC_INTEL_BDW_RT5677_MACH
> tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && GPIOLIB && I2C
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5677
> @@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
>
> config SND_SOC_INTEL_BROADWELL_MACH
> tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT286
> @@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
>
> config SND_SOC_INTEL_BYT_MAX98090_MACH
> tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
>
> config SND_SOC_INTEL_BYT_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5640
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5651_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5651
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5670
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5645
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
>
> config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_MAX98090
> select SND_SOC_TS3A227E
> @@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
>
> config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_DA7213
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
>
> config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_ES8316
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
>
> config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> select SND_SST_IPC_ACPI
> @@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
>
> config SND_SOC_INTEL_SKL_RT286_MACH
> tristate "ASoC Audio driver for SKL with RT286 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT286
> @@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
> tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_DA7219
> @@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_RT298_MACH
> tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT298
> @@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && SPI
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> create an alsa sound card for RT5663 + RT5514 + MAX98927.
> Say Y if you have such a device.
> If unsure select "N".
> +
> +endif
> --
> 2.9.0
>
--
~Vinod
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency
2017-11-08 14:10 ` Arnd Bergmann
@ 2017-11-08 14:24 ` Vinod Koul
0 siblings, 0 replies; 15+ messages in thread
From: Vinod Koul @ 2017-11-08 14:24 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Mark Brown, Liam Girdwood, Jaroslav Kysela, Takashi Iwai,
Pierre-Louis Bossart, Andy Shevchenko, Harsha Priya N,
Daniel Drake, Naveen M, alsa-devel, Linux Kernel Mailing List
On Wed, Nov 08, 2017 at 03:10:18PM +0100, Arnd Bergmann wrote:
> On Wed, Nov 8, 2017 at 3:09 PM, Vinod Koul <vinod.koul@intel.com> wrote:
> > On Wed, Nov 08, 2017 at 02:03:19PM +0100, Arnd Bergmann wrote:
> >> As pointed out by Pierre-Louis Bossart, the depndency I added
> >
> > /s/depndency/dependency
>
> Mark, can you fix that up when applying, or should I resend?
>
> >> diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
> >> index e18118209b75..bb8be10b8437 100644
> >> --- a/sound/soc/intel/Kconfig
> >> +++ b/sound/soc/intel/Kconfig
> >> @@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
> >> config SND_SOC_INTEL_SST_TOPLEVEL
> >> tristate "Intel ASoC SST drivers"
> >> depends on X86 || COMPILE_TEST
> >> - depends on DMADEVICES
> >> select SND_SOC_INTEL_MACH
> >> select SND_SOC_INTEL_COMMON
> >>
> >> config SND_SOC_INTEL_HASWELL
> >> tristate "Intel ASoC SST driver for Haswell/Broadwell"
> >> depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
> >> + depends on DMADEVICES
> >> select SND_SOC_INTEL_SST
> >> select SND_SOC_INTEL_SST_FIRMWARE
> >>
>
> >> --- a/sound/soc/intel/boards/Kconfig
> >> +++ b/sound/soc/intel/boards/Kconfig
> >> @@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
> >> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> >> depends on SND_SOC_INTEL_MACH
> >> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> >> - depends on DMADEVICES
> >
> > Per Liam this seems to be required for Haswell
> >
> >
> >> depends on SND_SOC_INTEL_HASWELL
>
> Correct, I just moved it to a different place above, we now get it through
> the SND_SOC_INTEL_HASWELL dependency.
Ah yes, sorry missed that part. This lgtm so:
Acked-By: Vinod Koul <vinod.koul@intel.com>
--
~Vinod
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [alsa-devel] [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
2017-11-08 13:03 ` Arnd Bergmann
(?)
(?)
@ 2017-11-08 16:39 ` Pierre-Louis Bossart
-1 siblings, 0 replies; 15+ messages in thread
From: Pierre-Louis Bossart @ 2017-11-08 16:39 UTC (permalink / raw)
To: Arnd Bergmann, Mark Brown
Cc: alsa-devel, Andy Shevchenko, Liam Girdwood, Vinod Koul,
linux-kernel, Takashi Iwai, Harsha Priya N, Naveen M
On 11/8/17 7:03 AM, Arnd Bergmann wrote:
> I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
> and SND_SOC_INTEL_MACH=y:
>
> ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
> ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
>
> The problem here is that the sound/soc/intel/common/ directory
> is then entered only for building modules, but the sst-acpi.o
> never gets built since it depends on a built-in Kconfig symbol.
>
> That configuration obviously makes no sense since all options
> below SND_SOC_INTEL_MACH also depend on something else that
> in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
>
> Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
> solves the build error. I notice we can also consolidate the
> 'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
> simplify it further and make sure the configuration stays sane.
Makes sense, thanks for the cleanup.
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> sound/soc/intel/boards/Kconfig | 26 +++++---------------------
> 1 file changed, 5 insertions(+), 21 deletions(-)
>
> diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
> index 5063f15b4ca4..6f754708a48c 100644
> --- a/sound/soc/intel/boards/Kconfig
> +++ b/sound/soc/intel/boards/Kconfig
> @@ -1,10 +1,12 @@
> config SND_SOC_INTEL_MACH
> tristate "Intel Audio machine drivers"
> + depends on SND_SOC_INTEL_SST_TOPLEVEL
> select SND_SOC_ACPI_INTEL_MATCH if ACPI
>
> +if SND_SOC_INTEL_MACH
> +
> config SND_MFLD_MACHINE
> tristate "SOC Machine Audio driver for Intel Medfield MID platform"
> - depends on SND_SOC_INTEL_MACH
> depends on INTEL_SCU_IPC
> select SND_SOC_SN95031
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
>
> config SND_SOC_INTEL_HASWELL_MACH
> tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5640
> @@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
>
> config SND_SOC_INTEL_BDW_RT5677_MACH
> tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && GPIOLIB && I2C
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT5677
> @@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
>
> config SND_SOC_INTEL_BROADWELL_MACH
> tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
> depends on SND_SOC_INTEL_HASWELL
> select SND_SOC_RT286
> @@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
>
> config SND_SOC_INTEL_BYT_MAX98090_MACH
> tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
>
> config SND_SOC_INTEL_BYT_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SST_IPC_ACPI = n
> depends on SND_SOC_INTEL_BAYTRAIL
> @@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5640_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5640
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
>
> config SND_SOC_INTEL_BYTCR_RT5651_MACH
> tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && I2C && ACPI
> select SND_SOC_RT5651
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5670
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
>
> config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_RT5645
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
>
> config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
> tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_MAX98090
> select SND_SOC_TS3A227E
> @@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
>
> config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_DA7213
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
>
> config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> select SND_SOC_ES8316
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> @@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
>
> config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
> tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && ACPI
> depends on SND_SST_ATOM_HIFI2_PLATFORM
> select SND_SST_IPC_ACPI
> @@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
>
> config SND_SOC_INTEL_SKL_RT286_MACH
> tristate "ASoC Audio driver for SKL with RT286 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT286
> @@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
>
> config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
> tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_NAU8825
> @@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
> tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_DA7219
> @@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
>
> config SND_SOC_INTEL_BXT_RT298_MACH
> tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86 && ACPI && I2C
> depends on SND_SOC_INTEL_SKYLAKE
> select SND_SOC_RT298
> @@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
>
> config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
> - depends on SND_SOC_INTEL_MACH
> depends on X86_INTEL_LPSS && I2C && SPI
> select SND_SOC_INTEL_SST
> depends on SND_SOC_INTEL_SKYLAKE
> @@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
> create an alsa sound card for RT5663 + RT5514 + MAX98927.
> Say Y if you have such a device.
> If unsure select "N".
> +
> +endif
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Applied "ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies" to the asoc tree
2017-11-08 13:03 ` Arnd Bergmann
@ 2017-11-08 18:26 ` Mark Brown
-1 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2017-11-08 18:26 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Mark Brown, Mark Brown, alsa-devel, Andy Shevchenko,
Liam Girdwood, Vinod Koul, linux-kernel, Pierre-Louis Bossart,
Takashi Iwai, Harsha Priya N, Naveen M, alsa-devel
The patch
ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
has been applied to the asoc tree at
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
>From 960115b842886999a64a87d8baadb81dce4293b4 Mon Sep 17 00:00:00 2001
From: Arnd Bergmann <arnd@arndb.de>
Date: Wed, 8 Nov 2017 14:03:20 +0100
Subject: [PATCH] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
and SND_SOC_INTEL_MACH=y:
ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
The problem here is that the sound/soc/intel/common/ directory
is then entered only for building modules, but the sst-acpi.o
never gets built since it depends on a built-in Kconfig symbol.
That configuration obviously makes no sense since all options
below SND_SOC_INTEL_MACH also depend on something else that
in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
solves the build error. I notice we can also consolidate the
'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
simplify it further and make sure the configuration stays sane.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
sound/soc/intel/boards/Kconfig | 26 +++++---------------------
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 5063f15b4ca4..6f754708a48c 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -1,10 +1,12 @@
config SND_SOC_INTEL_MACH
tristate "Intel Audio machine drivers"
+ depends on SND_SOC_INTEL_SST_TOPLEVEL
select SND_SOC_ACPI_INTEL_MATCH if ACPI
+if SND_SOC_INTEL_MACH
+
config SND_MFLD_MACHINE
tristate "SOC Machine Audio driver for Intel Medfield MID platform"
- depends on SND_SOC_INTEL_MACH
depends on INTEL_SCU_IPC
select SND_SOC_SN95031
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
@@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
@@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
@@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5640
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5651_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5651
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5670
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5645
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_MAX98090
select SND_SOC_TS3A227E
@@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_DA7213
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_ES8316
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
depends on SND_SST_ATOM_HIFI2_PLATFORM
select SND_SST_IPC_ACPI
@@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
config SND_SOC_INTEL_SKL_RT286_MACH
tristate "ASoC Audio driver for SKL with RT286 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT286
@@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_DA7219
@@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
config SND_SOC_INTEL_BXT_RT298_MACH
tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT298
@@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && SPI
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
create an alsa sound card for RT5663 + RT5514 + MAX98927.
Say Y if you have such a device.
If unsure select "N".
+
+endif
--
2.15.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Applied "ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies" to the asoc tree
@ 2017-11-08 18:26 ` Mark Brown
0 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2017-11-08 18:26 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Pierre-Louis Bossart, alsa-devel, Vinod Koul, Takashi Iwai,
Liam Girdwood, linux-kernel, Harsha Priya N, Mark Brown,
Naveen M, Andy Shevchenko
The patch
ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
has been applied to the asoc tree at
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
>From 960115b842886999a64a87d8baadb81dce4293b4 Mon Sep 17 00:00:00 2001
From: Arnd Bergmann <arnd@arndb.de>
Date: Wed, 8 Nov 2017 14:03:20 +0100
Subject: [PATCH] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies
I ran into a build error with CONFIG_SND_SOC_INTEL_COMMON=m
and SND_SOC_INTEL_MACH=y:
ERROR: "snd_soc_acpi_intel_broadwell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_haswell_machines" [sound/soc/intel/common/snd-soc-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_cherrytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
ERROR: "snd_soc_acpi_intel_baytrail_machines" [sound/soc/intel/atom/sst/snd-intel-sst-acpi.ko] undefined!
The problem here is that the sound/soc/intel/common/ directory
is then entered only for building modules, but the sst-acpi.o
never gets built since it depends on a built-in Kconfig symbol.
That configuration obviously makes no sense since all options
below SND_SOC_INTEL_MACH also depend on something else that
in turn depends on CONFIG_SND_SOC_INTEL_COMMON.
Adding a SND_SOC_INTEL_SST_TOPLEVEL dependency to SND_SOC_INTEL_MACH
solves the build error. I notice we can also consolidate the
'depends on SND_SOC_INTEL_MACH' lines by using an 'if' block to
simplify it further and make sure the configuration stays sane.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
sound/soc/intel/boards/Kconfig | 26 +++++---------------------
1 file changed, 5 insertions(+), 21 deletions(-)
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 5063f15b4ca4..6f754708a48c 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -1,10 +1,12 @@
config SND_SOC_INTEL_MACH
tristate "Intel Audio machine drivers"
+ depends on SND_SOC_INTEL_SST_TOPLEVEL
select SND_SOC_ACPI_INTEL_MATCH if ACPI
+if SND_SOC_INTEL_MACH
+
config SND_MFLD_MACHINE
tristate "SOC Machine Audio driver for Intel Medfield MID platform"
- depends on SND_SOC_INTEL_MACH
depends on INTEL_SCU_IPC
select SND_SOC_SN95031
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -17,7 +19,6 @@ config SND_MFLD_MACHINE
config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
@@ -29,7 +30,6 @@ config SND_SOC_INTEL_HASWELL_MACH
config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
@@ -39,7 +39,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
@@ -51,7 +50,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -62,7 +60,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
@@ -74,7 +71,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5640 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5640
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -87,7 +83,6 @@ config SND_SOC_INTEL_BYTCR_RT5640_MACH
config SND_SOC_INTEL_BYTCR_RT5651_MACH
tristate "ASoC Audio driver for Intel Baytrail and Baytrail-CR with RT5651 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86 && I2C && ACPI
select SND_SOC_RT5651
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -100,7 +95,6 @@ config SND_SOC_INTEL_BYTCR_RT5651_MACH
config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5672 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5670
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -113,7 +107,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5672_MACH
config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with RT5645/5650 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_RT5645
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -125,7 +118,6 @@ config SND_SOC_INTEL_CHT_BSW_RT5645_MACH
config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
tristate "ASoC Audio driver for Intel Cherrytrail & Braswell with MAX98090 & TI codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_MAX98090
select SND_SOC_TS3A227E
@@ -138,7 +130,6 @@ config SND_SOC_INTEL_CHT_BSW_MAX98090_TI_MACH
config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with DA7212/7213 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_DA7213
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -150,7 +141,6 @@ config SND_SOC_INTEL_BYT_CHT_DA7213_MACH
config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail with ES8316 codec"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
select SND_SOC_ES8316
depends on SND_SST_ATOM_HIFI2_PLATFORM
@@ -162,7 +152,6 @@ config SND_SOC_INTEL_BYT_CHT_ES8316_MACH
config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
tristate "ASoC Audio driver for Intel Baytrail & Cherrytrail platform with no codec (MinnowBoard MAX, Up)"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && ACPI
depends on SND_SST_ATOM_HIFI2_PLATFORM
select SND_SST_IPC_ACPI
@@ -174,7 +163,6 @@ config SND_SOC_INTEL_BYT_CHT_NOCODEC_MACH
config SND_SOC_INTEL_SKL_RT286_MACH
tristate "ASoC Audio driver for SKL with RT286 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT286
@@ -188,7 +176,6 @@ config SND_SOC_INTEL_SKL_RT286_MACH
config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and SSM4567 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -203,7 +190,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_SSM4567_MACH
config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
tristate "ASoC Audio driver for SKL with NAU88L25 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_NAU8825
@@ -218,7 +204,6 @@ config SND_SOC_INTEL_SKL_NAU88L25_MAX98357A_MACH
config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
tristate "ASoC Audio driver for Broxton with DA7219 and MAX98357A in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_DA7219
@@ -234,7 +219,6 @@ config SND_SOC_INTEL_BXT_DA7219_MAX98357A_MACH
config SND_SOC_INTEL_BXT_RT298_MACH
tristate "ASoC Audio driver for Broxton with RT298 I2S mode"
- depends on SND_SOC_INTEL_MACH
depends on X86 && ACPI && I2C
depends on SND_SOC_INTEL_SKYLAKE
select SND_SOC_RT298
@@ -249,7 +233,6 @@ config SND_SOC_INTEL_BXT_RT298_MACH
config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -265,7 +248,6 @@ config SND_SOC_INTEL_KBL_RT5663_MAX98927_MACH
config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
tristate "ASoC Audio driver for KBL with RT5663, RT5514 and MAX98927 in I2S Mode"
- depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && SPI
select SND_SOC_INTEL_SST
depends on SND_SOC_INTEL_SKYLAKE
@@ -279,3 +261,5 @@ config SND_SOC_INTEL_KBL_RT5663_RT5514_MAX98927_MACH
create an alsa sound card for RT5663 + RT5514 + MAX98927.
Say Y if you have such a device.
If unsure select "N".
+
+endif
--
2.15.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Applied "ASoC: Intel: improve DMADEVICES dependency" to the asoc tree
2017-11-08 13:03 ` Arnd Bergmann
@ 2017-11-08 18:26 ` Mark Brown
-1 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2017-11-08 18:26 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Mark Brown, Mark Brown, alsa-devel, Andy Shevchenko,
Liam Girdwood, Vinod Koul, linux-kernel, Pierre-Louis Bossart,
Takashi Iwai, Harsha Priya N, Naveen M, Daniel Drake, alsa-devel
The patch
ASoC: Intel: improve DMADEVICES dependency
has been applied to the asoc tree at
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
>From 326c4aa27a801918136df15d507f74968c7093fb Mon Sep 17 00:00:00 2001
From: Arnd Bergmann <arnd@arndb.de>
Date: Wed, 8 Nov 2017 14:03:19 +0100
Subject: [PATCH] ASoC: Intel: improve DMADEVICES dependency
As pointed out by Pierre-Louis Bossart, the dependency I added
was broader than necessary, only Baytrail and Haswell/Broadwell
actually need it, the others don't.
At the same time, we have individual entries for the codecs
that all have the 'select' statement but now don't need it
any more.
Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
sound/soc/intel/Kconfig | 3 ++-
sound/soc/intel/boards/Kconfig | 5 -----
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index e18118209b75..bb8be10b8437 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
config SND_SOC_INTEL_SST_TOPLEVEL
tristate "Intel ASoC SST drivers"
depends on X86 || COMPILE_TEST
- depends on DMADEVICES
select SND_SOC_INTEL_MACH
select SND_SOC_INTEL_COMMON
config SND_SOC_INTEL_HASWELL
tristate "Intel ASoC SST driver for Haswell/Broadwell"
depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
config SND_SOC_INTEL_BAYTRAIL
tristate "Intel ASoC SST driver for Baytrail (legacy)"
depends on SND_SOC_INTEL_SST_TOPLEVEL
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 449bc8baaa60..5063f15b4ca4 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
help
@@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
help
@@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
help
@@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_MAX98090
@@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_RT5640
--
2.15.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Applied "ASoC: Intel: improve DMADEVICES dependency" to the asoc tree
@ 2017-11-08 18:26 ` Mark Brown
0 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2017-11-08 18:26 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: Mark Brown
The patch
ASoC: Intel: improve DMADEVICES dependency
has been applied to the asoc tree at
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
Thanks,
Mark
>From 326c4aa27a801918136df15d507f74968c7093fb Mon Sep 17 00:00:00 2001
From: Arnd Bergmann <arnd@arndb.de>
Date: Wed, 8 Nov 2017 14:03:19 +0100
Subject: [PATCH] ASoC: Intel: improve DMADEVICES dependency
As pointed out by Pierre-Louis Bossart, the dependency I added
was broader than necessary, only Baytrail and Haswell/Broadwell
actually need it, the others don't.
At the same time, we have individual entries for the codecs
that all have the 'select' statement but now don't need it
any more.
Fixes: f7a88db6fffd ("ASoC: Intel: fix Kconfig dependencies")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-By: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
---
sound/soc/intel/Kconfig | 3 ++-
sound/soc/intel/boards/Kconfig | 5 -----
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig
index e18118209b75..bb8be10b8437 100644
--- a/sound/soc/intel/Kconfig
+++ b/sound/soc/intel/Kconfig
@@ -32,19 +32,20 @@ config SND_SOC_ACPI_INTEL_MATCH
config SND_SOC_INTEL_SST_TOPLEVEL
tristate "Intel ASoC SST drivers"
depends on X86 || COMPILE_TEST
- depends on DMADEVICES
select SND_SOC_INTEL_MACH
select SND_SOC_INTEL_COMMON
config SND_SOC_INTEL_HASWELL
tristate "Intel ASoC SST driver for Haswell/Broadwell"
depends on SND_SOC_INTEL_SST_TOPLEVEL && SND_DMA_SGBUF
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
config SND_SOC_INTEL_BAYTRAIL
tristate "Intel ASoC SST driver for Baytrail (legacy)"
depends on SND_SOC_INTEL_SST_TOPLEVEL
+ depends on DMADEVICES
select SND_SOC_INTEL_SST
select SND_SOC_INTEL_SST_FIRMWARE
diff --git a/sound/soc/intel/boards/Kconfig b/sound/soc/intel/boards/Kconfig
index 449bc8baaa60..5063f15b4ca4 100644
--- a/sound/soc/intel/boards/Kconfig
+++ b/sound/soc/intel/boards/Kconfig
@@ -19,7 +19,6 @@ config SND_SOC_INTEL_HASWELL_MACH
tristate "ASoC Audio DSP support for Intel Haswell Lynxpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5640
help
@@ -32,7 +31,6 @@ config SND_SOC_INTEL_BDW_RT5677_MACH
tristate "ASoC Audio driver for Intel Broadwell with RT5677 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && GPIOLIB && I2C
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT5677
help
@@ -43,7 +41,6 @@ config SND_SOC_INTEL_BROADWELL_MACH
tristate "ASoC Audio DSP support for Intel Broadwell Wildcatpoint"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C && I2C_DESIGNWARE_PLATFORM
- depends on DMADEVICES
depends on SND_SOC_INTEL_HASWELL
select SND_SOC_RT286
help
@@ -56,7 +53,6 @@ config SND_SOC_INTEL_BYT_MAX98090_MACH
tristate "ASoC Audio driver for Intel Baytrail with MAX98090 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_MAX98090
@@ -68,7 +64,6 @@ config SND_SOC_INTEL_BYT_RT5640_MACH
tristate "ASoC Audio driver for Intel Baytrail with RT5640 codec"
depends on SND_SOC_INTEL_MACH
depends on X86_INTEL_LPSS && I2C
- depends on DMADEVICES
depends on SND_SST_IPC_ACPI = n
depends on SND_SOC_INTEL_BAYTRAIL
select SND_SOC_RT5640
--
2.15.0
^ permalink raw reply related [flat|nested] 15+ messages in thread
end of thread, other threads:[~2017-11-08 18:26 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-08 13:03 [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency Arnd Bergmann
2017-11-08 13:03 ` Arnd Bergmann
2017-11-08 13:03 ` [PATCH 2/2] ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies Arnd Bergmann
2017-11-08 13:03 ` Arnd Bergmann
2017-11-08 14:11 ` Vinod Koul
2017-11-08 14:11 ` Vinod Koul
2017-11-08 16:39 ` [alsa-devel] " Pierre-Louis Bossart
2017-11-08 18:26 ` Applied "ASoC: Intel: improve SND_SOC_INTEL_MACH dependencies" to the asoc tree Mark Brown
2017-11-08 18:26 ` Mark Brown
2017-11-08 14:09 ` [PATCH 1/2] ASoC: Intel: improve DMADEVICES dependency Vinod Koul
2017-11-08 14:09 ` Vinod Koul
2017-11-08 14:10 ` Arnd Bergmann
2017-11-08 14:24 ` Vinod Koul
2017-11-08 18:26 ` Applied "ASoC: Intel: improve DMADEVICES dependency" to the asoc tree Mark Brown
2017-11-08 18:26 ` Mark Brown
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.