| Message ID | 8a05ea84-28e6-4d76-4f6d-55fb0a0cdf24@omprussia.ru (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Geert Uytterhoeven |
| Headers | show |
| Series | i2c: rcar: add IRQ check | expand |
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
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
> + priv->irq = ret = platform_get_irq(pdev, 0);
Please no double assignments. Otherwise good catch!
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
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);
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(-)