Commit 58fb7c64 authored by Sergey Shtylyov's avatar Sergey Shtylyov Committed by Wolfram Sang

i2c: mt65xx: fix IRQ check

Iff platform_get_irq() returns 0, the driver's probe() method will return 0
early (as if the method's call was successful).  Let's consider IRQ0 valid
for simplicity -- devm_request_irq() can always override that decision...

Fixes: ce38815d ("I2C: mediatek: Add driver for MediaTek I2C controller")
Signed-off-by: default avatarSergey Shtylyov <s.shtylyov@omprussia.ru>
Reviewed-by: default avatarQii Wang <qii.wang@mediatek.com>
Signed-off-by: default avatarWolfram Sang <wsa@kernel.org>
parent 3cfc8838
...@@ -1211,7 +1211,7 @@ static int mtk_i2c_probe(struct platform_device *pdev) ...@@ -1211,7 +1211,7 @@ static int mtk_i2c_probe(struct platform_device *pdev)
return PTR_ERR(i2c->pdmabase); return PTR_ERR(i2c->pdmabase);
irq = platform_get_irq(pdev, 0); irq = platform_get_irq(pdev, 0);
if (irq <= 0) if (irq < 0)
return irq; return irq;
init_completion(&i2c->msg_complete); init_completion(&i2c->msg_complete);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment