2017-11-19 03:56:46

by Arvind Yadav

[permalink] [raw]
Subject: [PATCH 6/6 v2] mmc: sunxi-mmc: Handle return value of platform_get_irq

platform_get_irq() can fail here and we must check its return value.

Signed-off-by: Arvind Yadav <[email protected]>
---
changes in v2 :
Add failure case '<= 0' instead of '< 0'. IRQ0 is not valid.

drivers/mmc/host/sunxi-mmc.c | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/drivers/mmc/host/sunxi-mmc.c b/drivers/mmc/host/sunxi-mmc.c
index cc98355d..ec2a16b 100644
--- a/drivers/mmc/host/sunxi-mmc.c
+++ b/drivers/mmc/host/sunxi-mmc.c
@@ -1255,6 +1255,11 @@ static int sunxi_mmc_resource_request(struct sunxi_mmc_host *host,
goto error_assert_reset;

host->irq = platform_get_irq(pdev, 0);
+ if (host->irq <= 0) {
+ ret = host->irq;
+ goto error_assert_reset;
+ }
+
return devm_request_threaded_irq(&pdev->dev, host->irq, sunxi_mmc_irq,
sunxi_mmc_handle_manual_stop, 0, "sunxi-mmc", host);

--
2.7.4


From 1585132247573597132@xxx Sun Nov 26 12:38:32 +0000 2017
X-GM-THRID: 1585132247573597132
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread