[meta-intel] [PATCH 3/6] alsa-state: ALSA config for intel-corei7-64 BSP

[Hart, Darren]

On 3/5/14, 20:56, "Kamble, Nitin A" <nitin.a.kamble at intel.com> wrote:

>
>On 3/5/2014 7:47 PM, Hart, Darren wrote:
>> On 3/5/14, 17:58, "Kamble, Nitin A" <nitin.a.kamble at intel.com> wrote:
>>
>>> From: Nitin A Kamble <nitin.a.kamble at intel.com>
>>>
>>> Provide ALSA config file for intel-corei7-64 BSP which enables
>>> ALSA sound on hardware like NUC.
>> This is HIGHLY machine specific unless I am sorely mistaken, this should
>> not be made part of a generic BSP.
>This is a static file, but there is another initscripts commit in this
>pull request, which changes this file as per the machine needs.
>

I saw the initiscript patch which dynamically rewrites asound.conf. It
seems to me there are a great deal of variables it can't really account
for. But independent of that, I don't think we want to use initscripts
that run at every boot to dynamically overwrite configuration files.

We should have a higher level design and planning discussion before we
start creating one-off solutions for each config file. I suspect if we
decide to have some kind of a runtime config, we'll want a run-once
mechanism, or possibly something that runs in the installer, not at every
boot.

Did you already have some broader scheme in mind as to how to address the
larger problem of supporting multiple platforms with a single image.

-- 
Darren Hart
Yocto Project - Linux Kernel
Intel Open Source Technology Center