From: zhang songyi <zhang.songyi@xxxxxxxxxx>As you're leaving only one statement in the if, you should drop the braces as well.
There is no need to call the dev_err() function directly to print a
custom message when handling an error from either the platform_get_irq()
or platform_get_irq_byname() functions as both are going to display an
appropriate error message in case of a failure.
/drivers/soc/pxa/ssp.c:150:2-9: line 150 is redundant because
platform_get_irq() already prints an error
Signed-off-by: zhang songyi <zhang.songyi@xxxxxxxxxx>
---
drivers/soc/pxa/ssp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/soc/pxa/ssp.c b/drivers/soc/pxa/ssp.c
index 93449fb3519e..7085901b4e4b 100644
--- a/drivers/soc/pxa/ssp.c
+++ b/drivers/soc/pxa/ssp.c
@@ -147,7 +147,6 @@ static int pxa_ssp_probe(struct platform_device *pdev)
ssp->irq = platform_get_irq(pdev, 0);
if (ssp->irq < 0) {
- dev_err(dev, "no IRQ resource defined\n");
return -ENODEV;
}