drm/i915: Remove the broken DP CRC support for g4x
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 14 Feb 2019 19:22:18 +0000 (21:22 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 20 Feb 2019 20:48:55 +0000 (22:48 +0200)
commit53039750bf0a859f3d10183b4e26bcafb155cb4a
tree58ae8f734cef63b7454dd1518de4bbd09707f5b9
parentb49aacc8b9eaab0f582f93f02b7f23a969832839
drm/i915: Remove the broken DP CRC support for g4x

DP CRCs don't really work on g4x. If you want any CRCs on DP you must
select the CRC source before the port is enabled, otherwise the CRC
source select bits simply ignore any writes to them. And once the port
is enabled we mustn't change the CRC source select until the port is
disabled. That almost works, but not quite :( Eventually the CRC source
select bits get permanently stuck one way or the other, and after that
a reboot (or possibly a display reset) is needed to get working CRCs
on that pipe (not matter which CRC source we try to use).

Additionally the DFT scrambler reset bits we're trying to use don't
seem to exist on g4x. There are some potentially relevant looking bits
in the pipe registers, but when I tried it I got stable looking CRCs
without setting any bits for this.

If there is a way to make DP CRCs work reliably on g4x, I wasn't
able to find it. So let's just remove the broken code we have.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190214192219.3858-3-ville.syrjala@linux.intel.com
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
drivers/gpu/drm/i915/intel_pipe_crc.c