]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "drm/i915: Mask reserved bits in display/sprite address registers"
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 24 Jan 2014 09:31:44 +0000 (10:31 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 24 Jan 2014 16:22:00 +0000 (17:22 +0100)
commitd931a8751aeeb38e7e7039232c888fb5ce5cf793
tree7fcb1962926051c8ae7a6dafc10ad93fd57b0470
parenteecc89d9df1b7ab1aec7f024ebdfe8576f721f3f
Revert "drm/i915: Mask reserved bits in display/sprite address registers"

This reverts commit 7dbcae8c7c1d74ed18fbb38f9fc503c55482e9e9.

I've left the masking in the pageflip code since that seems to be some
useful piece of preemptive robustness.

Iirc I've merged this patch under the assumption that the BIOS leaves
some random gunk in the lower bits and gets unhappy if we trample on
them. We have quite a few case like this, so this made sense.

Now I've just learned that there's actual hardware features bits in
the low 12 bits, and the kernel needs to preserve them to allow a
userspace blob to do its job. Given Dave Airlie's clear stance on
userspace blob drivers I've quickly chatted with him and he doesn't
seem too happy. So let's revert this.

If there are indeed bits that we must preserve in this range then we
can ressurrect this patch, but with proper documentation for those
bits supplied. And we probably also need to think a bit about
interactions with our driver.

Cc: Armin Reese <armin.c.reese@intel.com>
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Dave Airlie <airlied@linux.ie>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/i915_reg.h
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/i915/intel_sprite.c