Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752427AbbKQGOU (ORCPT ); Tue, 17 Nov 2015 01:14:20 -0500 Received: from mailout2.samsung.com ([203.254.224.25]:56081 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750882AbbKQGOS (ORCPT ); Tue, 17 Nov 2015 01:14:18 -0500 X-AuditID: cbfee68e-f791c6d000001498-e3-564ac5b63d2f From: Pankaj Dubey To: linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org Cc: kgene.kim@samsung.com, k.kozlowski@samsung.com, thomas.ab@samsung.com, amitdanielk@gmail.com, olof@lixom.net, khilman@linaro.org, arnd@arndb.de, Pankaj Dubey Subject: [PATCH v6 8/9] ARM: EXYNOS: rearrange static and non-static functions of PMU driver Date: Tue, 17 Nov 2015 11:35:50 +0530 Message-id: <1447740351-14246-9-git-send-email-pankaj.dubey@samsung.com> X-Mailer: git-send-email 2.4.5 In-reply-to: <1447740351-14246-1-git-send-email-pankaj.dubey@samsung.com> References: <1447740351-14246-1-git-send-email-pankaj.dubey@samsung.com> X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrDLMWRmVeSWpSXmKPExsWyRsSkWnfbUa8wg2PbBCw+rn7JZvF30jF2 i9cvDC16F1xls/h6eAWjxabH11gtLu+aw2bxufcIo8WM8/uYLE5d/8xmsWjrF3aLjmWMDjwe v39NYvTYOesuu8eda3vYPDYvqfe4cqKJ1aNvyypGj8+b5ALYo7hsUlJzMstSi/TtErgyvrQf Zi3YzVcx9eYLtgbG/9xdjJwcEgImErsmvmaBsMUkLtxbz9bFyMUhJLCCUWLz1ofMMEW7d7xl h0jMYpS4tWU6M4TTyiTx+kkvK0gVm4CuxJP3c8ESIgKtjBLL9v9nAnGYBQ4wSuxY94oRpEpY IE7i3rsHYAtZBFQl2rctAIvzCnhIPLv5DMjmANonJ7G0uRYkzCngKbFj6REmkLAQUMm7E+og IyUELrFLLF09iw1ijIDEt8mHWCBaZSU2HYC6WlLi4IobLBMYhRcwMqxiFE0tSC4oTkovMtIr TswtLs1L10vOz93ECIyU0/+e9e1gvHnA+hCjAAejEg9vw1/PMCHWxLLiytxDjKZAGyYyS4km 5wPjMa8k3tDYzMjC1MTU2Mjc0kxJnDdB6mewkEB6YklqdmpqQWpRfFFpTmrxIUYmDk6pBsbi +2Yuc4LY0ts/XXLXe7Fu+RObF2zZCQ3XPWe13LoesnfimkkS9lNiVp49HCcVH7Xv66wZHNox b2zSO+yWTAjpPsBi8LVz35UNxWsyX1oAHVa6gMGcscvM6ZmRhLK71MqZwRn/ZL7uZ2QSNszN +sh00elOYOozx4mPDn/vC/7+/31q4P5/75VYijMSDbWYi4oTAfkP9i+PAgAA X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrJIsWRmVeSWpSXmKPExsVy+t9jQd1tR73CDA5sZbP4uPolm8XfScfY LV6/MLToXXCVzeLr4RWMFpseX2O1uLxrDpvF594jjBYzzu9jsjh1/TObxaKtX9gtOpYxOvB4 /P41idFj56y77B53ru1h89i8pN7jyokmVo++LasYPT5vkgtgj2pgtMlITUxJLVJIzUvOT8nM S7dV8g6Od443NTMw1DW0tDBXUshLzE21VXLxCdB1y8wBulNJoSwxpxQoFJBYXKykb4dpQmiI m64FTGOErm9IEFyPkQEaSFjDmPGl/TBrwW6+iqk3X7A1MP7n7mLk5JAQMJHYveMtO4QtJnHh 3nq2LkYuDiGBWYwSt7ZMZ4ZwWpkkXj/pZQWpYhPQlXjyfi5YQkSglVFi2f7/TCAOs8ABRokd 614xglQJC8RJ3Hv3gAXEZhFQlWjftgAszivgIfHs5jMgmwNon5zE0uZakDCngKfEjqVHmEDC QkAl706oT2DkXcDIsIpRIrUguaA4KT3XMC+1XK84Mbe4NC9dLzk/dxMjOBqfSe1gPLjL/RCj AAejEg/vjiNeYUKsiWXFlbmHGCU4mJVEeJ8cAgrxpiRWVqUW5ccXleakFh9iNAU6ayKzlGhy PjBR5JXEGxqbmJsam1qaWJiYWSqJ8+p7GoUJCaQnlqRmp6YWpBbB9DFxcEo1MF4UZThd9uxj zO6lM5K+Xgpae8c+4Z3meau1cw10XdbxXD4SUFr37R77nfVn1+rPb/h2dnnxxgzJyG9/RLXV /rIpHDp8lq873/GC8Xm2yQ5nHPyPrcyJb3g4Ky4rwG+ByRxOK8spQi4/2woW67dtsT98Qod5 xcFT3z4H/D61180qiyVC0FIpYq4SS3FGoqEWc1FxIgBLWKHt3AIAAA== DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2029 Lines: 72 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); + 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) + ; +} + static int pmu_restart_notify(struct notifier_block *this, unsigned long code, void *unused) { -- 2.4.5 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/