pinctrl: zynqmp: Avoid using uninitialised variable

In zynqmp_pinconf_set if param is PIN_CFG_IOSTANDARD or
PIN_CONFIG_POWER_SOURCE and zynqmp_pm_pinctrl_get_config returns an
error then value will not be assigned to when its value is tested to be
not equal to arg. Add code to only test value not equal to arg if ret is
false.

This issue was found by Smatch.

Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
Link: https://lore.kernel.org/r/20250807-pinctrl_misc-v1-4-eeb564a1b032@linaro.org
Signed-off-by: Michal Simek <michal.simek@amd.com>

diff --git a/drivers/pinctrl/pinctrl-zynqmp.c b/drivers/pinctrl/pinctrl-zynqmp.c
index 27dadbff8cacca17640436c95b1929545526052d..665b76a7d4d68b50aab89c6c04e0f4c94cd5b56e 100644 (file)
--- a/drivers/pinctrl/pinctrl-zynqmp.c
+++ b/drivers/pinctrl/pinctrl-zynqmp.c
@@ -460,14 +460,14 @@ static int zynqmp_pinconf_set(struct udevice *dev, unsigned int pin,
        case PIN_CFG_IOSTANDARD:
                param = PM_PINCTRL_CONFIG_VOLTAGE_STATUS;
                ret = zynqmp_pm_pinctrl_get_config(pin, param, &value);
-               if (arg != value)
+               if (!ret && arg != value)
                        dev_warn(dev, "Invalid IO Standard requested for pin %d\n",
                                 pin);
                break;
        case PIN_CONFIG_POWER_SOURCE:
                param = PM_PINCTRL_CONFIG_VOLTAGE_STATUS;
                ret = zynqmp_pm_pinctrl_get_config(pin, param, &value);
-               if (arg != value)
+               if (!ret && arg != value)
                        dev_warn(dev, "Invalid IO Standard requested for pin %d\n",
                                 pin);
                break;