]> 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)
commit2f885a78e27cd79815f65344dc476b7a47120d52
tree7fcb1962926051c8ae7a6dafc10ad93fd57b0470
parent4f8e9db4ba4e73600e6d33037737a809fe3a64e2
Revert "drm/i915: Mask reserved bits in display/sprite address registers"

This reverts commit f91008fb09962444433bf34660ba397aa54336dd.

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