ASoC: codec: cs42l56: Convert to GPIO descriptors

of_gpio.h is deprecated, update the driver to use GPIO descriptors.
 - Use devm_gpiod_get_optional to get GPIO descriptor with default
   polarity GPIOD_OUT_LOW, set consumer name.
 - Use gpiod_set_value_cansleep to configure output value.

Checking the current driver using legacy GPIO API, the
nreset value is first output HIGH, then LOW, then HIGH.

Checking the datasheet, nreset is should be held low after power
on, when nreset is high, it starts to work.

Since the driver has been here for quite long time and no complain on
the nreset flow, still follow original flow when using GPIOD
descriptors.

Commit 944004eb56dc ("gpiolib: of: add a quirk for reset line for Cirrus
CS42L56 codec") added quirks, so the gpio request API will work as before.

Per datasheet, the DTS polarity should be GPIOD_ACTIVE_LOW. The binding
example use value 0(GPIOD_ACTIVE_HIGH) which seems wrong. There is
no in-tree DTS has the device, so all should be fine.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
Link: https://patch.msgid.link/20250506-csl42x-v3-3-e9496db544c4@nxp.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/cs42l56.c b/sound/soc/codecs/cs42l56.c
index 9c58d4f08692871c5baf2702c73367287735cf24..98fa812bc07b87fae717d601561d4ab3b9ee6bdd 100644 (file)
--- a/sound/soc/codecs/cs42l56.c
+++ b/sound/soc/codecs/cs42l56.c
@@ -8,6 +8,7 @@
  */
 
 #include <linux/delay.h>
+#include <linux/gpio/consumer.h>
 #include <linux/i2c.h>
 #include <linux/init.h>
 #include <linux/input.h>
@@ -15,7 +16,6 @@
 #include <linux/module.h>
 #include <linux/moduleparam.h>
 #include <linux/of.h>
-#include <linux/of_gpio.h>
 #include <linux/platform_device.h>
 #include <linux/pm.h>
 #include <linux/regmap.h>
@@ -35,7 +35,7 @@
 
 struct cs42l56_platform_data {
        /* GPIO for Reset */
-       unsigned int gpio_nreset;
+       struct gpio_desc *gpio_nreset;
 
        /* MICBIAS Level. Check datasheet Pg48 */
        unsigned int micbias_lvl;
@@ -1193,7 +1193,13 @@ static int cs42l56_handle_of_data(struct i2c_client *i2c_client,
        if (of_property_read_u32(np, "cirrus,hpf-left-freq", &val32) >= 0)
                pdata->hpfb_freq = val32;
 
-       pdata->gpio_nreset = of_get_named_gpio(np, "cirrus,gpio-nreset", 0);
+       pdata->gpio_nreset = devm_gpiod_get_optional(&i2c_client->dev, "cirrus,gpio-nreset",
+                                                    GPIOD_OUT_LOW);
+
+       if (IS_ERR(pdata->gpio_nreset))
+               return PTR_ERR(pdata->gpio_nreset);
+
+       gpiod_set_consumer_name(pdata->gpio_nreset, "CS42L56 /RST");
 
        return 0;
 }
@@ -1225,19 +1231,10 @@ static int cs42l56_i2c_probe(struct i2c_client *i2c_client)
        }
 
        if (cs42l56->pdata.gpio_nreset) {
-               ret = gpio_request_one(cs42l56->pdata.gpio_nreset,
-                                      GPIOF_OUT_INIT_HIGH, "CS42L56 /RST");
-               if (ret < 0) {
-                       dev_err(&i2c_client->dev,
-                               "Failed to request /RST %d: %d\n",
-                               cs42l56->pdata.gpio_nreset, ret);
-                       return ret;
-               }
-               gpio_set_value_cansleep(cs42l56->pdata.gpio_nreset, 0);
-               gpio_set_value_cansleep(cs42l56->pdata.gpio_nreset, 1);
+               gpiod_set_value_cansleep(cs42l56->pdata.gpio_nreset, 1);
+               gpiod_set_value_cansleep(cs42l56->pdata.gpio_nreset, 0);
        }
 
-
        i2c_set_clientdata(i2c_client, cs42l56);
 
        for (i = 0; i < ARRAY_SIZE(cs42l56->supplies); i++)