From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko =?iso-8859-15?q?St=FCbner?= Subject: Re: [PATCH 12/12] ARM: restart: S3C24XX: use new restart hook Date: Mon, 2 Jan 2012 14:14:16 +0100 Message-ID: <201201021414.17551.heiko@sntech.de> References: <1324385316-6052-1-git-send-email-kgene.kim@samsung.com> <201112201418.07935.heiko@sntech.de> <4EF08F02.9010604@samsung.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset=iso-8859-15 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from h1778886.stratoserver.net ([85.214.133.74]:53042 "EHLO h1778886.stratoserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751386Ab2ABNO1 convert rfc822-to-8bit (ORCPT ); Mon, 2 Jan 2012 08:14:27 -0500 In-Reply-To: <4EF08F02.9010604@samsung.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Kukjin Kim Cc: linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, ben-linux@fluff.org Hi Kgene, Am Dienstag, 20. Dezember 2011, 14:34:58 schrieb Kukjin Kim: > On 12/20/11 22:18, Heiko St=FCbner wrote: > > Am Dienstag, 20. Dezember 2011, 13:48:36 schrieb Kukjin Kim: > >> Hook these platforms restart code into the new restart hook rather > >> than using arch_reset(). > >> And adds local header file, common.h in arch/arm/mach-s3c2410/ and > >> arch/arm/mach-s3c2440/ directories. > >>=20 > >> Cc: Ben Dooks > >> Cc: Russell King > >> Signed-off-by: Kukjin Kim > >=20 > > just for my understanding would it not also be possible to move the > > function prototype to plat-samsung/include/plat/s3c2410.h and s3c24= 4x.h? >=20 > Yeah, we have many choices about that and I will sort out the > mach-s3c24.. directories to mach-s3c24xx or just mach-s3c... >=20 > > Or should these files vanish in the long run? >=20 > Yes, as per Russell's suggestion, I think we don't need that and as a > note, I'm working on removing plat-s3c24xx and plat-s5p, just keeping > one plat-samsung. I'm currently working on introducing a power-domain for the usb-phy of = the=20 S3C2443/S3C2416/S3C2450. This is one step mimicing the work of Mark Bro= wn to=20 get better idle handling later on, and also again reduces the number of= =20 architecture-calls the hsudc driver does, as requested by Felipe Balbi. Therefore I'm introducing a new s3c2443-pm-common.c. Question now is, s= hould=20 this move directly to plat-samsung? Thanks Heiko From mboxrd@z Thu Jan 1 00:00:00 1970 From: heiko@sntech.de (Heiko =?iso-8859-15?q?St=FCbner?=) Date: Mon, 2 Jan 2012 14:14:16 +0100 Subject: [PATCH 12/12] ARM: restart: S3C24XX: use new restart hook In-Reply-To: <4EF08F02.9010604@samsung.com> References: <1324385316-6052-1-git-send-email-kgene.kim@samsung.com> <201112201418.07935.heiko@sntech.de> <4EF08F02.9010604@samsung.com> Message-ID: <201201021414.17551.heiko@sntech.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Kgene, Am Dienstag, 20. Dezember 2011, 14:34:58 schrieb Kukjin Kim: > On 12/20/11 22:18, Heiko St?bner wrote: > > Am Dienstag, 20. Dezember 2011, 13:48:36 schrieb Kukjin Kim: > >> Hook these platforms restart code into the new restart hook rather > >> than using arch_reset(). > >> And adds local header file, common.h in arch/arm/mach-s3c2410/ and > >> arch/arm/mach-s3c2440/ directories. > >> > >> Cc: Ben Dooks > >> Cc: Russell King > >> Signed-off-by: Kukjin Kim > > > > just for my understanding would it not also be possible to move the > > function prototype to plat-samsung/include/plat/s3c2410.h and s3c244x.h? > > Yeah, we have many choices about that and I will sort out the > mach-s3c24.. directories to mach-s3c24xx or just mach-s3c... > > > Or should these files vanish in the long run? > > Yes, as per Russell's suggestion, I think we don't need that and as a > note, I'm working on removing plat-s3c24xx and plat-s5p, just keeping > one plat-samsung. I'm currently working on introducing a power-domain for the usb-phy of the S3C2443/S3C2416/S3C2450. This is one step mimicing the work of Mark Brown to get better idle handling later on, and also again reduces the number of architecture-calls the hsudc driver does, as requested by Felipe Balbi. Therefore I'm introducing a new s3c2443-pm-common.c. Question now is, should this move directly to plat-samsung? Thanks Heiko