[PATCH 9/9] drm/i915/xe3lpd: Extend WA 16023981245 for display 30.02

[Bhadane, Dnyaneshwar]

> -----Original Message-----
> From: Vivekanandan, Balasubramani
> <balasubramani.vivekanandan at intel.com>
> Sent: Monday, June 9, 2025 12:48 PM
> To: Bhadane, Dnyaneshwar <dnyaneshwar.bhadane at intel.com>; intel-
> xe at lists.freedesktop.org
> Subject: Re: [PATCH 9/9] drm/i915/xe3lpd: Extend WA 16023981245 for display
> 30.02
> 
> On 07.06.2025 01:21, Dnyaneshwar Bhadane wrote:
> > wa_16023981245 need to be exteded for display version 30.02
> >
> > Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane at intel.com>
> > ---
> >  drivers/gpu/drm/i915/display/intel_atomic_plane.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > index 15ede7678636..e0336c79c294 100644
> > --- a/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > +++ b/drivers/gpu/drm/i915/display/intel_atomic_plane.c
> > @@ -1085,7 +1085,8 @@ int intel_plane_check_src_coordinates(struct
> intel_plane_state *plane_state)
> >
> >  		/* Wa_16023981245 */
> >  		if ((DISPLAY_VERx100(display) == 2000 ||
> > -		     DISPLAY_VERx100(display) == 3000) &&
> > +		     DISPLAY_VERx100(display) == 3000 ||
> > +		     DISPLAY_VERx100(display) == 3002) &&
> You can simply the check to `DISPLAY_VER(display) == 30`
Hi, 
Thanks,
This is good idea to merge it as `DISPLAY_VER(display) == 30`. I will do in next revision.

Dnyaneshwar

> 
> Regards,
> Bala
> >  		     src_x % 2 != 0)
> >  			hsub = 2;
> >  	} else {
> > --
> > 2.34.1
> >