ASoC: tas2562: fix deprecated 'shut-down' GPIO always cleared after lookup

In tas2562_parse_dt(), the fallback lookup for the deprecated
"shut-down" GPIO property is broken due to a missing pair of braces.

The code intends to reset sdz_gpio to NULL only when the lookup
returns an error that is not -EPROBE_DEFER (so the driver gracefully
continues without a GPIO). However, without braces the statement:

    tas2562->sdz_gpio = NULL;

falls outside the IS_ERR() check and is executed unconditionally
for every path through the if block, including a successful GPIO
lookup.

This means any device using the deprecated 'shut-down' DT property
will always have sdz_gpio == NULL after probe, making the GPIO
completely non-functional.

Fix this by adding the missing braces to scope the NULL assignment
inside the IS_ERR() branch, matching the pattern already used for
the primary 'shutdown' GPIO lookup above.

Fixes: f78a97003b8b ("ASoC: tas2562: Update shutdown GPIO property")
Signed-off-by: Uday Khare <udaykhare77@gmail.com>
Link: https://patch.msgid.link/20260706153109.10953-1-udaykhare77@gmail.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/codecs/tas2562.c b/sound/soc/codecs/tas2562.c
index 2f7cfc2be970b513bc9064fea0a022bd7dde35f3..e1d62f30418ac2b9e08f0547a133edd92cddbf50 100644 (file)
--- a/sound/soc/codecs/tas2562.c
+++ b/sound/soc/codecs/tas2562.c
@@ -675,11 +675,12 @@ static int tas2562_parse_dt(struct tas2562_data *tas2562)
        if (tas2562->sdz_gpio == NULL) {
                tas2562->sdz_gpio = devm_gpiod_get_optional(dev, "shut-down",
                                                              GPIOD_OUT_HIGH);
-               if (IS_ERR(tas2562->sdz_gpio))
+               if (IS_ERR(tas2562->sdz_gpio)) {
                        if (PTR_ERR(tas2562->sdz_gpio) == -EPROBE_DEFER)
                                return -EPROBE_DEFER;
 
-               tas2562->sdz_gpio = NULL;
+                       tas2562->sdz_gpio = NULL;
+               }
        }
 
        if (tas2562->model_id == TAS2110)