[pulseaudio-discuss] [PATCH] Add HDMI 5.1 and 7.1 profiles to default.conf as well

[Alexander E. Patrakov]

2014-07-27 17:24 GMT+06:00 Tanu Kaskinen <tanu.kaskinen at linux.intel.com>:
> On Tue, 2014-07-22 at 15:11 +0200, David Henningsson wrote:
>>
>> On 2014-07-13 16:07, Tanu Kaskinen wrote:
>> > If they're added to default.conf, they should be added to
>> > extra-hdmi.conf (it only has 5.1), force-speaker.conf and
>> > force-speaker-and-int-mic.conf too. And before you say anything, yes,
>> > this duplication is bad, feel free to fix it. I believe it would work if
>> > the other files would include default.conf, the syntax is
>> >
>> > .include default.conf
>>
>> Slightly off-topic:
>>
>> Not sure it would work for force-speaker.conf and
>> force-speaker-and-int-mic.conf (e g analog-stereo would be present in
>> both, and they're different), and I didn't know about this syntax feature.
>
> You're right, default.conf can't be included as is. We could have a
> common.conf that contains the parts that are the same in every profile
> set. analog-stereo would not be included in common.conf.
>
>> Anyway, these profile files are only needed for Ubuntu 12.04 3.2 kernel
>> and the 3.4 LTS kernel. We could consider removing that workaround at
>> some time...
>
> Oh, ok, I didn't know they're unnecessary on newer kernels. Yes, we
> should drop them at some point, but I guess now is not the time.

Due to these complications, I don't think that the entire "include"
idea makes sense. I will try an alternative approach - a shell script
that writes the conf files that share the common parts.

-- 
Alexander E. Patrakov