drm/i915: Add a policy note for removing workarounds
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 17 Nov 2017 10:26:35 +0000 (10:26 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 17 Nov 2017 18:20:36 +0000 (18:20 +0000)
commit6a7a6a982a22091b9d67cb3ff33d7782494aa2f8
tree552dc278bad2e07290f6479e403f12bef442d808
parent223c73a3667a5afe33245d423f867f4c4b1b39ea
drm/i915: Add a policy note for removing workarounds

Rodrigo gave a persuasive argument for keeping workarounds: that they
serve as a good guide for the bring up of the next generation. Not only
do workarounds persist into the early revisions, they show where the
workarounds were previously added to the code flow and sometimes the old
workarounds have an explanation that give insight into their wider
implications.

Based on his suggestion, document the policy that we want to keep the
workarounds from the current generation to guide the next. Older
preproduction workarounds we still want to remove to keep the code
clean.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Jani Nikula <jani.nikula@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20171117102635.8689-1-chris@chris-wilson.co.uk
Acked-by: Jani Nikula <jani.nikula@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/i915/i915_drv.c