diff mbox series

i2c: rcar: add IRQ check

Message ID 8a05ea84-28e6-4d76-4f6d-55fb0a0cdf24@omprussia.ru
State New
Headers show
Series i2c: rcar: add IRQ check | expand

Commit Message

Sergey Shtylyov April 4, 2021, 5:52 p.m. UTC
The driver neglects to check the result of platform_get_irq()'s call and
blithely passes the negative error codes to devm_request_irq() (which
takes *unsigned* IRQ #), causing it to fail with -EINVAL, overriding
an original error code.  Stop calling devm_request_irq() with the
invalid IRQ #s.

Fixes: 6ccbe607132b ("i2c: add Renesas R-Car I2C driver")
Signed-off-by: Sergey Shtylyov <s.shtylyov@omprussia.ru>

---
 drivers/i2c/busses/i2c-rcar.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

Comments

Sergey Shtylyov April 5, 2021, 2:34 p.m. UTC | #1
On 4/4/21 8:52 PM, Sergey Shtylyov wrote:

> The driver neglects to check the result of platform_get_irq()'s call and

> blithely passes the negative error codes to devm_request_irq() (which

> takes *unsigned* IRQ #), causing it to fail with -EINVAL, overriding

> an original error code.  Stop calling devm_request_irq() with the

> invalid IRQ #s.

> 

> Fixes: 6ccbe607132b ("i2c: add Renesas R-Car I2C driver")

> Signed-off-by: Sergey Shtylyov <s.shtylyov@omprussia.ru>


   Sorry, forgot to mention that this patch is against the 'master' branch of
Wolfram's 'linux.git' repo...

MBR, Sergey
Geert Uytterhoeven April 6, 2021, 7:37 a.m. UTC | #2
On Sun, Apr 4, 2021 at 7:53 PM Sergey Shtylyov <s.shtylyov@omprussia.ru> wrote:
> The driver neglects to check the result of platform_get_irq()'s call and

> blithely passes the negative error codes to devm_request_irq() (which

> takes *unsigned* IRQ #), causing it to fail with -EINVAL, overriding

> an original error code.  Stop calling devm_request_irq() with the

> invalid IRQ #s.

>

> Fixes: 6ccbe607132b ("i2c: add Renesas R-Car I2C driver")

> Signed-off-by: Sergey Shtylyov <s.shtylyov@omprussia.ru>


Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>


Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Wolfram Sang April 8, 2021, 9:04 p.m. UTC | #3
> +	priv->irq = ret = platform_get_irq(pdev, 0);


Please no double assignments. Otherwise good catch!
Sergey Shtylyov April 9, 2021, 7:33 p.m. UTC | #4
On 4/9/21 12:04 AM, Wolfram Sang wrote:

>> +	priv->irq = ret = platform_get_irq(pdev, 0);

> 

> Please no double assignments. Otherwise good catch!


   OK, I'll come back with 5 more patches for the similar problems. :-)

MBR, Sergei
diff mbox series

Patch

Index: linux/drivers/i2c/busses/i2c-rcar.c
===================================================================
--- linux.orig/drivers/i2c/busses/i2c-rcar.c
+++ linux/drivers/i2c/busses/i2c-rcar.c
@@ -1027,7 +1027,9 @@  static int rcar_i2c_probe(struct platfor
 	if (of_property_read_bool(dev->of_node, "smbus"))
 		priv->flags |= ID_P_HOST_NOTIFY;
 
-	priv->irq = platform_get_irq(pdev, 0);
+	priv->irq = ret = platform_get_irq(pdev, 0);
+	if (ret < 0)
+		goto out_pm_disable;
 	ret = devm_request_irq(dev, priv->irq, irqhandler, irqflags, dev_name(dev), priv);
 	if (ret < 0) {
 		dev_err(dev, "cannot get irq %d\n", priv->irq);