* [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers
@ 2018-02-01 8:54 Olivier Moysan
2018-02-01 9:10 ` [alsa-devel] " Ladislav Michl
0 siblings, 1 reply; 4+ messages in thread
From: Olivier Moysan @ 2018-02-01 8:54 UTC (permalink / raw)
To: lgirdwood, broonie, perex, tiwai, mcoquelin.stm32,
alexandre.torgue, alsa-devel, linux-arm-kernel, kernel,
linux-kernel, olivier.moysan
Cc: arnaud.pouliquen, benjamin.gaignard
Add of dependency for STM32 ASoC drivers.
DFSDM of dependency is already inherited
from STM32_DFSDM_ADC dependency.
Signed-off-by: olivier moysan <olivier.moysan@st.com>
---
sound/soc/stm/Kconfig | 3 +++
1 file changed, 3 insertions(+)
diff --git a/sound/soc/stm/Kconfig b/sound/soc/stm/Kconfig
index 3ad881f..b5375f9 100644
--- a/sound/soc/stm/Kconfig
+++ b/sound/soc/stm/Kconfig
@@ -3,6 +3,7 @@ menu "STMicroelectronics STM32 SOC audio support"
config SND_SOC_STM32_SAI
tristate "STM32 SAI interface (Serial Audio Interface) support"
depends on ARCH_STM32 || COMPILE_TEST
+ depends on OF
depends on SND_SOC
select SND_SOC_GENERIC_DMAENGINE_PCM
select REGMAP_MMIO
@@ -12,6 +13,7 @@ config SND_SOC_STM32_SAI
config SND_SOC_STM32_I2S
tristate "STM32 I2S interface (SPI/I2S block) support"
depends on ARCH_STM32 || COMPILE_TEST
+ depends on OF
depends on SND_SOC
select SND_SOC_GENERIC_DMAENGINE_PCM
select REGMAP_MMIO
@@ -21,6 +23,7 @@ config SND_SOC_STM32_I2S
config SND_SOC_STM32_SPDIFRX
tristate "STM32 S/PDIF receiver (SPDIFRX) support"
depends on ARCH_STM32 || COMPILE_TEST
+ depends on OF
depends on SND_SOC
select SND_SOC_GENERIC_DMAENGINE_PCM
select REGMAP_MMIO
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [alsa-devel] [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers
2018-02-01 8:54 [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers Olivier Moysan
@ 2018-02-01 9:10 ` Ladislav Michl
2018-02-01 13:19 ` Olivier MOYSAN
0 siblings, 1 reply; 4+ messages in thread
From: Ladislav Michl @ 2018-02-01 9:10 UTC (permalink / raw)
To: Olivier Moysan
Cc: lgirdwood, broonie, perex, tiwai, mcoquelin.stm32,
alexandre.torgue, alsa-devel, linux-arm-kernel, kernel,
linux-kernel, arnaud.pouliquen, benjamin.gaignard
On Thu, Feb 01, 2018 at 09:54:41AM +0100, Olivier Moysan wrote:
> Add of dependency for STM32 ASoC drivers.
> DFSDM of dependency is already inherited
> from STM32_DFSDM_ADC dependency.
>
> Signed-off-by: olivier moysan <olivier.moysan@st.com>
> ---
> sound/soc/stm/Kconfig | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/sound/soc/stm/Kconfig b/sound/soc/stm/Kconfig
> index 3ad881f..b5375f9 100644
> --- a/sound/soc/stm/Kconfig
> +++ b/sound/soc/stm/Kconfig
> @@ -3,6 +3,7 @@ menu "STMicroelectronics STM32 SOC audio support"
> config SND_SOC_STM32_SAI
> tristate "STM32 SAI interface (Serial Audio Interface) support"
> depends on ARCH_STM32 || COMPILE_TEST
> + depends on OF
depends on (ARCH_STM32 && OF) || COMPILE_TEST ?
> depends on SND_SOC
> select SND_SOC_GENERIC_DMAENGINE_PCM
> select REGMAP_MMIO
> @@ -12,6 +13,7 @@ config SND_SOC_STM32_SAI
> config SND_SOC_STM32_I2S
> tristate "STM32 I2S interface (SPI/I2S block) support"
> depends on ARCH_STM32 || COMPILE_TEST
> + depends on OF
> depends on SND_SOC
> select SND_SOC_GENERIC_DMAENGINE_PCM
> select REGMAP_MMIO
> @@ -21,6 +23,7 @@ config SND_SOC_STM32_I2S
> config SND_SOC_STM32_SPDIFRX
> tristate "STM32 S/PDIF receiver (SPDIFRX) support"
> depends on ARCH_STM32 || COMPILE_TEST
> + depends on OF
> depends on SND_SOC
> select SND_SOC_GENERIC_DMAENGINE_PCM
> select REGMAP_MMIO
> --
> 1.9.1
>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [alsa-devel] [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers
2018-02-01 9:10 ` [alsa-devel] " Ladislav Michl
@ 2018-02-01 13:19 ` Olivier MOYSAN
2018-02-01 14:24 ` Mark Brown
0 siblings, 1 reply; 4+ messages in thread
From: Olivier MOYSAN @ 2018-02-01 13:19 UTC (permalink / raw)
To: Ladislav Michl
Cc: lgirdwood, broonie, perex, tiwai, mcoquelin.stm32,
Alexandre TORGUE, alsa-devel, linux-arm-kernel, kernel,
linux-kernel, Arnaud POULIQUEN, Benjamin GAIGNARD
hello,
On 02/01/2018 10:10 AM, Ladislav Michl wrote:
> On Thu, Feb 01, 2018 at 09:54:41AM +0100, Olivier Moysan wrote:
>> Add of dependency for STM32 ASoC drivers.
>> DFSDM of dependency is already inherited
>> from STM32_DFSDM_ADC dependency.
>>
>> Signed-off-by: olivier moysan <olivier.moysan@st.com>
>> ---
>> sound/soc/stm/Kconfig | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/sound/soc/stm/Kconfig b/sound/soc/stm/Kconfig
>> index 3ad881f..b5375f9 100644
>> --- a/sound/soc/stm/Kconfig
>> +++ b/sound/soc/stm/Kconfig
>> @@ -3,6 +3,7 @@ menu "STMicroelectronics STM32 SOC audio support"
>> config SND_SOC_STM32_SAI
>> tristate "STM32 SAI interface (Serial Audio Interface) support"
>> depends on ARCH_STM32 || COMPILE_TEST
>> + depends on OF
>
> depends on (ARCH_STM32 && OF) || COMPILE_TEST ?
>
I can find in many configs "depends on OF && (ARCH_X || COMPILE_TEST)"
This seems reasonable to me, as the driver always requires OF
dependency, regardless the compilation context.
In fact, I cannot see why OF would not have to be selected
for COMPILE_TEST.
BRs
>> depends on SND_SOC
>> select SND_SOC_GENERIC_DMAENGINE_PCM
>> select REGMAP_MMIO
>> @@ -12,6 +13,7 @@ config SND_SOC_STM32_SAI
>> config SND_SOC_STM32_I2S
>> tristate "STM32 I2S interface (SPI/I2S block) support"
>> depends on ARCH_STM32 || COMPILE_TEST
>> + depends on OF
>> depends on SND_SOC
>> select SND_SOC_GENERIC_DMAENGINE_PCM
>> select REGMAP_MMIO
>> @@ -21,6 +23,7 @@ config SND_SOC_STM32_I2S
>> config SND_SOC_STM32_SPDIFRX
>> tristate "STM32 S/PDIF receiver (SPDIFRX) support"
>> depends on ARCH_STM32 || COMPILE_TEST
>> + depends on OF
>> depends on SND_SOC
>> select SND_SOC_GENERIC_DMAENGINE_PCM
>> select REGMAP_MMIO
>> --
>> 1.9.1
>>
>> _______________________________________________
>> Alsa-devel mailing list
>> Alsa-devel@alsa-project.org
>> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [alsa-devel] [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers
2018-02-01 13:19 ` Olivier MOYSAN
@ 2018-02-01 14:24 ` Mark Brown
0 siblings, 0 replies; 4+ messages in thread
From: Mark Brown @ 2018-02-01 14:24 UTC (permalink / raw)
To: Olivier MOYSAN
Cc: Ladislav Michl, lgirdwood, perex, tiwai, mcoquelin.stm32,
Alexandre TORGUE, alsa-devel, linux-arm-kernel, kernel,
linux-kernel, Arnaud POULIQUEN, Benjamin GAIGNARD
[-- Attachment #1: Type: text/plain, Size: 747 bytes --]
On Thu, Feb 01, 2018 at 01:19:28PM +0000, Olivier MOYSAN wrote:
> On 02/01/2018 10:10 AM, Ladislav Michl wrote:
> > depends on (ARCH_STM32 && OF) || COMPILE_TEST ?
> I can find in many configs "depends on OF && (ARCH_X || COMPILE_TEST)"
> This seems reasonable to me, as the driver always requires OF
> dependency, regardless the compilation context.
> In fact, I cannot see why OF would not have to be selected
> for COMPILE_TEST.
There are some OF functions that are stubbed out when OF is disabled and
some that aren't, if the driver is using the more obscure APIs that
aren't stubbed it needs the hard dependency but if it's using the other
bits that do have the stubs then it can be compile tested without OF
being abled.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-02-01 14:26 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-01 8:54 [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers Olivier Moysan
2018-02-01 9:10 ` [alsa-devel] " Ladislav Michl
2018-02-01 13:19 ` Olivier MOYSAN
2018-02-01 14:24 ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).