Re: [PATCH v2 01/13] conf/ucm: bytcr-rt5645: Use the generic bytcr/PlatformEnableSeq.conf

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




- what is the license for those UCM files? I keep getting questions
and I don't have the answer. It'd be odd to have then inherit the LGPL
license from alsa-lib, it's expected that people will have to do minor
modifications left and right and handle a number of derivatives,
e.g. because the perceived volume is too low due to some
mechanics/acoustics issue and requiring them to share is a bit
cumbersome.
Unless explicitly specified, it should inherit from the whole alsa-lib
license, i.e. LGPL.  But I guess we have no problem for re-licensing
(or dual-license) with a more relaxed one for UCM profiles, once when
they are split to an individual repo.

Since the whole projects were moved to github, we can do it easily, I
suppose.

I think we should have UCM and topology files in the same repo since they are related (control names, etc).

BTW that's also one improvement that I forgot about. alsa-lib currently bails when it cannot find a control in an UCM file, and that's painful when control names change over time. I have e.g. older UCM profiles for Chromebooks that aren't compatible any longer with the upstream kernel for HDMI paths, but work fine with analog paths. It'd be really nice to downgrade the error to a warning.


- how are we going to handle the changes for SOF, we added an "sof-"
prefix to make it explicit that the platform/DSP driver side is
different, but having a different board name will interfere with
search and includes.
Sorry, it's not clear about the question.  You need to use a different
UCM profile when the device is bound with SOF, right?

It's only different for the DSP parts, the codec settings are completely identical so I was hoping to avoid duplication of all the profiles and use some sort of run-time rule to only include the changed DSP settings. I don't have a turn-key solution to suggest, just a desire to avoid more maintenance work just because of a prefix added.


_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux