[PATCH] drm/i915/scaler: Fix for WA_14011503117

[Kandpal, Suraj]

> Subject: RE: [PATCH] drm/i915/scaler: Fix for WA_14011503117
> 
> 
> > Subject: [PATCH] drm/i915/scaler: Fix for WA_14011503117
> 
> We already have a commit with that name don't really want many commits
> with the same subject
> 
> >
> > Remove the redundant check statement.
> 
> The is statement is not redundant just wrong will need to fix the commit
> description
> 
> >
> > v2: Add scaler_id check [Jani]
> >
> > Fixes: 73309ed9d598 ("drm/i915/display: WA_14011503117")
> > Signed-off-by: Nemesa Garg <nemesa.garg at intel.com>
> > ---
> >  drivers/gpu/drm/i915/display/skl_scaler.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/skl_scaler.c
> > b/drivers/gpu/drm/i915/display/skl_scaler.c
> > index cd7ebbeb9508..6891a0b00f41 100644
> > --- a/drivers/gpu/drm/i915/display/skl_scaler.c
> > +++ b/drivers/gpu/drm/i915/display/skl_scaler.c
> > @@ -960,7 +960,7 @@ 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)
> > +	if (scaler_state->scaler_id == -1)
> 
> Hmm but what if scaler_state ends up being null now
> 
> This should have been scaler_state && scaler_state->scaler_id == Also please
> add people who had bought this issue up in cc to keep them in loop in this case
> that would be Jani.
> It's very easy to miss fixes like this in all the mails that end up getting sent
> upstream more so if they are not cc'ed

Also rather than all this I would like the condition to be something like
!scaler_state || scaler_state->scaler_id < 0
Looks cleaner and also takes care of scaler state is null case
There is a good chance that if scaler state really ends up being null it will blow elsewhere in code
Before reaching here but still good thing to have in place

Regards,
Suraj Kandpal
> 
> Regards,
> Suraj Kandpal
> >  		return;
> >
> >  	intel_de_write_fw(display,
> > --
> > 2.25.1