drm/i915: Use spatio-temporal dithering on PCH
authorAdam Jackson <ajax@redhat.com>
Mon, 19 Apr 2010 19:57:25 +0000 (15:57 -0400)
committerEric Anholt <eric@anholt.net>
Fri, 7 May 2010 20:59:26 +0000 (13:59 -0700)
commit0a31a448659d48cbc38f5e7520d8a65f8f1f8276
treed02870a48aaf5669a95906948bdfbc175bad45f1
parent9e51159c14c29ebd485a45ba56f148e180d62c29
drm/i915: Use spatio-temporal dithering on PCH

Spatial dither is better than nothing, but ST is even better.

(from ajax's followup message:)
  I noticed this with:

  http://ajax.fedorapeople.org/YellowFlower.jpg

  set as my desktop background in Gnome on a 1280x800 machine (in
  particular, a Sony Vaio VPCB1 with 6-bit panel and a rather bright black
  level).  Easiest way to test this is by poking at PIPEACONF with
  intel_reg_write directly:

  % sudo intel_reg_write 0x70008 0xc0000040 # no dither
  % sudo intel_reg_write 0x70008 0xc0000050 # spatial
  % sudo intel_reg_write 0x70008 0xc0000054 # ST

  I notice it especially strongly in the relatively flat dark area in the
  top left.  Closer than about 18" I can see a noticeable checkerboard
  pattern with plain spatial dithering.  ST smooths that out; I can still
  tell that it's lacking color precision, but it's not offensive.

Signed-off-by: Adam Jackson <ajax@redhat.com>
Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/i915/i915_reg.h
drivers/gpu/drm/i915/intel_display.c