diff options
author | Stephen Boyd <swboyd@chromium.org> | 2019-07-30 11:15:27 -0700 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2019-08-12 11:29:47 +0100 |
commit | 802d9bd4fac70be2ea61fa83660a87a57d06bab0 (patch) | |
tree | 6172e94b70b5d7eb4d323178bdf6cdbdea0feb43 /drivers/mfd/fsl-imx25-tsadc.c | |
parent | b620c17672b9c162bbc1e480eaf43a825345cb2a (diff) | |
download | linux-802d9bd4fac70be2ea61fa83660a87a57d06bab0.tar.gz |
mfd: Remove dev_err() usage after platform_get_irq()
We don't need dev_err() messages when platform_get_irq() fails now that platform_get_irq() prints an error message itself when something goes wrong. Let's remove these prints with a simple semantic patch. // <smpl> @@ expression ret; struct platform_device *E; @@ ret = ( platform_get_irq(E, ...) | platform_get_irq_byname(E, ...) ); if ( \( ret < 0 \| ret <= 0 \) ) { ( -if (ret != -EPROBE_DEFER) -{ ... -dev_err(...); -... } | ... -dev_err(...); ) ... } // </smpl> While we're here, remove braces on if statements that only have one statement (manually). Signed-off-by: Stephen Boyd <swboyd@chromium.org> Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/mfd/fsl-imx25-tsadc.c')
-rw-r--r-- | drivers/mfd/fsl-imx25-tsadc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/mfd/fsl-imx25-tsadc.c b/drivers/mfd/fsl-imx25-tsadc.c index 20791cab7263..a016b39fe9b0 100644 --- a/drivers/mfd/fsl-imx25-tsadc.c +++ b/drivers/mfd/fsl-imx25-tsadc.c @@ -69,10 +69,8 @@ static int mx25_tsadc_setup_irq(struct platform_device *pdev, int irq; irq = platform_get_irq(pdev, 0); - if (irq <= 0) { - dev_err(dev, "Failed to get irq\n"); + if (irq <= 0) return irq; - } tsadc->domain = irq_domain_add_simple(np, 2, 0, &mx25_tsadc_domain_ops, tsadc); |