]> git.hungrycats.org Git - linux/commitdiff
cpufreq: ti-cpufreq: Fix an incorrect error return value
authorSuman Anna <s-anna@ti.com>
Thu, 31 May 2018 22:21:43 +0000 (17:21 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 25 Jun 2018 23:54:02 +0000 (07:54 +0800)
commit e5d295b06d69a1924665a16a4987be475addd00f upstream.

Commit 05829d9431df (cpufreq: ti-cpufreq: kfree opp_data when
failure) has fixed a memory leak in the failure path, however
the patch returned a positive value on get_cpu_device() failure
instead of the previous negative value. Fix this incorrect error
return value properly.

Fixes: 05829d9431df (cpufreq: ti-cpufreq: kfree opp_data when failure)
Cc: 4.14+ <stable@vger.kernel.org> # v4.14+
Signed-off-by: Suman Anna <s-anna@ti.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/cpufreq/ti-cpufreq.c

index a099b7bf74cd9ff741afc21ff0417958280784c3..46d1ab2dea87de569f957e58e72017f5c0f9c84b 100644 (file)
@@ -226,7 +226,7 @@ static int ti_cpufreq_probe(struct platform_device *pdev)
        opp_data->cpu_dev = get_cpu_device(0);
        if (!opp_data->cpu_dev) {
                pr_err("%s: Failed to get device for CPU0\n", __func__);
-               ret = ENODEV;
+               ret = -ENODEV;
                goto free_opp_data;
        }