Re: [PATCH v3 00/12] ASoC: add and use new snd_soc_register_component()

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

On 7/6/2026 4:01 AM, Kuninori Morimoto wrote:
> Hi
> 
> Current component has component->name. But snd_soc_register_component()
> user can't setup it, because component itself is alloced in that function.
> 
> So, to setup it, user need to use snd_soc_component_initialize() /
> snd_soc_add_component() directly instead of using
> snd_soc_register_component().
> 
> In the same time, Component will be capsuled soon.
> 
> Let's tidyup around here.
> All strange code can be gone if we have both below style.
> 
> Normal case
> 	snd_soc_register_component(dev, ...);
> 
> Want to setup case
> 	component = snd_soc_component_alloc(...);
> 
> 	snd_soc_component_set_xxx(component, ...);
> 	snd_soc_component_set_xxx(component, ...);
> 
> (A)	snd_soc_register_component(component, ...);
> 
> This patch-set adds new snd_soc_register_component() which allows to use
> component (A).
> 
> Originally, this patch-set was posted as
> 	v1: ASoC: intel: use component_name for Component name
> 	v2: ASoC: add [devm_]snd_soc_register_component_with_name()
> 
> Link: https://lore.kernel.org/r/87fr29esth.wl-kuninori.morimoto.gx@renesas.com
> Link: https://lore.kernel.org/r/87v7ayxk3s.wl-kuninori.morimoto.gx@renesas.com
Thank you for good discussion, addressing some and providing technical 
arguments for those comments we didn't square on.  I've spent quite a 
while analyzing the subject and the outcome is:  Is there something I 
can help with, to make the conversion easier?

And obviously:
Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com>