]> www.infradead.org Git - users/hch/misc.git/commitdiff
iio: irsd200: Remove print of error code from dev_err_probe
authorWaqar Hameed <waqar.hameed@axis.com>
Fri, 16 May 2025 17:18:56 +0000 (19:18 +0200)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 9 Jun 2025 06:45:34 +0000 (07:45 +0100)
Since `dev_err_probe()` already prints the error code, there is no need
to additionally print it in the error message. Therefore, just remove
them.

Signed-off-by: Waqar Hameed <waqar.hameed@axis.com>
Link: https://patch.msgid.link/83c5ed21654b1b98437247d0fef823237af641b4.1747415559.git.waqar.hameed@axis.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/proximity/irsd200.c

index 0d30b91dbcbc20c8837a61ca9b4db631f80b263d..5e751fb0b12fb2167f4d11e814915cb2761ab9bf 100644 (file)
@@ -885,9 +885,8 @@ static int irsd200_probe(struct i2c_client *client)
 
        ret = devm_regulator_get_enable(data->dev, "vdd");
        if (ret)
-               return dev_err_probe(
-                       data->dev, ret,
-                       "Could not get and enable regulator (%d)\n", ret);
+               return dev_err_probe(data->dev, ret,
+                                    "Could not get and enable regulator\n");
 
        ret = irsd200_setup(data);
        if (ret)
@@ -905,17 +904,15 @@ static int irsd200_probe(struct i2c_client *client)
        ret = devm_iio_triggered_buffer_setup(data->dev, indio_dev, NULL,
                                              irsd200_trigger_handler, NULL);
        if (ret)
-               return dev_err_probe(
-                       data->dev, ret,
-                       "Could not setup iio triggered buffer (%d)\n", ret);
+               return dev_err_probe(data->dev, ret,
+                                    "Could not setup iio triggered buffer\n");
 
        ret = devm_request_threaded_irq(data->dev, client->irq, NULL,
                                        irsd200_irq_thread,
                                        IRQF_TRIGGER_RISING | IRQF_ONESHOT,
                                        NULL, indio_dev);
        if (ret)
-               return dev_err_probe(data->dev, ret,
-                                    "Could not request irq (%d)\n", ret);
+               return dev_err_probe(data->dev, ret, "Could not request irq\n");
 
        trigger = devm_iio_trigger_alloc(data->dev, "%s-dev%d", indio_dev->name,
                                         iio_device_id(indio_dev));
@@ -929,14 +926,12 @@ static int irsd200_probe(struct i2c_client *client)
        ret = devm_iio_trigger_register(data->dev, trigger);
        if (ret)
                return dev_err_probe(data->dev, ret,
-                                    "Could not register iio trigger (%d)\n",
-                                    ret);
+                                    "Could not register iio trigger\n");
 
        ret = devm_iio_device_register(data->dev, indio_dev);
        if (ret)
                return dev_err_probe(data->dev, ret,
-                                    "Could not register iio device (%d)\n",
-                                    ret);
+                                    "Could not register iio device\n");
 
        return 0;
 }