From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935466AbcCQJtj (ORCPT ); Thu, 17 Mar 2016 05:49:39 -0400 Received: from down.free-electrons.com ([37.187.137.238]:41792 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S935334AbcCQJtg (ORCPT ); Thu, 17 Mar 2016 05:49:36 -0400 Date: Thu, 17 Mar 2016 10:49:34 +0100 From: Boris Brezillon To: Herbert Xu , "David S. Miller" , linux-crypto@vger.kernel.org Cc: Lior Amsalem , Nadav Haklai , Thomas Petazzoni , Gregory CLEMENT , Jason Cooper , Sebastian Hesselbarth , Andrew Lunn , Romain Perier , Arnaud Ebalard , Russell King - ARM Linux , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Subject: Re: [PATCH] crypto: marvell/cesa - forward devm_ioremap_resource() error code Message-ID: <20160317104934.63e9e396@bbrezillon> In-Reply-To: <1458208031-13463-2-git-send-email-boris.brezillon@free-electrons.com> References: <1458208031-13463-1-git-send-email-boris.brezillon@free-electrons.com> <1458208031-13463-2-git-send-email-boris.brezillon@free-electrons.com> X-Mailer: Claws Mail 3.12.0 (GTK+ 2.24.28; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Sorry for the noise, just sent twice the same patch :-/. Please ignore this one. On Thu, 17 Mar 2016 10:47:11 +0100 Boris Brezillon wrote: > Forward devm_ioremap_resource() error code instead of returning > -ENOMEM. > > Signed-off-by: Boris Brezillon > Reported-by: Russell King - ARM Linux > Fixes: f63601fd616a ("crypto: marvell/cesa - add a new driver for Marvell's CESA") > Cc: # 4.2+ > --- > drivers/crypto/marvell/cesa.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/crypto/marvell/cesa.c b/drivers/crypto/marvell/cesa.c > index c0656e7..80239ae 100644 > --- a/drivers/crypto/marvell/cesa.c > +++ b/drivers/crypto/marvell/cesa.c > @@ -420,7 +420,7 @@ static int mv_cesa_probe(struct platform_device *pdev) > res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "regs"); > cesa->regs = devm_ioremap_resource(dev, res); > if (IS_ERR(cesa->regs)) > - return -ENOMEM; > + return PTR_ERR(cesa->regs); > > ret = mv_cesa_dev_dma_init(cesa); > if (ret) -- Boris Brezillon, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com