diff options
| author | Takashi Iwai <tiwai@suse.de> | 2012-04-19 10:04:03 (GMT) |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2012-04-19 10:04:03 (GMT) |
| commit | 3e843196c697ee2c319d96e861980fb4c3e04e24 (patch) | |
| tree | 9a065c1dfe64353d9954bafd9ffbf647c8d783f0 | |
| parent | 118cb4a408e1c4021ac85d6c05da66bb6f57e556 (diff) | |
| download | linux-fsl-qoriq-3e843196c697ee2c319d96e861980fb4c3e04e24.tar.xz | |
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>
| -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 { |