On Fri, Jan 27, 2023 at 02:43:13PM -0800, Matt Roper wrote: > All production DG1 hardware has graphics stepping B0; there is no such > thing as C0. As such, we can simplify > > IS_DG1_GRAPHICS_STEP(uncore->i915, STEP_A0, STEP_C0) > > to just match DG1 in general. > > Bspec: 44463 > Signed-off-by: Matt Roper <[email protected]>
Reviewed-by: Rodrigo Vivi <[email protected]> > --- > drivers/gpu/drm/i915/gt/intel_region_lmem.c | 2 +- > drivers/gpu/drm/i915/i915_drv.h | 3 --- > 2 files changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/gpu/drm/i915/gt/intel_region_lmem.c > b/drivers/gpu/drm/i915/gt/intel_region_lmem.c > index f3ad93db0b21..89fdfc67f8d1 100644 > --- a/drivers/gpu/drm/i915/gt/intel_region_lmem.c > +++ b/drivers/gpu/drm/i915/gt/intel_region_lmem.c > @@ -158,7 +158,7 @@ static const struct intel_memory_region_ops > intel_region_lmem_ops = { > static bool get_legacy_lowmem_region(struct intel_uncore *uncore, > u64 *start, u32 *size) > { > - if (!IS_DG1_GRAPHICS_STEP(uncore->i915, STEP_A0, STEP_C0)) > + if (!IS_DG1(uncore->i915)) > return false; > > *start = 0; > diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h > index 57b84dbca084..495788e18b77 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -656,9 +656,6 @@ IS_SUBPLATFORM(const struct drm_i915_private *i915, > #define IS_RKL_DISPLAY_STEP(p, since, until) \ > (IS_ROCKETLAKE(p) && IS_DISPLAY_STEP(p, since, until)) > > -#define IS_DG1_GRAPHICS_STEP(p, since, until) \ > - (IS_DG1(p) && IS_GRAPHICS_STEP(p, since, until)) > - > #define IS_ADLS_DISPLAY_STEP(__i915, since, until) \ > (IS_ALDERLAKE_S(__i915) && \ > IS_DISPLAY_STEP(__i915, since, until)) > -- > 2.39.1 >
