On Wed, 06 Aug 2025, Nemesa Garg <nemesa.g...@intel.com> wrote:
> Remove the redundant check statement.
What if scaler_state->scaler_id == -1?
>
> Fixes: 73309ed9d598 ("drm/i915/display: WA_14011503117")
> Signed-off-by: Nemesa Garg <nemesa.g...@intel.com>
> ---
> drivers/gpu/drm/i915/display/skl_scaler.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/skl_scaler.c
> b/drivers/gpu/drm/i915/display/skl_scaler.c
> index cd7ebbeb9508..602198798174 100644
> --- a/drivers/gpu/drm/i915/display/skl_scaler.c
> +++ b/drivers/gpu/drm/i915/display/skl_scaler.c
> @@ -960,9 +960,6 @@ void adl_scaler_ecc_unmask(const struct intel_crtc_state
> *crtc_state)
> const struct intel_crtc_scaler_state *scaler_state =
> &crtc_state->scaler_state;
>
> - if (!scaler_state && scaler_state->scaler_id == -1)
> - return;
> -
> intel_de_write_fw(display,
> SKL_PS_ECC_STAT(crtc->pipe, scaler_state->scaler_id),
> 1);
--
Jani Nikula, Intel
