]> www.infradead.org Git - users/hch/misc.git/commitdiff
ALSA: hda/intel: Use guard() for mutex locks
authorTakashi Iwai <tiwai@suse.de>
Wed, 27 Aug 2025 07:28:58 +0000 (09:28 +0200)
committerTakashi Iwai <tiwai@suse.de>
Fri, 29 Aug 2025 09:52:15 +0000 (11:52 +0200)
Replace the manual mutex lock/unlock pairs with guard().

Only code refactoring, and no behavior change.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Link: https://patch.msgid.link/20250827072916.31933-19-tiwai@suse.de
sound/hda/controllers/intel.c

index 1bb3ff55b1151db8d12aaef184acd273b67714f6..1e13678428d26a1b51194f47f116a2fad9752ec4 100644 (file)
@@ -915,17 +915,17 @@ static void azx_shutdown_chip(struct azx *chip)
 static void azx_add_card_list(struct azx *chip)
 {
        struct hda_intel *hda = container_of(chip, struct hda_intel, chip);
-       mutex_lock(&card_list_lock);
+
+       guard(mutex)(&card_list_lock);
        list_add(&hda->list, &card_list);
-       mutex_unlock(&card_list_lock);
 }
 
 static void azx_del_card_list(struct azx *chip)
 {
        struct hda_intel *hda = container_of(chip, struct hda_intel, chip);
-       mutex_lock(&card_list_lock);
+
+       guard(mutex)(&card_list_lock);
        list_del_init(&hda->list);
-       mutex_unlock(&card_list_lock);
 }
 
 /* trigger power-save check at writing parameter */
@@ -942,7 +942,7 @@ static int __maybe_unused param_set_xint(const char *val, const struct kernel_pa
        if (pm_blacklist > 0)
                return 0;
 
-       mutex_lock(&card_list_lock);
+       guard(mutex)(&card_list_lock);
        list_for_each_entry(hda, &card_list, list) {
                chip = &hda->chip;
                if (!hda->probe_continued || chip->disabled ||
@@ -950,7 +950,6 @@ static int __maybe_unused param_set_xint(const char *val, const struct kernel_pa
                        continue;
                snd_hda_set_power_save(&chip->bus, power_save * 1000);
        }
-       mutex_unlock(&card_list_lock);
        return 0;
 }