Re: [alsa-devel] [PATCH 1/1] ASoC: stm32: add of dependency for stm32 drivers
From: Olivier MOYSAN
Date: Thu Feb 01 2018 - 08:22:09 EST
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@xxxxxx>
>> ---
>> 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@xxxxxxxxxxxxxxxx
>> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel