| Message ID | 20250110-dpst-v7-14-605cb0271162@intel.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Display Global Histogram | expand |
> -----Original Message----- > From: Murthy, Arun R <arun.r.murthy@intel.com> > Sent: Friday, January 10, 2025 1:16 AM > To: dri-devel@lists.freedesktop.org; intel-gfx@lists.freedesktop.org; intel- > xe@lists.freedesktop.org > Cc: dmitry.baryshkov@linaro.org; Kandpal, Suraj <suraj.kandpal@intel.com>; > Shankar, Uma <uma.shankar@intel.com>; "Imported from f20241218-dpst- > v7-0-81bfe7d08c2d"@intel.com; 20240705091333.328322-1- > mohammed.thasleem@intel.com; Murthy, Arun R > <arun.r.murthy@intel.com> > Subject: [PATCH v7 14/14] drm/i915/histogram: Enable pipe dithering > > Enable pipe dithering while enabling histogram to overcome some atrifacts > seen on the screen. > > Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com> LGTM, Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com> > --- > drivers/gpu/drm/i915/display/intel_histogram.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/drivers/gpu/drm/i915/display/intel_histogram.c > b/drivers/gpu/drm/i915/display/intel_histogram.c > index > d015350b57ed5c8e9aaab71311159bf51e15e9c7..7d0c5d07042c5eb0e33c > 95e7cadac5c0d1fda379 100644 > --- a/drivers/gpu/drm/i915/display/intel_histogram.c > +++ b/drivers/gpu/drm/i915/display/intel_histogram.c > @@ -22,6 +22,13 @@ > #define HISTOGRAM_BIN_READ_RETRY_COUNT 5 #define > IET_SAMPLE_FORMAT_1_INT_9_FRACT 0x1000009 > > +static void intel_histogram_enable_dithering(struct intel_display *display, > + enum pipe pipe) > +{ > + intel_de_rmw(display, PIPE_MISC(pipe), PIPE_MISC_DITHER_ENABLE, > + PIPE_MISC_DITHER_ENABLE); > +} > + > static void set_bin_index_0(struct intel_display *display, enum pipe pipe) { > if (DISPLAY_VER(display) >= 20) > @@ -200,6 +207,10 @@ static int intel_histogram_enable(struct intel_crtc > *intel_crtc, u8 mode) > > if (histogram->enable) > return 0; > + > + /* Pipe Dithering should be enabled with histogram */ > + intel_histogram_enable_dithering(display, pipe); > + > /* enable histogram, clear DPST_BIN reg and select TC function */ > if (DISPLAY_VER(display) >= 20) > intel_de_rmw(display, DPST_CTL(pipe), > > -- > 2.25.1
diff --git a/drivers/gpu/drm/i915/display/intel_histogram.c b/drivers/gpu/drm/i915/display/intel_histogram.c index d015350b57ed5c8e9aaab71311159bf51e15e9c7..7d0c5d07042c5eb0e33c95e7cadac5c0d1fda379 100644 --- a/drivers/gpu/drm/i915/display/intel_histogram.c +++ b/drivers/gpu/drm/i915/display/intel_histogram.c @@ -22,6 +22,13 @@ #define HISTOGRAM_BIN_READ_RETRY_COUNT 5 #define IET_SAMPLE_FORMAT_1_INT_9_FRACT 0x1000009 +static void intel_histogram_enable_dithering(struct intel_display *display, + enum pipe pipe) +{ + intel_de_rmw(display, PIPE_MISC(pipe), PIPE_MISC_DITHER_ENABLE, + PIPE_MISC_DITHER_ENABLE); +} + static void set_bin_index_0(struct intel_display *display, enum pipe pipe) { if (DISPLAY_VER(display) >= 20) @@ -200,6 +207,10 @@ static int intel_histogram_enable(struct intel_crtc *intel_crtc, u8 mode) if (histogram->enable) return 0; + + /* Pipe Dithering should be enabled with histogram */ + intel_histogram_enable_dithering(display, pipe); + /* enable histogram, clear DPST_BIN reg and select TC function */ if (DISPLAY_VER(display) >= 20) intel_de_rmw(display, DPST_CTL(pipe),
Enable pipe dithering while enabling histogram to overcome some atrifacts seen on the screen. Signed-off-by: Arun R Murthy <arun.r.murthy@intel.com> --- drivers/gpu/drm/i915/display/intel_histogram.c | 11 +++++++++++ 1 file changed, 11 insertions(+)