]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
iio: adc: Drop unnecessary -ENOMEM messages
authorDixit Parmar <dixitparmar19@gmail.com>
Fri, 22 Aug 2025 03:49:50 +0000 (09:19 +0530)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 25 Aug 2025 11:32:13 +0000 (12:32 +0100)
The drivers do not require their own error messages for error
-ENOMEM, memory allocation failures. So remove the dev_err()
messages from the probe().

Signed-off-by: Dixit Parmar <dixitparmar19@gmail.com>
Link: https://patch.msgid.link/20250822-enomam_logs-v1-2-db87f2974552@gmail.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
18 files changed:
drivers/iio/adc/ad7949.c
drivers/iio/adc/bcm_iproc_adc.c
drivers/iio/adc/cpcap-adc.c
drivers/iio/adc/da9150-gpadc.c
drivers/iio/adc/dln2-adc.c
drivers/iio/adc/exynos_adc.c
drivers/iio/adc/imx7d_adc.c
drivers/iio/adc/imx8qxp-adc.c
drivers/iio/adc/mxs-lradc-adc.c
drivers/iio/adc/palmas_gpadc.c
drivers/iio/adc/rn5t618-adc.c
drivers/iio/adc/stm32-dfsdm-adc.c
drivers/iio/adc/stmpe-adc.c
drivers/iio/adc/ti-adc084s021.c
drivers/iio/adc/ti-ads131e08.c
drivers/iio/adc/ti_am335x_adc.c
drivers/iio/adc/twl4030-madc.c
drivers/iio/adc/viperboard_adc.c

index 202561cad4012b67e23ad6d3623c913cfef35e68..b35d299a3977f1bdbf9d5113b3239f9df34462d5 100644 (file)
@@ -316,10 +316,8 @@ static int ad7949_spi_probe(struct spi_device *spi)
        int ret;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*ad7949_adc));
-       if (!indio_dev) {
-               dev_err(dev, "can not allocate iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        indio_dev->info = &ad7949_spi_info;
        indio_dev->name = spi_get_device_id(spi)->name;
index f258668b0dc71ee8f4de4ab01a6819a39250f333..6426c9e6ccc92517b5ce1fe3fb972aff857997d6 100644 (file)
@@ -511,10 +511,8 @@ static int iproc_adc_probe(struct platform_device *pdev)
 
        indio_dev = devm_iio_device_alloc(&pdev->dev,
                                        sizeof(*adc_priv));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed to allocate iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        adc_priv = iio_priv(indio_dev);
        platform_set_drvdata(pdev, indio_dev);
index ba7cbd3b48222543aa82f14054b4e20e260e8d69..d9ee2ea116a78b46e6490aeb43144eed3fc1790d 100644 (file)
@@ -953,11 +953,9 @@ static int cpcap_adc_probe(struct platform_device *pdev)
        int error;
 
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*ddata));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed to allocate iio device\n");
-
+       if (!indio_dev)
                return -ENOMEM;
-       }
+
        ddata = iio_priv(indio_dev);
        ddata->ato = device_get_match_data(&pdev->dev);
        if (!ddata->ato)
index b99291ce2a45ff235feeff19f329ba4505dd1a52..625e3a8e4d0367ab44c787c35fadae4ccaec3cad 100644 (file)
@@ -308,10 +308,9 @@ static int da9150_gpadc_probe(struct platform_device *pdev)
        int irq, ret;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*gpadc));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "Failed to allocate IIO device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
+
        gpadc = iio_priv(indio_dev);
 
        gpadc->da9150 = da9150;
index 5aea7644780f8b32ca6ed80b79afd07220d46a5c..eb902a946efea6acbf38e9ef917cbc0bf8207f6e 100644 (file)
@@ -584,10 +584,8 @@ static int dln2_adc_probe(struct platform_device *pdev)
        int i, ret, chans;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*dln2));
-       if (!indio_dev) {
-               dev_err(dev, "failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        dln2 = iio_priv(indio_dev);
        dln2->pdev = pdev;
@@ -628,10 +626,9 @@ static int dln2_adc_probe(struct platform_device *pdev)
        dln2->trig = devm_iio_trigger_alloc(dev, "%s-dev%d",
                                            indio_dev->name,
                                            iio_device_id(indio_dev));
-       if (!dln2->trig) {
-               dev_err(dev, "failed to allocate trigger\n");
+       if (!dln2->trig)
                return -ENOMEM;
-       }
+
        iio_trigger_set_drvdata(dln2->trig, dln2);
        ret = devm_iio_trigger_register(dev, dln2->trig);
        if (ret) {
index 4614cf848535944a41e0ab19f35c5ca0546000c0..32cfc4002b1bce06d875c5fc645b0da2a85e6916 100644 (file)
@@ -798,10 +798,8 @@ static int exynos_adc_probe(struct platform_device *pdev)
        int irq;
 
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(struct exynos_adc));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        info = iio_priv(indio_dev);
 
index 09ce71f6e941e5676cbea3c49bbae74122085304..039c0387da23798bb60fb10dfc1779be21b28470 100644 (file)
@@ -482,10 +482,8 @@ static int imx7d_adc_probe(struct platform_device *pdev)
        int ret;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*info));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "Failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        info = iio_priv(indio_dev);
        info->dev = dev;
index be13a6ed7e00b986ad4984d6582298d8d817f4d9..427ee9f244084289336b9ebad4a907029251a005 100644 (file)
@@ -315,10 +315,8 @@ static int imx8qxp_adc_probe(struct platform_device *pdev)
        int ret;
 
        indio_dev = devm_iio_device_alloc(dev, sizeof(*adc));
-       if (!indio_dev) {
-               dev_err(dev, "Failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        adc = iio_priv(indio_dev);
        adc->dev = dev;
index 92baf3f5f5601b863c694eb03b6d8f287e4fe6ab..dda5182a5076a6bd4c0da03e617bf0d8510769a9 100644 (file)
@@ -697,10 +697,8 @@ static int mxs_lradc_adc_probe(struct platform_device *pdev)
 
        /* Allocate the IIO device. */
        iio = devm_iio_device_alloc(dev, sizeof(*adc));
-       if (!iio) {
-               dev_err(dev, "Failed to allocate IIO device\n");
+       if (!iio)
                return -ENOMEM;
-       }
 
        adc = iio_priv(iio);
        adc->lradc = lradc;
index 7c01e33be04c4cc89c3749706515f214336ec495..3f433064618e471046033ef3c52d34f6d7f1319c 100644 (file)
@@ -885,10 +885,8 @@ static int palmas_gpadc_probe(struct platform_device *pdev)
                return -EINVAL;
 
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*adc));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "iio_device_alloc failed\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        adc = iio_priv(indio_dev);
        adc->dev = &pdev->dev;
index d6f6b351f2af85b2102b8f6fd51c3975e44a1ffa..f78fc795b69a702d006fe184814b78a79e9d2dd9 100644 (file)
@@ -199,10 +199,8 @@ static int rn5t618_adc_probe(struct platform_device *pdev)
        struct rn5t618 *rn5t618 = dev_get_drvdata(pdev->dev.parent);
 
        iio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*adc));
-       if (!iio_dev) {
-               dev_err(&pdev->dev, "failed allocating iio device\n");
+       if (!iio_dev)
                return -ENOMEM;
-       }
 
        adc = iio_priv(iio_dev);
        adc->dev = &pdev->dev;
index c2d21eecafe796a8c54aa2ad89f262dd691b92ca..74b1b4dc6e81d228465d875d2c617438932c3104 100644 (file)
@@ -1764,10 +1764,8 @@ static int stm32_dfsdm_adc_probe(struct platform_device *pdev)
 
        dev_data = of_device_get_match_data(dev);
        iio = devm_iio_device_alloc(dev, sizeof(*adc));
-       if (!iio) {
-               dev_err(dev, "%s: Failed to allocate IIO\n", __func__);
+       if (!iio)
                return -ENOMEM;
-       }
 
        adc = iio_priv(iio);
        adc->dfsdm = dev_get_drvdata(dev->parent);
index b0add5a2eab532df82125d328a742f629c11adaa..8e26c47edc080c776ea6cfc242b237d5816e04cd 100644 (file)
@@ -267,10 +267,8 @@ static int stmpe_adc_probe(struct platform_device *pdev)
                return irq_adc;
 
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(struct stmpe_adc));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        info = iio_priv(indio_dev);
        mutex_init(&info->lock);
index 50a474f4d9f55f6a85673d7c5f429b8c656be8d9..a100f770fa1c67c5e359c8088abd04bb5ee1c601 100644 (file)
@@ -200,10 +200,8 @@ static int adc084s021_probe(struct spi_device *spi)
        int ret;
 
        indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*adc));
-       if (!indio_dev) {
-               dev_err(&spi->dev, "Failed to allocate IIO device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        adc = iio_priv(indio_dev);
        adc->spi = spi;
index b18f30d3fdbe56b53ae1dfba84ac8086940c9adb..742acc6d8cf919981f33883de8da9733dc8a2642 100644 (file)
@@ -807,10 +807,8 @@ static int ads131e08_probe(struct spi_device *spi)
        }
 
        indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*st));
-       if (!indio_dev) {
-               dev_err(&spi->dev, "failed to allocate IIO device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        st = iio_priv(indio_dev);
        st->info = info;
@@ -841,10 +839,8 @@ static int ads131e08_probe(struct spi_device *spi)
 
        st->trig = devm_iio_trigger_alloc(&spi->dev, "%s-dev%d",
                indio_dev->name, iio_device_id(indio_dev));
-       if (!st->trig) {
-               dev_err(&spi->dev, "failed to allocate IIO trigger\n");
+       if (!st->trig)
                return -ENOMEM;
-       }
 
        st->trig->ops = &ads131e08_trigger_ops;
        st->trig->dev.parent = &spi->dev;
index fe1509d3b1e766cf13ea77b1d6b0781f54515dba..99f274adc870dc8a21c161345172a0321616b8e3 100644 (file)
@@ -631,10 +631,9 @@ static int tiadc_probe(struct platform_device *pdev)
        }
 
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*adc_dev));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed to allocate iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
+
        adc_dev = iio_priv(indio_dev);
 
        adc_dev->mfd_tscadc = ti_tscadc_dev_get(pdev);
index 0ea51ddeaa0a0a9bc22caa75b5001584993bfc73..fe3b31ec976ef00ad0a8ba0ea52ac868b3cd5c53 100644 (file)
@@ -758,10 +758,8 @@ static int twl4030_madc_probe(struct platform_device *pdev)
        }
 
        iio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*madc));
-       if (!iio_dev) {
-               dev_err(&pdev->dev, "failed allocating iio device\n");
+       if (!iio_dev)
                return -ENOMEM;
-       }
 
        madc = iio_priv(iio_dev);
        madc->dev = &pdev->dev;
index 1028b101cf568d709bb95f1d4ec8b1f25ed15c15..9bb0b83c8f6764d83bd2c38a3cc1b2d865998cb5 100644 (file)
@@ -113,10 +113,8 @@ static int vprbrd_adc_probe(struct platform_device *pdev)
 
        /* registering iio */
        indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*adc));
-       if (!indio_dev) {
-               dev_err(&pdev->dev, "failed allocating iio device\n");
+       if (!indio_dev)
                return -ENOMEM;
-       }
 
        adc = iio_priv(indio_dev);
        adc->vb = vb;