[PATCH 3/3] ARM: OMAP3+: PM: VP: check only the VPINIDLE status bit

Nishanth Menon nm at ti.com
Thu May 31 19:39:20 EDT 2012


On 16:27-20120531, Kevin Hilman wrote:
> Nishanth Menon <nm at ti.com> writes:
> 
> > Currently we check against the entire 32 bits of the status register
> > Where, bits 1-31 are marked as reserved and mentioned in TRM as read
> > returns undefined values. 
> 
> ouch
> 
> > Hence, check against purely the vpinidle status bit.
> >
> > Cc: Tony Lindgren <tony at atomide.com>
> > Cc: Kevin Hilman <khilman at ti.com>
> > Cc: linux-omap at vger.kernel.org
> > Cc: linux-arm-kernel at lists.infradead.org
> >
> > Reported-by: Vinay Amancha <vinaykumar at ti.com>
> > Signed-off-by: Nishanth Menon <nm at ti.com>
> 
> Thanks for the fix.
> 
> Looking at this combined with the previous patches, since this same
> thing is done several places, it looks like we need a helper function
> 'omap_vp_wait_for_idle()' or similar.  That function will do the check,
> wait for timeout, and return an error if failure.  The callers can than
> do appropriate error handling upon failure.
> 
yep - i think a static inline should do just fine here considering just
a test_timeout is involved.
will post out a new series in a few mins.
-- 
Regards,
Nishanth Menon



More information about the linux-arm-kernel mailing list