Re: [PATCH 01/15] ASoC: soc-core: support snd_soc_dai_link_component for aux_dev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On 8/10/19 12:16 PM, Jarkko Nikula wrote:
> Hi
> 
> On 8/8/19 8:52 AM, Kuninori Morimoto wrote:
>> From: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
>>
>> To find aux_dev, ASoC is using .name, codec_name, codec_of_node.
>> Here, .name is used to fallback in case of no codec.
>>
>> But, we already have this kind of component finding method by
>> snd_soc_dai_link_component and soc_find_component().
>> We shouldn't have duplicated implementation to do same things.
>> This patch adds snd_soc_dai_link_component support to finding aux_dev.
>>
>> Now, no driver is using only .name.
>> All drivers are using codec_name and/or codec_of_node.
>> This means no driver is finding component from .name so far.
>> (Actually almost all drivers are using .name as just "device name",
>>  not for finding component...)
>>
>> This patch
>> 1) add snd_soc_dai_link_component support for aux_dev. legacy style will
>>    be removed if all drivers are switched to new style.
>> 2) try to find component via snd_soc_dai_link_component.
>>    Then, it doesn't try to find via .name, because no driver is using
>>    it so far.
>>
>> Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>
>> ---
>>  include/sound/soc.h  |  7 +++++++
>>  sound/soc/soc-core.c | 36 ++++++++++--------------------------
>>  2 files changed, 17 insertions(+), 26 deletions(-)
...

> Bike-shedding: maybe comment here should say legacy style binding etc? I
> know these lines are removed by patch 13/15 but here yet comment is not
> valid :-)
> 
I forgot to add

Tested-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxx>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux