From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lukasz Majewski Subject: Re: [PATCH] cpufreq:exynos-cpufreq - Fix for memory leak in case SOC name does not match. Date: Tue, 09 Jun 2015 09:07:22 +0200 Message-ID: <20150609090722.43965fa0@amdc2363> References: <1432519780-2910-1-git-send-email-shailendra.capricorn@gmail.com> <20150525073149.GF9992@linux> <20150608103100.57f0473b@amdc2363> <8344426.BL3KyufMos@vostro.rjw.lan> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from mailout1.samsung.com ([203.254.224.24]:47551 "EHLO mailout1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752586AbbFIHIF (ORCPT ); Tue, 9 Jun 2015 03:08:05 -0400 In-reply-to: <8344426.BL3KyufMos@vostro.rjw.lan> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: "Rafael J. Wysocki" Cc: Krzysztof Kozlowski , linux-samsung-soc@vger.kernel.org, linux-pm@vger.kernel.org, Viresh Kumar , linux-kernel@vger.kernel.org, Kukjin Kim , Shailendra Verma , linux-arm-kernel@lists.infradead.org Hi Rafael, > On Monday, June 08, 2015 10:31:00 AM Lukasz Majewski wrote: > > Hi Viresh, > > > > > On 25-05-15, 07:39, Shailendra Verma wrote: > > > > During probe free the memory allocated to "exynos_info" in case > > > > of unknown SOC type. > > > > > > > > Signed-off-by: Shailendra Verma > > > > --- > > > > drivers/cpufreq/exynos-cpufreq.c | 6 ++++-- > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/drivers/cpufreq/exynos-cpufreq.c > > > > b/drivers/cpufreq/exynos-cpufreq.c index 82d2fbb..8682378 100644 > > > > --- a/drivers/cpufreq/exynos-cpufreq.c > > > > +++ b/drivers/cpufreq/exynos-cpufreq.c > > > > @@ -182,7 +182,7 @@ static int exynos_cpufreq_probe(struct > > > > platform_device *pdev) ret = > > > > exynos5250_cpufreq_init(exynos_info); } else { > > > > pr_err("%s: Unknown SoC type\n", __func__); > > > > - return -ENODEV; > > > > + ret = -ENODEV; > > > > } > > > > > > > > if (ret) > > > > @@ -190,12 +190,14 @@ static int exynos_cpufreq_probe(struct > > > > platform_device *pdev) > > > > if (exynos_info->set_freq == NULL) { > > > > dev_err(&pdev->dev, "No set_freq function > > > > (ERR)\n"); > > > > + ret = -EINVAL; > > > > goto err_vdd_arm; > > > > } > > > > > > > > arm_regulator = regulator_get(NULL, "vdd_arm"); > > > > if (IS_ERR(arm_regulator)) { > > > > dev_err(&pdev->dev, "failed to get resource > > > > vdd_arm\n"); > > > > + ret = -EINVAL; > > > > goto err_vdd_arm; > > > > } > > > > > > > > @@ -227,7 +229,7 @@ err_cpufreq_reg: > > > > regulator_put(arm_regulator); > > > > err_vdd_arm: > > > > kfree(exynos_info); > > > > - return -EINVAL; > > > > + return ret; > > > > } > > > > > > > > static struct platform_driver exynos_cpufreq_platdrv = { > > > > > > Acked-by: Viresh Kumar > > > > > > > Acked-by: Lukasz Majewski > > I'm assuming that it will go it through the Samsung tree. > > That would be the easiest way to proceed. Krzystof would you like to grab this patch? -- Best regards, Lukasz Majewski Samsung R&D Institute Poland (SRPOL) | Linux Platform Group