iio: adc: qcom-spmi-adc5: Tidy up adc5_get_fw_data() error messages
authorBjorn Andersson <bjorn.andersson@oss.qualcomm.com>
Fri, 4 Oct 2024 23:11:01 +0000 (16:11 -0700)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 5 Oct 2024 11:19:01 +0000 (12:19 +0100)
In the event that no channels (child nodes) are defined, the adc5 driver
will provide a generic error message indicating that adc5_get_fw_data()
returned -EINVAL. In all other error cases we get two error messages,
one helpful and the generic one.

Add a specific error message for the no channels case, and drop the
generic one, in order to improve the generates log prints in both cases.

Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com>
Link: https://patch.msgid.link/20241004-spmi-adc5-no-channel-error-v1-1-1a43d13ae967@oss.qualcomm.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/qcom-spmi-adc5.c

index 9b69f40beed8eacf3e8e7b4c1cac4c0c5a57164f..af3c2f659f5e90e330fc272aa6741472fc488d57 100644 (file)
@@ -830,7 +830,7 @@ static int adc5_get_fw_data(struct adc5_chip *adc)
 
        adc->nchannels = device_get_child_node_count(adc->dev);
        if (!adc->nchannels)
-               return -EINVAL;
+               return dev_err_probe(adc->dev, -EINVAL, "no channels defined\n");
 
        adc->iio_chans = devm_kcalloc(adc->dev, adc->nchannels,
                                       sizeof(*adc->iio_chans), GFP_KERNEL);
@@ -903,7 +903,7 @@ static int adc5_probe(struct platform_device *pdev)
 
        ret = adc5_get_fw_data(adc);
        if (ret)
-               return dev_err_probe(dev, ret, "adc get dt data failed\n");
+               return ret;
 
        irq_eoc = platform_get_irq(pdev, 0);
        if (irq_eoc < 0) {