On Thu, 15 Jun 2023, Dnyaneshwar Bhadane <dnyaneshwar.bhadane@xxxxxxxxx> wrote: > Follow consistent naming convention. Replace JSL with > JASPERLAKE. > > Signed-off-by: Dnyaneshwar Bhadane <dnyaneshwar.bhadane@xxxxxxxxx> > -#define IS_JSL_EHL(i915) (IS_PLATFORM(i915, INTEL_JASPERLAKE) || \ > +#define IS_JASPERLAKE_EHL(i915) (IS_PLATFORM(i915, INTEL_JASPERLAKE) || \ > IS_PLATFORM(i915, INTEL_ELKHARTLAKE)) The new name for this is just dumb. This matches two platforms, JSL and EHL, and there's no point in one of them being an acronym and the other one not. And IS_JASPERLAKE_ELKHARTLAKE() would be too long. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center