All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: Kukjin Kim <kgene.kim@samsung.com>
Cc: linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, ben-linux@fluff.org
Subject: Re: [PATCH 12/12] ARM: restart: S3C24XX: use new restart hook
Date: Mon, 2 Jan 2012 14:14:16 +0100	[thread overview]
Message-ID: <201201021414.17551.heiko@sntech.de> (raw)
In-Reply-To: <4EF08F02.9010604@samsung.com>

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<ben-linux@fluff.org>
> >> Cc: Russell King<rmk+kernel@arm.linux.org.uk>
> >> Signed-off-by: Kukjin Kim<kgene.kim@samsung.com>
> > 
> > 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

WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stübner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 12/12] ARM: restart: S3C24XX: use new restart hook
Date: Mon, 2 Jan 2012 14:14:16 +0100	[thread overview]
Message-ID: <201201021414.17551.heiko@sntech.de> (raw)
In-Reply-To: <4EF08F02.9010604@samsung.com>

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<ben-linux@fluff.org>
> >> Cc: Russell King<rmk+kernel@arm.linux.org.uk>
> >> Signed-off-by: Kukjin Kim<kgene.kim@samsung.com>
> > 
> > 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

  reply	other threads:[~2012-01-02 13:14 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-20 12:48 [PATCH 00/12] ARM: restart: SAMSUNG: use new restart hook Kukjin Kim
2011-12-20 12:48 ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 01/12] ARM: S3C64XX: introduce arch/arm/mach-s3c64xx/common.[ch] Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 02/12] ARM: restart: S3C64XX: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-21  1:45   ` Kukjin Kim
2011-12-21  1:45     ` Kukjin Kim
2012-01-03 11:51   ` Russell King - ARM Linux
2012-01-03 11:51     ` Russell King - ARM Linux
2011-12-20 12:48 ` [PATCH 03/12] ARM: S5P64X0: introduce arch/arm/mach-s5p64x0/common.[ch] Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 04/12] ARM: restart: S5P64X0: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-21  1:38   ` Kukjin Kim
2011-12-21  1:38     ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 05/12] ARM: S5PC100: introduce arch/arm/mach-s5pc100/common.[ch] Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 06/12] ARM: restart: S5PC100: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-21  1:39   ` Kukjin Kim
2011-12-21  1:39     ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 07/12] ARM: S5PV210: introduce arch/arm/mach-s5pv210/common.[ch] Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-23 19:14   ` Russell King - ARM Linux
2011-12-23 19:14     ` Russell King - ARM Linux
2011-12-20 12:48 ` [PATCH 08/12] ARM: restart: S5PV210: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-21  1:40   ` Kukjin Kim
2011-12-21  1:40     ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 09/12] ARM: EXYNOS: introduce arch/arm/mach-exynos/common.[ch] Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-23 19:19   ` Russell King - ARM Linux
2011-12-23 19:19     ` Russell King - ARM Linux
2011-12-24  1:10     ` Kukjin Kim
2011-12-24  1:10       ` Kukjin Kim
2011-12-27  7:31       ` Kukjin Kim
2011-12-27  7:31         ` Kukjin Kim
2012-01-03 10:41         ` Russell King - ARM Linux
2012-01-03 10:41           ` Russell King - ARM Linux
2012-01-03 10:49           ` Kukjin Kim
2012-01-03 10:49             ` Kukjin Kim
2012-01-03 10:54             ` Russell King - ARM Linux
2012-01-03 10:54               ` Russell King - ARM Linux
2012-01-03 11:01               ` Kukjin Kim
2012-01-03 11:01                 ` Kukjin Kim
2012-01-03 11:20                 ` Russell King - ARM Linux
2012-01-03 11:20                   ` Russell King - ARM Linux
2012-01-03 11:54                   ` Kukjin Kim
2012-01-03 11:54                     ` Kukjin Kim
2012-01-03 12:00                     ` Russell King - ARM Linux
2012-01-03 12:00                       ` Russell King - ARM Linux
2012-01-03 12:07                       ` Kukjin Kim
2012-01-03 12:07                         ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 10/12] ARM: restart: EXYNOS: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-21  1:42   ` Kukjin Kim
2011-12-21  1:42     ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 11/12] ARM: restart: S3C24XX: move SWRST based S3C platforms to machine_desc hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-20 12:48 ` [PATCH 12/12] ARM: restart: S3C24XX: use new restart hook Kukjin Kim
2011-12-20 12:48   ` Kukjin Kim
2011-12-20 13:18   ` Heiko Stübner
2011-12-20 13:18     ` Heiko Stübner
2011-12-20 13:34     ` Kukjin Kim
2011-12-20 13:34       ` Kukjin Kim
2012-01-02 13:14       ` Heiko Stübner [this message]
2012-01-02 13:14         ` Heiko Stübner
2012-01-02 13:26         ` Mark Brown
2012-01-02 13:26           ` Mark Brown
2012-01-02 13:33           ` Heiko Stübner
2012-01-02 13:33             ` Heiko Stübner
2012-01-02 13:39             ` Mark Brown
2012-01-02 13:39               ` Mark Brown
2012-01-03  6:08               ` Kukjin Kim
2012-01-03  6:08                 ` Kukjin Kim
2011-12-21  1:44   ` Kukjin Kim
2011-12-21  1:44     ` Kukjin Kim
2011-12-20 21:00 ` [PATCH 00/12] ARM: restart: SAMSUNG: " Russell King - ARM Linux
2011-12-20 21:00   ` Russell King - ARM Linux
2011-12-21  1:37   ` Kukjin Kim
2011-12-21  1:37     ` Kukjin Kim

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201201021414.17551.heiko@sntech.de \
    --to=heiko@sntech.de \
    --cc=ben-linux@fluff.org \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.