lkml.org 
[lkml]   [2020]   [Jun]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v4 08/23] ASoC: soc-core: Fix component name_prefix parsing
From
Date


On 6/29/2020 6:08 AM, Kuninori Morimoto wrote:
> External email: Use caution opening links or attachments
>
>
> Hi Sameer
>
> Thank you for your patch
>
> # I guess there was ML registering magic until v3 ?
> # This is 1st time for me to get this patch series...

Until v3 the series was mostly about registering ASoC components. Based
on the discussions in the previous series and threads, I have updated
drivers to work with DPCM and added patches for simple-card driver to
test complete audio path. Yes, simple-card driver patches are new and
were not part of earlier series. Sorry if I was not clear in the cover
letter.

Thank you for the review.
>
>> The "prefix" can be defined in DAI link node or it can be specified as
>> part of the component node itself. Currently "sound-name-prefix" defined
>> in a component is not taking effect. Actually the property is not getting
>> parsed. It can be fixed by parsing "sound-name-prefix" property whenever
>> "prefix" is missing in DAI link Codec node.
>>
>> Signed-off-by: Sameer Pujar <spujar@nvidia.com>
> (snip)
>> @@ -1111,8 +1111,10 @@ static void soc_set_name_prefix(struct snd_soc_card *card,
>> struct snd_soc_codec_conf *map = &card->codec_conf[i];
>>
>> if (snd_soc_is_matching_component(&map->dlc, component)) {
>> - component->name_prefix = map->name_prefix;
>> - return;
>> + if (map->name_prefix) {
>> + component->name_prefix = map->name_prefix;
>> + return;
>> + }
>> }
>> }
> This is nit-pick but it can be like this ?
>
> if (snd_soc_is_matching_component(&map->dlc, component) &&
> map->name_prefix) {
> ...
> }

Sounds fine. Will update.
>
> Thank you for your help !!
>
> Best regards
> ---
> Kuninori Morimoto

\
 
 \ /
  Last update: 2020-06-29 20:53    [W:1.262 / U:0.580 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site