drm/i915/pps: Introduce pps_delays_valid()

Add a small helper that determines if the PPS delays have been
initialized or not.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220510104242.6099-5-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>

diff --git a/drivers/gpu/drm/i915/display/intel_pps.c b/drivers/gpu/drm/i915/display/intel_pps.c
index bc56f0fe8e6d5c0db3e9b945481c9ed0404670f2..2ef6502703c29dd27484988a4218204e51fab1f9 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_pps.c
+++ b/drivers/gpu/drm/i915/display/intel_pps.c
@@ -1159,6 +1159,12 @@ intel_pps_verify_state(struct intel_dp *intel_dp)
        }
 }
 
+static bool pps_delays_valid(struct edp_power_seq *delays)
+{
+       return delays->t1_t3 || delays->t8 || delays->t9 ||
+               delays->t10 || delays->t11_t12;
+}
+
 static void pps_init_delays_cur(struct intel_dp *intel_dp,
                                struct edp_power_seq *cur)
 {
@@ -1230,7 +1236,7 @@ static void pps_init_delays(struct intel_dp *intel_dp)
        lockdep_assert_held(&dev_priv->pps_mutex);
 
        /* already initialized? */
-       if (final->t11_t12 != 0)
+       if (pps_delays_valid(final))
                return;
 
        pps_init_delays_cur(intel_dp, &cur);