On Fri, 11 Feb 2022, Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx> wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > Give names to the SSKPD/MLTR fields, and use the > REG_GENMASK* and REG_FIELD_GET*. > > Also drop the bogus non-mirrored SSKP register define. > > Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_reg.h | 27 ++++++++++++--------------- > drivers/gpu/drm/i915/intel_pm.c | 24 ++++++++++++------------ > 2 files changed, 24 insertions(+), 27 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index 87c92314ee26..278c9cbc6f3c 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -1896,10 +1896,17 @@ > #define MAD_DIMM_A_SIZE_SHIFT 0 > #define MAD_DIMM_A_SIZE_MASK (0xff << MAD_DIMM_A_SIZE_SHIFT) > > -/* snb MCH registers for priority tuning */ > #define MCH_SSKPD _MMIO(MCHBAR_MIRROR_BASE_SNB + 0x5d10) > -#define MCH_SSKPD_WM0_MASK 0x3f > -#define MCH_SSKPD_WM0_VAL 0xc > +#define SSKPD_NEW_WM0_MASK_HSW REG_GENMASK64(63, 56) > +#define SSKPD_WM4_MASK_HSW REG_GENMASK64(40, 32) > +#define SSKPD_WM3_MASK_HSW REG_GENMASK64(28, 20) > +#define SSKPD_WM2_MASK_HSW REG_GENMASK64(19, 12) > +#define SSKPD_WM1_MASK_HSW REG_GENMASK64(11, 4) > +#define SSKPD_OLD_WM0_MASK_HSW REG_GENMASK64(3, 0) > +#define SSKPD_WM3_MASK_SNB REG_GENMASK(29, 24) > +#define SSKPD_WM2_MASK_SNB REG_GENMASK(21, 16) > +#define SSKPD_WM1_MASK_SNB REG_GENMASK(13, 8) > +#define SSKPD_WM0_MASK_SNB REG_GENMASK(5, 0) > > /* Clocking configuration register */ > #define CLKCFG _MMIO(MCHBAR_MIRROR_BASE + 0xc00) > @@ -4321,19 +4328,9 @@ > > /* Memory latency timer register */ > #define MLTR_ILK _MMIO(0x11222) > -#define MLTR_WM1_SHIFT 0 > -#define MLTR_WM2_SHIFT 8 > /* the unit of memory self-refresh latency time is 0.5us */ > -#define ILK_SRLT_MASK 0x3f > - > - > -/* the address where we get all kinds of latency value */ > -#define SSKPD _MMIO(0x5d10) > -#define SSKPD_WM_MASK 0x3f > -#define SSKPD_WM0_SHIFT 0 > -#define SSKPD_WM1_SHIFT 8 > -#define SSKPD_WM2_SHIFT 16 > -#define SSKPD_WM3_SHIFT 24 > +#define MLTR_WM2_MASK REG_GENMASK(13, 8) > +#define MLTR_WM1_MASK REG_GENMASK(5, 0) > > /* > * The two pipe frame counter registers are not synchronized, so > diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c > index 34e46a9b8300..605944551e1b 100644 > --- a/drivers/gpu/drm/i915/intel_pm.c > +++ b/drivers/gpu/drm/i915/intel_pm.c > @@ -2946,27 +2946,27 @@ static void intel_read_wm_latency(struct drm_i915_private *dev_priv, > } else if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) { > u64 sskpd = intel_uncore_read64(uncore, MCH_SSKPD); > > - wm[0] = (sskpd >> 56) & 0xFF; > + wm[0] = REG_FIELD_GET64(SSKPD_NEW_WM0_MASK_HSW, sskpd); > if (wm[0] == 0) > - wm[0] = sskpd & 0xF; > - wm[1] = (sskpd >> 4) & 0xFF; > - wm[2] = (sskpd >> 12) & 0xFF; > - wm[3] = (sskpd >> 20) & 0x1FF; > - wm[4] = (sskpd >> 32) & 0x1FF; > + wm[0] = REG_FIELD_GET64(SSKPD_OLD_WM0_MASK_HSW, sskpd); > + wm[1] = REG_FIELD_GET64(SSKPD_WM1_MASK_HSW, sskpd); > + wm[2] = REG_FIELD_GET64(SSKPD_WM2_MASK_HSW, sskpd); > + wm[3] = REG_FIELD_GET64(SSKPD_WM3_MASK_HSW, sskpd); > + wm[4] = REG_FIELD_GET64(SSKPD_WM4_MASK_HSW, sskpd); > } else if (DISPLAY_VER(dev_priv) >= 6) { > u32 sskpd = intel_uncore_read(uncore, MCH_SSKPD); > > - wm[0] = (sskpd >> SSKPD_WM0_SHIFT) & SSKPD_WM_MASK; > - wm[1] = (sskpd >> SSKPD_WM1_SHIFT) & SSKPD_WM_MASK; > - wm[2] = (sskpd >> SSKPD_WM2_SHIFT) & SSKPD_WM_MASK; > - wm[3] = (sskpd >> SSKPD_WM3_SHIFT) & SSKPD_WM_MASK; > + wm[0] = REG_FIELD_GET(SSKPD_WM0_MASK_SNB, sskpd); > + wm[1] = REG_FIELD_GET(SSKPD_WM1_MASK_SNB, sskpd); > + wm[2] = REG_FIELD_GET(SSKPD_WM2_MASK_SNB, sskpd); > + wm[3] = REG_FIELD_GET(SSKPD_WM3_MASK_SNB, sskpd); > } else if (DISPLAY_VER(dev_priv) >= 5) { > u32 mltr = intel_uncore_read(uncore, MLTR_ILK); > > /* ILK primary LP0 latency is 700 ns */ > wm[0] = 7; > - wm[1] = (mltr >> MLTR_WM1_SHIFT) & ILK_SRLT_MASK; > - wm[2] = (mltr >> MLTR_WM2_SHIFT) & ILK_SRLT_MASK; > + wm[1] = REG_FIELD_GET(MLTR_WM1_MASK, mltr); > + wm[2] = REG_FIELD_GET(MLTR_WM2_MASK, mltr); > } else { > MISSING_CASE(INTEL_DEVID(dev_priv)); > } -- Jani Nikula, Intel Open Source Graphics Center