[alsa-devel] [PATCH 2/2] ALSA: hda - Remove NO_PRESENCE bit override for Dell 1420n Laptop

[Takashi Iwai]

At Tue, 23 Jul 2013 15:50:29 +0800,
Raymond Yau wrote:
> 
> > The quirk for Dell laptops with STAC9228 overrides the pin default
> > config of NID 0x0f to the value with AC_DEFCFG_MISC_NO_PRESENCE bit
> > on.  I'm not quite sure why this was done so, but can guess that this
> > was introduced for avoiding this to be muted by another headphone
> > plug.  Now, after transition to the generic parser, this workaround
> > rather causes a problem (notably as unexpected speaker mutes) because
> > the pin is seen as if it's always plugged in.
> >
> 
> 
> Is there any side effect when the driver perform pin fixup on line out pin
> complex which pincap does not support HP to headphone phone complex
> 
> Refer to STAC9228 datasheet , port F does not support headphone out

Yes, but we checked that Windows show in the same way (the secondary
headphone), so we just take it as is.


Takashi

> 
> 
> • Ports A, B, and D support
> • Headphone Out
> • Line Out
> • Line In
> • Microphone with
> 
> 
> Node 0x0f [Pin Complex] wcaps 0x400181: Stereo
>   Control: name="Front Headphone Front Phantom Jack", index=0, device=0
>   Pincap 0x00001737: IN OUT Detect Trigger ImpSense
>     Vref caps: HIZ 50 GRD 80
>   Pin Default 0x02011020: [Jack] Line Out at Ext Front
>     Conn = 1/8, Color = Black
>     DefAssociation = 0x2, Sequence = 0x0
>   Pin-ctls: 0x40: OUT VREF_HIZ
>   Unsolicited: tag=00, enabled=0
>   Connection: 1
>      0x050
> 
> 
>  diff --git a/sound/pci/hda/patch_sigmatel.c
> b/sound/pci/hda/patch_sigmatel.c
> 
> > index 766e567..92b9b43 100644
> > --- a/sound/pci/hda/patch_sigmatel.c
> > +++ b/sound/pci/hda/patch_sigmatel.c
> > @@ -3233,7 +3233,7 @@ static const struct hda_fixup stac927x_fixups[] = {
> >                         /* configure the analog microphone on some laptops
> > */
> >                         { 0x0c, 0x90a79130 },
> >                         /* correct the front output jack as a hp out */
> > -                       { 0x0f, 0x0227011f },
> > +                       { 0x0f, 0x0221101f },
> >                         /* correct the front input jack as a mic */
> >                         { 0x0e, 0x02a79130 },
> >                         {}
> >
> >