On Fri, Feb 03, 2012 at 01:34:55PM +0530, Shilimkar, Santosh wrote: > On Fri, Feb 3, 2012 at 12:58 PM, Felipe Balbi <balbi@xxxxxx> wrote: > > The way the code is written, pwrdm_pre_transion() > > calls _pwrdm_pre_transition_cb() (for each powerdomain) > > which, in turn, calls pwrdm_clear_all_prev_pwrst(). > > > > It looks unnecessary to clear all previous power > > states twice, so drop the initial set. > > > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> > > --- > > > > Hasn't been tested yet. Compile tested only. It looks > > like we can safely remove those calls, but I would > > like to hear from the PM folks first :-) > > > Bit late :) > > Paul already cleaned this up. > http://www.spinics.net/lists/linux-omap/msg63602.html that's too bad. Missed an oportunity to have a commit on pm code, heh. Anyway, thanks -- balbi
Attachment:
signature.asc
Description: Digital signature