From mboxrd@z Thu Jan 1 00:00:00 1970 From: Catalin Marinas Subject: Re: [RESUBMIT RFC PATCH v2 3/3] drivers: mfd: Add support for Exynos PMU driver Date: Mon, 28 Apr 2014 18:37:43 +0100 Message-ID: <20140428173743.GA14853@arm.com> References: <1398687607-4554-1-git-send-email-pankaj.dubey@samsung.com> <1398687607-4554-4-git-send-email-pankaj.dubey@samsung.com> <20140428122646.GF21279@lee--X1> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20140428122646.GF21279@lee--X1> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Lee Jones Cc: kgene.kim@samsung.com, linux@arm.linux.org.uk, Samuel Ortiz , b.zolnierkie@samsung.com, Sangbeom Kim , Pankaj Dubey , t.figa@samsung.com, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, yg1004.jang@samsung.com, vikas.sajjan@samsung.com, chow.kim@samsung.com, linux-arm-kernel@lists.infradead.org List-Id: linux-samsung-soc@vger.kernel.org On Mon, Apr 28, 2014 at 01:26:46PM +0100, Lee Jones wrote: > > This patch moves Exynos PMU driver implementation from > > "arm/mach-exynos" to "drivers/mfd". > > This driver is mainly used for setting misc bits of register from PMU IP > > of Exynos SoC which will be required to configure before Suspend/Resume. > > Currently all these settings are done in "arch/arm/mach-exynos/pmu.c" but > > moving ahead for ARM64 based SoC support, there is a need of DT based > > implementation of PMU driver. > > This driver uses already existing DT binding information. > > > > CC: Sangbeom Kim > > CC: Samuel Ortiz > > CC: Lee Jones > > Signed-off-by: Pankaj Dubey > > --- > > arch/arm/mach-exynos/Kconfig | 2 ++ > > arch/arm/mach-exynos/Makefile | 2 -- > > drivers/mfd/Kconfig | 9 +++++++++ > > drivers/mfd/Makefile | 1 + > > arch/arm/mach-exynos/pmu.c => drivers/mfd/exynos-pmu.c | 0 > > 5 files changed, 12 insertions(+), 2 deletions(-) > > rename arch/arm/mach-exynos/pmu.c => drivers/mfd/exynos-pmu.c (100%) > > So I just took a look at the code as zero changes looks suspicious to > me. The driver can not simply be copied and pasted into the MFD > subsystem in its current state. > > The fundamental question is; is this chip actually an MFD? What does > it do besides Power Management? I looked at the code briefly as well and I don't think it matches the mfd idea. Maybe it could be merged together with arch/arm/mach-exynos/pm.c and moved to drivers/power/ or a more appropriate directory for platform_suspend_ops. -- Catalin