[PATCH RESEND] mtd: denali: fix error handling for devm_ioremap_nocache()

[PATCH RESEND] mtd: denali: fix error handling for devm_ioremap_nocache()

[Xi Wang]

It is incorrect to use `!res' for error handling since `res' must be
non-null.  Use the return value `ptr' instead.

Signed-off-by: Xi Wang <xi.wang@gmail.com>
---
 drivers/mtd/nand/denali_dt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/denali_dt.c b/drivers/mtd/nand/denali_dt.c
index 546f8cb..02988b0 100644
--- a/drivers/mtd/nand/denali_dt.c
+++ b/drivers/mtd/nand/denali_dt.c
@@ -42,7 +42,7 @@ static void __iomem *request_and_map(struct device *dev,
 	}
 
 	ptr = devm_ioremap_nocache(dev, res->start, resource_size(res));
-	if (!res)
+	if (!ptr)
 		dev_err(dev, "ioremap_nocache of %s failed!", res->name);
 
 	return ptr;
-- 
1.8.1.2

Re: [PATCH RESEND] mtd: denali: fix error handling for devm_ioremap_nocache()

[Artem Bityutskiy]

On Fri, 2013-05-03 at 00:37 -0400, Xi Wang wrote:
> It is incorrect to use `!res' for error handling since `res' must be
> non-null.  Use the return value `ptr' instead.
> 
> Signed-off-by: Xi Wang <xi.wang@gmail.com>
> ---

This is already fixed upstream:

commit 6c67050ab108bcce2576fa1d7a0557eb0376520a
Author: Sachin Kamat <sachin.kamat@linaro.org>
Date:   Mon Mar 18 15:11:11 2013 +0530

    mtd: denali_dt: Fix incorrect error check
    
    The return value of devm_ioremap_nocache should be checked here
instead
    of res.
    
    Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
    Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
    Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>

-- 
Best Regards,
Artem Bityutskiy