From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750849AbbLJAsZ (ORCPT ); Wed, 9 Dec 2015 19:48:25 -0500 Received: from mailout2.w1.samsung.com ([210.118.77.12]:24220 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751043AbbLJAsX (ORCPT ); Wed, 9 Dec 2015 19:48:23 -0500 X-AuditID: cbfec7f5-f79b16d000005389-89-5668cbd4e01a Subject: Re: [PATCH 3.16.y-ckt 059/126] thermal: exynos: Fix unbalanced regulator disable on probe failure To: Luis Henriques References: <1449653896-5236-1-git-send-email-luis.henriques@canonical.com> <1449653896-5236-60-git-send-email-luis.henriques@canonical.com> <56682B1D.9090707@samsung.com> <20151209135952.GA4797@ares> Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, kernel-team@lists.ubuntu.com, k.kozlowski.k@gmail.com, Eduardo Valentin From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <5668CBD0.9010503@samsung.com> Date: Thu, 10 Dec 2015 09:48:16 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.4.0 MIME-version: 1.0 In-reply-to: <20151209135952.GA4797@ares> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrCLMWRmVeSWpSXmKPExsVy+t/xy7pXTmeEGVyYrWUx/8o1Vovn/36w W7x+YWgx59EmdovLu+awWTzd94jJYsHGR4wO7B6zGnrZPHbOusvuce3GfRaPvi2rGD0+b5IL YI3isklJzcksSy3St0vgyri/6yBzwWTBis9XV7M2ME7m62Lk5JAQMJG4OL+THcIWk7hwbz1b FyMXh5DAUkaJpa9PMYEkhASeMkr0bPIBsYUFciT+Hj/ECmKLCBhKnF0zgQWi4QyjxPlPz5hB HGaB2YwSl/7eYgapYhMwlti8fAkbxAo5id7uSUAdHBy8AloSO96DbWYRUJU4P2kXI0hYVCBC YtGOTJAwr4CgxI/J91hAbE4BTYnDN7aDlTAL6Encv6gFEmYWkJfYvOYt8wRGwVlIOmYhVM1C UrWAkXkVo2hqaXJBcVJ6rpFecWJucWleul5yfu4mRkjYf93BuPSY1SFGAQ5GJR7eCqf0MCHW xLLiytxDjBIczEoivFtPZYQJ8aYkVlalFuXHF5XmpBYfYpTmYFES5525632IkEB6Yklqdmpq QWoRTJaJg1OqgTHAIiXM+L6Bpa6SRhr/q6S35e2LL3F43FsfcW2pX7f6o87oCTpKv/cL7lu+ ru2r9bvZxre2ujVlP+rhZJKZEhv5wt7IgV93eUJZ9bycvZqzqp/4JLUJVaQ+McwI674vJnFc J7T9e9BhG8meA4tOlKkVrAl9sq9ujd+nk4Ee/NFi7soTT6u/UGIpzkg01GIuKk4EALm/2GV3 AgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09.12.2015 22:59, Luis Henriques wrote: > On Wed, Dec 09, 2015 at 10:22:37PM +0900, Krzysztof Kozlowski wrote: >> W dniu 09.12.2015 o 18:37, Luis Henriques pisze: >>> 3.16.7-ckt21 -stable review patch. If anyone has any objections, please let me know. >>> >>> ------------------ >>> >>> From: Krzysztof Kozlowski >>> >>> commit 824ead03b78403a21449cb7eb153a4344cd3b4c8 upstream. >>> >>> During probe if the regulator could not be enabled, the error exit path >>> would still disable it. This could lead to unbalanced counter of >>> regulator enable/disable. >>> >>> The patch moves code for getting and enabling the regulator from >>> exynos_map_dt_data() to probe function because it is really not a part >>> of getting Device Tree properties. >>> >>> Acked-by: Lukasz Majewski >>> Tested-by: Lukasz Majewski >>> Reviewed-by: Alim Akhtar >>> Signed-off-by: Krzysztof Kozlowski >>> Fixes: 5f09a5cbd14a ("thermal: exynos: Disable the regulator on probe failure") >>> Signed-off-by: Eduardo Valentin >>> [ luis: backported to 3.16: adjusted context ] >>> Signed-off-by: Luis Henriques >> >> This patch shouldn't be backported before 4.2. It fixes a bug introduced >> in 4.2: 5f09a5cbd14a. >> >> For kernels <4.2 this bug does not exist but fortunately the patch looks >> harmless. Anyway still for <4.2 I think it shouldn't be ported. >> >> Best regards, >> Krzysztof >> > > Thank you Krzysztof. > > I understand that this fixes an issue with commit 5f09a5cbd14a ("thermal: > exynos: Disable the regulator on probe failure"), which was included in > 4.2. However, since this commit was also tagged for stable, it has also > been backported to some stable kernels (at least 3.16.y-ckt and > 3.19.y-ckt). > > Would you agree that, for this reason, this patch is in fact applicable to > the 3.16 (and 3.19) kernel? Ahh, you are right! The 5f09a5cbd14a was backported so this applies to backporting as well. Sorry for the noise. Best regards, Krzysztof