[PATCH 3/3] S3C2443: add power domain for usb phy

[Felipe Balbi]

On Tue, Jan 24, 2012 at 02:27:21PM +0100, Heiko Stübner wrote:
> Am Dienstag, 24. Januar 2012, 14:07:56 schrieb Felipe Balbi:
> > Hi,
> > 
> > On Tue, Jan 24, 2012 at 10:01:33PM +0900, Kukjin Kim wrote:
> > > Heiko Stübner wrote:
> > > > Am Dienstag, 24. Januar 2012, 11:15:11 schrieb Felipe Balbi:
> > > > > On Tue, Jan 24, 2012 at 11:13:00AM +0100, Heiko Stübner wrote:
> > > > > > sorry for mail spam :-)
> > > > > > 
> > > > > > Am Dienstag, 24. Januar 2012, 10:51:31 schrieb Felipe Balbi:
> > > > > > > On Tue, Jan 24, 2012 at 10:50:14AM +0100, Heiko Stübner wrote:
> > > > > > > > Hi Felipe,
> > > > > > > > 
> > > > > > > > Am Dienstag, 24. Januar 2012, 10:38:32 schrieb Felipe Balbi:
> > > > > > > > > Hi,
> > > > > > > > > 
> > > > > > > > > On Sun, Jan 08, 2012 at 09:59:02PM +0100, Heiko Stübner wrote:
> > > > > > > > > > The phy-power-handling is common to S3C2443/2416/2450, so
> > > > > > > > > > introduce a s3c2443-pm-common.c to handle this for all of
> > > > > > > > > > them.
> > > > > > > > > > 
> > > > > > > > > > This makes it possible to remove the raw write to the
> > > > > > > > > > PWRCFG-register from the s3c-hsudc driver.
> > > > > > > > > > 
> > > > > > > > > > Signed-off-by: Heiko Stuebner <heiko at sntech.de>
> > > > > > > > > 
> > > > > > > > > I believe this doesn't really depend on previous patches (??)
> > > > > > > > > In that
> > > > > > > > 
> > > > > > > > > case, it can be carried with s3c's tree:
> > > > > > > > it does depend on the previous patch [2/3], as the device power
> > > > > > > > domain is controlled through the runtime-pm subsystem whose
> > > > > > > > calls were added in it.
> > > > > > > 
> > > > > > > then I will need proper acks to carry this patch.
> > > > > > 
> > > > > > another possibility would be to have patch 2 and 3 get in through
> > > > > > the samsung tree, as the majority of the change lies on this side
> > > > > > (i.e. introducing a new file in plat-s3c24xx)
> > > > > 
> > > > > yet another one, would be to defer patch 3/3 by one merge window ;-)
> > > > 
> > > > also fine by me ;-)
> > > > 
> > > > But let's see first what Kgene says.
> > > 
> > > Hi all,
> > > 
> > > Sorry for join lately on this.
> > > 
> > > I think we need to implement more pm domain for s3c24xx as well
> > > usbphy. So if Felipe is ok, I'd like to handle Heiko's 2nd and 3rd
> > > patches in my tree. Of course, I need Felipe's ack before that.
> > > BTW, frankly, I'm not sure we can finish supporting pm domain for all
> > > of s3c24xx for v3.4 :)
> > 
> > In that case, how about we take things slow ? I can carry pm-runtime
> > patch (2/3) and by 3.5 or 3.6 you will only need to touch
> > arch/arm/-related code.
> > 
> > If you guarantee that it won't make it in 3.4, I'm ok carrying
> > pm-runtime patch as it won't break anything anyway (I suppose).
> 
> Seems reasonable to me. The pm-runtime patch does not break anything - it 
> simply does nothing without either some runtime-pm functions or the power 
> domain work.
> 
> So I would second that we handle the power domain stuff after Kgene got the 
> mach directories merged for 3.4, i.e. drop the patch (3/3) for now and I will 
> resubmit it when the time comes.

sounds great to me :-) Thanks guys

-- 
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120124/904bea41/attachment-0001.sig>