diff mbox

[v3,01/10] ASoC: qcom: make lpass driver depend on OF

Message ID 1432245161-8990-1-git-send-email-srinivas.kandagatla@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Srinivas Kandagatla May 21, 2015, 9:52 p.m. UTC 
lpass driver only support DT so make this explicit in the kconfig.

Tested-by: Kenneth Westfield <kwestfie@codeaurora.org>
Acked-by: Kenneth Westfield <kwestfie@codeaurora.org>
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
---
 sound/soc/qcom/Kconfig | 2 ++
 1 file changed, 2 insertions(+)

Comments

Arnd Bergmann May 22, 2015, 7:24 a.m. UTC | #1 
On Thursday 21 May 2015 22:52:41 Srinivas Kandagatla wrote:
> 
>  config SND_SOC_LPASS_CPU
>         tristate
> +       depends on OF
>         select REGMAP_MMIO
>  
>  config SND_SOC_LPASS_PLATFORM
>         tristate
> +       depends on OF
>         select REGMAP_MMIO
>  
>  config SND_SOC_LPASS_IPQ806X
> 

Could you instead make the drivers compile without OF being set?

I see that patch 7 and 8 introduces another two options doing


+config SND_SOC_LPASS_APQ8016
+       tristate
+       depends on SND_SOC_QCOM
+       select SND_SOC_LPASS_CPU
+       select SND_SOC_LPASS_PLATFORM
+

+config SND_SOC_APQ8016_SBC
+       tristate "SoC Audio support for APQ8016 SBC platforms"
+       depends on (ARCH_QCOM && SND_SOC_QCOM) || COMPILE_TEST
+       select SND_SOC_LPASS_APQ8016
+       help

so you get an obvious build failure here when SND_SOC_APQ8016_SBC
gets selected with COMPILE_TEST on a platform that does not support OF.

	Arnd
Srinivas Kandagatla May 22, 2015, 11:53 a.m. UTC | #2 
Thanks for looking at this patch.

On 22/05/15 08:24, Arnd Bergmann wrote:
> On Thursday 21 May 2015 22:52:41 Srinivas Kandagatla wrote:
>>
>>   config SND_SOC_LPASS_CPU
>>          tristate
>> +       depends on OF
>>          select REGMAP_MMIO
>>
>>   config SND_SOC_LPASS_PLATFORM
>>          tristate
>> +       depends on OF
>>          select REGMAP_MMIO
>>
>>   config SND_SOC_LPASS_IPQ806X
>>
>
> Could you instead make the drivers compile without OF being set?
>
> I see that patch 7 and 8 introduces another two options doing
>
>
> +config SND_SOC_LPASS_APQ8016
> +       tristate
> +       depends on SND_SOC_QCOM
> +       select SND_SOC_LPASS_CPU
> +       select SND_SOC_LPASS_PLATFORM
> +
>
> +config SND_SOC_APQ8016_SBC
> +       tristate "SoC Audio support for APQ8016 SBC platforms"
> +       depends on (ARCH_QCOM && SND_SOC_QCOM) || COMPILE_TEST
> +       select SND_SOC_LPASS_APQ8016
> +       help
>
> so you get an obvious build failure here when SND_SOC_APQ8016_SBC
> gets selected with COMPILE_TEST on a platform that does not support OF.
>

Yes, this would fail.
I think moving of_device_id structures in drivers under #ifdef CONFIG_OF 
and removing the depends on OF would fix it.

--srini


> 	Arnd
>
>
Arnd Bergmann May 22, 2015, 1:21 p.m. UTC | #3 
On Friday 22 May 2015 12:53:44 Srinivas Kandagatla wrote:
> Thanks for looking at this patch.
> 
> On 22/05/15 08:24, Arnd Bergmann wrote:
> > On Thursday 21 May 2015 22:52:41 Srinivas Kandagatla wrote:
> >>
> >>   config SND_SOC_LPASS_CPU
> >>          tristate
> >> +       depends on OF
> >>          select REGMAP_MMIO
> >>
> >>   config SND_SOC_LPASS_PLATFORM
> >>          tristate
> >> +       depends on OF
> >>          select REGMAP_MMIO
> >>
> >>   config SND_SOC_LPASS_IPQ806X
> >>
> >
> > Could you instead make the drivers compile without OF being set?
> >
> > I see that patch 7 and 8 introduces another two options doing
> >
> >
> > +config SND_SOC_LPASS_APQ8016
> > +       tristate
> > +       depends on SND_SOC_QCOM
> > +       select SND_SOC_LPASS_CPU
> > +       select SND_SOC_LPASS_PLATFORM
> > +
> >
> > +config SND_SOC_APQ8016_SBC
> > +       tristate "SoC Audio support for APQ8016 SBC platforms"
> > +       depends on (ARCH_QCOM && SND_SOC_QCOM) || COMPILE_TEST
> > +       select SND_SOC_LPASS_APQ8016
> > +       help
> >
> > so you get an obvious build failure here when SND_SOC_APQ8016_SBC
> > gets selected with COMPILE_TEST on a platform that does not support OF.
> >
> 
> Yes, this would fail.
> I think moving of_device_id structures in drivers under #ifdef CONFIG_OF 
> and removing the depends on OF would fix it.
> 

Why?

	Arnd
Srinivas Kandagatla May 22, 2015, 1:50 p.m. UTC | #4 
On 22/05/15 14:21, Arnd Bergmann wrote:
> On Friday 22 May 2015 12:53:44 Srinivas Kandagatla wrote:
>> Thanks for looking at this patch.
>>
>> On 22/05/15 08:24, Arnd Bergmann wrote:
>>> On Thursday 21 May 2015 22:52:41 Srinivas Kandagatla wrote:
>>>>
>>>>    config SND_SOC_LPASS_CPU
>>>>           tristate
>>>> +       depends on OF
>>>>           select REGMAP_MMIO
>>>>
>>>>    config SND_SOC_LPASS_PLATFORM
>>>>           tristate
>>>> +       depends on OF
>>>>           select REGMAP_MMIO
>>>>
>>>>    config SND_SOC_LPASS_IPQ806X
>>>>
>>>
>>> Could you instead make the drivers compile without OF being set?
>>>
>>> I see that patch 7 and 8 introduces another two options doing
>>>
>>>
>>> +config SND_SOC_LPASS_APQ8016
>>> +       tristate
>>> +       depends on SND_SOC_QCOM
>>> +       select SND_SOC_LPASS_CPU
>>> +       select SND_SOC_LPASS_PLATFORM
>>> +
>>>
>>> +config SND_SOC_APQ8016_SBC
>>> +       tristate "SoC Audio support for APQ8016 SBC platforms"
>>> +       depends on (ARCH_QCOM && SND_SOC_QCOM) || COMPILE_TEST
>>> +       select SND_SOC_LPASS_APQ8016
>>> +       help
>>>
>>> so you get an obvious build failure here when SND_SOC_APQ8016_SBC
>>> gets selected with COMPILE_TEST on a platform that does not support OF.
>>>
>>
>> Yes, this would fail.
>> I think moving of_device_id structures in drivers under #ifdef CONFIG_OF
>> and removing the depends on OF would fix it.
>>
>
> Why?

Yes, you are correct we don't need the ifdef's anymore. I will send a 
fixup patch on top of what Mark has already applied.

--srini
>
> 	Arnd
>
>
diff mbox

Patch

diff --git a/sound/soc/qcom/Kconfig b/sound/soc/qcom/Kconfig
index b30c2ba..190c27d 100644
--- a/sound/soc/qcom/Kconfig
+++ b/sound/soc/qcom/Kconfig
@@ -6,10 +6,12 @@  config SND_SOC_QCOM
 
 config SND_SOC_LPASS_CPU
 	tristate
+	depends on OF
 	select REGMAP_MMIO
 
 config SND_SOC_LPASS_PLATFORM
 	tristate
+	depends on OF
 	select REGMAP_MMIO
 
 config SND_SOC_LPASS_IPQ806X