ALSA: hda: cs35l41: Don't dereference fwnode handle
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Tue, 12 Jul 2022 15:35:16 +0000 (18:35 +0300)
committerTakashi Iwai <tiwai@suse.de>
Wed, 13 Jul 2022 06:28:12 +0000 (08:28 +0200)
Use acpi_fwnode_handle() instead of dereferencing an fwnode handle directly,
which is a better coding practice.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20220712153519.35692-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/cs35l41_hda.c

index 49b2543..1a1afa0 100644 (file)
@@ -347,7 +347,7 @@ static int cs35l41_hda_read_acpi(struct cs35l41_hda *cs35l41, const char *hid, i
        /* To use the same release code for all laptop variants we can't use devm_ version of
         * gpiod_get here, as CLSA010* don't have a fully functional bios with an _DSD node
         */
-       cs35l41->reset_gpio = fwnode_gpiod_get_index(&adev->fwnode, "reset", cs35l41->index,
+       cs35l41->reset_gpio = fwnode_gpiod_get_index(acpi_fwnode_handle(adev), "reset", cs35l41->index,
                                                     GPIOD_OUT_LOW, "cs35l41-reset");
 
        property = "cirrus,speaker-position";