diff options
author | Takashi Iwai <tiwai@suse.de> | 2012-04-19 12:04:03 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2012-04-19 12:04:03 +0200 |
commit | 3e843196c697ee2c319d96e861980fb4c3e04e24 (patch) | |
tree | 9a065c1dfe64353d9954bafd9ffbf647c8d783f0 /sound/pci | |
parent | 118cb4a408e1c4021ac85d6c05da66bb6f57e556 (diff) | |
download | op-kernel-dev-3e843196c697ee2c319d96e861980fb4c3e04e24.zip op-kernel-dev-3e843196c697ee2c319d96e861980fb4c3e04e24.tar.gz |
ALSA: hda/sigmatel - Fix inverted mute LED
While refactoring the mute-LED handling for HP laptops, I messed up
the polarity check in a wrong way. The red (or the mute-LED if any)
should appear in the muted state, corresponding to GPIO on.
Reported-by: Mikko Vinni <mmvinni@yahoo.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci')
-rw-r--r-- | sound/pci/hda/patch_sigmatel.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.c index 33a9946..4742cac 100644 --- a/sound/pci/hda/patch_sigmatel.c +++ b/sound/pci/hda/patch_sigmatel.c @@ -5063,12 +5063,11 @@ static void stac92xx_update_led_status(struct hda_codec *codec, int enabled) if (spec->gpio_led_polarity) muted = !muted; - /*polarity defines *not* muted state level*/ if (!spec->vref_mute_led_nid) { if (muted) - spec->gpio_data &= ~spec->gpio_led; /* orange */ + spec->gpio_data |= spec->gpio_led; else - spec->gpio_data |= spec->gpio_led; /* white */ + spec->gpio_data &= ~spec->gpio_led; stac_gpio_set(codec, spec->gpio_mask, spec->gpio_dir, spec->gpio_data); } else { |