diff options
| author | Takashi Iwai <tiwai@suse.de> | 2009-11-02 14:23:15 +0100 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2009-11-02 14:23:15 +0100 |
| commit | ad87c64f00e01a694bf90bddc2b4a6c90796d13c (patch) | |
| tree | d79ca7cd5e34a9213570dc64d77067a18119a8af /sound/pci | |
| parent | ALSA: hda: Use quirk mask for Dell Inspiron Mini9/Vostro A90 using ALC268 (diff) | |
| download | linux-ad87c64f00e01a694bf90bddc2b4a6c90796d13c.tar.xz linux-ad87c64f00e01a694bf90bddc2b4a6c90796d13c.zip | |
ALSA: hda - Don't check invalid HP pin
alc_automute_pin() might be called even if any HP pin is defined, and
it will result in verbs with NID=0.
This patch adds a check for the validity of HP widget before issuing
any verbs.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci')
| -rw-r--r-- | sound/pci/hda/patch_realtek.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 148734d16132..ff20048504b6 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -965,6 +965,8 @@ static void alc_automute_pin(struct hda_codec *codec) unsigned int nid = spec->autocfg.hp_pins[0]; int i; + if (!nid) + return; pincap = snd_hda_query_pin_caps(codec, nid); if (pincap & AC_PINCAP_TRIG_REQ) /* need trigger? */ snd_hda_codec_read(codec, nid, 0, AC_VERB_SET_PIN_SENSE, 0); |