* [PATCH] i2c: don't print error when adding adapter fails
@ 2016-08-09 11:36 Wolfram Sang
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-14 22:17 ` Wolfram Sang
0 siblings, 2 replies; 21+ messages in thread
From: Wolfram Sang @ 2016-08-09 11:36 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang,
adi-buildroot-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
kernel-F5mvAk5X5gdBDgjK7y7TUQ,
linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-omap-u79uwXL29TY76Z2rM5mHXA,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
Since v4.8-rc1, the I2C core will print detailed information when adding an I2C
adapter fails. So, drivers can skip this now.
I am still undecided if I apply this as a single patch or break it out. But for
reviewing, avoiding the patch bomb is probably a good thing.
Should go via subsystem tree, I'd think.
Wolfram Sang (1):
i2c: don't print error when adding adapter fails
drivers/i2c/busses/i2c-amd756.c | 5 +----
drivers/i2c/busses/i2c-at91.c | 2 --
drivers/i2c/busses/i2c-axxia.c | 8 +-------
drivers/i2c/busses/i2c-bcm-iproc.c | 8 +-------
drivers/i2c/busses/i2c-bcm-kona.c | 4 +---
drivers/i2c/busses/i2c-bfin-twi.c | 4 +---
drivers/i2c/busses/i2c-brcmstb.c | 4 +---
drivers/i2c/busses/i2c-cadence.c | 4 +---
drivers/i2c/busses/i2c-cpm.c | 4 +---
drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +---
drivers/i2c/busses/i2c-davinci.c | 4 +---
drivers/i2c/busses/i2c-diolan-u2c.c | 4 +---
drivers/i2c/busses/i2c-dln2.c | 4 +---
drivers/i2c/busses/i2c-efm32.c | 1 -
drivers/i2c/busses/i2c-exynos5.c | 4 +---
drivers/i2c/busses/i2c-hix5hd2.c | 4 +---
drivers/i2c/busses/i2c-i801.c | 1 -
drivers/i2c/busses/i2c-ibm_iic.c | 4 +---
drivers/i2c/busses/i2c-img-scb.c | 4 +---
drivers/i2c/busses/i2c-imx.c | 4 +---
drivers/i2c/busses/i2c-isch.c | 4 +---
drivers/i2c/busses/i2c-ismt.c | 4 +---
drivers/i2c/busses/i2c-jz4780.c | 4 +---
drivers/i2c/busses/i2c-lpc2k.c | 4 +---
drivers/i2c/busses/i2c-meson.c | 1 -
drivers/i2c/busses/i2c-mpc.c | 4 +---
drivers/i2c/busses/i2c-mt65xx.c | 4 +---
drivers/i2c/busses/i2c-mxs.c | 1 -
drivers/i2c/busses/i2c-nforce2.c | 1 -
drivers/i2c/busses/i2c-nomadik.c | 4 +---
drivers/i2c/busses/i2c-ocores.c | 4 +---
drivers/i2c/busses/i2c-octeon.c | 4 +---
drivers/i2c/busses/i2c-omap.c | 4 +---
drivers/i2c/busses/i2c-piix4.c | 1 -
drivers/i2c/busses/i2c-pmcmsp.c | 4 +---
drivers/i2c/busses/i2c-pnx.c | 4 +---
drivers/i2c/busses/i2c-puv3.c | 5 +----
drivers/i2c/busses/i2c-pxa.c | 4 +---
drivers/i2c/busses/i2c-rcar.c | 4 +---
drivers/i2c/busses/i2c-riic.c | 4 +---
drivers/i2c/busses/i2c-rk3x.c | 4 +---
drivers/i2c/busses/i2c-s3c2410.c | 1 -
drivers/i2c/busses/i2c-sh7760.c | 4 +---
drivers/i2c/busses/i2c-sh_mobile.c | 1 -
drivers/i2c/busses/i2c-sirf.c | 4 +---
drivers/i2c/busses/i2c-st.c | 4 +---
drivers/i2c/busses/i2c-stu300.c | 5 +----
drivers/i2c/busses/i2c-tegra.c | 4 +---
drivers/i2c/busses/i2c-uniphier-f.c | 7 +------
drivers/i2c/busses/i2c-uniphier.c | 7 +------
drivers/i2c/busses/i2c-wmt.c | 4 +---
drivers/i2c/busses/i2c-xgene-slimpro.c | 1 -
drivers/i2c/busses/i2c-xiic.c | 1 -
drivers/i2c/busses/i2c-xlp9xx.c | 4 +---
drivers/i2c/busses/i2c-xlr.c | 4 +---
55 files changed, 44 insertions(+), 161 deletions(-)
--
2.8.1
^ permalink raw reply [flat|nested] 21+ messages in thread
* [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
@ 2016-08-09 11:36 ` Wolfram Sang
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
` (6 more replies)
2016-08-09 16:45 ` Joachim Eastwood
2016-08-10 6:50 ` [STLinux Kernel] " Patrice Chotard
2 siblings, 7 replies; 21+ messages in thread
From: Wolfram Sang @ 2016-08-09 11:36 UTC (permalink / raw)
To: linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, Wolfram Sang, Jean Delvare,
Wolfram Sang, Ludovic Desroches, Ray Jui, Scott Branden,
Jon Mason, bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w,
Florian Fainelli, Sonic Zhang, Brian Norris, Gregory Fong,
Michal Simek, Sören Brinkmann, Jochen Friedrich, Sekhar Nori,
Kevin Hilman, Guenter Roeck, Uwe Kleine-König, Kukjin Kim,
Krzysztof Kozlowski
The core will do this for us now.
Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
---
drivers/i2c/busses/i2c-amd756.c | 5 +----
drivers/i2c/busses/i2c-at91.c | 2 --
drivers/i2c/busses/i2c-axxia.c | 8 +-------
drivers/i2c/busses/i2c-bcm-iproc.c | 8 +-------
drivers/i2c/busses/i2c-bcm-kona.c | 4 +---
drivers/i2c/busses/i2c-bfin-twi.c | 4 +---
drivers/i2c/busses/i2c-brcmstb.c | 4 +---
drivers/i2c/busses/i2c-cadence.c | 4 +---
drivers/i2c/busses/i2c-cpm.c | 4 +---
drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +---
drivers/i2c/busses/i2c-davinci.c | 4 +---
drivers/i2c/busses/i2c-diolan-u2c.c | 4 +---
drivers/i2c/busses/i2c-dln2.c | 4 +---
drivers/i2c/busses/i2c-efm32.c | 1 -
drivers/i2c/busses/i2c-exynos5.c | 4 +---
drivers/i2c/busses/i2c-hix5hd2.c | 4 +---
drivers/i2c/busses/i2c-i801.c | 1 -
drivers/i2c/busses/i2c-ibm_iic.c | 4 +---
drivers/i2c/busses/i2c-img-scb.c | 4 +---
drivers/i2c/busses/i2c-imx.c | 4 +---
drivers/i2c/busses/i2c-isch.c | 4 +---
drivers/i2c/busses/i2c-ismt.c | 4 +---
drivers/i2c/busses/i2c-jz4780.c | 4 +---
drivers/i2c/busses/i2c-lpc2k.c | 4 +---
drivers/i2c/busses/i2c-meson.c | 1 -
drivers/i2c/busses/i2c-mpc.c | 4 +---
drivers/i2c/busses/i2c-mt65xx.c | 4 +---
drivers/i2c/busses/i2c-mxs.c | 1 -
drivers/i2c/busses/i2c-nforce2.c | 1 -
drivers/i2c/busses/i2c-nomadik.c | 4 +---
drivers/i2c/busses/i2c-ocores.c | 4 +---
drivers/i2c/busses/i2c-octeon.c | 4 +---
drivers/i2c/busses/i2c-omap.c | 4 +---
drivers/i2c/busses/i2c-piix4.c | 1 -
drivers/i2c/busses/i2c-pmcmsp.c | 4 +---
drivers/i2c/busses/i2c-pnx.c | 4 +---
drivers/i2c/busses/i2c-puv3.c | 5 +----
drivers/i2c/busses/i2c-pxa.c | 4 +---
drivers/i2c/busses/i2c-rcar.c | 4 +---
drivers/i2c/busses/i2c-riic.c | 4 +---
drivers/i2c/busses/i2c-rk3x.c | 4 +---
drivers/i2c/busses/i2c-s3c2410.c | 1 -
drivers/i2c/busses/i2c-sh7760.c | 4 +---
drivers/i2c/busses/i2c-sh_mobile.c | 1 -
drivers/i2c/busses/i2c-sirf.c | 4 +---
drivers/i2c/busses/i2c-st.c | 4 +---
drivers/i2c/busses/i2c-stu300.c | 5 +----
drivers/i2c/busses/i2c-tegra.c | 4 +---
drivers/i2c/busses/i2c-uniphier-f.c | 7 +------
drivers/i2c/busses/i2c-uniphier.c | 7 +------
drivers/i2c/busses/i2c-wmt.c | 4 +---
drivers/i2c/busses/i2c-xgene-slimpro.c | 1 -
drivers/i2c/busses/i2c-xiic.c | 1 -
drivers/i2c/busses/i2c-xlp9xx.c | 4 +---
drivers/i2c/busses/i2c-xlr.c | 4 +---
55 files changed, 44 insertions(+), 161 deletions(-)
diff --git a/drivers/i2c/busses/i2c-amd756.c b/drivers/i2c/busses/i2c-amd756.c
index 6c7113d990f882..274908cd1fdeab 100644
--- a/drivers/i2c/busses/i2c-amd756.c
+++ b/drivers/i2c/busses/i2c-amd756.c
@@ -378,11 +378,8 @@ static int amd756_probe(struct pci_dev *pdev, const struct pci_device_id *id)
amd756_ioport);
error = i2c_add_adapter(&amd756_smbus);
- if (error) {
- dev_err(&pdev->dev,
- "Adapter registration failed, module not inserted\n");
+ if (error)
goto out_err;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-at91.c b/drivers/i2c/busses/i2c-at91.c
index f23372669f770f..03a519d29616d7 100644
--- a/drivers/i2c/busses/i2c-at91.c
+++ b/drivers/i2c/busses/i2c-at91.c
@@ -1118,8 +1118,6 @@ static int at91_twi_probe(struct platform_device *pdev)
rc = i2c_add_numbered_adapter(&dev->adapter);
if (rc) {
- dev_err(dev->dev, "Adapter %s registration failed\n",
- dev->adapter.name);
clk_disable_unprepare(dev->clk);
pm_runtime_disable(dev->dev);
diff --git a/drivers/i2c/busses/i2c-axxia.c b/drivers/i2c/busses/i2c-axxia.c
index c335cc7852f94e..d3bcaf4ab095d0 100644
--- a/drivers/i2c/busses/i2c-axxia.c
+++ b/drivers/i2c/busses/i2c-axxia.c
@@ -558,13 +558,7 @@ static int axxia_i2c_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, idev);
- ret = i2c_add_adapter(&idev->adapter);
- if (ret) {
- dev_err(&pdev->dev, "failed to add adapter\n");
- return ret;
- }
-
- return 0;
+ return i2c_add_adapter(&idev->adapter);
}
static int axxia_i2c_remove(struct platform_device *pdev)
diff --git a/drivers/i2c/busses/i2c-bcm-iproc.c b/drivers/i2c/busses/i2c-bcm-iproc.c
index 19c843828fe2ca..8e3477f9297eda 100644
--- a/drivers/i2c/busses/i2c-bcm-iproc.c
+++ b/drivers/i2c/busses/i2c-bcm-iproc.c
@@ -488,13 +488,7 @@ static int bcm_iproc_i2c_probe(struct platform_device *pdev)
adap->dev.parent = &pdev->dev;
adap->dev.of_node = pdev->dev.of_node;
- ret = i2c_add_adapter(adap);
- if (ret) {
- dev_err(iproc_i2c->device, "failed to add adapter\n");
- return ret;
- }
-
- return 0;
+ return i2c_add_adapter(adap);
}
static int bcm_iproc_i2c_remove(struct platform_device *pdev)
diff --git a/drivers/i2c/busses/i2c-bcm-kona.c b/drivers/i2c/busses/i2c-bcm-kona.c
index ac9f47679c3a4b..a5c9098507896b 100644
--- a/drivers/i2c/busses/i2c-bcm-kona.c
+++ b/drivers/i2c/busses/i2c-bcm-kona.c
@@ -858,10 +858,8 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev)
adap->dev.of_node = pdev->dev.of_node;
rc = i2c_add_adapter(adap);
- if (rc) {
- dev_err(dev->device, "failed to add adapter\n");
+ if (rc)
return rc;
- }
dev_info(dev->device, "device registered successfully\n");
diff --git a/drivers/i2c/busses/i2c-bfin-twi.c b/drivers/i2c/busses/i2c-bfin-twi.c
index 025686d4164058..29d00c4f7824c2 100644
--- a/drivers/i2c/busses/i2c-bfin-twi.c
+++ b/drivers/i2c/busses/i2c-bfin-twi.c
@@ -685,10 +685,8 @@ static int i2c_bfin_twi_probe(struct platform_device *pdev)
write_CONTROL(iface, read_CONTROL(iface) | TWI_ENA);
rc = i2c_add_numbered_adapter(p_adap);
- if (rc < 0) {
- dev_err(&pdev->dev, "Can't add i2c adapter!\n");
+ if (rc < 0)
goto out_error;
- }
platform_set_drvdata(pdev, iface);
diff --git a/drivers/i2c/busses/i2c-brcmstb.c b/drivers/i2c/busses/i2c-brcmstb.c
index 3f5a4d71d3bf32..bc5c176e984aea 100644
--- a/drivers/i2c/busses/i2c-brcmstb.c
+++ b/drivers/i2c/busses/i2c-brcmstb.c
@@ -648,10 +648,8 @@ static int brcmstb_i2c_probe(struct platform_device *pdev)
adap->dev.parent = &pdev->dev;
adap->dev.of_node = pdev->dev.of_node;
rc = i2c_add_adapter(adap);
- if (rc) {
- dev_err(dev->device, "failed to add adapter\n");
+ if (rc)
goto probe_errorout;
- }
dev_info(dev->device, "%s@%dhz registered in %s mode\n",
int_name ? int_name : " ", dev->clk_freq_hz,
diff --git a/drivers/i2c/busses/i2c-cadence.c b/drivers/i2c/busses/i2c-cadence.c
index 90bbd9f9dd8f77..1cc1e79e958d56 100644
--- a/drivers/i2c/busses/i2c-cadence.c
+++ b/drivers/i2c/busses/i2c-cadence.c
@@ -963,10 +963,8 @@ static int cdns_i2c_probe(struct platform_device *pdev)
}
ret = i2c_add_adapter(&id->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
+ if (ret < 0)
goto err_clk_dis;
- }
/*
* Cadence I2C controller has a bug wherein it generates
diff --git a/drivers/i2c/busses/i2c-cpm.c b/drivers/i2c/busses/i2c-cpm.c
index ee57c1e865e271..d89bde2c5da256 100644
--- a/drivers/i2c/busses/i2c-cpm.c
+++ b/drivers/i2c/busses/i2c-cpm.c
@@ -665,10 +665,8 @@ static int cpm_i2c_probe(struct platform_device *ofdev)
cpm->adap.nr = (data && len == 4) ? be32_to_cpup(data) : -1;
result = i2c_add_numbered_adapter(&cpm->adap);
- if (result < 0) {
- dev_err(&ofdev->dev, "Unable to register with I2C\n");
+ if (result < 0)
goto out_shut;
- }
dev_dbg(&ofdev->dev, "hw routines for %s registered.\n",
cpm->adap.name);
diff --git a/drivers/i2c/busses/i2c-cros-ec-tunnel.c b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
index a0d95ff682ae12..cf334a9ec22cb7 100644
--- a/drivers/i2c/busses/i2c-cros-ec-tunnel.c
+++ b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
@@ -281,10 +281,8 @@ static int ec_i2c_probe(struct platform_device *pdev)
bus->adap.retries = I2C_MAX_RETRIES;
err = i2c_add_adapter(&bus->adap);
- if (err) {
- dev_err(dev, "cannot register i2c adapter\n");
+ if (err)
return err;
- }
platform_set_drvdata(pdev, bus);
return err;
diff --git a/drivers/i2c/busses/i2c-davinci.c b/drivers/i2c/busses/i2c-davinci.c
index a8bdcb5292f5b3..9e7ef5cf5d495d 100644
--- a/drivers/i2c/busses/i2c-davinci.c
+++ b/drivers/i2c/busses/i2c-davinci.c
@@ -846,10 +846,8 @@ static int davinci_i2c_probe(struct platform_device *pdev)
adap->nr = pdev->id;
r = i2c_add_numbered_adapter(adap);
- if (r) {
- dev_err(&pdev->dev, "failure adding adapter\n");
+ if (r)
goto err_unuse_clocks;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-diolan-u2c.c b/drivers/i2c/busses/i2c-diolan-u2c.c
index b19a310bf9b3e8..f718ee4e3332fe 100644
--- a/drivers/i2c/busses/i2c-diolan-u2c.c
+++ b/drivers/i2c/busses/i2c-diolan-u2c.c
@@ -487,10 +487,8 @@ static int diolan_u2c_probe(struct usb_interface *interface,
/* and finally attach to i2c layer */
ret = i2c_add_adapter(&dev->adapter);
- if (ret < 0) {
- dev_err(&interface->dev, "failed to add I2C adapter\n");
+ if (ret < 0)
goto error_free;
- }
dev_dbg(&interface->dev, "connected " DRIVER_NAME "\n");
diff --git a/drivers/i2c/busses/i2c-dln2.c b/drivers/i2c/busses/i2c-dln2.c
index f2eb4f76591fbe..8acda2aa15583c 100644
--- a/drivers/i2c/busses/i2c-dln2.c
+++ b/drivers/i2c/busses/i2c-dln2.c
@@ -228,10 +228,8 @@ static int dln2_i2c_probe(struct platform_device *pdev)
/* and finally attach to i2c layer */
ret = i2c_add_adapter(&dln2->adapter);
- if (ret < 0) {
- dev_err(dev, "failed to add I2C adapter: %d\n", ret);
+ if (ret < 0)
goto out_disable;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-efm32.c b/drivers/i2c/busses/i2c-efm32.c
index e253598d764c0d..aa336ba89aa3fe 100644
--- a/drivers/i2c/busses/i2c-efm32.c
+++ b/drivers/i2c/busses/i2c-efm32.c
@@ -438,7 +438,6 @@ static int efm32_i2c_probe(struct platform_device *pdev)
ret = i2c_add_adapter(&ddata->adapter);
if (ret) {
- dev_err(&pdev->dev, "failed to add i2c adapter (%d)\n", ret);
free_irq(ddata->irq, ddata);
err_disable_clk:
diff --git a/drivers/i2c/busses/i2c-exynos5.c b/drivers/i2c/busses/i2c-exynos5.c
index c0e3ada028763d..bea607149972d4 100644
--- a/drivers/i2c/busses/i2c-exynos5.c
+++ b/drivers/i2c/busses/i2c-exynos5.c
@@ -796,10 +796,8 @@ static int exynos5_i2c_probe(struct platform_device *pdev)
exynos5_i2c_reset(i2c);
ret = i2c_add_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "failed to add bus to i2c core\n");
+ if (ret < 0)
goto err_clk;
- }
platform_set_drvdata(pdev, i2c);
diff --git a/drivers/i2c/busses/i2c-hix5hd2.c b/drivers/i2c/busses/i2c-hix5hd2.c
index 7c6966434ee7b9..ae7f3180f7e887 100644
--- a/drivers/i2c/busses/i2c-hix5hd2.c
+++ b/drivers/i2c/busses/i2c-hix5hd2.c
@@ -478,10 +478,8 @@ static int hix5hd2_i2c_probe(struct platform_device *pdev)
pm_runtime_enable(priv->dev);
ret = i2c_add_adapter(&priv->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "failed to add bus to i2c core\n");
+ if (ret < 0)
goto err_runtime;
- }
return ret;
diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
index 5ef9b733d153fa..22a0ed4de80da0 100644
--- a/drivers/i2c/busses/i2c-i801.c
+++ b/drivers/i2c/busses/i2c-i801.c
@@ -1613,7 +1613,6 @@ static int i801_probe(struct pci_dev *dev, const struct pci_device_id *id)
"SMBus I801 adapter at %04lx", priv->smba);
err = i2c_add_adapter(&priv->adapter);
if (err) {
- dev_err(&dev->dev, "Failed to add SMBus adapter\n");
i801_acpi_remove(priv);
return err;
}
diff --git a/drivers/i2c/busses/i2c-ibm_iic.c b/drivers/i2c/busses/i2c-ibm_iic.c
index cdaa7be2cd1b94..412b91d255ad1d 100644
--- a/drivers/i2c/busses/i2c-ibm_iic.c
+++ b/drivers/i2c/busses/i2c-ibm_iic.c
@@ -751,10 +751,8 @@ static int iic_probe(struct platform_device *ofdev)
adap->timeout = HZ;
ret = i2c_add_adapter(adap);
- if (ret < 0) {
- dev_err(&ofdev->dev, "failed to register i2c adapter\n");
+ if (ret < 0)
goto error_cleanup;
- }
dev_info(&ofdev->dev, "using %s mode\n",
dev->fast_mode ? "fast (400 kHz)" : "standard (100 kHz)");
diff --git a/drivers/i2c/busses/i2c-img-scb.c b/drivers/i2c/busses/i2c-img-scb.c
index ea20425b697217..db8e8b40569d98 100644
--- a/drivers/i2c/busses/i2c-img-scb.c
+++ b/drivers/i2c/busses/i2c-img-scb.c
@@ -1394,10 +1394,8 @@ static int img_i2c_probe(struct platform_device *pdev)
goto disable_clk;
ret = i2c_add_numbered_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "failed to add adapter\n");
+ if (ret < 0)
goto disable_clk;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c
index 1844bc9f7cd5a2..cb11eee9567318 100644
--- a/drivers/i2c/busses/i2c-imx.c
+++ b/drivers/i2c/busses/i2c-imx.c
@@ -1129,10 +1129,8 @@ static int i2c_imx_probe(struct platform_device *pdev)
/* Add I2C adapter */
ret = i2c_add_numbered_adapter(&i2c_imx->adapter);
- if (ret < 0) {
- dev_err(&pdev->dev, "registration failed\n");
+ if (ret < 0)
goto rpm_disable;
- }
pm_runtime_mark_last_busy(&pdev->dev);
pm_runtime_put_autosuspend(&pdev->dev);
diff --git a/drivers/i2c/busses/i2c-isch.c b/drivers/i2c/busses/i2c-isch.c
index c2f25f19d76f9b..0cf1379f4e8091 100644
--- a/drivers/i2c/busses/i2c-isch.c
+++ b/drivers/i2c/busses/i2c-isch.c
@@ -288,10 +288,8 @@ static int smbus_sch_probe(struct platform_device *dev)
"SMBus SCH adapter at %04x", sch_smba);
retval = i2c_add_adapter(&sch_adapter);
- if (retval) {
- dev_err(&dev->dev, "Couldn't register adapter!\n");
+ if (retval)
sch_smba = 0;
- }
return retval;
}
diff --git a/drivers/i2c/busses/i2c-ismt.c b/drivers/i2c/busses/i2c-ismt.c
index 1c870771009840..f573448d213216 100644
--- a/drivers/i2c/busses/i2c-ismt.c
+++ b/drivers/i2c/busses/i2c-ismt.c
@@ -922,10 +922,8 @@ ismt_probe(struct pci_dev *pdev, const struct pci_device_id *id)
return err;
err = i2c_add_adapter(&priv->adapter);
- if (err) {
- dev_err(&pdev->dev, "Failed to add SMBus iSMT adapter\n");
+ if (err)
return -ENODEV;
- }
return 0;
}
diff --git a/drivers/i2c/busses/i2c-jz4780.c b/drivers/i2c/busses/i2c-jz4780.c
index cd9872594fe2a0..b8ea62105f42c9 100644
--- a/drivers/i2c/busses/i2c-jz4780.c
+++ b/drivers/i2c/busses/i2c-jz4780.c
@@ -798,10 +798,8 @@ static int jz4780_i2c_probe(struct platform_device *pdev)
goto err;
ret = i2c_add_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "Failed to add bus\n");
+ if (ret < 0)
goto err;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-lpc2k.c b/drivers/i2c/busses/i2c-lpc2k.c
index 586a15205e61de..9b1fef455a895d 100644
--- a/drivers/i2c/busses/i2c-lpc2k.c
+++ b/drivers/i2c/busses/i2c-lpc2k.c
@@ -432,10 +432,8 @@ static int i2c_lpc2k_probe(struct platform_device *pdev)
i2c->adap.dev.of_node = pdev->dev.of_node;
ret = i2c_add_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "failed to add adapter!\n");
+ if (ret < 0)
goto fail_clk;
- }
dev_info(&pdev->dev, "LPC2K I2C adapter\n");
diff --git a/drivers/i2c/busses/i2c-meson.c b/drivers/i2c/busses/i2c-meson.c
index 71d3929adf54ed..102d2b91771f35 100644
--- a/drivers/i2c/busses/i2c-meson.c
+++ b/drivers/i2c/busses/i2c-meson.c
@@ -453,7 +453,6 @@ static int meson_i2c_probe(struct platform_device *pdev)
ret = i2c_add_adapter(&i2c->adap);
if (ret < 0) {
- dev_err(&pdev->dev, "can't register adapter\n");
clk_unprepare(i2c->clk);
return ret;
}
diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
index 48ecffecc0eda3..565a49a0c56410 100644
--- a/drivers/i2c/busses/i2c-mpc.c
+++ b/drivers/i2c/busses/i2c-mpc.c
@@ -737,10 +737,8 @@ static int fsl_i2c_probe(struct platform_device *op)
i2c->adap.dev.of_node = of_node_get(op->dev.of_node);
result = i2c_add_adapter(&i2c->adap);
- if (result < 0) {
- dev_err(i2c->dev, "failed to add adapter\n");
+ if (result < 0)
goto fail_add;
- }
return result;
diff --git a/drivers/i2c/busses/i2c-mt65xx.c b/drivers/i2c/busses/i2c-mt65xx.c
index d9373e60be8ae3..4a7d9bc2142ba3 100644
--- a/drivers/i2c/busses/i2c-mt65xx.c
+++ b/drivers/i2c/busses/i2c-mt65xx.c
@@ -786,10 +786,8 @@ static int mtk_i2c_probe(struct platform_device *pdev)
i2c_set_adapdata(&i2c->adap, i2c);
ret = i2c_add_adapter(&i2c->adap);
- if (ret) {
- dev_err(&pdev->dev, "Failed to add i2c bus to i2c core\n");
+ if (ret)
return ret;
- }
platform_set_drvdata(pdev, i2c);
diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c
index 033846cdf26622..5738556b6aac20 100644
--- a/drivers/i2c/busses/i2c-mxs.c
+++ b/drivers/i2c/busses/i2c-mxs.c
@@ -868,7 +868,6 @@ static int mxs_i2c_probe(struct platform_device *pdev)
i2c_set_adapdata(adap, i2c);
err = i2c_add_numbered_adapter(adap);
if (err) {
- dev_err(dev, "Failed to add adapter (%d)\n", err);
writel(MXS_I2C_CTRL0_SFTRST,
i2c->regs + MXS_I2C_CTRL0_SET);
return err;
diff --git a/drivers/i2c/busses/i2c-nforce2.c b/drivers/i2c/busses/i2c-nforce2.c
index 42fcc9458432f8..374b35e7e450ce 100644
--- a/drivers/i2c/busses/i2c-nforce2.c
+++ b/drivers/i2c/busses/i2c-nforce2.c
@@ -366,7 +366,6 @@ static int nforce2_probe_smb(struct pci_dev *dev, int bar, int alt_reg,
error = i2c_add_adapter(&smbus->adapter);
if (error) {
- dev_err(&smbus->adapter.dev, "Failed to register adapter.\n");
release_region(smbus->base, smbus->size);
return error;
}
diff --git a/drivers/i2c/busses/i2c-nomadik.c b/drivers/i2c/busses/i2c-nomadik.c
index bcd17e8cbcb4b3..da6609d628487b 100644
--- a/drivers/i2c/busses/i2c-nomadik.c
+++ b/drivers/i2c/busses/i2c-nomadik.c
@@ -1046,10 +1046,8 @@ static int nmk_i2c_probe(struct amba_device *adev, const struct amba_id *id)
adap->name, dev->virtbase);
ret = i2c_add_adapter(adap);
- if (ret) {
- dev_err(&adev->dev, "failed to add adapter\n");
+ if (ret)
goto err_no_adap;
- }
pm_runtime_put(&adev->dev);
diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c
index dfa7a4b4a91d78..92692349d0c328 100644
--- a/drivers/i2c/busses/i2c-ocores.c
+++ b/drivers/i2c/busses/i2c-ocores.c
@@ -492,10 +492,8 @@ static int ocores_i2c_probe(struct platform_device *pdev)
/* add i2c adapter to i2c tree */
ret = i2c_add_adapter(&i2c->adap);
- if (ret) {
- dev_err(&pdev->dev, "Failed to add adapter\n");
+ if (ret)
return ret;
- }
/* add in known devices to the bus */
if (pdata) {
diff --git a/drivers/i2c/busses/i2c-octeon.c b/drivers/i2c/busses/i2c-octeon.c
index 30ae35146723a4..b75d2da79b27c5 100644
--- a/drivers/i2c/busses/i2c-octeon.c
+++ b/drivers/i2c/busses/i2c-octeon.c
@@ -1213,10 +1213,8 @@ static int octeon_i2c_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, i2c);
result = i2c_add_adapter(&i2c->adap);
- if (result < 0) {
- dev_err(i2c->dev, "failed to add adapter\n");
+ if (result < 0)
goto out;
- }
dev_info(i2c->dev, "probed\n");
return 0;
diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
index ab1279b8e2407a..c7da0c42baeefa 100644
--- a/drivers/i2c/busses/i2c-omap.c
+++ b/drivers/i2c/busses/i2c-omap.c
@@ -1425,10 +1425,8 @@ omap_i2c_probe(struct platform_device *pdev)
/* i2c device drivers may be active on return from add_adapter() */
adap->nr = pdev->id;
r = i2c_add_numbered_adapter(adap);
- if (r) {
- dev_err(omap->dev, "failure adding adapter\n");
+ if (r)
goto err_unuse_clocks;
- }
dev_info(omap->dev, "bus %d rev%d.%d at %d kHz\n", adap->nr,
major, minor, omap->speed);
diff --git a/drivers/i2c/busses/i2c-piix4.c b/drivers/i2c/busses/i2c-piix4.c
index 23d1c167b5d752..c2268cdf38e823 100644
--- a/drivers/i2c/busses/i2c-piix4.c
+++ b/drivers/i2c/busses/i2c-piix4.c
@@ -694,7 +694,6 @@ static int piix4_add_adapter(struct pci_dev *dev, unsigned short smba,
retval = i2c_add_adapter(adap);
if (retval) {
- dev_err(&dev->dev, "Couldn't register adapter!\n");
kfree(adapdata);
kfree(adap);
release_region(smba, SMBIOSIZE);
diff --git a/drivers/i2c/busses/i2c-pmcmsp.c b/drivers/i2c/busses/i2c-pmcmsp.c
index 2c40edbf6224eb..217c78711d6501 100644
--- a/drivers/i2c/busses/i2c-pmcmsp.c
+++ b/drivers/i2c/busses/i2c-pmcmsp.c
@@ -329,10 +329,8 @@ static int pmcmsptwi_probe(struct platform_device *pldev)
i2c_set_adapdata(&pmcmsptwi_adapter, &pmcmsptwi_data);
rc = i2c_add_adapter(&pmcmsptwi_adapter);
- if (rc) {
- dev_err(&pldev->dev, "Unable to register I2C adapter\n");
+ if (rc)
goto ret_unmap;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-pnx.c b/drivers/i2c/busses/i2c-pnx.c
index 7ea67aa46fb730..fd5f9d2bf6d94e 100644
--- a/drivers/i2c/busses/i2c-pnx.c
+++ b/drivers/i2c/busses/i2c-pnx.c
@@ -714,10 +714,8 @@ static int i2c_pnx_probe(struct platform_device *pdev)
/* Register this adapter with the I2C subsystem */
ret = i2c_add_numbered_adapter(&alg_data->adapter);
- if (ret < 0) {
- dev_err(&pdev->dev, "I2C: Failed to add bus\n");
+ if (ret < 0)
goto out_clock;
- }
dev_dbg(&pdev->dev, "%s: Master at %#8x, irq %d.\n",
alg_data->adapter.name, res->start, alg_data->irq);
diff --git a/drivers/i2c/busses/i2c-puv3.c b/drivers/i2c/busses/i2c-puv3.c
index 82b6f02544dae7..0c8b1571886d5b 100644
--- a/drivers/i2c/busses/i2c-puv3.c
+++ b/drivers/i2c/busses/i2c-puv3.c
@@ -212,11 +212,8 @@ static int puv3_i2c_probe(struct platform_device *pdev)
adapter->nr = pdev->id;
rc = i2c_add_numbered_adapter(adapter);
- if (rc) {
- dev_err(&pdev->dev, "Adapter '%s' registration failed\n",
- adapter->name);
+ if (rc)
goto fail_add_adapter;
- }
dev_info(&pdev->dev, "PKUnity v3 i2c bus adapter.\n");
return 0;
diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
index 0d351954db02b9..e28b825b043378 100644
--- a/drivers/i2c/busses/i2c-pxa.c
+++ b/drivers/i2c/busses/i2c-pxa.c
@@ -1292,10 +1292,8 @@ static int i2c_pxa_probe(struct platform_device *dev)
#endif
ret = i2c_add_numbered_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&dev->dev, "failed to add bus: %d\n", ret);
+ if (ret < 0)
goto ereqirq;
- }
platform_set_drvdata(dev, i2c);
diff --git a/drivers/i2c/busses/i2c-rcar.c b/drivers/i2c/busses/i2c-rcar.c
index 52407f3c9e1cce..400e344704d273 100644
--- a/drivers/i2c/busses/i2c-rcar.c
+++ b/drivers/i2c/busses/i2c-rcar.c
@@ -875,10 +875,8 @@ static int rcar_i2c_probe(struct platform_device *pdev)
platform_set_drvdata(pdev, priv);
ret = i2c_add_numbered_adapter(adap);
- if (ret < 0) {
- dev_err(dev, "reg adap failed: %d\n", ret);
+ if (ret < 0)
goto out_pm_disable;
- }
dev_info(dev, "probed\n");
diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
index d7e3af67154373..6263ea82d6ac3b 100644
--- a/drivers/i2c/busses/i2c-riic.c
+++ b/drivers/i2c/busses/i2c-riic.c
@@ -383,10 +383,8 @@ static int riic_i2c_probe(struct platform_device *pdev)
ret = i2c_add_adapter(adap);
- if (ret) {
- dev_err(&pdev->dev, "failed to add adapter\n");
+ if (ret)
return ret;
- }
platform_set_drvdata(pdev, riic);
diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c
index 2bc8b01153d619..3b87afe82f6394 100644
--- a/drivers/i2c/busses/i2c-rk3x.c
+++ b/drivers/i2c/busses/i2c-rk3x.c
@@ -1303,10 +1303,8 @@ static int rk3x_i2c_probe(struct platform_device *pdev)
rk3x_i2c_adapt_div(i2c, clk_rate);
ret = i2c_add_adapter(&i2c->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "Could not register adapter\n");
+ if (ret < 0)
goto err_clk_notifier;
- }
dev_info(&pdev->dev, "Initialized RK3xxx I2C bus at %p\n", i2c->regs);
diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
index 38dc1cacfd8bba..499af26e736e7e 100644
--- a/drivers/i2c/busses/i2c-s3c2410.c
+++ b/drivers/i2c/busses/i2c-s3c2410.c
@@ -1215,7 +1215,6 @@ static int s3c24xx_i2c_probe(struct platform_device *pdev)
ret = i2c_add_numbered_adapter(&i2c->adap);
if (ret < 0) {
- dev_err(&pdev->dev, "failed to add bus to i2c core\n");
pm_runtime_disable(&pdev->dev);
s3c24xx_i2c_deregister_cpufreq(i2c);
clk_unprepare(i2c->clk);
diff --git a/drivers/i2c/busses/i2c-sh7760.c b/drivers/i2c/busses/i2c-sh7760.c
index 24968384b4014f..c2005c789d2b09 100644
--- a/drivers/i2c/busses/i2c-sh7760.c
+++ b/drivers/i2c/busses/i2c-sh7760.c
@@ -510,10 +510,8 @@ static int sh7760_i2c_probe(struct platform_device *pdev)
}
ret = i2c_add_numbered_adapter(&id->adap);
- if (ret < 0) {
- dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
+ if (ret < 0)
goto out4;
- }
platform_set_drvdata(pdev, id);
diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-sh_mobile.c
index 6fb3e264599229..50f276e8d17967 100644
--- a/drivers/i2c/busses/i2c-sh_mobile.c
+++ b/drivers/i2c/busses/i2c-sh_mobile.c
@@ -981,7 +981,6 @@ static int sh_mobile_i2c_probe(struct platform_device *dev)
ret = i2c_add_numbered_adapter(adap);
if (ret < 0) {
sh_mobile_i2c_release_dma(pd);
- dev_err(&dev->dev, "cannot add numbered adapter\n");
return ret;
}
diff --git a/drivers/i2c/busses/i2c-sirf.c b/drivers/i2c/busses/i2c-sirf.c
index 792a42bdd335b9..95e81d0f72b4d2 100644
--- a/drivers/i2c/busses/i2c-sirf.c
+++ b/drivers/i2c/busses/i2c-sirf.c
@@ -387,10 +387,8 @@ static int i2c_sirfsoc_probe(struct platform_device *pdev)
writel(regval, siic->base + SIRFSOC_I2C_SDA_DELAY);
err = i2c_add_numbered_adapter(adap);
- if (err < 0) {
- dev_err(&pdev->dev, "Can't add new i2c adapter\n");
+ if (err < 0)
goto out;
- }
clk_disable(clk);
diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c
index 944ec420508487..1371547ce1a3a8 100644
--- a/drivers/i2c/busses/i2c-st.c
+++ b/drivers/i2c/busses/i2c-st.c
@@ -874,10 +874,8 @@ static int st_i2c_probe(struct platform_device *pdev)
init_completion(&i2c_dev->complete);
ret = i2c_add_adapter(adap);
- if (ret) {
- dev_err(&pdev->dev, "Failed to add adapter\n");
+ if (ret)
return ret;
- }
platform_set_drvdata(pdev, i2c_dev);
diff --git a/drivers/i2c/busses/i2c-stu300.c b/drivers/i2c/busses/i2c-stu300.c
index 460c134832ac30..dc63236b45b216 100644
--- a/drivers/i2c/busses/i2c-stu300.c
+++ b/drivers/i2c/busses/i2c-stu300.c
@@ -920,11 +920,8 @@ static int stu300_probe(struct platform_device *pdev)
/* i2c device drivers may be active on return from add_adapter() */
ret = i2c_add_numbered_adapter(adap);
- if (ret) {
- dev_err(&pdev->dev, "failure adding ST Micro DDC "
- "I2C adapter\n");
+ if (ret)
return ret;
- }
platform_set_drvdata(pdev, dev);
dev_info(&pdev->dev, "ST DDC I2C @ %p, irq %d\n",
diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c
index b126dbaa47e370..d9979da11485ae 100644
--- a/drivers/i2c/busses/i2c-tegra.c
+++ b/drivers/i2c/busses/i2c-tegra.c
@@ -932,10 +932,8 @@ static int tegra_i2c_probe(struct platform_device *pdev)
i2c_dev->adapter.dev.of_node = pdev->dev.of_node;
ret = i2c_add_numbered_adapter(&i2c_dev->adapter);
- if (ret) {
- dev_err(&pdev->dev, "Failed to add I2C adapter\n");
+ if (ret)
goto disable_div_clk;
- }
return 0;
diff --git a/drivers/i2c/busses/i2c-uniphier-f.c b/drivers/i2c/busses/i2c-uniphier-f.c
index aeead0d27d1007..64318e69089439 100644
--- a/drivers/i2c/busses/i2c-uniphier-f.c
+++ b/drivers/i2c/busses/i2c-uniphier-f.c
@@ -550,15 +550,10 @@ static int uniphier_fi2c_probe(struct platform_device *pdev)
}
ret = i2c_add_adapter(&priv->adap);
- if (ret) {
- dev_err(dev, "failed to add I2C adapter\n");
- goto err;
- }
-
-err:
if (ret)
clk_disable_unprepare(priv->clk);
+ err:
return ret;
}
diff --git a/drivers/i2c/busses/i2c-uniphier.c b/drivers/i2c/busses/i2c-uniphier.c
index 475a5eb514e215..94f64cccfdef08 100644
--- a/drivers/i2c/busses/i2c-uniphier.c
+++ b/drivers/i2c/busses/i2c-uniphier.c
@@ -407,15 +407,10 @@ static int uniphier_i2c_probe(struct platform_device *pdev)
}
ret = i2c_add_adapter(&priv->adap);
- if (ret) {
- dev_err(dev, "failed to add I2C adapter\n");
- goto err;
- }
-
-err:
if (ret)
clk_disable_unprepare(priv->clk);
+ err:
return ret;
}
diff --git a/drivers/i2c/busses/i2c-wmt.c b/drivers/i2c/busses/i2c-wmt.c
index e1e3a85596c562..fbd0fd59f31239 100644
--- a/drivers/i2c/busses/i2c-wmt.c
+++ b/drivers/i2c/busses/i2c-wmt.c
@@ -432,10 +432,8 @@ static int wmt_i2c_probe(struct platform_device *pdev)
}
err = i2c_add_adapter(adap);
- if (err) {
- dev_err(&pdev->dev, "failed to add adapter\n");
+ if (err)
return err;
- }
platform_set_drvdata(pdev, i2c_dev);
diff --git a/drivers/i2c/busses/i2c-xgene-slimpro.c b/drivers/i2c/busses/i2c-xgene-slimpro.c
index 4233f5695352fd..263685c7a51287 100644
--- a/drivers/i2c/busses/i2c-xgene-slimpro.c
+++ b/drivers/i2c/busses/i2c-xgene-slimpro.c
@@ -418,7 +418,6 @@ static int xgene_slimpro_i2c_probe(struct platform_device *pdev)
i2c_set_adapdata(adapter, ctx);
rc = i2c_add_adapter(adapter);
if (rc) {
- dev_err(&pdev->dev, "Adapter registeration failed\n");
mbox_free_channel(ctx->mbox_chan);
return rc;
}
diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c
index 74f54f2f471fa7..66bce3b311a199 100644
--- a/drivers/i2c/busses/i2c-xiic.c
+++ b/drivers/i2c/busses/i2c-xiic.c
@@ -804,7 +804,6 @@ static int xiic_i2c_probe(struct platform_device *pdev)
/* add i2c adapter to i2c tree */
ret = i2c_add_adapter(&i2c->adap);
if (ret) {
- dev_err(&pdev->dev, "Failed to add adapter\n");
xiic_deinit(i2c);
goto err_clk_dis;
}
diff --git a/drivers/i2c/busses/i2c-xlp9xx.c b/drivers/i2c/busses/i2c-xlp9xx.c
index 55a7bef1b2e1df..2a972ed7aa0df1 100644
--- a/drivers/i2c/busses/i2c-xlp9xx.c
+++ b/drivers/i2c/busses/i2c-xlp9xx.c
@@ -400,10 +400,8 @@ static int xlp9xx_i2c_probe(struct platform_device *pdev)
i2c_set_adapdata(&priv->adapter, priv);
err = i2c_add_adapter(&priv->adapter);
- if (err) {
- dev_err(&pdev->dev, "failed to add I2C adapter!\n");
+ if (err)
return err;
- }
platform_set_drvdata(pdev, priv);
dev_dbg(&pdev->dev, "I2C bus:%d added\n", priv->adapter.nr);
diff --git a/drivers/i2c/busses/i2c-xlr.c b/drivers/i2c/busses/i2c-xlr.c
index 613c3a4f2c5142..0968f59b6df586 100644
--- a/drivers/i2c/busses/i2c-xlr.c
+++ b/drivers/i2c/busses/i2c-xlr.c
@@ -432,10 +432,8 @@ static int xlr_i2c_probe(struct platform_device *pdev)
i2c_set_adapdata(&priv->adap, priv);
ret = i2c_add_numbered_adapter(&priv->adap);
- if (ret < 0) {
- dev_err(&priv->adap.dev, "Failed to add i2c bus.\n");
+ if (ret < 0)
return ret;
- }
platform_set_drvdata(pdev, priv);
dev_info(&priv->adap.dev, "Added I2C Bus.\n");
--
2.8.1
^ permalink raw reply related [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
@ 2016-08-09 11:47 ` Grygorii Strashko
2016-08-09 11:48 ` Peter Korsgaard
` (5 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Grygorii Strashko @ 2016-08-09 11:47 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Krzysztof Kozlowski, Heiko Stuebner, Wolfram Sang, Tony Lindgren,
Linus Walleij, Sekhar Nori, Jochen Friedrich, Masahiro Yamada,
Laxman Dewangan, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guan Xuetao,
Peter Korsgaard, Sonic Zhang, kernel-F5mvAk5X5gdBDgjK7y7TUQ,
Alexandre Courbot, Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Kukjin Kim,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Sylvain Lemieux, Guenter Roeck
On 08/09/2016 02:36 PM, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
> ---
For:
> drivers/i2c/busses/i2c-davinci.c | 4 +---
> drivers/i2c/busses/i2c-omap.c | 4 +---
Reviewed-by: Grygorii Strashko <grygorii.strashko-l0cyMroinI0@public.gmane.org>
--
regards,
-grygorii
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:47 ` Grygorii Strashko
@ 2016-08-09 11:48 ` Peter Korsgaard
2016-08-09 12:58 ` Thierry Reding
` (4 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Peter Korsgaard @ 2016-08-09 11:48 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w, Florian Fainelli,
Sonic Zhang, Brian Norris, Gregory Fong, Michal Simek,
Sören Brinkmann, Jochen Friedrich, Sekhar Nori, Kevin Hilman,
Guenter Roeck, Uwe Kleine-König, Kukjin Kim,
Krzysztof Kozlowski
>>>>> "Wolfram" == Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org> writes:
> The core will do this for us now.
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
> ---
> drivers/i2c/busses/i2c-ocores.c | 4 +---
For i2c-ocores.c:
Acked-by: Peter Korsgaard <peter-+2lRwdCCLRT2eFz/2MeuCQ@public.gmane.org>
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
@ 2016-08-09 11:57 ` Uwe Kleine-König
2016-08-09 13:59 ` Wolfram Sang
2016-08-09 12:13 ` Heiko Stübner
` (4 subsequent siblings)
6 siblings, 1 reply; 21+ messages in thread
From: Uwe Kleine-König @ 2016-08-09 11:57 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel, linux-i2c, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list, Florian Fainelli, Sonic Zhang,
Brian Norris, Gregory Fong, Michal Simek, Sören Brinkmann,
Jochen Friedrich, Sekhar Nori, Kevin Hilman, Guenter Roeck,
Uwe Kleine-König, Kukjin Kim, Krzysztof Kozlowski
Hello Wolfram,
On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
> The core will do this for us now.
Would be nice to point out the relevant commit that changed the core in
the commit log.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:57 ` Uwe Kleine-König
@ 2016-08-09 12:13 ` Heiko Stübner
2016-08-09 12:55 ` Neil Horman
` (3 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Heiko Stübner @ 2016-08-09 12:13 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel, linux-i2c, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list, Florian Fainelli, Sonic Zhang,
Brian Norris, Gregory Fong, Michal Simek, Sören Brinkmann,
Jochen Friedrich, Sekhar Nori, Kevin Hilman, Guenter Roeck,
Uwe Kleine-König, Kukjin Kim, Krzysztof Kozlowski
Am Dienstag, 9. August 2016, 13:36:17 schrieb Wolfram Sang:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
> ---
[...]
> diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c
> index 2bc8b01153d619..3b87afe82f6394 100644
> --- a/drivers/i2c/busses/i2c-rk3x.c
> +++ b/drivers/i2c/busses/i2c-rk3x.c
> @@ -1303,10 +1303,8 @@ static int rk3x_i2c_probe(struct platform_device
> *pdev) rk3x_i2c_adapt_div(i2c, clk_rate);
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "Could not register adapter\n");
> + if (ret < 0)
> goto err_clk_notifier;
> - }
>
> dev_info(&pdev->dev, "Initialized RK3xxx I2C bus at %p\n", i2c->regs);
for Rockchip
Acked-by: Heiko Stuebner <heiko@sntech.de>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
` (2 preceding siblings ...)
2016-08-09 12:13 ` Heiko Stübner
@ 2016-08-09 12:55 ` Neil Horman
[not found] ` <20160809125522.GA15676-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2016-08-09 13:31 ` Guenter Roeck
` (2 subsequent siblings)
6 siblings, 1 reply; 21+ messages in thread
From: Neil Horman @ 2016-08-09 12:55 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel, linux-i2c, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list, Florian Fainelli, Sonic Zhang,
Brian Norris, Gregory Fong, Michal Simek, Sören Brinkmann,
Jochen Friedrich, Sekhar Nori, Kevin Hilman, Guenter Roeck,
Uwe Kleine-König, Kukjin Kim, Krzysztof Kozlowski
On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
> ---
> drivers/i2c/busses/i2c-amd756.c | 5 +----
> drivers/i2c/busses/i2c-at91.c | 2 --
> drivers/i2c/busses/i2c-axxia.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-iproc.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-kona.c | 4 +---
> drivers/i2c/busses/i2c-bfin-twi.c | 4 +---
> drivers/i2c/busses/i2c-brcmstb.c | 4 +---
> drivers/i2c/busses/i2c-cadence.c | 4 +---
> drivers/i2c/busses/i2c-cpm.c | 4 +---
> drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +---
> drivers/i2c/busses/i2c-davinci.c | 4 +---
> drivers/i2c/busses/i2c-diolan-u2c.c | 4 +---
> drivers/i2c/busses/i2c-dln2.c | 4 +---
> drivers/i2c/busses/i2c-efm32.c | 1 -
> drivers/i2c/busses/i2c-exynos5.c | 4 +---
> drivers/i2c/busses/i2c-hix5hd2.c | 4 +---
> drivers/i2c/busses/i2c-i801.c | 1 -
> drivers/i2c/busses/i2c-ibm_iic.c | 4 +---
> drivers/i2c/busses/i2c-img-scb.c | 4 +---
> drivers/i2c/busses/i2c-imx.c | 4 +---
> drivers/i2c/busses/i2c-isch.c | 4 +---
> drivers/i2c/busses/i2c-ismt.c | 4 +---
> drivers/i2c/busses/i2c-jz4780.c | 4 +---
> drivers/i2c/busses/i2c-lpc2k.c | 4 +---
> drivers/i2c/busses/i2c-meson.c | 1 -
> drivers/i2c/busses/i2c-mpc.c | 4 +---
> drivers/i2c/busses/i2c-mt65xx.c | 4 +---
> drivers/i2c/busses/i2c-mxs.c | 1 -
> drivers/i2c/busses/i2c-nforce2.c | 1 -
> drivers/i2c/busses/i2c-nomadik.c | 4 +---
> drivers/i2c/busses/i2c-ocores.c | 4 +---
> drivers/i2c/busses/i2c-octeon.c | 4 +---
> drivers/i2c/busses/i2c-omap.c | 4 +---
> drivers/i2c/busses/i2c-piix4.c | 1 -
> drivers/i2c/busses/i2c-pmcmsp.c | 4 +---
> drivers/i2c/busses/i2c-pnx.c | 4 +---
> drivers/i2c/busses/i2c-puv3.c | 5 +----
> drivers/i2c/busses/i2c-pxa.c | 4 +---
> drivers/i2c/busses/i2c-rcar.c | 4 +---
> drivers/i2c/busses/i2c-riic.c | 4 +---
> drivers/i2c/busses/i2c-rk3x.c | 4 +---
> drivers/i2c/busses/i2c-s3c2410.c | 1 -
> drivers/i2c/busses/i2c-sh7760.c | 4 +---
> drivers/i2c/busses/i2c-sh_mobile.c | 1 -
> drivers/i2c/busses/i2c-sirf.c | 4 +---
> drivers/i2c/busses/i2c-st.c | 4 +---
> drivers/i2c/busses/i2c-stu300.c | 5 +----
> drivers/i2c/busses/i2c-tegra.c | 4 +---
> drivers/i2c/busses/i2c-uniphier-f.c | 7 +------
> drivers/i2c/busses/i2c-uniphier.c | 7 +------
> drivers/i2c/busses/i2c-wmt.c | 4 +---
> drivers/i2c/busses/i2c-xgene-slimpro.c | 1 -
> drivers/i2c/busses/i2c-xiic.c | 1 -
> drivers/i2c/busses/i2c-xlp9xx.c | 4 +---
> drivers/i2c/busses/i2c-xlr.c | 4 +---
> 55 files changed, 44 insertions(+), 161 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-amd756.c b/drivers/i2c/busses/i2c-amd756.c
> index 6c7113d990f882..274908cd1fdeab 100644
> --- a/drivers/i2c/busses/i2c-amd756.c
> +++ b/drivers/i2c/busses/i2c-amd756.c
> @@ -378,11 +378,8 @@ static int amd756_probe(struct pci_dev *pdev, const struct pci_device_id *id)
> amd756_ioport);
>
> error = i2c_add_adapter(&amd756_smbus);
> - if (error) {
> - dev_err(&pdev->dev,
> - "Adapter registration failed, module not inserted\n");
> + if (error)
> goto out_err;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-at91.c b/drivers/i2c/busses/i2c-at91.c
> index f23372669f770f..03a519d29616d7 100644
> --- a/drivers/i2c/busses/i2c-at91.c
> +++ b/drivers/i2c/busses/i2c-at91.c
> @@ -1118,8 +1118,6 @@ static int at91_twi_probe(struct platform_device *pdev)
>
> rc = i2c_add_numbered_adapter(&dev->adapter);
> if (rc) {
> - dev_err(dev->dev, "Adapter %s registration failed\n",
> - dev->adapter.name);
> clk_disable_unprepare(dev->clk);
>
> pm_runtime_disable(dev->dev);
> diff --git a/drivers/i2c/busses/i2c-axxia.c b/drivers/i2c/busses/i2c-axxia.c
> index c335cc7852f94e..d3bcaf4ab095d0 100644
> --- a/drivers/i2c/busses/i2c-axxia.c
> +++ b/drivers/i2c/busses/i2c-axxia.c
> @@ -558,13 +558,7 @@ static int axxia_i2c_probe(struct platform_device *pdev)
>
> platform_set_drvdata(pdev, idev);
>
> - ret = i2c_add_adapter(&idev->adapter);
> - if (ret) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> - return ret;
> - }
> -
> - return 0;
> + return i2c_add_adapter(&idev->adapter);
> }
>
> static int axxia_i2c_remove(struct platform_device *pdev)
> diff --git a/drivers/i2c/busses/i2c-bcm-iproc.c b/drivers/i2c/busses/i2c-bcm-iproc.c
> index 19c843828fe2ca..8e3477f9297eda 100644
> --- a/drivers/i2c/busses/i2c-bcm-iproc.c
> +++ b/drivers/i2c/busses/i2c-bcm-iproc.c
> @@ -488,13 +488,7 @@ static int bcm_iproc_i2c_probe(struct platform_device *pdev)
> adap->dev.parent = &pdev->dev;
> adap->dev.of_node = pdev->dev.of_node;
>
> - ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(iproc_i2c->device, "failed to add adapter\n");
> - return ret;
> - }
> -
> - return 0;
> + return i2c_add_adapter(adap);
> }
>
> static int bcm_iproc_i2c_remove(struct platform_device *pdev)
> diff --git a/drivers/i2c/busses/i2c-bcm-kona.c b/drivers/i2c/busses/i2c-bcm-kona.c
> index ac9f47679c3a4b..a5c9098507896b 100644
> --- a/drivers/i2c/busses/i2c-bcm-kona.c
> +++ b/drivers/i2c/busses/i2c-bcm-kona.c
> @@ -858,10 +858,8 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev)
> adap->dev.of_node = pdev->dev.of_node;
>
> rc = i2c_add_adapter(adap);
> - if (rc) {
> - dev_err(dev->device, "failed to add adapter\n");
> + if (rc)
> return rc;
> - }
>
> dev_info(dev->device, "device registered successfully\n");
>
> diff --git a/drivers/i2c/busses/i2c-bfin-twi.c b/drivers/i2c/busses/i2c-bfin-twi.c
> index 025686d4164058..29d00c4f7824c2 100644
> --- a/drivers/i2c/busses/i2c-bfin-twi.c
> +++ b/drivers/i2c/busses/i2c-bfin-twi.c
> @@ -685,10 +685,8 @@ static int i2c_bfin_twi_probe(struct platform_device *pdev)
> write_CONTROL(iface, read_CONTROL(iface) | TWI_ENA);
>
> rc = i2c_add_numbered_adapter(p_adap);
> - if (rc < 0) {
> - dev_err(&pdev->dev, "Can't add i2c adapter!\n");
> + if (rc < 0)
> goto out_error;
> - }
>
> platform_set_drvdata(pdev, iface);
>
> diff --git a/drivers/i2c/busses/i2c-brcmstb.c b/drivers/i2c/busses/i2c-brcmstb.c
> index 3f5a4d71d3bf32..bc5c176e984aea 100644
> --- a/drivers/i2c/busses/i2c-brcmstb.c
> +++ b/drivers/i2c/busses/i2c-brcmstb.c
> @@ -648,10 +648,8 @@ static int brcmstb_i2c_probe(struct platform_device *pdev)
> adap->dev.parent = &pdev->dev;
> adap->dev.of_node = pdev->dev.of_node;
> rc = i2c_add_adapter(adap);
> - if (rc) {
> - dev_err(dev->device, "failed to add adapter\n");
> + if (rc)
> goto probe_errorout;
> - }
>
> dev_info(dev->device, "%s@%dhz registered in %s mode\n",
> int_name ? int_name : " ", dev->clk_freq_hz,
> diff --git a/drivers/i2c/busses/i2c-cadence.c b/drivers/i2c/busses/i2c-cadence.c
> index 90bbd9f9dd8f77..1cc1e79e958d56 100644
> --- a/drivers/i2c/busses/i2c-cadence.c
> +++ b/drivers/i2c/busses/i2c-cadence.c
> @@ -963,10 +963,8 @@ static int cdns_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&id->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto err_clk_dis;
> - }
>
> /*
> * Cadence I2C controller has a bug wherein it generates
> diff --git a/drivers/i2c/busses/i2c-cpm.c b/drivers/i2c/busses/i2c-cpm.c
> index ee57c1e865e271..d89bde2c5da256 100644
> --- a/drivers/i2c/busses/i2c-cpm.c
> +++ b/drivers/i2c/busses/i2c-cpm.c
> @@ -665,10 +665,8 @@ static int cpm_i2c_probe(struct platform_device *ofdev)
> cpm->adap.nr = (data && len == 4) ? be32_to_cpup(data) : -1;
> result = i2c_add_numbered_adapter(&cpm->adap);
>
> - if (result < 0) {
> - dev_err(&ofdev->dev, "Unable to register with I2C\n");
> + if (result < 0)
> goto out_shut;
> - }
>
> dev_dbg(&ofdev->dev, "hw routines for %s registered.\n",
> cpm->adap.name);
> diff --git a/drivers/i2c/busses/i2c-cros-ec-tunnel.c b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> index a0d95ff682ae12..cf334a9ec22cb7 100644
> --- a/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +++ b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> @@ -281,10 +281,8 @@ static int ec_i2c_probe(struct platform_device *pdev)
> bus->adap.retries = I2C_MAX_RETRIES;
>
> err = i2c_add_adapter(&bus->adap);
> - if (err) {
> - dev_err(dev, "cannot register i2c adapter\n");
> + if (err)
> return err;
> - }
> platform_set_drvdata(pdev, bus);
>
> return err;
> diff --git a/drivers/i2c/busses/i2c-davinci.c b/drivers/i2c/busses/i2c-davinci.c
> index a8bdcb5292f5b3..9e7ef5cf5d495d 100644
> --- a/drivers/i2c/busses/i2c-davinci.c
> +++ b/drivers/i2c/busses/i2c-davinci.c
> @@ -846,10 +846,8 @@ static int davinci_i2c_probe(struct platform_device *pdev)
>
> adap->nr = pdev->id;
> r = i2c_add_numbered_adapter(adap);
> - if (r) {
> - dev_err(&pdev->dev, "failure adding adapter\n");
> + if (r)
> goto err_unuse_clocks;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-diolan-u2c.c b/drivers/i2c/busses/i2c-diolan-u2c.c
> index b19a310bf9b3e8..f718ee4e3332fe 100644
> --- a/drivers/i2c/busses/i2c-diolan-u2c.c
> +++ b/drivers/i2c/busses/i2c-diolan-u2c.c
> @@ -487,10 +487,8 @@ static int diolan_u2c_probe(struct usb_interface *interface,
>
> /* and finally attach to i2c layer */
> ret = i2c_add_adapter(&dev->adapter);
> - if (ret < 0) {
> - dev_err(&interface->dev, "failed to add I2C adapter\n");
> + if (ret < 0)
> goto error_free;
> - }
>
> dev_dbg(&interface->dev, "connected " DRIVER_NAME "\n");
>
> diff --git a/drivers/i2c/busses/i2c-dln2.c b/drivers/i2c/busses/i2c-dln2.c
> index f2eb4f76591fbe..8acda2aa15583c 100644
> --- a/drivers/i2c/busses/i2c-dln2.c
> +++ b/drivers/i2c/busses/i2c-dln2.c
> @@ -228,10 +228,8 @@ static int dln2_i2c_probe(struct platform_device *pdev)
>
> /* and finally attach to i2c layer */
> ret = i2c_add_adapter(&dln2->adapter);
> - if (ret < 0) {
> - dev_err(dev, "failed to add I2C adapter: %d\n", ret);
> + if (ret < 0)
> goto out_disable;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-efm32.c b/drivers/i2c/busses/i2c-efm32.c
> index e253598d764c0d..aa336ba89aa3fe 100644
> --- a/drivers/i2c/busses/i2c-efm32.c
> +++ b/drivers/i2c/busses/i2c-efm32.c
> @@ -438,7 +438,6 @@ static int efm32_i2c_probe(struct platform_device *pdev)
>
> ret = i2c_add_adapter(&ddata->adapter);
> if (ret) {
> - dev_err(&pdev->dev, "failed to add i2c adapter (%d)\n", ret);
> free_irq(ddata->irq, ddata);
>
> err_disable_clk:
> diff --git a/drivers/i2c/busses/i2c-exynos5.c b/drivers/i2c/busses/i2c-exynos5.c
> index c0e3ada028763d..bea607149972d4 100644
> --- a/drivers/i2c/busses/i2c-exynos5.c
> +++ b/drivers/i2c/busses/i2c-exynos5.c
> @@ -796,10 +796,8 @@ static int exynos5_i2c_probe(struct platform_device *pdev)
> exynos5_i2c_reset(i2c);
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> + if (ret < 0)
> goto err_clk;
> - }
>
> platform_set_drvdata(pdev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-hix5hd2.c b/drivers/i2c/busses/i2c-hix5hd2.c
> index 7c6966434ee7b9..ae7f3180f7e887 100644
> --- a/drivers/i2c/busses/i2c-hix5hd2.c
> +++ b/drivers/i2c/busses/i2c-hix5hd2.c
> @@ -478,10 +478,8 @@ static int hix5hd2_i2c_probe(struct platform_device *pdev)
> pm_runtime_enable(priv->dev);
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> + if (ret < 0)
> goto err_runtime;
> - }
>
> return ret;
>
> diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> index 5ef9b733d153fa..22a0ed4de80da0 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1613,7 +1613,6 @@ static int i801_probe(struct pci_dev *dev, const struct pci_device_id *id)
> "SMBus I801 adapter at %04lx", priv->smba);
> err = i2c_add_adapter(&priv->adapter);
> if (err) {
> - dev_err(&dev->dev, "Failed to add SMBus adapter\n");
> i801_acpi_remove(priv);
> return err;
> }
> diff --git a/drivers/i2c/busses/i2c-ibm_iic.c b/drivers/i2c/busses/i2c-ibm_iic.c
> index cdaa7be2cd1b94..412b91d255ad1d 100644
> --- a/drivers/i2c/busses/i2c-ibm_iic.c
> +++ b/drivers/i2c/busses/i2c-ibm_iic.c
> @@ -751,10 +751,8 @@ static int iic_probe(struct platform_device *ofdev)
> adap->timeout = HZ;
>
> ret = i2c_add_adapter(adap);
> - if (ret < 0) {
> - dev_err(&ofdev->dev, "failed to register i2c adapter\n");
> + if (ret < 0)
> goto error_cleanup;
> - }
>
> dev_info(&ofdev->dev, "using %s mode\n",
> dev->fast_mode ? "fast (400 kHz)" : "standard (100 kHz)");
> diff --git a/drivers/i2c/busses/i2c-img-scb.c b/drivers/i2c/busses/i2c-img-scb.c
> index ea20425b697217..db8e8b40569d98 100644
> --- a/drivers/i2c/busses/i2c-img-scb.c
> +++ b/drivers/i2c/busses/i2c-img-scb.c
> @@ -1394,10 +1394,8 @@ static int img_i2c_probe(struct platform_device *pdev)
> goto disable_clk;
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (ret < 0)
> goto disable_clk;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c
> index 1844bc9f7cd5a2..cb11eee9567318 100644
> --- a/drivers/i2c/busses/i2c-imx.c
> +++ b/drivers/i2c/busses/i2c-imx.c
> @@ -1129,10 +1129,8 @@ static int i2c_imx_probe(struct platform_device *pdev)
>
> /* Add I2C adapter */
> ret = i2c_add_numbered_adapter(&i2c_imx->adapter);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "registration failed\n");
> + if (ret < 0)
> goto rpm_disable;
> - }
>
> pm_runtime_mark_last_busy(&pdev->dev);
> pm_runtime_put_autosuspend(&pdev->dev);
> diff --git a/drivers/i2c/busses/i2c-isch.c b/drivers/i2c/busses/i2c-isch.c
> index c2f25f19d76f9b..0cf1379f4e8091 100644
> --- a/drivers/i2c/busses/i2c-isch.c
> +++ b/drivers/i2c/busses/i2c-isch.c
> @@ -288,10 +288,8 @@ static int smbus_sch_probe(struct platform_device *dev)
> "SMBus SCH adapter at %04x", sch_smba);
>
> retval = i2c_add_adapter(&sch_adapter);
> - if (retval) {
> - dev_err(&dev->dev, "Couldn't register adapter!\n");
> + if (retval)
> sch_smba = 0;
> - }
>
> return retval;
> }
> diff --git a/drivers/i2c/busses/i2c-ismt.c b/drivers/i2c/busses/i2c-ismt.c
> index 1c870771009840..f573448d213216 100644
> --- a/drivers/i2c/busses/i2c-ismt.c
> +++ b/drivers/i2c/busses/i2c-ismt.c
> @@ -922,10 +922,8 @@ ismt_probe(struct pci_dev *pdev, const struct pci_device_id *id)
> return err;
>
> err = i2c_add_adapter(&priv->adapter);
> - if (err) {
> - dev_err(&pdev->dev, "Failed to add SMBus iSMT adapter\n");
> + if (err)
> return -ENODEV;
> - }
> return 0;
> }
>
> diff --git a/drivers/i2c/busses/i2c-jz4780.c b/drivers/i2c/busses/i2c-jz4780.c
> index cd9872594fe2a0..b8ea62105f42c9 100644
> --- a/drivers/i2c/busses/i2c-jz4780.c
> +++ b/drivers/i2c/busses/i2c-jz4780.c
> @@ -798,10 +798,8 @@ static int jz4780_i2c_probe(struct platform_device *pdev)
> goto err;
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "Failed to add bus\n");
> + if (ret < 0)
> goto err;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-lpc2k.c b/drivers/i2c/busses/i2c-lpc2k.c
> index 586a15205e61de..9b1fef455a895d 100644
> --- a/drivers/i2c/busses/i2c-lpc2k.c
> +++ b/drivers/i2c/busses/i2c-lpc2k.c
> @@ -432,10 +432,8 @@ static int i2c_lpc2k_probe(struct platform_device *pdev)
> i2c->adap.dev.of_node = pdev->dev.of_node;
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add adapter!\n");
> + if (ret < 0)
> goto fail_clk;
> - }
>
> dev_info(&pdev->dev, "LPC2K I2C adapter\n");
>
> diff --git a/drivers/i2c/busses/i2c-meson.c b/drivers/i2c/busses/i2c-meson.c
> index 71d3929adf54ed..102d2b91771f35 100644
> --- a/drivers/i2c/busses/i2c-meson.c
> +++ b/drivers/i2c/busses/i2c-meson.c
> @@ -453,7 +453,6 @@ static int meson_i2c_probe(struct platform_device *pdev)
>
> ret = i2c_add_adapter(&i2c->adap);
> if (ret < 0) {
> - dev_err(&pdev->dev, "can't register adapter\n");
> clk_unprepare(i2c->clk);
> return ret;
> }
> diff --git a/drivers/i2c/busses/i2c-mpc.c b/drivers/i2c/busses/i2c-mpc.c
> index 48ecffecc0eda3..565a49a0c56410 100644
> --- a/drivers/i2c/busses/i2c-mpc.c
> +++ b/drivers/i2c/busses/i2c-mpc.c
> @@ -737,10 +737,8 @@ static int fsl_i2c_probe(struct platform_device *op)
> i2c->adap.dev.of_node = of_node_get(op->dev.of_node);
>
> result = i2c_add_adapter(&i2c->adap);
> - if (result < 0) {
> - dev_err(i2c->dev, "failed to add adapter\n");
> + if (result < 0)
> goto fail_add;
> - }
>
> return result;
>
> diff --git a/drivers/i2c/busses/i2c-mt65xx.c b/drivers/i2c/busses/i2c-mt65xx.c
> index d9373e60be8ae3..4a7d9bc2142ba3 100644
> --- a/drivers/i2c/busses/i2c-mt65xx.c
> +++ b/drivers/i2c/busses/i2c-mt65xx.c
> @@ -786,10 +786,8 @@ static int mtk_i2c_probe(struct platform_device *pdev)
>
> i2c_set_adapdata(&i2c->adap, i2c);
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add i2c bus to i2c core\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-mxs.c b/drivers/i2c/busses/i2c-mxs.c
> index 033846cdf26622..5738556b6aac20 100644
> --- a/drivers/i2c/busses/i2c-mxs.c
> +++ b/drivers/i2c/busses/i2c-mxs.c
> @@ -868,7 +868,6 @@ static int mxs_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(adap, i2c);
> err = i2c_add_numbered_adapter(adap);
> if (err) {
> - dev_err(dev, "Failed to add adapter (%d)\n", err);
> writel(MXS_I2C_CTRL0_SFTRST,
> i2c->regs + MXS_I2C_CTRL0_SET);
> return err;
> diff --git a/drivers/i2c/busses/i2c-nforce2.c b/drivers/i2c/busses/i2c-nforce2.c
> index 42fcc9458432f8..374b35e7e450ce 100644
> --- a/drivers/i2c/busses/i2c-nforce2.c
> +++ b/drivers/i2c/busses/i2c-nforce2.c
> @@ -366,7 +366,6 @@ static int nforce2_probe_smb(struct pci_dev *dev, int bar, int alt_reg,
>
> error = i2c_add_adapter(&smbus->adapter);
> if (error) {
> - dev_err(&smbus->adapter.dev, "Failed to register adapter.\n");
> release_region(smbus->base, smbus->size);
> return error;
> }
> diff --git a/drivers/i2c/busses/i2c-nomadik.c b/drivers/i2c/busses/i2c-nomadik.c
> index bcd17e8cbcb4b3..da6609d628487b 100644
> --- a/drivers/i2c/busses/i2c-nomadik.c
> +++ b/drivers/i2c/busses/i2c-nomadik.c
> @@ -1046,10 +1046,8 @@ static int nmk_i2c_probe(struct amba_device *adev, const struct amba_id *id)
> adap->name, dev->virtbase);
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&adev->dev, "failed to add adapter\n");
> + if (ret)
> goto err_no_adap;
> - }
>
> pm_runtime_put(&adev->dev);
>
> diff --git a/drivers/i2c/busses/i2c-ocores.c b/drivers/i2c/busses/i2c-ocores.c
> index dfa7a4b4a91d78..92692349d0c328 100644
> --- a/drivers/i2c/busses/i2c-ocores.c
> +++ b/drivers/i2c/busses/i2c-ocores.c
> @@ -492,10 +492,8 @@ static int ocores_i2c_probe(struct platform_device *pdev)
>
> /* add i2c adapter to i2c tree */
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> /* add in known devices to the bus */
> if (pdata) {
> diff --git a/drivers/i2c/busses/i2c-octeon.c b/drivers/i2c/busses/i2c-octeon.c
> index 30ae35146723a4..b75d2da79b27c5 100644
> --- a/drivers/i2c/busses/i2c-octeon.c
> +++ b/drivers/i2c/busses/i2c-octeon.c
> @@ -1213,10 +1213,8 @@ static int octeon_i2c_probe(struct platform_device *pdev)
> platform_set_drvdata(pdev, i2c);
>
> result = i2c_add_adapter(&i2c->adap);
> - if (result < 0) {
> - dev_err(i2c->dev, "failed to add adapter\n");
> + if (result < 0)
> goto out;
> - }
> dev_info(i2c->dev, "probed\n");
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-omap.c b/drivers/i2c/busses/i2c-omap.c
> index ab1279b8e2407a..c7da0c42baeefa 100644
> --- a/drivers/i2c/busses/i2c-omap.c
> +++ b/drivers/i2c/busses/i2c-omap.c
> @@ -1425,10 +1425,8 @@ omap_i2c_probe(struct platform_device *pdev)
> /* i2c device drivers may be active on return from add_adapter() */
> adap->nr = pdev->id;
> r = i2c_add_numbered_adapter(adap);
> - if (r) {
> - dev_err(omap->dev, "failure adding adapter\n");
> + if (r)
> goto err_unuse_clocks;
> - }
>
> dev_info(omap->dev, "bus %d rev%d.%d at %d kHz\n", adap->nr,
> major, minor, omap->speed);
> diff --git a/drivers/i2c/busses/i2c-piix4.c b/drivers/i2c/busses/i2c-piix4.c
> index 23d1c167b5d752..c2268cdf38e823 100644
> --- a/drivers/i2c/busses/i2c-piix4.c
> +++ b/drivers/i2c/busses/i2c-piix4.c
> @@ -694,7 +694,6 @@ static int piix4_add_adapter(struct pci_dev *dev, unsigned short smba,
>
> retval = i2c_add_adapter(adap);
> if (retval) {
> - dev_err(&dev->dev, "Couldn't register adapter!\n");
> kfree(adapdata);
> kfree(adap);
> release_region(smba, SMBIOSIZE);
> diff --git a/drivers/i2c/busses/i2c-pmcmsp.c b/drivers/i2c/busses/i2c-pmcmsp.c
> index 2c40edbf6224eb..217c78711d6501 100644
> --- a/drivers/i2c/busses/i2c-pmcmsp.c
> +++ b/drivers/i2c/busses/i2c-pmcmsp.c
> @@ -329,10 +329,8 @@ static int pmcmsptwi_probe(struct platform_device *pldev)
> i2c_set_adapdata(&pmcmsptwi_adapter, &pmcmsptwi_data);
>
> rc = i2c_add_adapter(&pmcmsptwi_adapter);
> - if (rc) {
> - dev_err(&pldev->dev, "Unable to register I2C adapter\n");
> + if (rc)
> goto ret_unmap;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-pnx.c b/drivers/i2c/busses/i2c-pnx.c
> index 7ea67aa46fb730..fd5f9d2bf6d94e 100644
> --- a/drivers/i2c/busses/i2c-pnx.c
> +++ b/drivers/i2c/busses/i2c-pnx.c
> @@ -714,10 +714,8 @@ static int i2c_pnx_probe(struct platform_device *pdev)
>
> /* Register this adapter with the I2C subsystem */
> ret = i2c_add_numbered_adapter(&alg_data->adapter);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "I2C: Failed to add bus\n");
> + if (ret < 0)
> goto out_clock;
> - }
>
> dev_dbg(&pdev->dev, "%s: Master at %#8x, irq %d.\n",
> alg_data->adapter.name, res->start, alg_data->irq);
> diff --git a/drivers/i2c/busses/i2c-puv3.c b/drivers/i2c/busses/i2c-puv3.c
> index 82b6f02544dae7..0c8b1571886d5b 100644
> --- a/drivers/i2c/busses/i2c-puv3.c
> +++ b/drivers/i2c/busses/i2c-puv3.c
> @@ -212,11 +212,8 @@ static int puv3_i2c_probe(struct platform_device *pdev)
>
> adapter->nr = pdev->id;
> rc = i2c_add_numbered_adapter(adapter);
> - if (rc) {
> - dev_err(&pdev->dev, "Adapter '%s' registration failed\n",
> - adapter->name);
> + if (rc)
> goto fail_add_adapter;
> - }
>
> dev_info(&pdev->dev, "PKUnity v3 i2c bus adapter.\n");
> return 0;
> diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c
> index 0d351954db02b9..e28b825b043378 100644
> --- a/drivers/i2c/busses/i2c-pxa.c
> +++ b/drivers/i2c/busses/i2c-pxa.c
> @@ -1292,10 +1292,8 @@ static int i2c_pxa_probe(struct platform_device *dev)
> #endif
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&dev->dev, "failed to add bus: %d\n", ret);
> + if (ret < 0)
> goto ereqirq;
> - }
>
> platform_set_drvdata(dev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-rcar.c b/drivers/i2c/busses/i2c-rcar.c
> index 52407f3c9e1cce..400e344704d273 100644
> --- a/drivers/i2c/busses/i2c-rcar.c
> +++ b/drivers/i2c/busses/i2c-rcar.c
> @@ -875,10 +875,8 @@ static int rcar_i2c_probe(struct platform_device *pdev)
> platform_set_drvdata(pdev, priv);
>
> ret = i2c_add_numbered_adapter(adap);
> - if (ret < 0) {
> - dev_err(dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto out_pm_disable;
> - }
>
> dev_info(dev, "probed\n");
>
> diff --git a/drivers/i2c/busses/i2c-riic.c b/drivers/i2c/busses/i2c-riic.c
> index d7e3af67154373..6263ea82d6ac3b 100644
> --- a/drivers/i2c/busses/i2c-riic.c
> +++ b/drivers/i2c/busses/i2c-riic.c
> @@ -383,10 +383,8 @@ static int riic_i2c_probe(struct platform_device *pdev)
>
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, riic);
>
> diff --git a/drivers/i2c/busses/i2c-rk3x.c b/drivers/i2c/busses/i2c-rk3x.c
> index 2bc8b01153d619..3b87afe82f6394 100644
> --- a/drivers/i2c/busses/i2c-rk3x.c
> +++ b/drivers/i2c/busses/i2c-rk3x.c
> @@ -1303,10 +1303,8 @@ static int rk3x_i2c_probe(struct platform_device *pdev)
> rk3x_i2c_adapt_div(i2c, clk_rate);
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "Could not register adapter\n");
> + if (ret < 0)
> goto err_clk_notifier;
> - }
>
> dev_info(&pdev->dev, "Initialized RK3xxx I2C bus at %p\n", i2c->regs);
>
> diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
> index 38dc1cacfd8bba..499af26e736e7e 100644
> --- a/drivers/i2c/busses/i2c-s3c2410.c
> +++ b/drivers/i2c/busses/i2c-s3c2410.c
> @@ -1215,7 +1215,6 @@ static int s3c24xx_i2c_probe(struct platform_device *pdev)
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> pm_runtime_disable(&pdev->dev);
> s3c24xx_i2c_deregister_cpufreq(i2c);
> clk_unprepare(i2c->clk);
> diff --git a/drivers/i2c/busses/i2c-sh7760.c b/drivers/i2c/busses/i2c-sh7760.c
> index 24968384b4014f..c2005c789d2b09 100644
> --- a/drivers/i2c/busses/i2c-sh7760.c
> +++ b/drivers/i2c/busses/i2c-sh7760.c
> @@ -510,10 +510,8 @@ static int sh7760_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_numbered_adapter(&id->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto out4;
> - }
>
> platform_set_drvdata(pdev, id);
>
> diff --git a/drivers/i2c/busses/i2c-sh_mobile.c b/drivers/i2c/busses/i2c-sh_mobile.c
> index 6fb3e264599229..50f276e8d17967 100644
> --- a/drivers/i2c/busses/i2c-sh_mobile.c
> +++ b/drivers/i2c/busses/i2c-sh_mobile.c
> @@ -981,7 +981,6 @@ static int sh_mobile_i2c_probe(struct platform_device *dev)
> ret = i2c_add_numbered_adapter(adap);
> if (ret < 0) {
> sh_mobile_i2c_release_dma(pd);
> - dev_err(&dev->dev, "cannot add numbered adapter\n");
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-sirf.c b/drivers/i2c/busses/i2c-sirf.c
> index 792a42bdd335b9..95e81d0f72b4d2 100644
> --- a/drivers/i2c/busses/i2c-sirf.c
> +++ b/drivers/i2c/busses/i2c-sirf.c
> @@ -387,10 +387,8 @@ static int i2c_sirfsoc_probe(struct platform_device *pdev)
> writel(regval, siic->base + SIRFSOC_I2C_SDA_DELAY);
>
> err = i2c_add_numbered_adapter(adap);
> - if (err < 0) {
> - dev_err(&pdev->dev, "Can't add new i2c adapter\n");
> + if (err < 0)
> goto out;
> - }
>
> clk_disable(clk);
>
> diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c
> index 944ec420508487..1371547ce1a3a8 100644
> --- a/drivers/i2c/busses/i2c-st.c
> +++ b/drivers/i2c/busses/i2c-st.c
> @@ -874,10 +874,8 @@ static int st_i2c_probe(struct platform_device *pdev)
> init_completion(&i2c_dev->complete);
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, i2c_dev);
>
> diff --git a/drivers/i2c/busses/i2c-stu300.c b/drivers/i2c/busses/i2c-stu300.c
> index 460c134832ac30..dc63236b45b216 100644
> --- a/drivers/i2c/busses/i2c-stu300.c
> +++ b/drivers/i2c/busses/i2c-stu300.c
> @@ -920,11 +920,8 @@ static int stu300_probe(struct platform_device *pdev)
>
> /* i2c device drivers may be active on return from add_adapter() */
> ret = i2c_add_numbered_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "failure adding ST Micro DDC "
> - "I2C adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, dev);
> dev_info(&pdev->dev, "ST DDC I2C @ %p, irq %d\n",
> diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c
> index b126dbaa47e370..d9979da11485ae 100644
> --- a/drivers/i2c/busses/i2c-tegra.c
> +++ b/drivers/i2c/busses/i2c-tegra.c
> @@ -932,10 +932,8 @@ static int tegra_i2c_probe(struct platform_device *pdev)
> i2c_dev->adapter.dev.of_node = pdev->dev.of_node;
>
> ret = i2c_add_numbered_adapter(&i2c_dev->adapter);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add I2C adapter\n");
> + if (ret)
> goto disable_div_clk;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-uniphier-f.c b/drivers/i2c/busses/i2c-uniphier-f.c
> index aeead0d27d1007..64318e69089439 100644
> --- a/drivers/i2c/busses/i2c-uniphier-f.c
> +++ b/drivers/i2c/busses/i2c-uniphier-f.c
> @@ -550,15 +550,10 @@ static int uniphier_fi2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-uniphier.c b/drivers/i2c/busses/i2c-uniphier.c
> index 475a5eb514e215..94f64cccfdef08 100644
> --- a/drivers/i2c/busses/i2c-uniphier.c
> +++ b/drivers/i2c/busses/i2c-uniphier.c
> @@ -407,15 +407,10 @@ static int uniphier_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-wmt.c b/drivers/i2c/busses/i2c-wmt.c
> index e1e3a85596c562..fbd0fd59f31239 100644
> --- a/drivers/i2c/busses/i2c-wmt.c
> +++ b/drivers/i2c/busses/i2c-wmt.c
> @@ -432,10 +432,8 @@ static int wmt_i2c_probe(struct platform_device *pdev)
> }
>
> err = i2c_add_adapter(adap);
> - if (err) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (err)
> return err;
> - }
>
> platform_set_drvdata(pdev, i2c_dev);
>
> diff --git a/drivers/i2c/busses/i2c-xgene-slimpro.c b/drivers/i2c/busses/i2c-xgene-slimpro.c
> index 4233f5695352fd..263685c7a51287 100644
> --- a/drivers/i2c/busses/i2c-xgene-slimpro.c
> +++ b/drivers/i2c/busses/i2c-xgene-slimpro.c
> @@ -418,7 +418,6 @@ static int xgene_slimpro_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(adapter, ctx);
> rc = i2c_add_adapter(adapter);
> if (rc) {
> - dev_err(&pdev->dev, "Adapter registeration failed\n");
> mbox_free_channel(ctx->mbox_chan);
> return rc;
> }
> diff --git a/drivers/i2c/busses/i2c-xiic.c b/drivers/i2c/busses/i2c-xiic.c
> index 74f54f2f471fa7..66bce3b311a199 100644
> --- a/drivers/i2c/busses/i2c-xiic.c
> +++ b/drivers/i2c/busses/i2c-xiic.c
> @@ -804,7 +804,6 @@ static int xiic_i2c_probe(struct platform_device *pdev)
> /* add i2c adapter to i2c tree */
> ret = i2c_add_adapter(&i2c->adap);
> if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> xiic_deinit(i2c);
> goto err_clk_dis;
> }
> diff --git a/drivers/i2c/busses/i2c-xlp9xx.c b/drivers/i2c/busses/i2c-xlp9xx.c
> index 55a7bef1b2e1df..2a972ed7aa0df1 100644
> --- a/drivers/i2c/busses/i2c-xlp9xx.c
> +++ b/drivers/i2c/busses/i2c-xlp9xx.c
> @@ -400,10 +400,8 @@ static int xlp9xx_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(&priv->adapter, priv);
>
> err = i2c_add_adapter(&priv->adapter);
> - if (err) {
> - dev_err(&pdev->dev, "failed to add I2C adapter!\n");
> + if (err)
> return err;
> - }
>
> platform_set_drvdata(pdev, priv);
> dev_dbg(&pdev->dev, "I2C bus:%d added\n", priv->adapter.nr);
> diff --git a/drivers/i2c/busses/i2c-xlr.c b/drivers/i2c/busses/i2c-xlr.c
> index 613c3a4f2c5142..0968f59b6df586 100644
> --- a/drivers/i2c/busses/i2c-xlr.c
> +++ b/drivers/i2c/busses/i2c-xlr.c
> @@ -432,10 +432,8 @@ static int xlr_i2c_probe(struct platform_device *pdev)
>
> i2c_set_adapdata(&priv->adap, priv);
> ret = i2c_add_numbered_adapter(&priv->adap);
> - if (ret < 0) {
> - dev_err(&priv->adap.dev, "Failed to add i2c bus.\n");
> + if (ret < 0)
> return ret;
> - }
>
> platform_set_drvdata(pdev, priv);
> dev_info(&priv->adap.dev, "Added I2C Bus.\n");
> --
> 2.8.1
>
>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:47 ` Grygorii Strashko
2016-08-09 11:48 ` Peter Korsgaard
@ 2016-08-09 12:58 ` Thierry Reding
2016-08-09 16:15 ` Ray Jui
` (3 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Thierry Reding @ 2016-08-09 12:58 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA,
linux-i2c-u79uwXL29TY76Z2rM5mHXA, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w, Florian Fainelli,
Sonic Zhang, Brian Norris, Gregory Fong, Michal Simek,
Sören Brinkmann, Jochen Friedrich, Sekhar Nori, Kevin Hilman,
Guenter Roeck, Uwe Kleine-König, Kukjin Kim,
Krzysztof Kozlowski
[-- Attachment #1: Type: text/plain, Size: 700 bytes --]
On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
[...]
> diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c
> index b126dbaa47e370..d9979da11485ae 100644
> --- a/drivers/i2c/busses/i2c-tegra.c
> +++ b/drivers/i2c/busses/i2c-tegra.c
> @@ -932,10 +932,8 @@ static int tegra_i2c_probe(struct platform_device *pdev)
> i2c_dev->adapter.dev.of_node = pdev->dev.of_node;
>
> ret = i2c_add_numbered_adapter(&i2c_dev->adapter);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add I2C adapter\n");
> + if (ret)
> goto disable_div_clk;
> - }
>
> return 0;
>
Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
` (3 preceding siblings ...)
2016-08-09 12:55 ` Neil Horman
@ 2016-08-09 13:31 ` Guenter Roeck
2016-08-09 14:39 ` Wolfram Sang
2016-08-09 16:36 ` Vladimir Zapolskiy
2016-08-10 8:32 ` Krzysztof Kozlowski
6 siblings, 1 reply; 21+ messages in thread
From: Guenter Roeck @ 2016-08-09 13:31 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel
Cc: linux-i2c, Wolfram Sang, linux-arm-kernel, adi-buildroot-devel,
linuxppc-dev, linux-samsung-soc, linux-amlogic, linux-omap,
linux-rockchip, kernel, linux-tegra@vger.kernel.org
On 08/09/2016 04:36 AM, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
> ---
[ ... ]
> diff --git a/drivers/i2c/busses/i2c-uniphier-f.c b/drivers/i2c/busses/i2c-uniphier-f.c
> index aeead0d27d1007..64318e69089439 100644
> --- a/drivers/i2c/busses/i2c-uniphier-f.c
> +++ b/drivers/i2c/busses/i2c-uniphier-f.c
> @@ -550,15 +550,10 @@ static int uniphier_fi2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
You sure about that one ? It leaves the clock enabled in some of the error paths.
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-uniphier.c b/drivers/i2c/busses/i2c-uniphier.c
> index 475a5eb514e215..94f64cccfdef08 100644
> --- a/drivers/i2c/busses/i2c-uniphier.c
> +++ b/drivers/i2c/busses/i2c-uniphier.c
> @@ -407,15 +407,10 @@ static int uniphier_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
Same as above.
Note: I dropped all individuals from Cc:; my mailer refused to accept the reply because there
were too many.
Guenter
> return ret;
> }
>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:57 ` Uwe Kleine-König
@ 2016-08-09 13:59 ` Wolfram Sang
0 siblings, 0 replies; 21+ messages in thread
From: Wolfram Sang @ 2016-08-09 13:59 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Wolfram Sang, linux-kernel, linux-i2c, Jean Delvare,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list, Florian Fainelli, Sonic Zhang,
Brian Norris, Gregory Fong, Michal Simek, Sören Brinkmann,
Jochen Friedrich, Sekhar Nori, Kevin Hilman, Guenter Roeck,
Uwe Kleine-König, Kukjin Kim, Krzysztof Kozlowski
[-- Attachment #1: Type: text/plain, Size: 424 bytes --]
On Tue, Aug 09, 2016 at 01:57:40PM +0200, Uwe Kleine-König wrote:
> Hello Wolfram,
>
> On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
> > The core will do this for us now.
>
> Would be nice to point out the relevant commit that changed the core in
> the commit log.
I agree, but it was a series of commits[1] so I skipped that idea.
[1] http://www.spinics.net/lists/linux-i2c/msg25496.html
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 13:31 ` Guenter Roeck
@ 2016-08-09 14:39 ` Wolfram Sang
0 siblings, 0 replies; 21+ messages in thread
From: Wolfram Sang @ 2016-08-09 14:39 UTC (permalink / raw)
To: Guenter Roeck
Cc: Wolfram Sang, linux-kernel, linux-i2c, linux-arm-kernel,
adi-buildroot-devel, linuxppc-dev, linux-samsung-soc,
linux-amlogic, linux-omap, linux-rockchip, kernel,
linux-tegra@vger.kernel.org
[-- Attachment #1: Type: text/plain, Size: 640 bytes --]
> >@@ -550,15 +550,10 @@ static int uniphier_fi2c_probe(struct platform_device *pdev)
> > }
> >
> > ret = i2c_add_adapter(&priv->adap);
> >- if (ret) {
> >- dev_err(dev, "failed to add I2C adapter\n");
> >- goto err;
> >- }
> >-
> >-err:
> > if (ret)
> > clk_disable_unprepare(priv->clk);
> >
> >+ err:
>
> You sure about that one ? It leaves the clock enabled in some of the error paths.
Uh, ehrm, where is the brown paper bag? Thanks, Guenter!
> Note: I dropped all individuals from Cc:; my mailer refused to accept the reply because there
> were too many.
Yes, the disadvantage of not breaking out.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
` (2 preceding siblings ...)
2016-08-09 12:58 ` Thierry Reding
@ 2016-08-09 16:15 ` Ray Jui
2016-08-10 6:17 ` Ludovic Desroches
` (2 subsequent siblings)
6 siblings, 0 replies; 21+ messages in thread
From: Ray Jui @ 2016-08-09 16:15 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA, Jean Delvare, Wolfram Sang,
Ludovic Desroches, Ray Jui, Scott Branden, Jon Mason,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w, Florian Fainelli,
Sonic Zhang, Brian Norris, Gregory Fong, Michal Simek,
Sören Brinkmann, Jochen Friedrich, Sekhar Nori, Kevin Hilman,
Guenter Roeck, Uwe Kleine-König, Kukjin Kim,
Krzysztof Kozlowski, Seth Heasley
Hi Wolfram,
On 8/9/2016 4:36 AM, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
[...]
> diff --git a/drivers/i2c/busses/i2c-bcm-iproc.c b/drivers/i2c/busses/i2c-bcm-iproc.c
> index 19c843828fe2ca..8e3477f9297eda 100644
> --- a/drivers/i2c/busses/i2c-bcm-iproc.c
> +++ b/drivers/i2c/busses/i2c-bcm-iproc.c
> @@ -488,13 +488,7 @@ static int bcm_iproc_i2c_probe(struct platform_device *pdev)
> adap->dev.parent = &pdev->dev;
> adap->dev.of_node = pdev->dev.of_node;
>
> - ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(iproc_i2c->device, "failed to add adapter\n");
> - return ret;
> - }
> -
> - return 0;
> + return i2c_add_adapter(adap);
> }
>
> static int bcm_iproc_i2c_remove(struct platform_device *pdev)
> diff --git a/drivers/i2c/busses/i2c-bcm-kona.c b/drivers/i2c/busses/i2c-bcm-kona.c
> index ac9f47679c3a4b..a5c9098507896b 100644
> --- a/drivers/i2c/busses/i2c-bcm-kona.c
> +++ b/drivers/i2c/busses/i2c-bcm-kona.c
> @@ -858,10 +858,8 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev)
> adap->dev.of_node = pdev->dev.of_node;
>
> rc = i2c_add_adapter(adap);
> - if (rc) {
> - dev_err(dev->device, "failed to add adapter\n");
> + if (rc)
> return rc;
> - }
>
> dev_info(dev->device, "device registered successfully\n");
>
For both i2c-bcm-iproc.c and i2c-bcm-kona.c:
Acked-by: Ray Jui <ray.jui-dY08KVG/lbpWk0Htik3J/w@public.gmane.org>
Thanks,
Ray
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
` (4 preceding siblings ...)
2016-08-09 13:31 ` Guenter Roeck
@ 2016-08-09 16:36 ` Vladimir Zapolskiy
2016-08-10 8:32 ` Krzysztof Kozlowski
6 siblings, 0 replies; 21+ messages in thread
From: Vladimir Zapolskiy @ 2016-08-09 16:36 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel
Cc: linux-i2c, Jean Delvare, Wolfram Sang, Ludovic Desroches, Ray Jui,
Scott Branden, Jon Mason, bcm-kernel-feedback-list,
Florian Fainelli, Sonic Zhang, Brian Norris, Gregory Fong,
Michal Simek, Sören Brinkmann, Jochen Friedrich, Sekhar Nori,
Kevin Hilman, Guenter Roeck, Uwe Kleine-König, Kukjin Kim,
Krzysztof Kozlowski, Seth Heasley
On 09.08.2016 14:36, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
> ---
For
> drivers/i2c/busses/i2c-pnx.c | 4 +---
[snip]
> diff --git a/drivers/i2c/busses/i2c-pnx.c b/drivers/i2c/busses/i2c-pnx.c
> index 7ea67aa46fb730..fd5f9d2bf6d94e 100644
> --- a/drivers/i2c/busses/i2c-pnx.c
> +++ b/drivers/i2c/busses/i2c-pnx.c
> @@ -714,10 +714,8 @@ static int i2c_pnx_probe(struct platform_device *pdev)
>
> /* Register this adapter with the I2C subsystem */
> ret = i2c_add_numbered_adapter(&alg_data->adapter);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "I2C: Failed to add bus\n");
> + if (ret < 0)
> goto out_clock;
> - }
>
> dev_dbg(&pdev->dev, "%s: Master at %#8x, irq %d.\n",
> alg_data->adapter.name, res->start, alg_data->irq);
Acked-by: Vladimir Zapolskiy <vz@mleia.com>
--
With best wishes,
Vladimir
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:36 ` Wolfram Sang
@ 2016-08-09 16:45 ` Joachim Eastwood
2016-08-10 6:50 ` [STLinux Kernel] " Patrice Chotard
2 siblings, 0 replies; 21+ messages in thread
From: Joachim Eastwood @ 2016-08-09 16:45 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
adi-buildroot-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
kernel-F5mvAk5X5gdBDgjK7y7TUQ,
linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-omap, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA
On 9 August 2016 at 13:36, Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org> wrote:
> Since v4.8-rc1, the I2C core will print detailed information when adding an I2C
> adapter fails. So, drivers can skip this now.
>
> I am still undecided if I apply this as a single patch or break it out. But for
> reviewing, avoiding the patch bomb is probably a good thing.
>
> Should go via subsystem tree, I'd think.
>
> Wolfram Sang (1):
> i2c: don't print error when adding adapter fails
For
> drivers/i2c/busses/i2c-lpc2k.c | 4 +---
Acked-by: Joachim Eastwood <manabian-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
regards,
Joachim Eastwood
^ permalink raw reply [flat|nested] 21+ messages in thread
* RE: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <20160809125522.GA15676-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
@ 2016-08-10 2:56 ` Zhang, Sonic
0 siblings, 0 replies; 21+ messages in thread
From: Zhang, Sonic @ 2016-08-10 2:56 UTC (permalink / raw)
To: Neil Horman, Wolfram Sang
Cc: Krzysztof Kozlowski, Heiko Stuebner, Wolfram Sang, Tony Lindgren,
Linus Walleij, Sekhar Nori, Jochen Friedrich, Masahiro Yamada,
Laxman Dewangan,
linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Peter Korsgaard, Guan Xuetao,
kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org, Alexandre Courbot,
Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Kukjin Kim,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org,
linux-arm-kernel
For i2c-bfin-twi.c
Acked-by: Sonic Zhang <sonic.zhang-OyLXuOCK7orQT0dZR+AlfA@public.gmane.org>
Sonic
-----Original Message-----
From: Neil Horman [mailto:nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org]
Sent: Tuesday, August 09, 2016 8:55 PM
To: Wolfram Sang
Cc: linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; Jean Delvare; Wolfram Sang; Ludovic Desroches; Ray Jui; Scott Branden; Jon Mason; bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w@public.gmane.org; Florian Fainelli; Zhang, Sonic; Brian Norris; Gregory Fong; Michal Simek; Sören Brinkmann; Jochen Friedrich; Sekhar Nori; Kevin Hilman; Guenter Roeck; Uwe Kleine-König; Kukjin Kim; Krzysztof Kozlowski; Seth Heasley; Joachim Eastwood; Carlo Caione; Alessandro Rubini; Linus Walleij; Peter Korsgaard; Tony Lindgren; Vladimir Zapolskiy; Sylvain Lemieux; Guan Xuetao; Heiko Stuebner; Barry Song; Patrice Chotard; Laxman Dewangan; Stephen Warren; Thierry Reding; Alexandre Courbot; Masahiro Yamada; Tony Prisk; linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; adi-buildroot-devel@lists.sourceforge.net; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org; linux-samsung-soc@vger.kernel.org; linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org; kernel-F5mvAk5X5gdBDgjK7y7TUQ@public.gmane.org; linux-tegra@vger.kernel.org
Subject: Re: [PATCH] i2c: don't print error when adding adapter fails
On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
> ---
> drivers/i2c/busses/i2c-amd756.c | 5 +----
> drivers/i2c/busses/i2c-at91.c | 2 --
> drivers/i2c/busses/i2c-axxia.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-iproc.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-kona.c | 4 +---
> drivers/i2c/busses/i2c-bfin-twi.c | 4 +---
> drivers/i2c/busses/i2c-brcmstb.c | 4 +---
> drivers/i2c/busses/i2c-cadence.c | 4 +---
> drivers/i2c/busses/i2c-cpm.c | 4 +---
> drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +---
> drivers/i2c/busses/i2c-davinci.c | 4 +---
> drivers/i2c/busses/i2c-diolan-u2c.c | 4 +---
> drivers/i2c/busses/i2c-dln2.c | 4 +---
> drivers/i2c/busses/i2c-efm32.c | 1 -
> drivers/i2c/busses/i2c-exynos5.c | 4 +---
> drivers/i2c/busses/i2c-hix5hd2.c | 4 +---
> drivers/i2c/busses/i2c-i801.c | 1 -
> drivers/i2c/busses/i2c-ibm_iic.c | 4 +---
> drivers/i2c/busses/i2c-img-scb.c | 4 +---
> drivers/i2c/busses/i2c-imx.c | 4 +---
> drivers/i2c/busses/i2c-isch.c | 4 +---
> drivers/i2c/busses/i2c-ismt.c | 4 +---
> drivers/i2c/busses/i2c-jz4780.c | 4 +---
> drivers/i2c/busses/i2c-lpc2k.c | 4 +---
> drivers/i2c/busses/i2c-meson.c | 1 -
> drivers/i2c/busses/i2c-mpc.c | 4 +---
> drivers/i2c/busses/i2c-mt65xx.c | 4 +---
> drivers/i2c/busses/i2c-mxs.c | 1 -
> drivers/i2c/busses/i2c-nforce2.c | 1 -
> drivers/i2c/busses/i2c-nomadik.c | 4 +---
> drivers/i2c/busses/i2c-ocores.c | 4 +---
> drivers/i2c/busses/i2c-octeon.c | 4 +---
> drivers/i2c/busses/i2c-omap.c | 4 +---
> drivers/i2c/busses/i2c-piix4.c | 1 -
> drivers/i2c/busses/i2c-pmcmsp.c | 4 +---
> drivers/i2c/busses/i2c-pnx.c | 4 +---
> drivers/i2c/busses/i2c-puv3.c | 5 +----
> drivers/i2c/busses/i2c-pxa.c | 4 +---
> drivers/i2c/busses/i2c-rcar.c | 4 +---
> drivers/i2c/busses/i2c-riic.c | 4 +---
> drivers/i2c/busses/i2c-rk3x.c | 4 +---
> drivers/i2c/busses/i2c-s3c2410.c | 1 -
> drivers/i2c/busses/i2c-sh7760.c | 4 +---
> drivers/i2c/busses/i2c-sh_mobile.c | 1 -
> drivers/i2c/busses/i2c-sirf.c | 4 +---
> drivers/i2c/busses/i2c-st.c | 4 +---
> drivers/i2c/busses/i2c-stu300.c | 5 +----
> drivers/i2c/busses/i2c-tegra.c | 4 +---
> drivers/i2c/busses/i2c-uniphier-f.c | 7 +------
> drivers/i2c/busses/i2c-uniphier.c | 7 +------
> drivers/i2c/busses/i2c-wmt.c | 4 +---
> drivers/i2c/busses/i2c-xgene-slimpro.c | 1 -
> drivers/i2c/busses/i2c-xiic.c | 1 -
> drivers/i2c/busses/i2c-xlp9xx.c | 4 +---
> drivers/i2c/busses/i2c-xlr.c | 4 +---
> 55 files changed, 44 insertions(+), 161 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-amd756.c
> b/drivers/i2c/busses/i2c-amd756.c index 6c7113d990f882..274908cd1fdeab
> 100644
> --- a/drivers/i2c/busses/i2c-amd756.c
> +++ b/drivers/i2c/busses/i2c-amd756.c
> @@ -378,11 +378,8 @@ static int amd756_probe(struct pci_dev *pdev, const struct pci_device_id *id)
> amd756_ioport);
>
> error = i2c_add_adapter(&amd756_smbus);
> - if (error) {
> - dev_err(&pdev->dev,
> - "Adapter registration failed, module not inserted\n");
> + if (error)
> goto out_err;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-at91.c
> b/drivers/i2c/busses/i2c-at91.c index f23372669f770f..03a519d29616d7
> 100644
> --- a/drivers/i2c/busses/i2c-at91.c
> +++ b/drivers/i2c/busses/i2c-at91.c
> @@ -1118,8 +1118,6 @@ static int at91_twi_probe(struct platform_device
> *pdev)
>
> rc = i2c_add_numbered_adapter(&dev->adapter);
> if (rc) {
> - dev_err(dev->dev, "Adapter %s registration failed\n",
> - dev->adapter.name);
> clk_disable_unprepare(dev->clk);
>
> pm_runtime_disable(dev->dev);
> diff --git a/drivers/i2c/busses/i2c-axxia.c
> b/drivers/i2c/busses/i2c-axxia.c index c335cc7852f94e..d3bcaf4ab095d0
> 100644
> --- a/drivers/i2c/busses/i2c-axxia.c
> +++ b/drivers/i2c/busses/i2c-axxia.c
> @@ -558,13 +558,7 @@ static int axxia_i2c_probe(struct platform_device
> *pdev)
>
> platform_set_drvdata(pdev, idev);
>
> - ret = i2c_add_adapter(&idev->adapter);
> - if (ret) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> - return ret;
> - }
> -
> - return 0;
> + return i2c_add_adapter(&idev->adapter);
> }
>
> static int axxia_i2c_remove(struct platform_device *pdev) diff --git
> a/drivers/i2c/busses/i2c-bcm-iproc.c
> b/drivers/i2c/busses/i2c-bcm-iproc.c
> index 19c843828fe2ca..8e3477f9297eda 100644
> --- a/drivers/i2c/busses/i2c-bcm-iproc.c
> +++ b/drivers/i2c/busses/i2c-bcm-iproc.c
> @@ -488,13 +488,7 @@ static int bcm_iproc_i2c_probe(struct platform_device *pdev)
> adap->dev.parent = &pdev->dev;
> adap->dev.of_node = pdev->dev.of_node;
>
> - ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(iproc_i2c->device, "failed to add adapter\n");
> - return ret;
> - }
> -
> - return 0;
> + return i2c_add_adapter(adap);
> }
>
> static int bcm_iproc_i2c_remove(struct platform_device *pdev) diff
> --git a/drivers/i2c/busses/i2c-bcm-kona.c
> b/drivers/i2c/busses/i2c-bcm-kona.c
> index ac9f47679c3a4b..a5c9098507896b 100644
> --- a/drivers/i2c/busses/i2c-bcm-kona.c
> +++ b/drivers/i2c/busses/i2c-bcm-kona.c
> @@ -858,10 +858,8 @@ static int bcm_kona_i2c_probe(struct platform_device *pdev)
> adap->dev.of_node = pdev->dev.of_node;
>
> rc = i2c_add_adapter(adap);
> - if (rc) {
> - dev_err(dev->device, "failed to add adapter\n");
> + if (rc)
> return rc;
> - }
>
> dev_info(dev->device, "device registered successfully\n");
>
> diff --git a/drivers/i2c/busses/i2c-bfin-twi.c
> b/drivers/i2c/busses/i2c-bfin-twi.c
> index 025686d4164058..29d00c4f7824c2 100644
> --- a/drivers/i2c/busses/i2c-bfin-twi.c
> +++ b/drivers/i2c/busses/i2c-bfin-twi.c
> @@ -685,10 +685,8 @@ static int i2c_bfin_twi_probe(struct platform_device *pdev)
> write_CONTROL(iface, read_CONTROL(iface) | TWI_ENA);
>
> rc = i2c_add_numbered_adapter(p_adap);
> - if (rc < 0) {
> - dev_err(&pdev->dev, "Can't add i2c adapter!\n");
> + if (rc < 0)
> goto out_error;
> - }
>
> platform_set_drvdata(pdev, iface);
>
> diff --git a/drivers/i2c/busses/i2c-brcmstb.c
> b/drivers/i2c/busses/i2c-brcmstb.c
> index 3f5a4d71d3bf32..bc5c176e984aea 100644
> --- a/drivers/i2c/busses/i2c-brcmstb.c
> +++ b/drivers/i2c/busses/i2c-brcmstb.c
> @@ -648,10 +648,8 @@ static int brcmstb_i2c_probe(struct platform_device *pdev)
> adap->dev.parent = &pdev->dev;
> adap->dev.of_node = pdev->dev.of_node;
> rc = i2c_add_adapter(adap);
> - if (rc) {
> - dev_err(dev->device, "failed to add adapter\n");
> + if (rc)
> goto probe_errorout;
> - }
>
> dev_info(dev->device, "%s@%dhz registered in %s mode\n",
> int_name ? int_name : " ", dev->clk_freq_hz, diff --git
> a/drivers/i2c/busses/i2c-cadence.c b/drivers/i2c/busses/i2c-cadence.c
> index 90bbd9f9dd8f77..1cc1e79e958d56 100644
> --- a/drivers/i2c/busses/i2c-cadence.c
> +++ b/drivers/i2c/busses/i2c-cadence.c
> @@ -963,10 +963,8 @@ static int cdns_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&id->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto err_clk_dis;
> - }
>
> /*
> * Cadence I2C controller has a bug wherein it generates diff --git
> a/drivers/i2c/busses/i2c-cpm.c b/drivers/i2c/busses/i2c-cpm.c index
> ee57c1e865e271..d89bde2c5da256 100644
> --- a/drivers/i2c/busses/i2c-cpm.c
> +++ b/drivers/i2c/busses/i2c-cpm.c
> @@ -665,10 +665,8 @@ static int cpm_i2c_probe(struct platform_device *ofdev)
> cpm->adap.nr = (data && len == 4) ? be32_to_cpup(data) : -1;
> result = i2c_add_numbered_adapter(&cpm->adap);
>
> - if (result < 0) {
> - dev_err(&ofdev->dev, "Unable to register with I2C\n");
> + if (result < 0)
> goto out_shut;
> - }
>
> dev_dbg(&ofdev->dev, "hw routines for %s registered.\n",
> cpm->adap.name);
> diff --git a/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> index a0d95ff682ae12..cf334a9ec22cb7 100644
> --- a/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> +++ b/drivers/i2c/busses/i2c-cros-ec-tunnel.c
> @@ -281,10 +281,8 @@ static int ec_i2c_probe(struct platform_device *pdev)
> bus->adap.retries = I2C_MAX_RETRIES;
>
> err = i2c_add_adapter(&bus->adap);
> - if (err) {
> - dev_err(dev, "cannot register i2c adapter\n");
> + if (err)
> return err;
> - }
> platform_set_drvdata(pdev, bus);
>
> return err;
> diff --git a/drivers/i2c/busses/i2c-davinci.c
> b/drivers/i2c/busses/i2c-davinci.c
> index a8bdcb5292f5b3..9e7ef5cf5d495d 100644
> --- a/drivers/i2c/busses/i2c-davinci.c
> +++ b/drivers/i2c/busses/i2c-davinci.c
> @@ -846,10 +846,8 @@ static int davinci_i2c_probe(struct
> platform_device *pdev)
>
> adap->nr = pdev->id;
> r = i2c_add_numbered_adapter(adap);
> - if (r) {
> - dev_err(&pdev->dev, "failure adding adapter\n");
> + if (r)
> goto err_unuse_clocks;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-diolan-u2c.c
> b/drivers/i2c/busses/i2c-diolan-u2c.c
> index b19a310bf9b3e8..f718ee4e3332fe 100644
> --- a/drivers/i2c/busses/i2c-diolan-u2c.c
> +++ b/drivers/i2c/busses/i2c-diolan-u2c.c
> @@ -487,10 +487,8 @@ static int diolan_u2c_probe(struct usb_interface
> *interface,
>
> /* and finally attach to i2c layer */
> ret = i2c_add_adapter(&dev->adapter);
> - if (ret < 0) {
> - dev_err(&interface->dev, "failed to add I2C adapter\n");
> + if (ret < 0)
> goto error_free;
> - }
>
> dev_dbg(&interface->dev, "connected " DRIVER_NAME "\n");
>
> diff --git a/drivers/i2c/busses/i2c-dln2.c
> b/drivers/i2c/busses/i2c-dln2.c index f2eb4f76591fbe..8acda2aa15583c
> 100644
> --- a/drivers/i2c/busses/i2c-dln2.c
> +++ b/drivers/i2c/busses/i2c-dln2.c
> @@ -228,10 +228,8 @@ static int dln2_i2c_probe(struct platform_device
> *pdev)
>
> /* and finally attach to i2c layer */
> ret = i2c_add_adapter(&dln2->adapter);
> - if (ret < 0) {
> - dev_err(dev, "failed to add I2C adapter: %d\n", ret);
> + if (ret < 0)
> goto out_disable;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-efm32.c
> b/drivers/i2c/busses/i2c-efm32.c index e253598d764c0d..aa336ba89aa3fe
> 100644
> --- a/drivers/i2c/busses/i2c-efm32.c
> +++ b/drivers/i2c/busses/i2c-efm32.c
> @@ -438,7 +438,6 @@ static int efm32_i2c_probe(struct platform_device
> *pdev)
>
> ret = i2c_add_adapter(&ddata->adapter);
> if (ret) {
> - dev_err(&pdev->dev, "failed to add i2c adapter (%d)\n", ret);
> free_irq(ddata->irq, ddata);
>
> err_disable_clk:
> diff --git a/drivers/i2c/busses/i2c-exynos5.c
> b/drivers/i2c/busses/i2c-exynos5.c
> index c0e3ada028763d..bea607149972d4 100644
> --- a/drivers/i2c/busses/i2c-exynos5.c
> +++ b/drivers/i2c/busses/i2c-exynos5.c
> @@ -796,10 +796,8 @@ static int exynos5_i2c_probe(struct platform_device *pdev)
> exynos5_i2c_reset(i2c);
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> + if (ret < 0)
> goto err_clk;
> - }
>
> platform_set_drvdata(pdev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-hix5hd2.c
> b/drivers/i2c/busses/i2c-hix5hd2.c
> index 7c6966434ee7b9..ae7f3180f7e887 100644
> --- a/drivers/i2c/busses/i2c-hix5hd2.c
> +++ b/drivers/i2c/busses/i2c-hix5hd2.c
> @@ -478,10 +478,8 @@ static int hix5hd2_i2c_probe(struct platform_device *pdev)
> pm_runtime_enable(priv->dev);
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> + if (ret < 0)
> goto err_runtime;
> - }
>
> return ret;
>
> diff --git a/drivers/i2c/busses/i2c-i801.c
> b/drivers/i2c/busses/i2c-i801.c index 5ef9b733d153fa..22a0ed4de80da0
> 100644
> --- a/drivers/i2c/busses/i2c-i801.c
> +++ b/drivers/i2c/busses/i2c-i801.c
> @@ -1613,7 +1613,6 @@ static int i801_probe(struct pci_dev *dev, const struct pci_device_id *id)
> "SMBus I801 adapter at %04lx", priv->smba);
> err = i2c_add_adapter(&priv->adapter);
> if (err) {
> - dev_err(&dev->dev, "Failed to add SMBus adapter\n");
> i801_acpi_remove(priv);
> return err;
> }
> diff --git a/drivers/i2c/busses/i2c-ibm_iic.c
> b/drivers/i2c/busses/i2c-ibm_iic.c
> index cdaa7be2cd1b94..412b91d255ad1d 100644
> --- a/drivers/i2c/busses/i2c-ibm_iic.c
> +++ b/drivers/i2c/busses/i2c-ibm_iic.c
> @@ -751,10 +751,8 @@ static int iic_probe(struct platform_device *ofdev)
> adap->timeout = HZ;
>
> ret = i2c_add_adapter(adap);
> - if (ret < 0) {
> - dev_err(&ofdev->dev, "failed to register i2c adapter\n");
> + if (ret < 0)
> goto error_cleanup;
> - }
>
> dev_info(&ofdev->dev, "using %s mode\n",
> dev->fast_mode ? "fast (400 kHz)" : "standard (100 kHz)"); diff
> --git a/drivers/i2c/busses/i2c-img-scb.c
> b/drivers/i2c/busses/i2c-img-scb.c
> index ea20425b697217..db8e8b40569d98 100644
> --- a/drivers/i2c/busses/i2c-img-scb.c
> +++ b/drivers/i2c/busses/i2c-img-scb.c
> @@ -1394,10 +1394,8 @@ static int img_i2c_probe(struct platform_device *pdev)
> goto disable_clk;
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (ret < 0)
> goto disable_clk;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-imx.c
> b/drivers/i2c/busses/i2c-imx.c index 1844bc9f7cd5a2..cb11eee9567318
> 100644
> --- a/drivers/i2c/busses/i2c-imx.c
> +++ b/drivers/i2c/busses/i2c-imx.c
> @@ -1129,10 +1129,8 @@ static int i2c_imx_probe(struct platform_device
> *pdev)
>
> /* Add I2C adapter */
> ret = i2c_add_numbered_adapter(&i2c_imx->adapter);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "registration failed\n");
> + if (ret < 0)
> goto rpm_disable;
> - }
>
> pm_runtime_mark_last_busy(&pdev->dev);
> pm_runtime_put_autosuspend(&pdev->dev);
> diff --git a/drivers/i2c/busses/i2c-isch.c
> b/drivers/i2c/busses/i2c-isch.c index c2f25f19d76f9b..0cf1379f4e8091
> 100644
> --- a/drivers/i2c/busses/i2c-isch.c
> +++ b/drivers/i2c/busses/i2c-isch.c
> @@ -288,10 +288,8 @@ static int smbus_sch_probe(struct platform_device *dev)
> "SMBus SCH adapter at %04x", sch_smba);
>
> retval = i2c_add_adapter(&sch_adapter);
> - if (retval) {
> - dev_err(&dev->dev, "Couldn't register adapter!\n");
> + if (retval)
> sch_smba = 0;
> - }
>
> return retval;
> }
> diff --git a/drivers/i2c/busses/i2c-ismt.c
> b/drivers/i2c/busses/i2c-ismt.c index 1c870771009840..f573448d213216
> 100644
> --- a/drivers/i2c/busses/i2c-ismt.c
> +++ b/drivers/i2c/busses/i2c-ismt.c
> @@ -922,10 +922,8 @@ ismt_probe(struct pci_dev *pdev, const struct pci_device_id *id)
> return err;
>
> err = i2c_add_adapter(&priv->adapter);
> - if (err) {
> - dev_err(&pdev->dev, "Failed to add SMBus iSMT adapter\n");
> + if (err)
> return -ENODEV;
> - }
> return 0;
> }
>
> diff --git a/drivers/i2c/busses/i2c-jz4780.c
> b/drivers/i2c/busses/i2c-jz4780.c index cd9872594fe2a0..b8ea62105f42c9
> 100644
> --- a/drivers/i2c/busses/i2c-jz4780.c
> +++ b/drivers/i2c/busses/i2c-jz4780.c
> @@ -798,10 +798,8 @@ static int jz4780_i2c_probe(struct platform_device *pdev)
> goto err;
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "Failed to add bus\n");
> + if (ret < 0)
> goto err;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-lpc2k.c
> b/drivers/i2c/busses/i2c-lpc2k.c index 586a15205e61de..9b1fef455a895d
> 100644
> --- a/drivers/i2c/busses/i2c-lpc2k.c
> +++ b/drivers/i2c/busses/i2c-lpc2k.c
> @@ -432,10 +432,8 @@ static int i2c_lpc2k_probe(struct platform_device *pdev)
> i2c->adap.dev.of_node = pdev->dev.of_node;
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add adapter!\n");
> + if (ret < 0)
> goto fail_clk;
> - }
>
> dev_info(&pdev->dev, "LPC2K I2C adapter\n");
>
> diff --git a/drivers/i2c/busses/i2c-meson.c
> b/drivers/i2c/busses/i2c-meson.c index 71d3929adf54ed..102d2b91771f35
> 100644
> --- a/drivers/i2c/busses/i2c-meson.c
> +++ b/drivers/i2c/busses/i2c-meson.c
> @@ -453,7 +453,6 @@ static int meson_i2c_probe(struct platform_device
> *pdev)
>
> ret = i2c_add_adapter(&i2c->adap);
> if (ret < 0) {
> - dev_err(&pdev->dev, "can't register adapter\n");
> clk_unprepare(i2c->clk);
> return ret;
> }
> diff --git a/drivers/i2c/busses/i2c-mpc.c
> b/drivers/i2c/busses/i2c-mpc.c index 48ecffecc0eda3..565a49a0c56410
> 100644
> --- a/drivers/i2c/busses/i2c-mpc.c
> +++ b/drivers/i2c/busses/i2c-mpc.c
> @@ -737,10 +737,8 @@ static int fsl_i2c_probe(struct platform_device *op)
> i2c->adap.dev.of_node = of_node_get(op->dev.of_node);
>
> result = i2c_add_adapter(&i2c->adap);
> - if (result < 0) {
> - dev_err(i2c->dev, "failed to add adapter\n");
> + if (result < 0)
> goto fail_add;
> - }
>
> return result;
>
> diff --git a/drivers/i2c/busses/i2c-mt65xx.c
> b/drivers/i2c/busses/i2c-mt65xx.c index d9373e60be8ae3..4a7d9bc2142ba3
> 100644
> --- a/drivers/i2c/busses/i2c-mt65xx.c
> +++ b/drivers/i2c/busses/i2c-mt65xx.c
> @@ -786,10 +786,8 @@ static int mtk_i2c_probe(struct platform_device
> *pdev)
>
> i2c_set_adapdata(&i2c->adap, i2c);
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add i2c bus to i2c core\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-mxs.c
> b/drivers/i2c/busses/i2c-mxs.c index 033846cdf26622..5738556b6aac20
> 100644
> --- a/drivers/i2c/busses/i2c-mxs.c
> +++ b/drivers/i2c/busses/i2c-mxs.c
> @@ -868,7 +868,6 @@ static int mxs_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(adap, i2c);
> err = i2c_add_numbered_adapter(adap);
> if (err) {
> - dev_err(dev, "Failed to add adapter (%d)\n", err);
> writel(MXS_I2C_CTRL0_SFTRST,
> i2c->regs + MXS_I2C_CTRL0_SET);
> return err;
> diff --git a/drivers/i2c/busses/i2c-nforce2.c
> b/drivers/i2c/busses/i2c-nforce2.c
> index 42fcc9458432f8..374b35e7e450ce 100644
> --- a/drivers/i2c/busses/i2c-nforce2.c
> +++ b/drivers/i2c/busses/i2c-nforce2.c
> @@ -366,7 +366,6 @@ static int nforce2_probe_smb(struct pci_dev *dev,
> int bar, int alt_reg,
>
> error = i2c_add_adapter(&smbus->adapter);
> if (error) {
> - dev_err(&smbus->adapter.dev, "Failed to register adapter.\n");
> release_region(smbus->base, smbus->size);
> return error;
> }
> diff --git a/drivers/i2c/busses/i2c-nomadik.c
> b/drivers/i2c/busses/i2c-nomadik.c
> index bcd17e8cbcb4b3..da6609d628487b 100644
> --- a/drivers/i2c/busses/i2c-nomadik.c
> +++ b/drivers/i2c/busses/i2c-nomadik.c
> @@ -1046,10 +1046,8 @@ static int nmk_i2c_probe(struct amba_device *adev, const struct amba_id *id)
> adap->name, dev->virtbase);
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&adev->dev, "failed to add adapter\n");
> + if (ret)
> goto err_no_adap;
> - }
>
> pm_runtime_put(&adev->dev);
>
> diff --git a/drivers/i2c/busses/i2c-ocores.c
> b/drivers/i2c/busses/i2c-ocores.c index dfa7a4b4a91d78..92692349d0c328
> 100644
> --- a/drivers/i2c/busses/i2c-ocores.c
> +++ b/drivers/i2c/busses/i2c-ocores.c
> @@ -492,10 +492,8 @@ static int ocores_i2c_probe(struct
> platform_device *pdev)
>
> /* add i2c adapter to i2c tree */
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> /* add in known devices to the bus */
> if (pdata) {
> diff --git a/drivers/i2c/busses/i2c-octeon.c
> b/drivers/i2c/busses/i2c-octeon.c index 30ae35146723a4..b75d2da79b27c5
> 100644
> --- a/drivers/i2c/busses/i2c-octeon.c
> +++ b/drivers/i2c/busses/i2c-octeon.c
> @@ -1213,10 +1213,8 @@ static int octeon_i2c_probe(struct platform_device *pdev)
> platform_set_drvdata(pdev, i2c);
>
> result = i2c_add_adapter(&i2c->adap);
> - if (result < 0) {
> - dev_err(i2c->dev, "failed to add adapter\n");
> + if (result < 0)
> goto out;
> - }
> dev_info(i2c->dev, "probed\n");
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-omap.c
> b/drivers/i2c/busses/i2c-omap.c index ab1279b8e2407a..c7da0c42baeefa
> 100644
> --- a/drivers/i2c/busses/i2c-omap.c
> +++ b/drivers/i2c/busses/i2c-omap.c
> @@ -1425,10 +1425,8 @@ omap_i2c_probe(struct platform_device *pdev)
> /* i2c device drivers may be active on return from add_adapter() */
> adap->nr = pdev->id;
> r = i2c_add_numbered_adapter(adap);
> - if (r) {
> - dev_err(omap->dev, "failure adding adapter\n");
> + if (r)
> goto err_unuse_clocks;
> - }
>
> dev_info(omap->dev, "bus %d rev%d.%d at %d kHz\n", adap->nr,
> major, minor, omap->speed);
> diff --git a/drivers/i2c/busses/i2c-piix4.c
> b/drivers/i2c/busses/i2c-piix4.c index 23d1c167b5d752..c2268cdf38e823
> 100644
> --- a/drivers/i2c/busses/i2c-piix4.c
> +++ b/drivers/i2c/busses/i2c-piix4.c
> @@ -694,7 +694,6 @@ static int piix4_add_adapter(struct pci_dev *dev,
> unsigned short smba,
>
> retval = i2c_add_adapter(adap);
> if (retval) {
> - dev_err(&dev->dev, "Couldn't register adapter!\n");
> kfree(adapdata);
> kfree(adap);
> release_region(smba, SMBIOSIZE);
> diff --git a/drivers/i2c/busses/i2c-pmcmsp.c
> b/drivers/i2c/busses/i2c-pmcmsp.c index 2c40edbf6224eb..217c78711d6501
> 100644
> --- a/drivers/i2c/busses/i2c-pmcmsp.c
> +++ b/drivers/i2c/busses/i2c-pmcmsp.c
> @@ -329,10 +329,8 @@ static int pmcmsptwi_probe(struct platform_device *pldev)
> i2c_set_adapdata(&pmcmsptwi_adapter, &pmcmsptwi_data);
>
> rc = i2c_add_adapter(&pmcmsptwi_adapter);
> - if (rc) {
> - dev_err(&pldev->dev, "Unable to register I2C adapter\n");
> + if (rc)
> goto ret_unmap;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-pnx.c
> b/drivers/i2c/busses/i2c-pnx.c index 7ea67aa46fb730..fd5f9d2bf6d94e
> 100644
> --- a/drivers/i2c/busses/i2c-pnx.c
> +++ b/drivers/i2c/busses/i2c-pnx.c
> @@ -714,10 +714,8 @@ static int i2c_pnx_probe(struct platform_device
> *pdev)
>
> /* Register this adapter with the I2C subsystem */
> ret = i2c_add_numbered_adapter(&alg_data->adapter);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "I2C: Failed to add bus\n");
> + if (ret < 0)
> goto out_clock;
> - }
>
> dev_dbg(&pdev->dev, "%s: Master at %#8x, irq %d.\n",
> alg_data->adapter.name, res->start, alg_data->irq); diff --git
> a/drivers/i2c/busses/i2c-puv3.c b/drivers/i2c/busses/i2c-puv3.c index
> 82b6f02544dae7..0c8b1571886d5b 100644
> --- a/drivers/i2c/busses/i2c-puv3.c
> +++ b/drivers/i2c/busses/i2c-puv3.c
> @@ -212,11 +212,8 @@ static int puv3_i2c_probe(struct platform_device
> *pdev)
>
> adapter->nr = pdev->id;
> rc = i2c_add_numbered_adapter(adapter);
> - if (rc) {
> - dev_err(&pdev->dev, "Adapter '%s' registration failed\n",
> - adapter->name);
> + if (rc)
> goto fail_add_adapter;
> - }
>
> dev_info(&pdev->dev, "PKUnity v3 i2c bus adapter.\n");
> return 0;
> diff --git a/drivers/i2c/busses/i2c-pxa.c
> b/drivers/i2c/busses/i2c-pxa.c index 0d351954db02b9..e28b825b043378
> 100644
> --- a/drivers/i2c/busses/i2c-pxa.c
> +++ b/drivers/i2c/busses/i2c-pxa.c
> @@ -1292,10 +1292,8 @@ static int i2c_pxa_probe(struct platform_device
> *dev) #endif
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&dev->dev, "failed to add bus: %d\n", ret);
> + if (ret < 0)
> goto ereqirq;
> - }
>
> platform_set_drvdata(dev, i2c);
>
> diff --git a/drivers/i2c/busses/i2c-rcar.c
> b/drivers/i2c/busses/i2c-rcar.c index 52407f3c9e1cce..400e344704d273
> 100644
> --- a/drivers/i2c/busses/i2c-rcar.c
> +++ b/drivers/i2c/busses/i2c-rcar.c
> @@ -875,10 +875,8 @@ static int rcar_i2c_probe(struct platform_device *pdev)
> platform_set_drvdata(pdev, priv);
>
> ret = i2c_add_numbered_adapter(adap);
> - if (ret < 0) {
> - dev_err(dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto out_pm_disable;
> - }
>
> dev_info(dev, "probed\n");
>
> diff --git a/drivers/i2c/busses/i2c-riic.c
> b/drivers/i2c/busses/i2c-riic.c index d7e3af67154373..6263ea82d6ac3b
> 100644
> --- a/drivers/i2c/busses/i2c-riic.c
> +++ b/drivers/i2c/busses/i2c-riic.c
> @@ -383,10 +383,8 @@ static int riic_i2c_probe(struct platform_device
> *pdev)
>
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, riic);
>
> diff --git a/drivers/i2c/busses/i2c-rk3x.c
> b/drivers/i2c/busses/i2c-rk3x.c index 2bc8b01153d619..3b87afe82f6394
> 100644
> --- a/drivers/i2c/busses/i2c-rk3x.c
> +++ b/drivers/i2c/busses/i2c-rk3x.c
> @@ -1303,10 +1303,8 @@ static int rk3x_i2c_probe(struct platform_device *pdev)
> rk3x_i2c_adapt_div(i2c, clk_rate);
>
> ret = i2c_add_adapter(&i2c->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "Could not register adapter\n");
> + if (ret < 0)
> goto err_clk_notifier;
> - }
>
> dev_info(&pdev->dev, "Initialized RK3xxx I2C bus at %p\n",
> i2c->regs);
>
> diff --git a/drivers/i2c/busses/i2c-s3c2410.c
> b/drivers/i2c/busses/i2c-s3c2410.c
> index 38dc1cacfd8bba..499af26e736e7e 100644
> --- a/drivers/i2c/busses/i2c-s3c2410.c
> +++ b/drivers/i2c/busses/i2c-s3c2410.c
> @@ -1215,7 +1215,6 @@ static int s3c24xx_i2c_probe(struct
> platform_device *pdev)
>
> ret = i2c_add_numbered_adapter(&i2c->adap);
> if (ret < 0) {
> - dev_err(&pdev->dev, "failed to add bus to i2c core\n");
> pm_runtime_disable(&pdev->dev);
> s3c24xx_i2c_deregister_cpufreq(i2c);
> clk_unprepare(i2c->clk);
> diff --git a/drivers/i2c/busses/i2c-sh7760.c
> b/drivers/i2c/busses/i2c-sh7760.c index 24968384b4014f..c2005c789d2b09
> 100644
> --- a/drivers/i2c/busses/i2c-sh7760.c
> +++ b/drivers/i2c/busses/i2c-sh7760.c
> @@ -510,10 +510,8 @@ static int sh7760_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_numbered_adapter(&id->adap);
> - if (ret < 0) {
> - dev_err(&pdev->dev, "reg adap failed: %d\n", ret);
> + if (ret < 0)
> goto out4;
> - }
>
> platform_set_drvdata(pdev, id);
>
> diff --git a/drivers/i2c/busses/i2c-sh_mobile.c
> b/drivers/i2c/busses/i2c-sh_mobile.c
> index 6fb3e264599229..50f276e8d17967 100644
> --- a/drivers/i2c/busses/i2c-sh_mobile.c
> +++ b/drivers/i2c/busses/i2c-sh_mobile.c
> @@ -981,7 +981,6 @@ static int sh_mobile_i2c_probe(struct platform_device *dev)
> ret = i2c_add_numbered_adapter(adap);
> if (ret < 0) {
> sh_mobile_i2c_release_dma(pd);
> - dev_err(&dev->dev, "cannot add numbered adapter\n");
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-sirf.c
> b/drivers/i2c/busses/i2c-sirf.c index 792a42bdd335b9..95e81d0f72b4d2
> 100644
> --- a/drivers/i2c/busses/i2c-sirf.c
> +++ b/drivers/i2c/busses/i2c-sirf.c
> @@ -387,10 +387,8 @@ static int i2c_sirfsoc_probe(struct platform_device *pdev)
> writel(regval, siic->base + SIRFSOC_I2C_SDA_DELAY);
>
> err = i2c_add_numbered_adapter(adap);
> - if (err < 0) {
> - dev_err(&pdev->dev, "Can't add new i2c adapter\n");
> + if (err < 0)
> goto out;
> - }
>
> clk_disable(clk);
>
> diff --git a/drivers/i2c/busses/i2c-st.c b/drivers/i2c/busses/i2c-st.c
> index 944ec420508487..1371547ce1a3a8 100644
> --- a/drivers/i2c/busses/i2c-st.c
> +++ b/drivers/i2c/busses/i2c-st.c
> @@ -874,10 +874,8 @@ static int st_i2c_probe(struct platform_device *pdev)
> init_completion(&i2c_dev->complete);
>
> ret = i2c_add_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, i2c_dev);
>
> diff --git a/drivers/i2c/busses/i2c-stu300.c
> b/drivers/i2c/busses/i2c-stu300.c index 460c134832ac30..dc63236b45b216
> 100644
> --- a/drivers/i2c/busses/i2c-stu300.c
> +++ b/drivers/i2c/busses/i2c-stu300.c
> @@ -920,11 +920,8 @@ static int stu300_probe(struct platform_device
> *pdev)
>
> /* i2c device drivers may be active on return from add_adapter() */
> ret = i2c_add_numbered_adapter(adap);
> - if (ret) {
> - dev_err(&pdev->dev, "failure adding ST Micro DDC "
> - "I2C adapter\n");
> + if (ret)
> return ret;
> - }
>
> platform_set_drvdata(pdev, dev);
> dev_info(&pdev->dev, "ST DDC I2C @ %p, irq %d\n", diff --git
> a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c
> index b126dbaa47e370..d9979da11485ae 100644
> --- a/drivers/i2c/busses/i2c-tegra.c
> +++ b/drivers/i2c/busses/i2c-tegra.c
> @@ -932,10 +932,8 @@ static int tegra_i2c_probe(struct platform_device *pdev)
> i2c_dev->adapter.dev.of_node = pdev->dev.of_node;
>
> ret = i2c_add_numbered_adapter(&i2c_dev->adapter);
> - if (ret) {
> - dev_err(&pdev->dev, "Failed to add I2C adapter\n");
> + if (ret)
> goto disable_div_clk;
> - }
>
> return 0;
>
> diff --git a/drivers/i2c/busses/i2c-uniphier-f.c
> b/drivers/i2c/busses/i2c-uniphier-f.c
> index aeead0d27d1007..64318e69089439 100644
> --- a/drivers/i2c/busses/i2c-uniphier-f.c
> +++ b/drivers/i2c/busses/i2c-uniphier-f.c
> @@ -550,15 +550,10 @@ static int uniphier_fi2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-uniphier.c
> b/drivers/i2c/busses/i2c-uniphier.c
> index 475a5eb514e215..94f64cccfdef08 100644
> --- a/drivers/i2c/busses/i2c-uniphier.c
> +++ b/drivers/i2c/busses/i2c-uniphier.c
> @@ -407,15 +407,10 @@ static int uniphier_i2c_probe(struct platform_device *pdev)
> }
>
> ret = i2c_add_adapter(&priv->adap);
> - if (ret) {
> - dev_err(dev, "failed to add I2C adapter\n");
> - goto err;
> - }
> -
> -err:
> if (ret)
> clk_disable_unprepare(priv->clk);
>
> + err:
> return ret;
> }
>
> diff --git a/drivers/i2c/busses/i2c-wmt.c
> b/drivers/i2c/busses/i2c-wmt.c index e1e3a85596c562..fbd0fd59f31239
> 100644
> --- a/drivers/i2c/busses/i2c-wmt.c
> +++ b/drivers/i2c/busses/i2c-wmt.c
> @@ -432,10 +432,8 @@ static int wmt_i2c_probe(struct platform_device *pdev)
> }
>
> err = i2c_add_adapter(adap);
> - if (err) {
> - dev_err(&pdev->dev, "failed to add adapter\n");
> + if (err)
> return err;
> - }
>
> platform_set_drvdata(pdev, i2c_dev);
>
> diff --git a/drivers/i2c/busses/i2c-xgene-slimpro.c
> b/drivers/i2c/busses/i2c-xgene-slimpro.c
> index 4233f5695352fd..263685c7a51287 100644
> --- a/drivers/i2c/busses/i2c-xgene-slimpro.c
> +++ b/drivers/i2c/busses/i2c-xgene-slimpro.c
> @@ -418,7 +418,6 @@ static int xgene_slimpro_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(adapter, ctx);
> rc = i2c_add_adapter(adapter);
> if (rc) {
> - dev_err(&pdev->dev, "Adapter registeration failed\n");
> mbox_free_channel(ctx->mbox_chan);
> return rc;
> }
> diff --git a/drivers/i2c/busses/i2c-xiic.c
> b/drivers/i2c/busses/i2c-xiic.c index 74f54f2f471fa7..66bce3b311a199
> 100644
> --- a/drivers/i2c/busses/i2c-xiic.c
> +++ b/drivers/i2c/busses/i2c-xiic.c
> @@ -804,7 +804,6 @@ static int xiic_i2c_probe(struct platform_device *pdev)
> /* add i2c adapter to i2c tree */
> ret = i2c_add_adapter(&i2c->adap);
> if (ret) {
> - dev_err(&pdev->dev, "Failed to add adapter\n");
> xiic_deinit(i2c);
> goto err_clk_dis;
> }
> diff --git a/drivers/i2c/busses/i2c-xlp9xx.c
> b/drivers/i2c/busses/i2c-xlp9xx.c index 55a7bef1b2e1df..2a972ed7aa0df1
> 100644
> --- a/drivers/i2c/busses/i2c-xlp9xx.c
> +++ b/drivers/i2c/busses/i2c-xlp9xx.c
> @@ -400,10 +400,8 @@ static int xlp9xx_i2c_probe(struct platform_device *pdev)
> i2c_set_adapdata(&priv->adapter, priv);
>
> err = i2c_add_adapter(&priv->adapter);
> - if (err) {
> - dev_err(&pdev->dev, "failed to add I2C adapter!\n");
> + if (err)
> return err;
> - }
>
> platform_set_drvdata(pdev, priv);
> dev_dbg(&pdev->dev, "I2C bus:%d added\n", priv->adapter.nr); diff
> --git a/drivers/i2c/busses/i2c-xlr.c b/drivers/i2c/busses/i2c-xlr.c
> index 613c3a4f2c5142..0968f59b6df586 100644
> --- a/drivers/i2c/busses/i2c-xlr.c
> +++ b/drivers/i2c/busses/i2c-xlr.c
> @@ -432,10 +432,8 @@ static int xlr_i2c_probe(struct platform_device
> *pdev)
>
> i2c_set_adapdata(&priv->adap, priv);
> ret = i2c_add_numbered_adapter(&priv->adap);
> - if (ret < 0) {
> - dev_err(&priv->adap.dev, "Failed to add i2c bus.\n");
> + if (ret < 0)
> return ret;
> - }
>
> platform_set_drvdata(pdev, priv);
> dev_info(&priv->adap.dev, "Added I2C Bus.\n");
> --
> 2.8.1
>
>
Acked-by: Neil Horman <nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
` (3 preceding siblings ...)
2016-08-09 16:15 ` Ray Jui
@ 2016-08-10 6:17 ` Ludovic Desroches
2016-08-10 8:34 ` Laxman Dewangan
2016-10-05 15:17 ` Jean Delvare
6 siblings, 0 replies; 21+ messages in thread
From: Ludovic Desroches @ 2016-08-10 6:17 UTC (permalink / raw)
To: Wolfram Sang
Cc: Krzysztof Kozlowski, Heiko Stuebner, Wolfram Sang, Tony Lindgren,
Linus Walleij, Sekhar Nori, Jochen Friedrich, Masahiro Yamada,
Laxman Dewangan, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guan Xuetao,
Peter Korsgaard, Sonic Zhang, kernel-F5mvAk5X5gdBDgjK7y7TUQ,
Alexandre Courbot, Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Kukjin Kim,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Sylvain Lemieux, Guenter Roeck
On Tue, Aug 09, 2016 at 01:36:17PM +0200, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
> ---
[...]
> drivers/i2c/busses/i2c-at91.c | 2 --
Acked-by: Ludovic Desroches <ludovic.desroches-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
[...]
> diff --git a/drivers/i2c/busses/i2c-at91.c b/drivers/i2c/busses/i2c-at91.c
> index f23372669f770f..03a519d29616d7 100644
> --- a/drivers/i2c/busses/i2c-at91.c
> +++ b/drivers/i2c/busses/i2c-at91.c
> @@ -1118,8 +1118,6 @@ static int at91_twi_probe(struct platform_device *pdev)
>
> rc = i2c_add_numbered_adapter(&dev->adapter);
> if (rc) {
> - dev_err(dev->dev, "Adapter %s registration failed\n",
> - dev->adapter.name);
> clk_disable_unprepare(dev->clk);
>
> pm_runtime_disable(dev->dev);
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [STLinux Kernel] [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:36 ` Wolfram Sang
2016-08-09 16:45 ` Joachim Eastwood
@ 2016-08-10 6:50 ` Patrice Chotard
2 siblings, 0 replies; 21+ messages in thread
From: Patrice Chotard @ 2016-08-10 6:50 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA,
kernel-F5mvAk5X5gdBDgjK7y7TUQ,
adi-buildroot-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-i2c-u79uwXL29TY76Z2rM5mHXA,
linux-tegra-u79uwXL29TY76Z2rM5mHXA,
linux-amlogic-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
linux-omap-u79uwXL29TY76Z2rM5mHXA,
linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r
On 08/09/2016 01:36 PM, Wolfram Sang wrote:
> Since v4.8-rc1, the I2C core will print detailed information when adding an I2C
> adapter fails. So, drivers can skip this now.
>
> I am still undecided if I apply this as a single patch or break it out. But for
> reviewing, avoiding the patch bomb is probably a good thing.
>
> Should go via subsystem tree, I'd think.
>
> Wolfram Sang (1):
> i2c: don't print error when adding adapter fails
>
> drivers/i2c/busses/i2c-st.c | 4 +---
>
For i2c-st.c
Acked-by: Patrice Chotard <patrice.chotard-qxv4g6HH51o@public.gmane.org>
Thanks
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 ` Wolfram Sang
` (5 preceding siblings ...)
2016-08-09 16:36 ` Vladimir Zapolskiy
@ 2016-08-10 8:32 ` Krzysztof Kozlowski
6 siblings, 0 replies; 21+ messages in thread
From: Krzysztof Kozlowski @ 2016-08-10 8:32 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel
Cc: Heiko Stuebner, Wolfram Sang, Tony Lindgren, Linus Walleij,
Sekhar Nori, Masahiro Yamada, Laxman Dewangan, linux-i2c,
Guan Xuetao, Peter Korsgaard, Sonic Zhang, kernel,
Alexandre Courbot, Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip, Kukjin Kim, bcm-kernel-feedback-list,
linux-arm-kernel, Sylvain Lemieux, Guenter Roeck,
Alessandro Rubini, Jean Delvare
On 08/09/2016 01:36 PM, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev@sang-engineering.com>
> ---
(...)
> drivers/i2c/busses/i2c-exynos5.c | 4 +---
> drivers/i2c/busses/i2c-s3c2410.c | 1 -
For Exynos and S3C2410:
Acked-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
` (4 preceding siblings ...)
2016-08-10 6:17 ` Ludovic Desroches
@ 2016-08-10 8:34 ` Laxman Dewangan
2016-10-05 15:17 ` Jean Delvare
6 siblings, 0 replies; 21+ messages in thread
From: Laxman Dewangan @ 2016-08-10 8:34 UTC (permalink / raw)
To: Wolfram Sang, linux-kernel-u79uwXL29TY76Z2rM5mHXA
Cc: Krzysztof Kozlowski, Heiko Stuebner, Wolfram Sang, Tony Lindgren,
Linus Walleij, Sekhar Nori, Jochen Friedrich, Masahiro Yamada,
Thierry Reding, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guan Xuetao,
Peter Korsgaard, Sonic Zhang, kernel-F5mvAk5X5gdBDgjK7y7TUQ,
Alexandre Courbot, Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Kukjin Kim,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Sylvain Lemieux, Guenter Roeck
On Tuesday 09 August 2016 05:06 PM, Wolfram Sang wrote:
> The core will do this for us now.
>
For i2c-tegra.c
Acked-by: Laxman Dewangan <ldewangan-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
2016-08-09 11:36 [PATCH] i2c: don't print error when adding adapter fails Wolfram Sang
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
@ 2016-08-14 22:17 ` Wolfram Sang
1 sibling, 0 replies; 21+ messages in thread
From: Wolfram Sang @ 2016-08-14 22:17 UTC (permalink / raw)
To: Wolfram Sang
Cc: linux-kernel, linux-i2c, adi-buildroot-devel, kernel,
linux-amlogic, linux-arm-kernel, linux-omap, linuxppc-dev,
linux-rockchip, linux-samsung-soc, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 511 bytes --]
On Tue, Aug 09, 2016 at 01:36:16PM +0200, Wolfram Sang wrote:
> Since v4.8-rc1, the I2C core will print detailed information when adding an I2C
> adapter fails. So, drivers can skip this now.
>
> I am still undecided if I apply this as a single patch or break it out. But for
> reviewing, avoiding the patch bomb is probably a good thing.
>
> Should go via subsystem tree, I'd think.
>
> Wolfram Sang (1):
> i2c: don't print error when adding adapter fails
>
Applied to for-next, thanks!
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 21+ messages in thread
* Re: [PATCH] i2c: don't print error when adding adapter fails
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
` (5 preceding siblings ...)
2016-08-10 8:34 ` Laxman Dewangan
@ 2016-10-05 15:17 ` Jean Delvare
6 siblings, 0 replies; 21+ messages in thread
From: Jean Delvare @ 2016-10-05 15:17 UTC (permalink / raw)
To: Wolfram Sang
Cc: Krzysztof Kozlowski, Heiko Stuebner, Wolfram Sang, Tony Lindgren,
Linus Walleij, Sekhar Nori, Jochen Friedrich, Masahiro Yamada,
Laxman Dewangan, linux-i2c-u79uwXL29TY76Z2rM5mHXA, Guan Xuetao,
Peter Korsgaard, Sonic Zhang, kernel-F5mvAk5X5gdBDgjK7y7TUQ,
Alexandre Courbot, Florian Fainelli, Kevin Hilman, Michal Simek,
linux-rockchip-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Kukjin Kim,
bcm-kernel-feedback-list-dY08KVG/lbpWk0Htik3J/w,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
Sylvain Lemieux, Guenter Roeck
On Tue, 9 Aug 2016 13:36:17 +0200, Wolfram Sang wrote:
> The core will do this for us now.
>
> Signed-off-by: Wolfram Sang <wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
> ---
> drivers/i2c/busses/i2c-amd756.c | 5 +----
> drivers/i2c/busses/i2c-at91.c | 2 --
> drivers/i2c/busses/i2c-axxia.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-iproc.c | 8 +-------
> drivers/i2c/busses/i2c-bcm-kona.c | 4 +---
> drivers/i2c/busses/i2c-bfin-twi.c | 4 +---
> drivers/i2c/busses/i2c-brcmstb.c | 4 +---
> drivers/i2c/busses/i2c-cadence.c | 4 +---
> drivers/i2c/busses/i2c-cpm.c | 4 +---
> drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +---
> drivers/i2c/busses/i2c-davinci.c | 4 +---
> drivers/i2c/busses/i2c-diolan-u2c.c | 4 +---
> drivers/i2c/busses/i2c-dln2.c | 4 +---
> drivers/i2c/busses/i2c-efm32.c | 1 -
> drivers/i2c/busses/i2c-exynos5.c | 4 +---
> drivers/i2c/busses/i2c-hix5hd2.c | 4 +---
> drivers/i2c/busses/i2c-i801.c | 1 -
> drivers/i2c/busses/i2c-ibm_iic.c | 4 +---
> drivers/i2c/busses/i2c-img-scb.c | 4 +---
> drivers/i2c/busses/i2c-imx.c | 4 +---
> drivers/i2c/busses/i2c-isch.c | 4 +---
> drivers/i2c/busses/i2c-ismt.c | 4 +---
> drivers/i2c/busses/i2c-jz4780.c | 4 +---
> drivers/i2c/busses/i2c-lpc2k.c | 4 +---
> drivers/i2c/busses/i2c-meson.c | 1 -
> drivers/i2c/busses/i2c-mpc.c | 4 +---
> drivers/i2c/busses/i2c-mt65xx.c | 4 +---
> drivers/i2c/busses/i2c-mxs.c | 1 -
> drivers/i2c/busses/i2c-nforce2.c | 1 -
> drivers/i2c/busses/i2c-nomadik.c | 4 +---
> drivers/i2c/busses/i2c-ocores.c | 4 +---
> drivers/i2c/busses/i2c-octeon.c | 4 +---
> drivers/i2c/busses/i2c-omap.c | 4 +---
> drivers/i2c/busses/i2c-piix4.c | 1 -
> drivers/i2c/busses/i2c-pmcmsp.c | 4 +---
> drivers/i2c/busses/i2c-pnx.c | 4 +---
> drivers/i2c/busses/i2c-puv3.c | 5 +----
> drivers/i2c/busses/i2c-pxa.c | 4 +---
> drivers/i2c/busses/i2c-rcar.c | 4 +---
> drivers/i2c/busses/i2c-riic.c | 4 +---
> drivers/i2c/busses/i2c-rk3x.c | 4 +---
> drivers/i2c/busses/i2c-s3c2410.c | 1 -
> drivers/i2c/busses/i2c-sh7760.c | 4 +---
> drivers/i2c/busses/i2c-sh_mobile.c | 1 -
> drivers/i2c/busses/i2c-sirf.c | 4 +---
> drivers/i2c/busses/i2c-st.c | 4 +---
> drivers/i2c/busses/i2c-stu300.c | 5 +----
> drivers/i2c/busses/i2c-tegra.c | 4 +---
> drivers/i2c/busses/i2c-uniphier-f.c | 7 +------
> drivers/i2c/busses/i2c-uniphier.c | 7 +------
> drivers/i2c/busses/i2c-wmt.c | 4 +---
> drivers/i2c/busses/i2c-xgene-slimpro.c | 1 -
> drivers/i2c/busses/i2c-xiic.c | 1 -
> drivers/i2c/busses/i2c-xlp9xx.c | 4 +---
> drivers/i2c/busses/i2c-xlr.c | 4 +---
> 55 files changed, 44 insertions(+), 161 deletions(-)
> (...)
I like the idea.
For i2c-amd756, i2c-diolan-u2c, i2c-i801, i2c-isch, i2c-ismt,
i2c-nforce and i2c-piix4:
Reviewed-by: Jean Delvare <jdelvare-l3A5Bk7waGM@public.gmane.org>
--
Jean Delvare
SUSE L3 Support
^ permalink raw reply [flat|nested] 21+ messages in thread
end of thread, other threads:[~2016-10-05 15:17 UTC | newest]
Thread overview: 21+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-09 11:36 [PATCH] i2c: don't print error when adding adapter fails Wolfram Sang
[not found] ` <1470742581-12971-1-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:36 ` Wolfram Sang
[not found] ` <1470742581-12971-2-git-send-email-wsa-dev-jBu1N2QxHDJrcw3mvpCnnVaTQe2KTcn/@public.gmane.org>
2016-08-09 11:47 ` Grygorii Strashko
2016-08-09 11:48 ` Peter Korsgaard
2016-08-09 12:58 ` Thierry Reding
2016-08-09 16:15 ` Ray Jui
2016-08-10 6:17 ` Ludovic Desroches
2016-08-10 8:34 ` Laxman Dewangan
2016-10-05 15:17 ` Jean Delvare
2016-08-09 11:57 ` Uwe Kleine-König
2016-08-09 13:59 ` Wolfram Sang
2016-08-09 12:13 ` Heiko Stübner
2016-08-09 12:55 ` Neil Horman
[not found] ` <20160809125522.GA15676-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2016-08-10 2:56 ` Zhang, Sonic
2016-08-09 13:31 ` Guenter Roeck
2016-08-09 14:39 ` Wolfram Sang
2016-08-09 16:36 ` Vladimir Zapolskiy
2016-08-10 8:32 ` Krzysztof Kozlowski
2016-08-09 16:45 ` Joachim Eastwood
2016-08-10 6:50 ` [STLinux Kernel] " Patrice Chotard
2016-08-14 22:17 ` Wolfram Sang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).