Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-12 Thread Felipe Tonello
Hi Takashi, On Mon, Aug 12, 2013 at 3:39 AM, Takashi Iwai wrote: > At Fri, 9 Aug 2013 10:36:04 -0700, > Felipe Tonello wrote: >> >> Hi Takashi, >> >> On Fri, Aug 9, 2013 at 6:52 AM, Takashi Iwai wrote: >> > At Thu, 8 Aug 2013 23:21:55 -0700, >> > Felipe F. Tonello wrote: >> >> >> >> From: "Feli

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-12 Thread Mark Brown
On Mon, Aug 12, 2013 at 12:39:10PM +0200, Takashi Iwai wrote: > But, don't get me wrong: I'm not against the action itself, the > removal of input jack support in HD-audio. I myself did propose this > once ago. Again, what's missing in your approach is the proper > process. > - Think whether r

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-12 Thread Takashi Iwai
At Fri, 9 Aug 2013 10:36:04 -0700, Felipe Tonello wrote: > > Hi Takashi, > > On Fri, Aug 9, 2013 at 6:52 AM, Takashi Iwai wrote: > > At Thu, 8 Aug 2013 23:21:55 -0700, > > Felipe F. Tonello wrote: > >> > >> From: "Felipe F. Tonello" > >> > >> This patch adds jack support for ALSA KControl. > >

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-09 Thread Felipe Tonello
Hi Takashi, On Fri, Aug 9, 2013 at 6:52 AM, Takashi Iwai wrote: > At Thu, 8 Aug 2013 23:21:55 -0700, > Felipe F. Tonello wrote: >> >> From: "Felipe F. Tonello" >> >> This patch adds jack support for ALSA KControl. >> >> This support is necessary since the new kcontrol is used by user-space >> d

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-09 Thread Takashi Iwai
At Fri, 9 Aug 2013 17:39:47 +0100, Mark Brown wrote: > > On Fri, Aug 09, 2013 at 03:52:04PM +0200, Takashi Iwai wrote: > > > Above all, the multiple indices don't work anyway with the snd_jack > > stuff in the current form. The index was introduced only for kjack, > > and for HD-audio, snd_jack

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-09 Thread Mark Brown
On Fri, Aug 09, 2013 at 03:52:04PM +0200, Takashi Iwai wrote: > Above all, the multiple indices don't work anyway with the snd_jack > stuff in the current form. The index was introduced only for kjack, > and for HD-audio, snd_jack is provided just for a compatibility > reason, thus it doesn't mat

Re: [PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-09 Thread Takashi Iwai
At Thu, 8 Aug 2013 23:21:55 -0700, Felipe F. Tonello wrote: > > From: "Felipe F. Tonello" > > This patch adds jack support for ALSA KControl. > > This support is necessary since the new kcontrol is used by user-space > daemons, such as PulseAudio(>=2.0), to do jack detection.) > > Also, HDA's

[PATCH v3 1/2] ALSA: Added jack detection KControl support

2013-08-08 Thread Felipe F. Tonello
From: "Felipe F. Tonello" This patch adds jack support for ALSA KControl. This support is necessary since the new kcontrol is used by user-space daemons, such as PulseAudio(>=2.0), to do jack detection.) Also, HDA's CONFIG_SND_HDA_INPUT_JACK is disabled because it causes to use standard snd_ja