Re: [PATCH] conf/ucm: GoogleNyan: Add configuration

[Tomeu Vizoso <tomeu.vizoso-ZGY8ohtN/8qB+jHODAdFcQ@public.gmane.org>]

On 01/14/2015 06:14 PM, Stephen Warren wrote:
> On 01/14/2015 09:57 AM, Tomeu Vizoso wrote:
>> Taken from the ChromeOS sources, this configuration should apply to all
>> Nyan boards from Google, so far HP Chromebook 14 (nyan-blaze) and Acer
>> Chromebook 13 (nyan-big).
> 
> How does user-space correlate the files added here:
> 
>>   src/conf/ucm/GoogleNyan/GoogleNyan.conf |  5 ++
>>   src/conf/ucm/GoogleNyan/HiFi.conf       | 83 +++++++++++++++++++++++++++++++++
>>   src/conf/ucm/GoogleNyan/Makefile.am     |  4 ++
> 
> ... with the (DT-provided) ASoC card name, or DT compatible value:
> 
>          sound {
>                  compatible = "nvidia,tegra-audio-max98090-nyan-big",
>                               "nvidia,tegra-audio-max98090";
>                  nvidia,model = "Acer Chromebook 13";
> 
> I would have expected either the file/directory names in this repo to 
> match the ASoC card name that the kernel exposes (derived from the model 
> string above), or that the files here contain some kind of indexable 
> string that contains the kernel's ASoC card name. Is that not how the 
> UCM repo usually works?
> 
> Otherwise, generic distro UCM init scripts aren't possible; UCM setup 
> scripts would need a huge table mapping ASoC card name to UCM repo 
> file/directory name, which doesn't seem scalable.

Yeah, a corresponding patch is coming to the kernel soon.

Regards,

Tomeu