public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH 2/2 v3] OMAP3: PM: SR: SmartReflex Refactor Rev4.0
@ 2009-10-26 15:13 Menon, Nishanth
  0 siblings, 0 replies; 16+ messages in thread
From: Menon, Nishanth @ 2009-10-26 15:13 UTC (permalink / raw)
  To: G, Manjunath Kondaiah, 'linux-omap@vger.kernel.org'
  Cc: 'guilhem.imberton@motorola.com',
	'mikechan@google.com', Nayak, Rajendra,
	'ext-roger.quadros@nokia.com',
	'ext-kalle.jokiniemi@nokia.com', Reddy, Teerth,
	'khilman@deeprootsystems.com', 'paul@pwsan.com',
	'jouni.hogander@nokia.com'

Trying not to top post.. Apologies before hand on my client restrictions. Anyways..

Manjunath,
Yes, the call sequences are common. We may consider using cpu_relax() in the context of dvfs calls.. Except it could result in race connditions not in intrest.

Do me a favor and flag the udelays u'd like to convert and send a patch for fix pls.

Regards,
Nishanth Menon

----- Original Message -----
From: G, Manjunath Kondaiah
To: Menon, Nishanth; linux-omap@vger.kernel.org <linux-omap@vger.kernel.org>
Cc: Imberton Guilhem <guilhem.imberton@motorola.com>; Mike Chan <mikechan@google.com>; Nayak, Rajendra; Roger Quadros <ext-roger.quadros@nokia.com>; Kalle Jokiniemi <ext-kalle.jokiniemi@nokia.com>; Reddy, Teerth; Kevin Hilman <khilman@deeprootsystems.com>; Paul Walmsley <paul@pwsan.com>; Hogander Jouni <jouni.hogander@nokia.com>
Sent: Mon Oct 26 10:09:07 2009
Subject: RE: [PATCH 2/2 v3] OMAP3: PM: SR: SmartReflex Refactor Rev4.0


> -----Original Message-----
> From: Menon, Nishanth 
> Sent: Monday, October 26, 2009 3:48 PM
> To: G, Manjunath Kondaiah; linux-omap@vger.kernel.org
> Cc: Imberton Guilhem; Mike Chan; Nayak, Rajendra; Roger 
> Quadros; Kalle Jokiniemi; Reddy, Teerth; Kevin Hilman; Paul 
> Walmsley; Hogander Jouni
> Subject: RE: [PATCH 2/2 v3] OMAP3: PM: SR: SmartReflex Refactor Rev4.0
> 
> > -----Original Message-----
> > From: G, Manjunath Kondaiah
> > Sent: Monday, October 26, 2009 3:40 AM
> > To: Menon, Nishanth; linux-omap@vger.kernel.org
> > Cc: Imberton Guilhem; Mike Chan; Nayak, Rajendra; Roger 
> Quadros; Kalle
> > Jokiniemi; Reddy, Teerth; Kevin Hilman; Paul Walmsley; 
> Hogander Jouni
> > Subject: RE: [PATCH 2/2 v3] OMAP3: PM: SR: SmartReflex 
> Refactor Rev4.0
> > 
> > 
> > As per your comments for other patches when ever there is 
> udelay usage,
> > cpu_relax is the better option. I see there are lot of 
> udelay(...) calls
> > used in this patch. Why can't you use cpu_relax() or schedule().
> > Any specific reason?
> > 
> Don’t really want to do cpu_relax in irq_locked context.. if 
> you look at the code flow, the call from cpu_idle is in 
> irq_locked.. Further this delay is part of bring up form 
> saved context where there is nothing else scheduled + we 
> don’t want anything else scheduled (and causing a change in 
> scheduling decision). So unfortunately, unlike standard 
> drivers, this cannot use the same reasoning.
> 

NAK. My understanding is that, SR functions will be called during voltage 
scaling also. Voltage scaling will happen in non IRQ locked context.

Please clarify if I am wrong.

-Manjunath

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2009-10-28  5:17 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <OMAP3:PM:SR: SmartReflex Refactor Rev3.0>
2009-10-24  5:15 ` [PATCH 0/2 v3] OMAP3: PM: refactor Smart Reflex Nishanth Menon
2009-10-24  5:15   ` [PATCH 1/2 v3] OMAP3: PM: SR: prepare: remove old SR code Nishanth Menon
2009-10-24  5:15     ` [PATCH 2/2 v3] OMAP3: PM: SR: SmartReflex Refactor Rev4.0 Nishanth Menon
2009-10-25 22:12       ` Cousson, Benoit
2009-10-25 22:59         ` Nishanth Menon
2009-10-26 13:05           ` Cousson, Benoit
2009-10-26 13:19             ` Menon, Nishanth
2009-10-26 17:06               ` Cousson, Benoit
     [not found]       ` <E0D41E29EB0DAC4E9F3FF173962E9E940253EFE01B@dbde02.ent.ti.com>
2009-10-26 10:18         ` Menon, Nishanth
2009-10-26 15:09           ` G, Manjunath Kondaiah
2009-10-27 23:36           ` Paul Walmsley
2009-10-28  5:17             ` Menon, Nishanth
     [not found]       ` <74583B8642AB8841B30447520659FCA9DDDFCC3A@dnce01.ent.ti.com>
2009-10-27 11:46         ` Imberton Guilhem-gimb01
2009-10-27 15:13       ` Kalle Jokiniemi
2009-10-28  5:05         ` Menon, Nishanth
2009-10-26 15:13 Menon, Nishanth

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox