[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