[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <fd6dac9b18d2ce37998901dc074c61b94e523969.1512497067.git.arvind.yadav.cs@gmail.com>
Date: Tue, 5 Dec 2017 23:41:52 +0530
From: Arvind Yadav <arvind.yadav.cs@...il.com>
To: gregkh@...uxfoundation.org, davem@...emloft.net,
dmitry.torokhov@...il.com, linus.walleij@...aro.org
Cc: linux-kernel@...r.kernel.org
Subject: [PATCH] driver-core: platform: Avoid to return IRQ 0 in platform_get_irq()
Function platform_get_irq() can return 0. Which means NO_IRQ.
So this change will not allow to return 0.
This change is help to use platform_get_irq() without this,
val = platform_get_irq();
if (val <= 0)
ret = val ? val : -ENODEV;
Signed-off-by: Arvind Yadav <arvind.yadav.cs@...il.com>
---
drivers/base/platform.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/base/platform.c b/drivers/base/platform.c
index c203fb9..7b3079c 100644
--- a/drivers/base/platform.c
+++ b/drivers/base/platform.c
@@ -127,7 +127,7 @@ int platform_get_irq(struct platform_device *dev, unsigned int num)
irqd_set_trigger_type(irqd, r->flags & IORESOURCE_BITS);
}
- return r ? r->start : -ENXIO;
+ return r && r->start ? r->start : -ENXIO;
#endif
}
EXPORT_SYMBOL_GPL(platform_get_irq);
--
2.7.4
Powered by blists - more mailing lists