]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
ALSA: hda/conexant: Clean up with the new GPIO helper
authorTakashi Iwai <tiwai@suse.de>
Thu, 9 Apr 2026 09:38:22 +0000 (11:38 +0200)
committerTakashi Iwai <tiwai@suse.de>
Thu, 9 Apr 2026 10:05:54 +0000 (12:05 +0200)
Use the new GPIO helper function to clean up the open code.

Merely a code refactoring, and no behavior change.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Link: https://patch.msgid.link/20260409093826.1317626-10-tiwai@suse.de
sound/hda/codecs/conexant.c

index aa726eb323ebe22f890cbb3cfaebc5f5a4cccbab..3a9717df39b4be4bdb733e3e38bede0485b2355a 100644 (file)
@@ -154,14 +154,8 @@ static void cxt_init_gpio_led(struct hda_codec *codec)
        struct conexant_spec *spec = codec->spec;
        unsigned int mask = spec->gpio_mute_led_mask | spec->gpio_mic_led_mask;
 
-       if (mask) {
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_MASK,
-                                   mask);
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_DIRECTION,
-                                   mask);
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_DATA,
-                                   spec->gpio_led);
-       }
+       if (mask)
+               snd_hda_codec_set_gpio(codec, mask, mask, spec->gpio_led, 0);
 }
 
 static void cx_fixup_headset_recog(struct hda_codec *codec)
@@ -775,9 +769,7 @@ static void cxt_setup_gpio_unmute(struct hda_codec *codec,
 {
        if (gpio_mute_mask) {
                // set gpio data to 0.
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_DATA, 0);
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_MASK, gpio_mute_mask);
-               snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_DIRECTION, gpio_mute_mask);
+               snd_hda_codec_set_gpio(codec, gpio_mute_mask, gpio_mute_mask, 0, 0);
                snd_hda_codec_write(codec, 0x01, 0, AC_VERB_SET_GPIO_STICKY_MASK, 0);
        }
 }