[PATCHv6 5/8] drm/i915/histogram: Add crtc properties for global histogram

[Dmitry Baryshkov]

On Fri, 22 Nov 2024 at 06:25, Murthy, Arun R <arun.r.murthy at intel.com> wrote:
>
> > On Thu, Nov 21, 2024 at 05:56:00PM +0530, Arun R Murthy wrote:
> > > CRTC properties have been added for enable/disable histogram, reading
> > > the histogram data and writing the IET data.
> > > "HISTOGRAM_EN" is the crtc property to enable/disable the global
> > > histogram and takes a value 0/1 accordingly.
> > > "Histogram" is a crtc property to read the binary histogram data.
> > > "Global IET" is a crtc property to write the IET binary LUT data.
> > >
> > > v2: Read the histogram blob data before sending uevent (Jani)
> > > v3: use drm_property_replace_blob_from_id (Vandita)
> > >     Add substruct for histogram in intel_crtc_state (Jani)
> > > v4:  Rebased after addressing comments on patch 1
> > > v5: histogram check with old/new crtc_state (Suraj)
> > > v6: Rebase
> > >
> > > Signed-off-by: Arun R Murthy <arun.r.murthy at intel.com>
> > > ---
> > >  drivers/gpu/drm/i915/display/intel_atomic.c   |   5 +
> > >  drivers/gpu/drm/i915/display/intel_crtc.c     | 166 +++++++++++++++++-
> > >  drivers/gpu/drm/i915/display/intel_crtc.h     |   5 +
> > >  drivers/gpu/drm/i915/display/intel_display.c  |  16 ++
> > >  .../drm/i915/display/intel_display_types.h    |  13 ++
> > >  .../gpu/drm/i915/display/intel_histogram.c    |   6 +
> > >  6 files changed, 210 insertions(+), 1 deletion(-)
> >
> > Please provide the documentation for these properties: binary format, etc. Also,
> > ideally such a properties should be defined in a vendor-neutral way, so that
> > other drivers also can implement support for similar functionality.
> >
> I assumed that there are no other users of this histogram. Hence had it internal to i915 driver.
> Will move these properties to drm-crtc, making it useable by other users as well.

There are no users for the Intel histogram format. However other
vendors can provide similar functionality to enhance visual quality
depending on the image contents and ambient light. Please define the
properties in a generic way, letting other vendors be able to
implement similar functionality.

-- 
With best wishes
Dmitry