On Tue, 2013-05-21 at 19:20 +0200, Daniel Vetter wrote: > We have another one of these in the wait_for register wait macro in > intel_drv.h Can you please amend your patch with that fixed up, too? I noticed it, but didn't change it since we don't need there the +1 adjustment to begin with. The time_after() check already makes sure we wait at least MS amount. So I think we should remove +1 from there.. --Imre