thermal/drivers/qcom-spmi-adc-tm5: Simplify with dev_err_probe()
authorKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Tue, 9 Jul 2024 12:59:39 +0000 (14:59 +0200)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Mon, 15 Jul 2024 11:31:41 +0000 (13:31 +0200)
Error handling in probe() can be a bit simpler with dev_err_probe().

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Bjorn Andersson <andersson@kernel.org>
Link: https://lore.kernel.org/r/20240709-thermal-probe-v1-9-241644e2b6e0@linaro.org
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
drivers/thermal/qcom/qcom-spmi-adc-tm5.c

index 756ac6842ff9107e442cc87cac68570ea3985317..7c9f4023babcfc75ae90d2c4b7370406036e8a4f 100644 (file)
@@ -829,12 +829,9 @@ static int adc_tm5_get_dt_channel_data(struct adc_tm5_chip *adc_tm,
 
        channel->iio = devm_fwnode_iio_channel_get_by_name(adc_tm->dev,
                                                           of_fwnode_handle(node), NULL);
-       if (IS_ERR(channel->iio)) {
-               ret = PTR_ERR(channel->iio);
-               if (ret != -EPROBE_DEFER)
-                       dev_err(dev, "%s: error getting channel: %d\n", name, ret);
-               return ret;
-       }
+       if (IS_ERR(channel->iio))
+               return dev_err_probe(dev, PTR_ERR(channel->iio), "%s: error getting channel\n",
+                                    name);
 
        ret = of_property_read_u32_array(node, "qcom,pre-scaling", varr, 2);
        if (!ret) {