[PATCH V2] gpu: host1x: handle the correct # of syncpt regs
Stephen Warren
swarren at wwwdotorg.org
Mon Apr 14 13:53:51 PDT 2014
On 04/04/2014 04:31 PM, Stephen Warren wrote:
> From: Stephen Warren <swarren at nvidia.com>
>
> BIT_WORD() truncates rather than rounds, so the loops in
> syncpt_thresh_isr() and _host1x_intr_disable_all_syncpt_intrs() use <=
> rather than < in an attempt to process the correct number of registers
> when rounding of the conversion of count of bits to count of words is
> necessary. However, when rounding isn't necessary because the value is
> already a multiple of the divisor (as is the case for all values of
> nb_pts the code actually sees), this causes one too many registers to
> be processed.
>
> Solve this by using and explicit DIV_ROUND_UP() call, rather than
> BIT_WORD(), and comparing with < rather than <=.
I don't see this in linux-next yet.
More information about the dri-devel
mailing list