From: Herbert Xu Subject: Re: [PATCH] crypto: caam: Introduce the use of the managed version of kzalloc Date: Sat, 21 Jun 2014 02:41:05 +0800 Message-ID: <20140620184105.GD3169@gondor.apana.org.au> References: <20140527182548.GA6333@himangi-Dell> <201406011856.41778.marex@denx.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Himangi Saraogi , "David S. Miller" , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, julia.lawall@lip6.fr To: Marek Vasut Return-path: Received: from helcar.apana.org.au ([209.40.204.226]:35203 "EHLO helcar.apana.org.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966182AbaFTSlO (ORCPT ); Fri, 20 Jun 2014 14:41:14 -0400 Content-Disposition: inline In-Reply-To: <201406011856.41778.marex@denx.de> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Sun, Jun 01, 2014 at 06:56:41PM +0200, Marek Vasut wrote: > On Tuesday, May 27, 2014 at 08:25:48 PM, Himangi Saraogi wrote: > > This patch moves data allocated using kzalloc to managed data allocated > > using devm_kzalloc and cleans now unnecessary kfrees in probe and remove > > functions. Also, linux/device.h is added to make sure the devm_*() > > routine declarations are unambiguously available. Earlier, in the probe > > function ctrlpriv was leaked on the failure of ctrl = of_iomap(nprop, 0); > > as well as on the failure of ctrlpriv->jrpdev = kzalloc(...); . These > > two bugs have been fixed by the patch. > > > > The following Coccinelle semantic patch was used for making the change: > > Reviewed-by: Marek Vasut Patch applied. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt