]> git.ipfire.org Git - thirdparty/linux.git/commit
drm/i915/dsb: Move the +1 usec adjustment into dsb_wait_usec()
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 7 Feb 2025 22:31:52 +0000 (00:31 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 12 Feb 2025 17:20:45 +0000 (19:20 +0200)
commit26b3a8b25e68da478965f3054e4e213a48519fb6
tree6ded370a75776a0dae5e43e3ed86da9499ade192
parent46e2ffbdd2a0835ec3918606cd183b158c3f0e91
drm/i915/dsb: Move the +1 usec adjustment into dsb_wait_usec()

The "wait usec" DSB command doesn't quite seem to able to
guarantee that it always waits at least the specified
amount of usecs. Some of that could be just because it
supposedly just does some kind of dumb timestamp comparison
internally. But I also see cases where two hardware timestamps
sampled on each side of the "wait usec" command come out one
less than expected. So it looks like we always need at least a
+1 to guarantee that we never wait less than specified. Always
apply that adjustment in dsb_wait_usec().

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250207223159.14132-2-ville.syrjala@linux.intel.com
Reviewed-by: Ankit Nautiyal <ankit.k.nautiyal@intel.com>
drivers/gpu/drm/i915/display/intel_dsb.c