Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753037AbbKLEXJ (ORCPT ); Wed, 11 Nov 2015 23:23:09 -0500 Received: from mailout1.w1.samsung.com ([210.118.77.11]:19378 "EHLO mailout1.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752668AbbKLEXG (ORCPT ); Wed, 11 Nov 2015 23:23:06 -0500 X-AuditID: cbfec7f4-f79c56d0000012ee-cc-56441423778e Subject: Re: [PATCH v4 4/9] ARM: EXYNOS: split up exynos3250 SoC specific PMU data To: Pankaj Dubey , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <1447155784-12667-1-git-send-email-pankaj.dubey@samsung.com> <1447155784-12667-5-git-send-email-pankaj.dubey@samsung.com> Cc: kgene.kim@samsung.com, thomas.ab@samsung.com, amitdanielk@gmail.com, olof@lixom.net, khilman@linaro.org, arnd@arndb.de From: Krzysztof Kozlowski X-Enigmail-Draft-Status: N1110 Message-id: <56441423.9000208@samsung.com> Date: Thu, 12 Nov 2015 13:22:59 +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: <1447155784-12667-5-git-send-email-pankaj.dubey@samsung.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFmpgkeLIzCtJLcpLzFFi42I5/e/4VV1lEZcwg8P7rSw+rn7JZvF30jF2 i9cvDC16F1xls/h6eAWjxabH11gtLu+aw2Yx4/w+JotT1z+zWSza+oXdomMZowO3x+9fkxg9 ds66y+5x59oeNo/NS+o9rpxoYvXo27KK0ePzJrkA9igum5TUnMyy1CJ9uwSujMM3brIWnBKs WH3Xp4FxB18XIweHhICJxN/ppl2MnECmmMSFe+vZQGwhgaWMErP3mncxcgHZXxglLn2/yw6S EBYIkTh87zcLSEJEYAqjxMWeZYwQVe2MEp0TfzOCVDEL1Eic/nQFzGYTMJbYvHwJG8QKOYne 7kksIDavgJbEhb1PwWpYBFQlzvafZQWxRQUiJCZOaGCFqBGU+DH5Hlg9p4CnROuW70wgVzML 6Encv6gFsUpeYvOat8wTGAVnIemYhVA1C0nVAkbmVYyiqaXJBcVJ6bmGesWJucWleel6yfm5 mxgh8fJlB+PiY1aHGAU4GJV4eCfMdA4TYk0sK67MPcQowcGsJMIbMA8oxJuSWFmVWpQfX1Sa k1p8iFGag0VJnHfurvchQgLpiSWp2ampBalFMFkmDk6pBkYR7z0yq/cbSh/aO0nup/4EIRf2 T+4rHs+NcdYPEzpR33L1W1hRd/ntZtHnGxZrGovYb+1vPJSeWeNS/e/ml2Wvtyn17WCJCImd 231Q8mNmjvunHVEHLrZOT0qR8/IWOsGgbtsWqrvqLZ+Mzb9SYz7DAMYy9mmsvLeF352ROtMT 6Rq2ZeqLF0osxRmJhlrMRcWJAGpg4umTAgAA Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2431 Lines: 69 On 10.11.2015 20:42, Pankaj Dubey wrote: > This patch splits up mach-exynos/pmu.c file, and moves exynos3250 PMU > configuration data and functions handing those data into exynos3250 > SoC specific PMU file mach-exynos/exynos3250-pmu.c. > > Signed-off-by: Pankaj Dubey > --- > arch/arm/mach-exynos/Makefile | 2 +- > arch/arm/mach-exynos/exynos-pmu.h | 40 ++++++++ > arch/arm/mach-exynos/exynos.c | 2 - > arch/arm/mach-exynos/exynos3250-pmu.c | 175 ++++++++++++++++++++++++++++++++ > arch/arm/mach-exynos/pmu.c | 184 +--------------------------------- > 5 files changed, 220 insertions(+), 183 deletions(-) > create mode 100644 arch/arm/mach-exynos/exynos-pmu.h > create mode 100644 arch/arm/mach-exynos/exynos3250-pmu.c > > diff --git a/arch/arm/mach-exynos/Makefile b/arch/arm/mach-exynos/Makefile > index 2f30676..e869f86 100644 > --- a/arch/arm/mach-exynos/Makefile > +++ b/arch/arm/mach-exynos/Makefile > @@ -9,7 +9,7 @@ ccflags-$(CONFIG_ARCH_MULTIPLATFORM) += -I$(srctree)/$(src)/include -I$(srctree) > > # Core > > -obj-$(CONFIG_ARCH_EXYNOS) += exynos.o pmu.o exynos-smc.o firmware.o > +obj-$(CONFIG_ARCH_EXYNOS) += exynos.o pmu.o exynos-smc.o firmware.o exynos3250-pmu.o > > obj-$(CONFIG_EXYNOS_CPU_SUSPEND) += pm.o sleep.o > obj-$(CONFIG_PM_SLEEP) += suspend.o > diff --git a/arch/arm/mach-exynos/exynos-pmu.h b/arch/arm/mach-exynos/exynos-pmu.h > new file mode 100644 > index 0000000..6f95c7d > --- /dev/null > +++ b/arch/arm/mach-exynos/exynos-pmu.h > @@ -0,0 +1,40 @@ > +/* > + * Copyright (c) 2015 Samsung Electronics Co., Ltd. > + * http://www.samsung.com > + * > + * Header for EXYNOS PMU Driver support > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + */ > + > +#ifndef __EXYNOS_PMU_H > +#define __EXYNOS_PMU_H > + > +#include > + > +#define PMU_TABLE_END (-1U) > + > + Unnecessary blank line. Rest looks good, so with this fix: Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof -- 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/