iio: dac: ds4424: ratelimit read errors and use device context

Replace pr_err() with dev_err_ratelimited() in the RAW read path to avoid
log spam on repeated I2C failures and to include the device context.

Use %pe to print errno names for faster debugging. Use the parent
device context to identify the physical hardware causing the error.

Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/dac/ds4424.c b/drivers/iio/dac/ds4424.c
index c15eb7b5eb96017f98334c65e4bd72c094b5e2a5..9d33a810336f78f8835a6ae74bc4f294870a9eb8 100644 (file)
--- a/drivers/iio/dac/ds4424.c
+++ b/drivers/iio/dac/ds4424.c
@@ -99,8 +99,9 @@ static int ds4424_read_raw(struct iio_dev *indio_dev,
        case IIO_CHAN_INFO_RAW:
                ret = ds4424_get_value(indio_dev, &regval, chan->channel);
                if (ret < 0) {
-                       pr_err("%s : ds4424_get_value returned %d\n",
-                                                       __func__, ret);
+                       dev_err_ratelimited(indio_dev->dev.parent,
+                                           "Failed to read channel %d: %pe\n",
+                                           chan->channel, ERR_PTR(ret));
                        return ret;
                }