[alsa-devel] [PATCH 1/1] ALSA: hda - Analog Devices AD1984A add HP Touchsmart model

[Miguel de Barros]

Thanks for your feedback,

here is a new version of the patch taking into account your comments. I
fully agree the mix source is not really useful.

I've put in the commit comment some info on what is included in the
patch. Tell me if more information is needed...

Miguel


Le lundi 28 septembre 2009 à 10:47 +0200, Takashi Iwai a écrit :
> At Sun, 27 Sep 2009 22:34:58 +0200,
> Miguel de Barros wrote:
> > 
> > Hello,
> > 
> > I would like to submit this patch, 
> > 
> > it will fix the bug I've submited :
> > https://bugtrack.alsa-project.org/alsa-bug/view.php?id=4614
> > 
> > 
> > Thanks,
> > 	Miguel de Barros
> > [2 0001-ALSA-hda-Analog-Devices-AD1984A-add-HP-Touchsmart.patch <text/x-patch; UTF-8 (7bit)>]
> > >From 8a106650b4f18c956ae40bf863fb3423b66d605f Mon Sep 17 00:00:00 2001
> > From: Miguel de Barros <miguel.de.barros at bluewin.ch>
> > Date: Sun, 27 Sep 2009 22:11:21 +0200
> > Subject: [PATCH 1/1] ALSA: hda - Analog Devices AD1984A add HP Touchsmart model
> >          modified:   Documentation/HD-Audio-Models.txt
> >          modified:   pci/hda/patch_analog.c
> > 
> > 
> > Signed-off-by: Miguel de Barros <miguel.de.barros at bluewin.ch>
> 
> Thanks for the patch.
> 
> Could you add a bit more description what you change and/or fix with
> this patch?  The reference to bug tracking can be also in the
> changelog text.
> OTOH, the list of modified files is superfluous since we can track
> them via git more easily.
> 
> 
> > +static struct hda_input_mux ad1984a_touchsmart_capture_source = {
> > +	.num_items = 3,
> > +	.items = {
> > +		{ "Mix", 0x3 }, /* Mixer */
> > +		{ "Mic", 0x4 }, /* port-E */
> > +		{ "Internal Mic", 0x5 }, /* Digital Mic */
> > +	},
> > +};
> > +
> > +/* switch to external mic if plugged */
> > +static void ad1984a_touchsmart_automic(struct hda_codec *codec)
> > +{
> > +	if (snd_hda_codec_read(codec, 0x1c, 0,
> > +				     AC_VERB_GET_PIN_SENSE, 0) & 0x80000000) {
> > +		snd_hda_codec_write(codec, 0x0c, 0,
> > +				     AC_VERB_SET_CONNECT_SEL, 0x4);
> > +	} else {
> > +		snd_hda_codec_write(codec, 0x0c, 0,
> > +				     AC_VERB_SET_CONNECT_SEL, 0x5);
> > +	}
> > +}
> 
> The auto-mic feature is actually conflicting with the capture_source
> setup.  That is, even if you choose "Internal Mic" manually as the
> capture source, it's changed (even without notification) to the mic
> jack when plugged in.
> 
> So, either use only the auto-mic and get rid of the manual "capture
> source" (I prefer), or disable auto-mic.  We can remove "Mix" item
> in the first case just for simplicity.
> 
> 
> Could you fix and repot?
> 
> thanks,
> 
> Takashi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-ALSA-hda-Analog-Devices-AD1984A-add-HP-Touchsmart.patch
Type: text/x-patch
Size: 0 bytes
Desc: not available
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20090928/f526f6e8/attachment-0001.patch