drm/i915/casf: Integrate the sharpness filter properly into the scaler code

The sharpness filter is just a special mode of the pipe scaler.
It doesn't warrant all this special casing everywhere. Just
integrate it properly into the scaler code so that it's treated
no different from the other pipe scaler uses (scaling,centering,
YCbCr 4:2:0 output).

v2: Also reject scaling_filter vs. sharpness

Reviewed-by: Nemesa Garg <nemesa.garg@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patch.msgid.link/20260407175244.19654-10-ville.syrjala@linux.intel.com

diff --git a/drivers/gpu/drm/i915/display/intel_casf.c b/drivers/gpu/drm/i915/display/intel_casf.c
index 21e84a4f9ff5a0bcf23ad0a70797a3d64e6f8655..c2d2746c5f04118fefd95585b664cc0385209d88 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_casf.c
+++ b/drivers/gpu/drm/i915/display/intel_casf.c
@@ -75,20 +75,6 @@ static void intel_casf_filter_lut_load(const struct intel_crtc_state *crtc_state
                               sharpness_lut[i]);
 }
 
-void intel_casf_update_strength(const struct intel_crtc_state *crtc_state)
-{
-       struct intel_display *display = to_intel_display(crtc_state);
-       struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       int win_size;
-
-       intel_de_rmw(display, SHARPNESS_CTL(crtc->pipe), FILTER_STRENGTH_MASK,
-                    FILTER_STRENGTH(crtc_state->pch_pfit.casf.strength));
-
-       win_size = intel_de_read(display, SKL_PS_WIN_SZ(crtc->pipe, 1));
-
-       intel_de_write_fw(display, SKL_PS_WIN_SZ(crtc->pipe, 1), win_size);
-}
-
 static void intel_casf_compute_win_size(struct intel_crtc_state *crtc_state)
 {
        const struct drm_display_mode *mode = &crtc_state->hw.adjusted_mode;
@@ -102,18 +88,14 @@ static void intel_casf_compute_win_size(struct intel_crtc_state *crtc_state)
                crtc_state->pch_pfit.casf.win_size = SHARPNESS_FILTER_SIZE_7X7;
 }
 
+static void intel_casf_scaler_compute_coef(struct intel_crtc_state *crtc_state);
+
 int intel_casf_compute_config(struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc_state);
 
-       if (!HAS_CASF(display))
-               return 0;
-
-       if (crtc_state->hw.sharpness_strength == 0) {
-               crtc_state->pch_pfit.casf.enable = false;
-               crtc_state->pch_pfit.casf.strength = 0;
+       if (crtc_state->hw.sharpness_strength == 0)
                return 0;
-       }
 
        /* CASF with joiner not supported in hardware */
        if (crtc_state->joiner_pipes) {
@@ -136,7 +118,7 @@ int intel_casf_compute_config(struct intel_crtc_state *crtc_state)
 
        intel_casf_compute_win_size(crtc_state);
 
-       intel_casf_scaler_compute_config(crtc_state);
+       intel_casf_scaler_compute_coef(crtc_state);
 
        return 0;
 }
@@ -161,14 +143,6 @@ void intel_casf_sharpness_get_config(struct intel_crtc_state *crtc_state)
        }
 }
 
-bool intel_casf_needs_scaler(const struct intel_crtc_state *crtc_state)
-{
-       if (crtc_state->pch_pfit.casf.enable)
-               return true;
-
-       return false;
-}
-
 static int casf_coeff_tap(int i)
 {
        return i % SCALER_FILTER_NUM_TAPS;
@@ -240,7 +214,7 @@ static void convert_sharpness_coef_binary(struct scaler_filter_coeff *coeff,
        }
 }
 
-void intel_casf_scaler_compute_config(struct intel_crtc_state *crtc_state)
+static void intel_casf_scaler_compute_coef(struct intel_crtc_state *crtc_state)
 {
        const u16 *filtercoeff;
        u16 filter_coeff[SCALER_FILTER_NUM_TAPS];
@@ -264,32 +238,8 @@ void intel_casf_scaler_compute_config(struct intel_crtc_state *crtc_state)
        }
 }
 
-void intel_casf_enable(const struct intel_crtc_state *crtc_state)
+void intel_casf_setup(const struct intel_crtc_state *crtc_state)
 {
-       struct intel_display *display = to_intel_display(crtc_state);
-       struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       u32 sharpness_ctl;
-
        intel_casf_filter_lut_load(crtc_state);
-
        intel_casf_write_coeff(crtc_state);
-
-       sharpness_ctl = FILTER_EN | FILTER_STRENGTH(crtc_state->pch_pfit.casf.strength);
-
-       sharpness_ctl |= crtc_state->pch_pfit.casf.win_size;
-
-       intel_de_write(display, SHARPNESS_CTL(crtc->pipe), sharpness_ctl);
-
-       skl_scaler_setup_casf(crtc_state);
-}
-
-void intel_casf_disable(const struct intel_crtc_state *crtc_state)
-{
-       struct intel_display *display = to_intel_display(crtc_state);
-       struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-
-       intel_de_write(display, SKL_PS_CTRL(crtc->pipe, 1), 0);
-       intel_de_write(display, SKL_PS_WIN_POS(crtc->pipe, 1), 0);
-       intel_de_write(display, SHARPNESS_CTL(crtc->pipe), 0);
-       intel_de_write(display, SKL_PS_WIN_SZ(crtc->pipe, 1), 0);
 }

diff --git a/drivers/gpu/drm/i915/display/intel_casf.h b/drivers/gpu/drm/i915/display/intel_casf.h
index c4f984b73348e1a2c2df2b03d25a6c0a3fd974bf..3ebb7522af0a4765cd926217510e2b510d0b573a 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_casf.h
+++ b/drivers/gpu/drm/i915/display/intel_casf.h
@@ -11,11 +11,7 @@
 struct intel_crtc_state;
 
 int intel_casf_compute_config(struct intel_crtc_state *crtc_state);
-void intel_casf_update_strength(const struct intel_crtc_state *new_crtc_state);
 void intel_casf_sharpness_get_config(struct intel_crtc_state *crtc_state);
-void intel_casf_enable(const struct intel_crtc_state *crtc_state);
-void intel_casf_disable(const struct intel_crtc_state *crtc_state);
-void intel_casf_scaler_compute_config(struct intel_crtc_state *crtc_state);
-bool intel_casf_needs_scaler(const struct intel_crtc_state *crtc_state);
+void intel_casf_setup(const struct intel_crtc_state *crtc_state);
 
 #endif /* __INTEL_CASF_H__ */

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index e02e69467871c3a109290fd1c2b9039edb9dc427..58a654ca0d20822f85d9139d5f0d58022abffebc 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -58,7 +58,6 @@
 #include "intel_audio.h"
 #include "intel_bo.h"
 #include "intel_bw.h"
-#include "intel_casf.h"
 #include "intel_cdclk.h"
 #include "intel_clock_gating.h"
 #include "intel_color.h"
@@ -988,24 +987,6 @@ static bool audio_disabling(const struct intel_crtc_state *old_crtc_state,
                 memcmp(old_crtc_state->eld, new_crtc_state->eld, MAX_ELD_BYTES) != 0);
 }
 
-static bool intel_casf_enabling(const struct intel_crtc_state *new_crtc_state,
-                               const struct intel_crtc_state *old_crtc_state)
-{
-       if (!new_crtc_state->hw.active)
-               return false;
-
-       return is_enabling(pch_pfit.casf.enable, old_crtc_state, new_crtc_state);
-}
-
-static bool intel_casf_disabling(const struct intel_crtc_state *old_crtc_state,
-                                const struct intel_crtc_state *new_crtc_state)
-{
-       if (!new_crtc_state->hw.active)
-               return false;
-
-       return is_disabling(pch_pfit.casf.enable, old_crtc_state, new_crtc_state);
-}
-
 static bool intel_crtc_lobf_enabling(const struct intel_crtc_state *old_crtc_state,
                                     const struct intel_crtc_state *new_crtc_state)
 {
@@ -1187,9 +1168,6 @@ static void intel_pre_plane_update(struct intel_atomic_state *state,
        if (audio_disabling(old_crtc_state, new_crtc_state))
                intel_encoders_audio_disable(state, crtc);
 
-       if (intel_casf_disabling(old_crtc_state, new_crtc_state))
-               intel_casf_disable(new_crtc_state);
-
        intel_drrs_deactivate(old_crtc_state);
 
        if (hsw_ips_pre_update(state, crtc))
@@ -4308,14 +4286,9 @@ static int intel_crtc_atomic_check(struct intel_atomic_state *state,
                return ret;
        }
 
-       ret = intel_casf_compute_config(crtc_state);
-       if (ret)
-               return ret;
-
        if (DISPLAY_VER(display) >= 9) {
                if (intel_crtc_needs_modeset(crtc_state) ||
-                   intel_crtc_needs_fastset(crtc_state) ||
-                   intel_casf_needs_scaler(crtc_state)) {
+                   intel_crtc_needs_fastset(crtc_state)) {
                        ret = skl_update_scaler_crtc(crtc_state);
                        if (ret)
                                return ret;
@@ -6817,11 +6790,6 @@ static void intel_pre_update_crtc(struct intel_atomic_state *state,
                        intel_vrr_set_transcoder_timings(new_crtc_state);
        }
 
-       if (intel_casf_enabling(new_crtc_state, old_crtc_state))
-               intel_casf_enable(new_crtc_state);
-       else if (new_crtc_state->pch_pfit.casf.strength != old_crtc_state->pch_pfit.casf.strength)
-               intel_casf_update_strength(new_crtc_state);
-
        intel_fbc_update(state, crtc);
 
        drm_WARN_ON(display->drm, !intel_display_power_is_enabled(display, POWER_DOMAIN_DC_OFF));

diff --git a/drivers/gpu/drm/i915/display/intel_pfit.c b/drivers/gpu/drm/i915/display/intel_pfit.c
index 2dec4ccf74ced3bf07bf3a2a40b3278c0b910b07..5b170c74a5100aca01b82652650124c149e717c5 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_pfit.c
+++ b/drivers/gpu/drm/i915/display/intel_pfit.c
@@ -196,7 +196,8 @@ static int pch_panel_fitting(struct intel_crtc_state *crtc_state,
        /* Native modes don't need fitting */
        if (adjusted_mode->crtc_hdisplay == pipe_src_w &&
            adjusted_mode->crtc_vdisplay == pipe_src_h &&
-           crtc_state->output_format != INTEL_OUTPUT_FORMAT_YCBCR420)
+           crtc_state->output_format != INTEL_OUTPUT_FORMAT_YCBCR420 &&
+           crtc_state->hw.sharpness_strength == 0)
                return 0;
 
        switch (conn_state->scaling_mode) {
@@ -260,6 +261,16 @@ static int pch_panel_fitting(struct intel_crtc_state *crtc_state,
                return -EINVAL;
        }
 
+       if (crtc_state->hw.sharpness_strength &&
+           (width != pipe_src_w || height != pipe_src_h ||
+            crtc_state->hw.scaling_filter != DRM_SCALING_FILTER_DEFAULT ||
+            crtc_state->output_format != INTEL_OUTPUT_FORMAT_RGB)) {
+               drm_dbg_kms(display->drm,
+                           "[CRTC:%d:%s] no scaling/YCbCr output with sharpness filter\n",
+                           crtc->base.base.id, crtc->base.name);
+               return -EINVAL;
+       }
+
        drm_rect_init(&crtc_state->pch_pfit.dst,
                      x, y, width, height);
        crtc_state->pch_pfit.enabled = true;

diff --git a/drivers/gpu/drm/i915/display/skl_scaler.c b/drivers/gpu/drm/i915/display/skl_scaler.c
index 762f4bb46c2d90ac4108663fbecb0f91b4c28db5..308b8d363bba0368e66f866563a87e9b4ba2968b 100644 (file)
--- a/drivers/gpu/drm/i915/display/skl_scaler.c
+++ b/drivers/gpu/drm/i915/display/skl_scaler.c
@@ -270,6 +270,11 @@ int skl_update_scaler_crtc(struct intel_crtc_state *crtc_state)
 {
        const struct drm_display_mode *pipe_mode = &crtc_state->hw.pipe_mode;
        int width, height;
+       int ret;
+
+       ret = intel_casf_compute_config(crtc_state);
+       if (ret)
+               return ret;
 
        if (crtc_state->pch_pfit.enabled) {
                width = drm_rect_width(&crtc_state->pch_pfit.dst);
@@ -284,8 +289,7 @@ int skl_update_scaler_crtc(struct intel_crtc_state *crtc_state)
                                 drm_rect_width(&crtc_state->pipe_src),
                                 drm_rect_height(&crtc_state->pipe_src),
                                 width, height, NULL, 0,
-                                crtc_state->pch_pfit.enabled ||
-                                intel_casf_needs_scaler(crtc_state));
+                                crtc_state->pch_pfit.enabled);
 }
 
 /**
@@ -534,14 +538,11 @@ static int setup_crtc_scaler(struct intel_atomic_state *state,
        struct intel_crtc_scaler_state *scaler_state =
                &crtc_state->scaler_state;
 
-       if (intel_casf_needs_scaler(crtc_state) && crtc_state->pch_pfit.enabled)
-               return -EINVAL;
-
        return intel_atomic_setup_scaler(crtc_state,
                                         hweight32(scaler_state->scaler_users),
                                         crtc, "CRTC", crtc->base.base.id,
                                         NULL, &scaler_state->scaler_id,
-                                        intel_casf_needs_scaler(crtc_state));
+                                        crtc_state->pch_pfit.casf.enable);
 }
 
 static int setup_plane_scaler(struct intel_atomic_state *state,
@@ -757,43 +758,14 @@ static void skl_scaler_setup_filter(struct intel_display *display,
        }
 }
 
-void skl_scaler_setup_casf(const struct intel_crtc_state *crtc_state)
+static u32 casf_sharpness_ctl(const struct intel_crtc_state *crtc_state)
 {
-       struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       struct intel_display *display = to_intel_display(crtc);
-       const struct drm_display_mode *adjusted_mode =
-               &crtc_state->hw.adjusted_mode;
-       const struct intel_crtc_scaler_state *scaler_state =
-               &crtc_state->scaler_state;
-       struct drm_rect src, dest;
-       int id, width, height;
-       int x = 0, y = 0;
-       enum pipe pipe = crtc->pipe;
-       u32 ps_ctrl;
-
-       width = adjusted_mode->crtc_hdisplay;
-       height = adjusted_mode->crtc_vdisplay;
-
-       drm_rect_init(&dest, x, y, width, height);
-
-       width = drm_rect_width(&dest);
-       height = drm_rect_height(&dest);
-       id = scaler_state->scaler_id;
-
-       drm_rect_init(&src, 0, 0,
-                     drm_rect_width(&crtc_state->pipe_src) << 16,
-                     drm_rect_height(&crtc_state->pipe_src) << 16);
-
-       trace_intel_pipe_scaler_update_arm(crtc, id, x, y, width, height);
-
-       ps_ctrl = PS_SCALER_EN | PS_BINDING_PIPE | scaler_state->scalers[id].mode |
-               skl_scaler_get_filter_select(crtc_state->hw.scaling_filter, true);
+       if (!crtc_state->pch_pfit.casf.enable)
+               return 0;
 
-       intel_de_write_fw(display, SKL_PS_CTRL(pipe, id), ps_ctrl);
-       intel_de_write_fw(display, SKL_PS_WIN_POS(pipe, id),
-                         PS_WIN_XPOS(x) | PS_WIN_YPOS(y));
-       intel_de_write_fw(display, SKL_PS_WIN_SZ(pipe, id),
-                         PS_WIN_XSIZE(width) | PS_WIN_YSIZE(height));
+       return FILTER_EN |
+               FILTER_STRENGTH(crtc_state->pch_pfit.casf.strength) |
+               crtc_state->pch_pfit.casf.win_size;
 }
 
 void skl_pfit_enable(const struct intel_crtc_state *crtc_state)
@@ -837,12 +809,20 @@ void skl_pfit_enable(const struct intel_crtc_state *crtc_state)
        id = scaler_state->scaler_id;
 
        ps_ctrl = PS_SCALER_EN | PS_BINDING_PIPE | scaler_state->scalers[id].mode |
-               skl_scaler_get_filter_select(crtc_state->hw.scaling_filter, false);
+               skl_scaler_get_filter_select(crtc_state->hw.scaling_filter,
+                                            crtc_state->pch_pfit.casf.enable);
 
        trace_intel_pipe_scaler_update_arm(crtc, id, x, y, width, height);
 
-       skl_scaler_setup_filter(display, NULL, pipe, id, 0,
-                               crtc_state->hw.scaling_filter);
+       if (crtc_state->pch_pfit.casf.enable)
+               intel_casf_setup(crtc_state);
+       else
+               skl_scaler_setup_filter(display, NULL, pipe, id, 0,
+                                       crtc_state->hw.scaling_filter);
+
+       if (scaler_has_casf(display, id))
+               intel_de_write_fw(display, SHARPNESS_CTL(crtc->pipe),
+                                 casf_sharpness_ctl(crtc_state));
 
        intel_de_write_fw(display, SKL_PS_CTRL(pipe, id), ps_ctrl);
 
@@ -930,6 +910,9 @@ static void skl_detach_scaler(struct intel_dsb *dsb,
 
        trace_intel_scaler_disable_arm(crtc, id);
 
+       if (scaler_has_casf(display, id))
+               intel_de_write_dsb(display, dsb, SHARPNESS_CTL(crtc->pipe), 0);
+
        intel_de_write_dsb(display, dsb, SKL_PS_CTRL(crtc->pipe, id), 0);
        intel_de_write_dsb(display, dsb, SKL_PS_WIN_POS(crtc->pipe, id), 0);
        intel_de_write_dsb(display, dsb, SKL_PS_WIN_SZ(crtc->pipe, id), 0);
@@ -983,18 +966,16 @@ void skl_scaler_get_config(struct intel_crtc_state *crtc_state)
                if (scaler_has_casf(display, i))
                        intel_casf_sharpness_get_config(crtc_state);
 
-               if (!crtc_state->pch_pfit.casf.enable)
-                       crtc_state->pch_pfit.enabled = true;
+               crtc_state->pch_pfit.enabled = true;
 
                pos = intel_de_read(display, SKL_PS_WIN_POS(crtc->pipe, i));
                size = intel_de_read(display, SKL_PS_WIN_SZ(crtc->pipe, i));
 
-               if (!crtc_state->pch_pfit.casf.enable)
-                       drm_rect_init(&crtc_state->pch_pfit.dst,
-                                     REG_FIELD_GET(PS_WIN_XPOS_MASK, pos),
-                                     REG_FIELD_GET(PS_WIN_YPOS_MASK, pos),
-                                     REG_FIELD_GET(PS_WIN_XSIZE_MASK, size),
-                                     REG_FIELD_GET(PS_WIN_YSIZE_MASK, size));
+               drm_rect_init(&crtc_state->pch_pfit.dst,
+                             REG_FIELD_GET(PS_WIN_XPOS_MASK, pos),
+                             REG_FIELD_GET(PS_WIN_YPOS_MASK, pos),
+                             REG_FIELD_GET(PS_WIN_XSIZE_MASK, size),
+                             REG_FIELD_GET(PS_WIN_YSIZE_MASK, size));
 
                scaler_state->scalers[i].in_use = true;
                break;

diff --git a/drivers/gpu/drm/i915/display/skl_scaler.h b/drivers/gpu/drm/i915/display/skl_scaler.h
index 20ecf373eb1963367896f5435f6b6fab132f67df..5deabca909e65fc665ae7437a0a6dd56fac23deb 100644 (file)
--- a/drivers/gpu/drm/i915/display/skl_scaler.h
+++ b/drivers/gpu/drm/i915/display/skl_scaler.h
@@ -36,8 +36,6 @@ void skl_scaler_disable(const struct intel_crtc_state *old_crtc_state);
 
 void skl_scaler_get_config(struct intel_crtc_state *crtc_state);
 
-void skl_scaler_setup_casf(const struct intel_crtc_state *crtc_state);
-
 enum drm_mode_status
 skl_scaler_mode_valid(struct intel_display *display,
                      const struct drm_display_mode *mode,