Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-02 Thread Alan Stern
On Thu, 2 Aug 2012, Magnus Damm wrote: > From my point of view this all depends on what the hardware supports. > If the USB host controller hardware can be powered down (power domain > off, clock off) when no USB hardware is connected, but still perform > hotplug detection of a newly added device,

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-02 Thread Alan Stern
On Wed, 1 Aug 2012 kuninori.morimoto...@renesas.com wrote: > It is possible to power-on on platform setup level > if Rafael and Magnus don't care this style. > Then, I don't need this patch on ehci/ohci driver. > > 1) power-on on platform setup timing > 2) use ehci/ohci driver >

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-01 Thread Kuninori Morimoto
Hi Alan, Magnus Thank you for your reply > > -- probe -- > >if (pdata->power) > > pdata->power(xxx, 1) > > > >usb_add_hcd() > > > >-- remove -- > >usb_remove_hcd() > > > >if (pdata->power) > > pdata->power(xxx, 0) > > From my point of view this all depend

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-01 Thread Magnus Damm
Hi Morimoto-san, On Thu, Aug 2, 2012 at 9:09 AM, wrote: > > Hi Alan, Rafael, Magnus > > Thank you for your reply > >> The problem is that the ehci-platform driver currently doesn't know >> anything about clocks or power sources. We need to add that >> information in a generic fashion, somehow.

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-01 Thread kuninori . morimoto . gx
Hi Alan, Rafael, Magnus Thank you for your reply > The problem is that the ehci-platform driver currently doesn't know > anything about clocks or power sources. We need to add that > information in a generic fashion, somehow. Until that's done, you > can't really use ehci-platform as your driv

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-08-01 Thread Alan Stern
On Tue, 31 Jul 2012, Kuninori Morimoto wrote: > Hi Alan > Cc Magnus, Rafael > > > > > this patch enable to use pm_runtime_xxx() on ehci-platform > > > > if .config has CONFIG_PM_RUNTIME, otherwise, these are ignored > > > > > > > > Signed-off-by: Kuninori Morimoto > > > > --- > > > > drivers/u

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-07-31 Thread Kuninori Morimoto
Hi Alan Cc Magnus, Rafael > > > this patch enable to use pm_runtime_xxx() on ehci-platform > > > if .config has CONFIG_PM_RUNTIME, otherwise, these are ignored > > > > > > Signed-off-by: Kuninori Morimoto > > > --- > > > drivers/usb/host/ehci-platform.c |6 ++ > > > 1 files changed, 6

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-07-31 Thread Kuninori Morimoto
Hi Alan Cc Magnus, Rafael > > this patch enable to use pm_runtime_xxx() on ehci-platform > > if .config has CONFIG_PM_RUNTIME, otherwise, these are ignored > > > > Signed-off-by: Kuninori Morimoto > > --- > > drivers/usb/host/ehci-platform.c |6 ++ > > 1 files changed, 6 insertions(+),

Re: [PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-07-31 Thread Alan Stern
On Mon, 30 Jul 2012, Kuninori Morimoto wrote: > this patch enable to use pm_runtime_xxx() on ehci-platform > if .config has CONFIG_PM_RUNTIME, otherwise, these are ignored > > Signed-off-by: Kuninori Morimoto > --- > drivers/usb/host/ehci-platform.c |6 ++ > 1 files changed, 6 insertion

[PATCH] usb: host: ehci-platform: add pm_runtime_xx()

2012-07-30 Thread Kuninori Morimoto
this patch enable to use pm_runtime_xxx() on ehci-platform if .config has CONFIG_PM_RUNTIME, otherwise, these are ignored Signed-off-by: Kuninori Morimoto --- drivers/usb/host/ehci-platform.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/usb/host/ehci-plat