From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751374AbbKTFKv (ORCPT ); Fri, 20 Nov 2015 00:10:51 -0500 Received: from mailout3.w1.samsung.com ([210.118.77.13]:29859 "EHLO mailout3.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751038AbbKTFKt (ORCPT ); Fri, 20 Nov 2015 00:10:49 -0500 X-AuditID: cbfec7f4-f79026d00000418a-ee-564eab55c8e6 Subject: Re: [PATCH v6 8/9] ARM: EXYNOS: rearrange static and non-static functions of PMU driver To: Manish Badarkhe , Pankaj Dubey References: <1447740351-14246-1-git-send-email-pankaj.dubey@samsung.com> <1447740351-14246-9-git-send-email-pankaj.dubey@samsung.com> Cc: "linux-samsung-soc@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , linux-pm@vger.kernel.org, "kgene.kim@samsung.com" , thomas.ab@samsung.com, amitdanielk@gmail.com, olof@lixom.net, khilman@linaro.org, arnd@arndb.de From: Krzysztof Kozlowski Message-id: <564EAB54.6020904@samsung.com> Date: Fri, 20 Nov 2015 14:10:44 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-version: 1.0 In-reply-to: Content-type: text/plain; charset=utf-8 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrPLMWRmVeSWpSXmKPExsVy+t/xq7qhq/3CDB6+tbT4uPolm8XfScfY LeZNfMtu8fqFoUXvgqtsFl8Pr2C02PT4GqvF5V1z2Cw+9x5htJhxfh+Txanrn9ksFm39wm7R sYzRgdfj969JjB47Z91l97hzbQ+bx+Yl9R5XTjSxevRtWcXo8XmTXAB7FJdNSmpOZllqkb5d AlfGuxWdbAV7BSsmPlzP3sB4i7eLkZNDQsBE4vDvV2wQtpjEhXvrgWwuDiGBpYwSSyfCOF8Y JVZcaGIHqRIWSJLofNjEAmKLCERJvH72BaroHKNE25s2dhCHWaCfWWJp43QmkCo2AWOJzcuX AFVxcPAKaEkcecUJEmYRUJV4Oucd2FBRgQiJiRMaWEFsXgFBiR+T77GAlHMKBEuc/ATWySyg LjFlSi5IBbOAvMTmNW+ZJzAKzELSMAuhahaSqgWMzKsYRVNLkwuKk9JzDfWKE3OLS/PS9ZLz czcxQqLlyw7GxcesDjEKcDAq8fA2iPuFCbEmlhVX5h5ilOBgVhLhXeoHFOJNSaysSi3Kjy8q zUktPsQozcGiJM47d9f7ECGB9MSS1OzU1ILUIpgsEwenVAPjotYVnXwBoW/F57J+j+atFpmS fFnM7OhM2bXVyeLTw3ayfBK5t7IzK/PPvqJ63ev5zz8vftEdELr5d+u0d4nPsgV3JgTfWLAo Oc3Ukn23xuJX7HUWvO3+fZWlky4tYX8fpnPt6MrLTWq125aY5grNL046bR9hvunEtDTLlTEO D7bUWza1apsrsRRnJBpqMRcVJwIAsrw9IpICAAA= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 20.11.2015 13:46, Manish Badarkhe wrote: > On Tue, Nov 17, 2015 at 11:35 AM, Pankaj Dubey wrote: >> This patch moves exynos_sys_powerdown_conf function above all >> static functions, to avoid confusion causing due to mixing of >> static-nonstatic-static functions and to improve readability of this >> driver. >> >> Signed-off-by: Pankaj Dubey >> Suggested-by: Krzysztof Kozlowski >> Reviewed-by: Krzysztof Kozlowski >> --- >> arch/arm/mach-exynos/pmu.c | 34 +++++++++++++++++----------------- >> 1 file changed, 17 insertions(+), 17 deletions(-) >> >> diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c >> index 01cb649..a7741d4 100644 >> --- a/arch/arm/mach-exynos/pmu.c >> +++ b/arch/arm/mach-exynos/pmu.c >> @@ -39,23 +39,6 @@ u32 pmu_raw_readl(u32 offset) >> return readl_relaxed(pmu_base_addr + offset); >> } >> >> -static void exynos_power_off(void) >> -{ >> - unsigned int tmp; >> - >> - pr_info("Power down.\n"); >> - tmp = pmu_raw_readl(EXYNOS_PS_HOLD_CONTROL); >> - tmp ^= (1 << 8); >> - pmu_raw_writel(tmp, EXYNOS_PS_HOLD_CONTROL); >> - >> - /* Wait a little so we don't give a false warning below */ >> - mdelay(100); >> - >> - pr_err("Power down failed, please power off system manually.\n"); >> - while (1) >> - ; >> -} >> - >> void exynos_sys_powerdown_conf(enum sys_powerdown mode) >> { >> unsigned int i; >> @@ -85,6 +68,23 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode) >> } >> } >> >> +static void exynos_power_off(void) >> +{ >> + unsigned int tmp; >> + >> + pr_info("Power down.\n"); >> + tmp = pmu_raw_readl(EXYNOS_PS_HOLD_CONTROL); >> + tmp ^= (1 << 8); > Can we have some define over here? to operate this bit. > 1. This will be removed by syscon-reboot/poweroff patches [0]. 2. This patch is only rename/move. Fixing stuff should go to separate patches. But IMHO fixing is not needed because of 1. [0] http://www.spinics.net/lists/devicetree/msg98858.html Best regards, Krzysztof