From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1163332AbbKTRRr (ORCPT ); Fri, 20 Nov 2015 12:17:47 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:57230 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1162917AbbKTRRp (ORCPT ); Fri, 20 Nov 2015 12:17:45 -0500 Date: Fri, 20 Nov 2015 09:17:44 -0800 From: Stephen Boyd To: Sudip Mukherjee Cc: Michael Turquette , linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org Subject: Re: [PATCH] clk: versatile: fix memory leak Message-ID: <20151120171744.GN32672@codeaurora.org> References: <1447681600-14120-1-git-send-email-sudipm.mukherjee@gmail.com> <20151118222446.GG32672@codeaurora.org> <20151120085815.GB4485@sudip-pc> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151120085815.GB4485@sudip-pc> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/20, Sudip Mukherjee wrote: > On Wed, Nov 18, 2015 at 02:24:46PM -0800, Stephen Boyd wrote: > > On 11/16, Sudip Mukherjee wrote: > > > If of_clk_parent_fill() fails then we printed an error message and > > > returned. But we missed freeing sp810. > > > > > > Signed-off-by: Sudip Mukherjee > > > --- > > > > Applied to clk-next > > Thanks. > > > > We don't need that error message on allocation failure either. > > I sent another patch to remove the error message on kzalloc failure. Did > you mean that one or did you mean the error message for > of_clk_parent_fill()? Yes I meant the one you removed. Thanks. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project