[alsa-devel] [External] Re: alsa-lib: Add ucm support for whiskeylake sof-skl_hda_card audio

[Jaroslav Kysela]

Dne 27. 09. 19 v 12:07 Jaska Uimonen napsal(a):
> On Tue, 2019-09-24 at 13:53 +0200, Jaroslav Kysela wrote:
>> Dne 19. 09. 19 v 17:15 Pierre-Louis Bossart napsal(a):
>>> On 9/19/19 9:54 AM, Mark Pearson wrote:
>>>>>
>>>>> Indeed UCM is required for all cases where SOF and PulseAudio
>>>>> are used.
>>>>>
>>>>> Our thinking was however to add this UCM file to the new
>>>>> repository outside
>>>>> of alsa-lib [1]. There is an on-going thread started by
>>>>> Jaroslav to move those
>>>>> files and relicense them as BSD-3-Clause [2]
>>>>>
>>>>> [1]
>>>>> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-
>>>>> July/153257.html
>>>>> [2]
>>>>> https://mailman.alsa-project.org/pipermail/alsa-devel/2019-
>>>>> September/155246.html
>>>>
>>>> Thanks Pierre.
>>>>
>>>> Do we have any approximate timelines on when and how this will
>>>> happen? (I'm new to this)
>>>>
>>>> One of my main aims is that we have a customer using Debian and
>>>> one of our platforms that require this change - I need to make
>>>> sure I understand how this would roll out and what actions they
>>>> need to take in the meantime if it's not going to be available in
>>>> Debian.
>>>
>>> I think the first order would be to have the file cleaned-up, with
>>> its 
>>> Intel origin clearly stated with a signed-off-by tag.
>>>
>>> Then once this is done, the Debian package creation needs to be
>>> handled 
>>> (using either the ALSA repo or the cloned version on SOF GitHub).
>>> I 
>>> don't have any experience with Debian packages so can't really
>>> comment 
>>> on the effort it would take.
>>
>> I did some cleanups here:
>>
>> https://github.com/alsa-project/alsa-ucm-conf/commit/f796f0852a097e23
>> 8fa9f5efb174e95b5ee6c8b7
>>
>> Pierre, could you confirm the original source and are you ok with
>> that?
> 
> Cleanup looks fine to me, we should add still UCM "PlaybackVolume" and
> "CaptureVolume" settings, because otherwise Pulseaudio will use SW
> volume only. This will make for example HDA led quirks useless...
> (and actually CaptureVolume and PlaybackVolume in pulseaudio ucm
> support is still not integrated, hopefully soon). Defining Capture and
> PlaybackVolume should not do any harm currently for user space.
> 
> I can do that, Jaroslav you want PR against github or patches here 
> to mailing list?

As you wish. Both ways are acceptable for me. Note that I did another cleanup
for 'Bass Speaker' for Carbon X1 7th and merged 'import' branch to 'master'
branch on github (so do the PR against master, if you like).

				Thanks,
					Jaroslav

-- 
Jaroslav Kysela <perex at perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.