[alsa-devel] [RFC PATCH] HDA: Generic input jack handling
david.henningsson at canonical.com
Sun Oct 9 10:38:57 CEST 2011
On 10/08/2011 09:11 AM, Takashi Iwai wrote:
> At Fri, 07 Oct 2011 18:04:37 +0200,
> David Henningsson wrote:
>> On 10/07/2011 05:27 PM, Takashi Iwai wrote:
>>> That being said, I'm inclined to delay it for 3.3. Now is so close to
>>> the merge window and we don't need to rush for a refactoring patch.
>>> It can be done more safely in the early development cycle.
>>> Sorry if it disappoints you.
>> This is much more than a refactoring patch - Realtek has no input jacks
>> for everything but headphones (and mics only if autoswitch is enabled),
>> so this is a good new feature. And my planned next step was to add it to
>> patch_via, which currently does not have input jacks at all.
>> To give you some background here. As you might know, I've written
>> patches for PulseAudio to use these input jacks, that's why they're
>> suddenly so much more important. Also, it is not certain whether the
>> next release of Ubuntu (12.04, released April next year) will use kernel
>> 3.2 or 3.3. Should it be 3.3, this is not a problem, but if it is 3.2,
>> it would be better if all this was working upstream. Should it not be
>> working, I'll need to apply it for Ubuntu only instead.
> Well, then really you shouldn't be rushing too much.
> As mentioned, the API function will be needed to be changed because
> it's wrong to call with jack_type. So, if you merge it in 3.2, you'll
> get anyway API/ABI incompatibility for further changes.
> That is, if it were a self-contained patch only in patch_realtek.c, I
> wouldn't mind too much. But it's an addition of API in HD-audio code
> code, so I do care about it.
I don't like copy-pasting the same code into several vendor's patch
files when the function should really be generic. Therefore I don't want
to make several self-contained patches.
So if you feel uncomfortable with having the generic function in 3.2,
queue it up for 3.3 (after I've changed the jack_type stuff to your
preference) and I'll try to backport it into Ubuntu's 3.2 kernel, should
it become necessary.
David Henningsson, Canonical Ltd.
More information about the Alsa-devel