Commit 2624ee66 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915/psr: Use intel_de_rmw()

Replace some hand rolled RMW stuff with intel_de_rmw().
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20230411191429.29895-5-ville.syrjala@linux.intel.comReviewed-by: default avatarJouni Högander <jouni.hogander@intel.com>
parent 1d9582dc
...@@ -232,13 +232,11 @@ void intel_psr_irq_handler(struct intel_dp *intel_dp, u32 psr_iir) ...@@ -232,13 +232,11 @@ void intel_psr_irq_handler(struct intel_dp *intel_dp, u32 psr_iir)
transcoder_name(cpu_transcoder)); transcoder_name(cpu_transcoder));
if (DISPLAY_VER(dev_priv) >= 9) { if (DISPLAY_VER(dev_priv) >= 9) {
u32 val = intel_de_read(dev_priv, u32 val;
PSR_EVENT(cpu_transcoder));
bool psr2_enabled = intel_dp->psr.psr2_enabled;
intel_de_write(dev_priv, PSR_EVENT(cpu_transcoder), val = intel_de_rmw(dev_priv, PSR_EVENT(cpu_transcoder), 0, 0);
val);
psr_event_print(dev_priv, val, psr2_enabled); psr_event_print(dev_priv, val, intel_dp->psr.psr2_enabled);
} }
} }
...@@ -493,9 +491,8 @@ static void hsw_activate_psr1(struct intel_dp *intel_dp) ...@@ -493,9 +491,8 @@ static void hsw_activate_psr1(struct intel_dp *intel_dp)
if (DISPLAY_VER(dev_priv) >= 8) if (DISPLAY_VER(dev_priv) >= 8)
val |= EDP_PSR_CRC_ENABLE; val |= EDP_PSR_CRC_ENABLE;
val |= (intel_de_read(dev_priv, EDP_PSR_CTL(intel_dp->psr.transcoder)) & intel_de_rmw(dev_priv, EDP_PSR_CTL(intel_dp->psr.transcoder),
EDP_PSR_RESTORE_PSR_ACTIVE_CTX_MASK); ~EDP_PSR_RESTORE_PSR_ACTIVE_CTX_MASK, val);
intel_de_write(dev_priv, EDP_PSR_CTL(intel_dp->psr.transcoder), val);
} }
static u32 intel_psr2_get_tp_time(struct intel_dp *intel_dp) static u32 intel_psr2_get_tp_time(struct intel_dp *intel_dp)
...@@ -1342,19 +1339,16 @@ static void intel_psr_exit(struct intel_dp *intel_dp) ...@@ -1342,19 +1339,16 @@ static void intel_psr_exit(struct intel_dp *intel_dp)
if (intel_dp->psr.psr2_enabled) { if (intel_dp->psr.psr2_enabled) {
tgl_disallow_dc3co_on_psr2_exit(intel_dp); tgl_disallow_dc3co_on_psr2_exit(intel_dp);
val = intel_de_read(dev_priv,
EDP_PSR2_CTL(intel_dp->psr.transcoder)); val = intel_de_rmw(dev_priv, EDP_PSR2_CTL(intel_dp->psr.transcoder),
EDP_PSR2_ENABLE, 0);
drm_WARN_ON(&dev_priv->drm, !(val & EDP_PSR2_ENABLE)); drm_WARN_ON(&dev_priv->drm, !(val & EDP_PSR2_ENABLE));
val &= ~EDP_PSR2_ENABLE;
intel_de_write(dev_priv,
EDP_PSR2_CTL(intel_dp->psr.transcoder), val);
} else { } else {
val = intel_de_read(dev_priv, val = intel_de_rmw(dev_priv, EDP_PSR_CTL(intel_dp->psr.transcoder),
EDP_PSR_CTL(intel_dp->psr.transcoder)); EDP_PSR_ENABLE, 0);
drm_WARN_ON(&dev_priv->drm, !(val & EDP_PSR_ENABLE)); drm_WARN_ON(&dev_priv->drm, !(val & EDP_PSR_ENABLE));
val &= ~EDP_PSR_ENABLE;
intel_de_write(dev_priv,
EDP_PSR_CTL(intel_dp->psr.transcoder), val);
} }
intel_dp->psr.active = false; intel_dp->psr.active = false;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment