diff options
author | Wu Fengguang <fengguang.wu@intel.com> | 2015-01-21 06:19:45 -0500 |
---|---|---|
committer | Ulf Hansson <ulf.hansson@linaro.org> | 2015-01-21 06:33:43 -0500 |
commit | 3e7cf90213e789bdc13f2638cc9d8b9993d5491e (patch) | |
tree | 213dc163464a206fc9e746bd67fc7d5883fe93ea | |
parent | 6bb836d6fabd18d5854fcf40debdfe11cd063e6f (diff) |
mmc: sdhci: host: fix odd_ptr_err.cocci warnings
drivers/mmc/host/sdhci_f_sdh30.c:143:5-11: inconsistent IS_ERR and PTR_ERR, PTR_ERR on line 144
PTR_ERR should access the value just tested by IS_ERR
Semantic patch information:
There can be false positives in the patch case, where it is the call
IS_ERR that is wrong.
Generated by: scripts/coccinelle/tests/odd_ptr_err.cocci
CC: Vincent Yang <vincent.yang.fujitsu@gmail.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
-rw-r--r-- | drivers/mmc/host/sdhci_f_sdh30.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/sdhci_f_sdh30.c b/drivers/mmc/host/sdhci_f_sdh30.c index c1bb156445e4..2fe8b91481b3 100644 --- a/drivers/mmc/host/sdhci_f_sdh30.c +++ b/drivers/mmc/host/sdhci_f_sdh30.c | |||
@@ -141,7 +141,7 @@ static int sdhci_f_sdh30_probe(struct platform_device *pdev) | |||
141 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 141 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
142 | host->ioaddr = devm_ioremap_resource(&pdev->dev, res); | 142 | host->ioaddr = devm_ioremap_resource(&pdev->dev, res); |
143 | if (IS_ERR(host->ioaddr)) { | 143 | if (IS_ERR(host->ioaddr)) { |
144 | ret = PTR_ERR(host); | 144 | ret = PTR_ERR(host->ioaddr); |
145 | goto err; | 145 | goto err; |
146 | } | 146 | } |
147 | 147 | ||