Skip to content

Commit 79b0003

Browse files
committed
Merge tag 'drm-fixes-2022-03-11' of git://anongit.freedesktop.org/drm/drm
Pull drm fixes from Dave Airlie: "As expected at this stage its pretty quiet, one sun4i mixer fix and one i915 display flicker fix: i915: - fix psr screen flicker sun4i: - mixer format fix" * tag 'drm-fixes-2022-03-11' of git://anongit.freedesktop.org/drm/drm: drm/sun4i: mixer: Fix P010 and P210 format numbers drm/i915/psr: Set "SF Partial Frame Enable" also on full update
2 parents dda64ea + 30eb13a commit 79b0003

3 files changed

Lines changed: 19 additions & 6 deletions

File tree

drivers/gpu/drm/i915/display/intel_psr.c

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1406,6 +1406,13 @@ static inline u32 man_trk_ctl_single_full_frame_bit_get(struct drm_i915_private
14061406
PSR2_MAN_TRK_CTL_SF_SINGLE_FULL_FRAME;
14071407
}
14081408

1409+
static inline u32 man_trk_ctl_partial_frame_bit_get(struct drm_i915_private *dev_priv)
1410+
{
1411+
return IS_ALDERLAKE_P(dev_priv) ?
1412+
ADLP_PSR2_MAN_TRK_CTL_SF_PARTIAL_FRAME_UPDATE :
1413+
PSR2_MAN_TRK_CTL_SF_PARTIAL_FRAME_UPDATE;
1414+
}
1415+
14091416
static void psr_force_hw_tracking_exit(struct intel_dp *intel_dp)
14101417
{
14111418
struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
@@ -1510,7 +1517,13 @@ static void psr2_man_trk_ctl_calc(struct intel_crtc_state *crtc_state,
15101517
{
15111518
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
15121519
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
1513-
u32 val = PSR2_MAN_TRK_CTL_ENABLE;
1520+
u32 val = 0;
1521+
1522+
if (!IS_ALDERLAKE_P(dev_priv))
1523+
val = PSR2_MAN_TRK_CTL_ENABLE;
1524+
1525+
/* SF partial frame enable has to be set even on full update */
1526+
val |= man_trk_ctl_partial_frame_bit_get(dev_priv);
15141527

15151528
if (full_update) {
15161529
/*
@@ -1530,7 +1543,6 @@ static void psr2_man_trk_ctl_calc(struct intel_crtc_state *crtc_state,
15301543
} else {
15311544
drm_WARN_ON(crtc_state->uapi.crtc->dev, clip->y1 % 4 || clip->y2 % 4);
15321545

1533-
val |= PSR2_MAN_TRK_CTL_SF_PARTIAL_FRAME_UPDATE;
15341546
val |= PSR2_MAN_TRK_CTL_SU_REGION_START_ADDR(clip->y1 / 4 + 1);
15351547
val |= PSR2_MAN_TRK_CTL_SU_REGION_END_ADDR(clip->y2 / 4 + 1);
15361548
}

drivers/gpu/drm/i915/i915_reg.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4829,6 +4829,7 @@ enum {
48294829
#define ADLP_PSR2_MAN_TRK_CTL_SU_REGION_START_ADDR(val) REG_FIELD_PREP(ADLP_PSR2_MAN_TRK_CTL_SU_REGION_START_ADDR_MASK, val)
48304830
#define ADLP_PSR2_MAN_TRK_CTL_SU_REGION_END_ADDR_MASK REG_GENMASK(12, 0)
48314831
#define ADLP_PSR2_MAN_TRK_CTL_SU_REGION_END_ADDR(val) REG_FIELD_PREP(ADLP_PSR2_MAN_TRK_CTL_SU_REGION_END_ADDR_MASK, val)
4832+
#define ADLP_PSR2_MAN_TRK_CTL_SF_PARTIAL_FRAME_UPDATE REG_BIT(31)
48324833
#define ADLP_PSR2_MAN_TRK_CTL_SF_SINGLE_FULL_FRAME REG_BIT(14)
48334834
#define ADLP_PSR2_MAN_TRK_CTL_SF_CONTINUOS_FULL_FRAME REG_BIT(13)
48344835

drivers/gpu/drm/sun4i/sun8i_mixer.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -111,10 +111,10 @@
111111
/* format 13 is semi-planar YUV411 VUVU */
112112
#define SUN8I_MIXER_FBFMT_YUV411 14
113113
/* format 15 doesn't exist */
114-
/* format 16 is P010 YVU */
115-
#define SUN8I_MIXER_FBFMT_P010_YUV 17
116-
/* format 18 is P210 YVU */
117-
#define SUN8I_MIXER_FBFMT_P210_YUV 19
114+
#define SUN8I_MIXER_FBFMT_P010_YUV 16
115+
/* format 17 is P010 YVU */
116+
#define SUN8I_MIXER_FBFMT_P210_YUV 18
117+
/* format 19 is P210 YVU */
118118
/* format 20 is packed YVU444 10-bit */
119119
/* format 21 is packed YUV444 10-bit */
120120

0 commit comments

Comments
 (0)