diff options
| author | Andrew Goodbody <andrew.goodbody@linaro.org> | 2025-07-22 14:40:24 +0100 |
|---|---|---|
| committer | Fabio Estevam <festevam@gmail.com> | 2025-07-24 08:00:08 -0300 |
| commit | 110a23e479d6a885ec9be322e8e63e102b194439 (patch) | |
| tree | 831565b3e610e4726d1d8acad0cd9655ec9a158c /drivers/adc/imx93-adc.c | |
| parent | e6e8c601ed78470a3fd0e04058fd53b8a17c3107 (diff) | |
imx93: adc: local variable ret should not be unsigned
Local variable ret is declared as unsigned but is used to receive the
return value of functions that return int. ret is then tested for being
negative which must always fail. Change ret to be an int.
This issue was found by Smatch.
Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
Diffstat (limited to 'drivers/adc/imx93-adc.c')
| -rw-r--r-- | drivers/adc/imx93-adc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/adc/imx93-adc.c b/drivers/adc/imx93-adc.c index f593fb6447b..d671df79f68 100644 --- a/drivers/adc/imx93-adc.c +++ b/drivers/adc/imx93-adc.c @@ -221,7 +221,7 @@ static int imx93_adc_stop(struct udevice *dev) static int imx93_adc_probe(struct udevice *dev) { struct imx93_adc_priv *adc = dev_get_priv(dev); - unsigned int ret; + int ret; ret = imx93_adc_calibration(adc); if (ret < 0) @@ -238,7 +238,7 @@ static int imx93_adc_of_to_plat(struct udevice *dev) { struct adc_uclass_plat *uc_pdata = dev_get_uclass_plat(dev); struct imx93_adc_priv *adc = dev_get_priv(dev); - unsigned int ret; + int ret; adc->regs = dev_read_addr_ptr(dev); if (adc->regs == (struct imx93_adc *)FDT_ADDR_T_NONE) { |