From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from eusmtp01.atmel.com ([212.144.249.242]:30521 "EHLO eusmtp01.atmel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753223Ab3HWDyZ (ORCPT ); Thu, 22 Aug 2013 23:54:25 -0400 Message-ID: <5216DCE6.60109@atmel.com> Date: Fri, 23 Aug 2013 11:54:14 +0800 From: Josh Wu MIME-Version: 1.0 Subject: Re: [PATCH -next] mtd: atmel_nand: fix error return code in atmel_nand_probe() References: In-Reply-To: Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: devicetree-owner@vger.kernel.org To: Wei Yongjun Cc: dwmw2@infradead.org, grant.likely@linaro.org, rob.herring@calxeda.com, artem.bityutskiy@linux.intel.com, plagnioj@jcrosoft.com, jg1.han@samsung.com, yongjun_wei@trendmicro.com.cn, linux-mtd@lists.infradead.org, devicetree@vger.kernel.org List-ID: Hi, Wei Yongjun On 8/23/2013 10:50 AM, Wei Yongjun wrote: > From: Wei Yongjun > > Fix to return a negative error code from the error handling > case instead of 0, as done elsewhere in this function. > > Signed-off-by: Wei Yongjun Thank you for catch this and Acked-by: Josh Wu > --- > drivers/mtd/nand/atmel_nand.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/mtd/nand/atmel_nand.c b/drivers/mtd/nand/atmel_nand.c > index 65b302c..093bcf8 100644 > --- a/drivers/mtd/nand/atmel_nand.c > +++ b/drivers/mtd/nand/atmel_nand.c > @@ -2047,6 +2047,7 @@ static int __init atmel_nand_probe(struct platform_device *pdev) > irq = platform_get_irq(pdev, 0); > if (irq < 0) { > dev_err(host->dev, "Cannot get HSMC irq!\n"); > + res = irq; > goto err_nand_ioremap; > } > > Best Regards, Josh Wu