From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: RE: [PATCHv3] ARM: EXYNOS4: Support for generic I/O power domains on EXYNOS4210 Date: Sat, 16 Jul 2011 12:19:19 +0900 Message-ID: <035201cc4367$2b165c80$81431580$%kim@samsung.com> References: <4E165C64.8090808@samsung.com> <000e01cc3d2f$eca5d530$c5f17f90$%kim@samsung.com> <4E16A171.7040509@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7BIT Return-path: Received: from mailout3.samsung.com ([203.254.224.33]:12038 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751420Ab1GPDT2 (ORCPT ); Fri, 15 Jul 2011 23:19:28 -0400 In-reply-to: <4E16A171.7040509@samsung.com> Content-language: ko Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: 'Chanwoo Choi' Cc: "'Rafael J. Wysocki'" , 'Kyungmin Park' , 'MyungJoo Ham' , 'Sylwester Nawrocki' , linux-pm@lists.linux-foundation.org, 'linux-samsung-soc' , 'linux-kernel' Chanwoo Choi wrote: > > Kukjin Kim wrote: (snip) > >> @@ -183,6 +183,7 @@ config MACH_NURI > >> select EXYNOS4_SETUP_SDHCI > >> select EXYNOS4_SETUP_USB_PHY > >> select SAMSUNG_DEV_PWM > >> + select PM_GENERIC_DOMAINS if PM > > > > Do you _really_ think this should be under MACH_NURI? > > > This patch apply the generic power-domain to NURI board. I mean, the PM_GENERIC_DOMAINS feature depends on CPU not board even though this is for only NURI board. (snip) > >> diff --git a/arch/arm/mach-exynos4/include/mach/pm-exynos4210.h > >> b/arch/arm/mach-exynos4/include/mach/pm-exynos4210.h > >> new file mode 100644 > >> index 0000000..ab09034 > >> --- /dev/null > >> +++ b/arch/arm/mach-exynos4/include/mach/pm-exynos4210.h > >> @@ -0,0 +1,53 @@ > >> +/* linux/arch/arm/mach-exynos4/include/mach/pm-exynos4.h > > > > According to your file name, should be linux/arch/arm/mach- > exynos4/include/mach/pm-exynos4210.h ? > I said just typo... +/* linux/arch/arm/mach-exynos4/include/mach/pm-exynos4210.h > This patch support only power-domain of EXYNOS4210, so I named "pm- > exynos4210.h" > If EXYNOS4 series board will be released, I will modify include file name > from "pm-exynos4210.h" to "pm-exynos4.h" to support all EXYNOS4 series. No need it, it's different. (snip) > > Would be better if you could add some common function for pd_power_up and > _down... > > What do you mean about common function? > I thought, pd_power_up and _down can be implemented in one function. (snip) > > I wonder why we need to add this for handling power domain not update current > pd. > See arch/arm/mach-exynos4/dev-pd.c I mean, we _really_ need to keep duplication files for handling power domain?... (snip) Thanks. Best regards, Kgene. -- Kukjin Kim , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.