i2c: highlander: add IRQ check
authorSergey Shtylyov <s.shtylyov@omp.ru>
Sun, 30 May 2021 19:13:45 +0000 (22:13 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 15 Sep 2021 07:50:36 +0000 (09:50 +0200)
commiteef8496579de75ee1482f717634fdb5378497bb1
tree0f2d4671dcdba66a1901db2a0c8fb6ece67aa178
parent11dd40c18918e7788121d25d23d853f8bf9e1fbc
i2c: highlander: add IRQ check

[ Upstream commit f16a3bb69aa6baabf8f0aca982c8cf21e2a4f6bc ]

The driver is written as if platform_get_irq() returns 0 on errors (while
actually it returns a negative error code), blithely passing these error
codes to request_irq() (which takes *unsigned* IRQ #) -- which fails with
-EINVAL. Add the necessary error check to the pre-existing *if* statement
forcing the driver into the polling mode...

Fixes: 4ad48e6ab18c ("i2c: Renesas Highlander FPGA SMBus support")
Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Wolfram Sang <wsa@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/i2c/busses/i2c-highlander.c