On Mon, Sep 27, 2010 at 04:48:04AM -0500, Sergei Shtylyov wrote:
Hello.
On 27-09-2010 10:07, Felipe Balbi wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
the runtime hooks be noops on the other platfo
Hello.
On 27-09-2010 10:07, Felipe Balbi wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
the runtime hooks be noops on the other platforms.
Agreed. We should focus on removing ->set_clock for .38 ac
On Fri, Sep 24, 2010 at 09:49:34AM -0500, Sergei Shtylyov wrote:
Hello.
Felipe Balbi wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
the runtime hooks be noops on the other platforms.
Agreed. We sh
Felipe Balbi writes:
> Hi,
>
> On Thu, Sep 23, 2010 at 12:33:58PM -0500, Kevin Hilman wrote:
>>I guess that's Felipe's call, but I don't like that option.
>>
>>I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
>>the runtime hooks be noops on the other platforms.
>
> Agreed.
Hello.
Felipe Balbi wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
the runtime hooks be noops on the other platforms.
Agreed. We should focus on removing ->set_clock for .38 actually. Is
DaVinci al
;Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Hi,
>
>On Fri, Sep 24, 2010 at 06:00:26AM -0500, Kalliguddi, Hema wrote:
>>then for now I can think of using set_clok platform function to call
>>the runtime pm apis, Is there any alternative you a
Hi,
On Fri, Sep 24, 2010 at 06:00:26AM -0500, Kalliguddi, Hema wrote:
then for now I can think of using set_clok platform function to call
the runtime pm apis, Is there any alternative you are thinking of?
how about not pushing a patch which isn't the best way to solve the
problem in the first
Hi,
On Thu, Sep 23, 2010 at 12:33:58PM -0500, Kevin Hilman wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP and
the runtime hooks be noops on the other platforms.
Agreed. We should focus on removing ->set_cl
Hi,
On Fri, Sep 24, 2010 at 12:34:09AM -0500, Kalliguddi, Hema wrote:
Here is the patch which is making use of already existing platform
set_clock functions pointer. With this we don't need to use #ifdefs.
If it looks good I will post it again along with series.
those weren't even used anymor
;Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Hi,
>
>On Fri, Sep 24, 2010 at 04:28:35AM -0500, Kalliguddi, Hema wrote:
>>I just noticed one more think in the code. there are suspend/resume
>>function pointers in platform_driver structure.
Hi,
On Fri, Sep 24, 2010 at 04:28:35AM -0500, Kalliguddi, Hema wrote:
I just noticed one more think in the code. there are suspend/resume
function pointers in platform_driver structure. Why can't we use these
for platform specific operations like enable/disable clocks, context
save/restore?
el.org; Basak, Partha; Tony Lindgren;
>Cousson, Benoit; Paul Walmsley
>Subject: RE: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Felipe,
>
>
>>-Original Message-
>>From: Balbi, Felipe
>>Sent: Friday, September 24, 2010 2:14 PM
>>To:
Walmsley
>Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Hi,
>
>On Fri, Sep 24, 2010 at 03:16:29AM -0500, Kalliguddi, Hema wrote:
>>>I guess that's Felipe's call, but I don't like that option.
>>>
>>>I think it's
Hi,
On Fri, Sep 24, 2010 at 03:16:29AM -0500, Kalliguddi, Hema wrote:
I guess that's Felipe's call, but I don't like that option.
I think it's cleaner to have the ->set_clock hook be a noop on OMAP
and the runtime hooks be noops on the other platforms.
I think the set_clock function was used
Benoit; Paul Walmsley
>Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>"Kalliguddi, Hema" writes:
>
>>
>>
>>>-Original Message-
>>>From: Kevin Hilman [mailto:khil...@deeprootsystems.com]
>>>Sent: Thursd
g;
>Basak, Partha; Tony Lindgren; Cousson, Benoit; Paul Walmsley
>Subject: RE: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>
>
>>-Original Message-
>>From: Kevin Hilman [mailto:khil...@deeprootsystems.com]
>>Sent: Thursday, September 23
ux-...@vger.kernel.org; Basak, Partha; Tony Lindgren;
>>Cousson, Benoit; Paul Walmsley
>>Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>>
>>Felipe Balbi writes:
>>
>>> Hi,
>>>
>>> On Wed, Sep 22, 2010 at 07:30:30P
Benoit; Paul Walmsley
>Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Felipe Balbi writes:
>
>> Hi,
>>
>> On Wed, Sep 22, 2010 at 07:30:30PM -0500, Kalliguddi, Hema wrote:
>>>Calling runtime pm APIs pm_runtime_put_sync() and
>pm_ru
Felipe Balbi writes:
> Hi,
>
> On Wed, Sep 22, 2010 at 07:30:30PM -0500, Kalliguddi, Hema wrote:
>>Calling runtime pm APIs pm_runtime_put_sync() and pm_runtime_get_sync()
>>for enabling/disabling the clocks,sysconfig settings.
>>
>>Also need to put the USB in force standby and force idle mode whe
Hi,
On Thu, Sep 23, 2010 at 06:11:22AM -0500, Kalliguddi, Hema wrote:
When there is global suspend initiated, driver's pm_ops functions
Musb_suspend api is called and the clocks are disabled.
Now when the system resumes after wakeup,
Musb_resume_noirq is called, now need to re-enable the clocks
gt;Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Hi,
>
>On Thu, Sep 23, 2010 at 02:51:12AM -0500, Kalliguddi, Hema wrote:
>>>these two should a separate series, otherwise it's difficult for
>>>different maintainers to decide what
Hi,
On Thu, Sep 23, 2010 at 02:51:12AM -0500, Kalliguddi, Hema wrote:
these two should a separate series, otherwise it's difficult for
different maintainers to decide what they need to pick up :-)
I don't mind
if you're re-sending already, please do split :-)
I would rather remove these, a
Partha; Tony Lindgren; Kevin Hilman; Cousson, Benoit;
>Paul Walmsley
>Subject: RE: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>
>
>>-Original Message-
>>From: Balbi, Felipe
>>Sent: Thursday, September 23, 2010 12:06 PM
>>
;Subject: Re: [PATCH 8/9 v3] usb : musb: Using runtime pm apis for musb.
>
>Hi,
>
>On Wed, Sep 22, 2010 at 07:30:30PM -0500, Kalliguddi, Hema wrote:
>>Calling runtime pm APIs pm_runtime_put_sync() and
>pm_runtime_get_sync()
>>for enabling/disabling the clocks,sysconf
Hi,
On Wed, Sep 22, 2010 at 07:30:30PM -0500, Kalliguddi, Hema wrote:
Calling runtime pm APIs pm_runtime_put_sync() and pm_runtime_get_sync()
for enabling/disabling the clocks,sysconfig settings.
Also need to put the USB in force standby and force idle mode when usb not used
and set it back to
Calling runtime pm APIs pm_runtime_put_sync() and pm_runtime_get_sync()
for enabling/disabling the clocks,sysconfig settings.
Also need to put the USB in force standby and force idle mode when usb not used
and set it back to no idle and no stndby after wakeup.
For OMAP3 auto idle bit has to be dis
26 matches
Mail list logo