[Intel-gfx] [v8][PATCH 00/10] drm/i915: adding state checker for gamma lut value

[Shankar, Uma]

>-----Original Message-----
>From: Sharma, Swati2
>Sent: Monday, August 26, 2019 11:56 AM
>To: intel-gfx at lists.freedesktop.org
>Cc: Nikula, Jani <jani.nikula at intel.com>; Sharma, Shashank
><shashank.sharma at intel.com>; Manna, Animesh <animesh.manna at intel.com>;
>Nautiyal, Ankit K <ankit.k.nautiyal at intel.com>; daniel.vetter at ffwll.ch;
>ville.syrjala at linux.intel.com; Shankar, Uma <uma.shankar at intel.com>; Sharma,
>Swati2 <swati2.sharma at intel.com>
>Subject: [v8][PATCH 00/10] drm/i915: adding state checker for gamma lut value

The series name is changed "value" instead of "values". This makes it a new series
for patchwork. Please fix that in the next version and lets maintain the name whatever
has been there from the beginning.

>In this patch series, added state checker to validate gamma (8BIT and 10BIT).
>This reads hardware state, and compares the originally requested state(s/w) to the
>state read from the hardware.
>This is done for legacy, i965, ilk, glk and their variant platforms.
>
>Intentionally, excluded bdw and ivb since they have spilt gamma mode; for which
>degamma read outs are required (which I think shouldn't be included in this patch
>series). Will include after degamma state checker is completed.
>
>v1: -Implementation done for legacy platforms
>     (removed all the placeholders) (Jani)
>v2: -Restructured code and created platform specific patch series for
>     gamma validation
>v3: -Rebase
>v4: -Minor changes-function name changes mainly
>v5: -Added degamma validation (Ville)
>v6: -Removed degamma changes, debugging was becoming difficult
>    -Added function to assign bit_precision for gamma/degamma
>     lut values /platform
>    -Added debug info into intel_dump_pipe_config() (Jani)
>v7: -Added platform specific functions to compute gamma bit precision
>     on the basis of GAMMA_MODE (Ville)
>    -Corrected checkpatch warnings
>v8: -Restructured code
>    -Removed bdw and ivb platform state checker
>
>Swati Sharma (10):
>  drm/i915/display: Add func to get gamma bit precision
>  drm/i915/display: Add debug log for color parameters
>  drm/i915/display: Add func to compare hw/sw gamma lut
>  drm/i915/display: Add macro to compare gamma hw/sw lut
>  drm/i915/display: Extract i9xx_read_luts()
>  drm/i91/display: Extract i965_read_luts()
>  drm/i915/display: Extract chv_read_luts()
>  drm/i915/display: Extract ilk_read_luts()
>  drm/i915/display: Extract glk_read_luts()
>  FOR_TESTING_ONLY: Print rgb values of hw and sw blobs
>
> drivers/gpu/drm/i915/display/intel_color.c   | 370 ++++++++++++++++++++++++++-
> drivers/gpu/drm/i915/display/intel_color.h   |   7 +
> drivers/gpu/drm/i915/display/intel_display.c |  34 +++
> drivers/gpu/drm/i915/i915_reg.h              |  15 ++
> 4 files changed, 423 insertions(+), 3 deletions(-)
>
>--
>1.9.1