Re: [PATCH v3 06/12] ASoC: intel: avs: use snd_soc_register_component()

[Cezary Rojewski <cezary.rojewski@intel.com>]

On 7/6/2026 4:02 AM, Kuninori Morimoto wrote:
> It is calling snd_soc_component_initialize() / snd_soc_add_component().
> We can now use snd_soc_register_component() instead.
> 
> It is using container_of() to get avs_soc_component from component, but
> will not be able to use it when capsuling has done.
> We can now use snd_soc_component_to_priv() instead.

...

> @@ -1387,25 +1387,33 @@ int avs_register_component(struct device *dev, const char *name,
>   			   struct snd_soc_dai_driver *cpu_dais, int num_cpu_dais)
>   {

...

> -	ret = snd_soc_component_initialize(&acomp->base, drv, dev);
> +	snd_soc_component_set_name(acomp->base, comp_name);
> +	snd_soc_component_set_priv(acomp->base, acomp);
> +
> +	ret = snd_soc_register_component(acomp->base, drv, cpu_dais, num_cpu_dais);
>   	if (ret < 0)
>   		return ret;
>   
> -	return snd_soc_add_component(&acomp->base, cpu_dais, num_cpu_dais);
> +	return 0;

Nitpick: you could just:
	return snd_soc_register_component(...);

Note: alone that's not a reason for a resend and v4.  I can do that 
later when doing my quarterly driver reviews/scans.

>   }
>   
>   static struct snd_soc_dai_driver dmic_cpu_dais[] = {