drm/i915/tgl: Update north display hotplug detection to TGL connections
authorJosé Roberto de Souza <jose.souza@intel.com>
Thu, 25 Jul 2019 23:48:12 +0000 (16:48 -0700)
committerLucas De Marchi <lucas.demarchi@intel.com>
Tue, 30 Jul 2019 19:31:08 +0000 (12:31 -0700)
TGL has 3 combophys and 6 TC/TBT ports, so it has 2 more TC/TBT ports
than ICL and the PORT_C on TGL is a combophy.
So here adding a new hpd north table and function to detect long
pulse for TGL.

Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Anusha Srivatsa <anusha.srivatsa@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190713010940.17711-6-lucas.demarchi@intel.com
Link: https://patchwork.freedesktop.org/patch/msgid/20190725234813.27179-3-lucas.demarchi@intel.com
drivers/gpu/drm/i915/i915_irq.c
drivers/gpu/drm/i915/i915_reg.h

index 34527cdd9388361966c693bae1dd72e90da6c840..e43582be24e6446d1fdeb6275f58d7c62756d19d 100644 (file)
@@ -58,6 +58,8 @@
  * and related files, but that will be described in separate chapters.
  */
 
+typedef bool (*long_pulse_detect_func)(enum hpd_pin pin, u32 val);
+
 static const u32 hpd_ilk[HPD_NUM_PINS] = {
        [HPD_PORT_A] = DE_DP_A_HOTPLUG,
 };
@@ -135,6 +137,15 @@ static const u32 hpd_gen11[HPD_NUM_PINS] = {
        [HPD_PORT_F] = GEN11_TC4_HOTPLUG | GEN11_TBT4_HOTPLUG
 };
 
+static const u32 hpd_gen12[HPD_NUM_PINS] = {
+       [HPD_PORT_D] = GEN11_TC1_HOTPLUG | GEN11_TBT1_HOTPLUG,
+       [HPD_PORT_E] = GEN11_TC2_HOTPLUG | GEN11_TBT2_HOTPLUG,
+       [HPD_PORT_F] = GEN11_TC3_HOTPLUG | GEN11_TBT3_HOTPLUG,
+       [HPD_PORT_G] = GEN11_TC4_HOTPLUG | GEN11_TBT4_HOTPLUG,
+       [HPD_PORT_H] = GEN12_TC5_HOTPLUG | GEN12_TBT5_HOTPLUG,
+       [HPD_PORT_I] = GEN12_TC6_HOTPLUG | GEN12_TBT6_HOTPLUG
+};
+
 static const u32 hpd_icp[HPD_NUM_PINS] = {
        [HPD_PORT_A] = SDE_DDIA_HOTPLUG_ICP,
        [HPD_PORT_B] = SDE_DDIB_HOTPLUG_ICP,
@@ -1694,6 +1705,26 @@ static bool gen11_port_hotplug_long_detect(enum hpd_pin pin, u32 val)
        }
 }
 
+static bool gen12_port_hotplug_long_detect(enum hpd_pin pin, u32 val)
+{
+       switch (pin) {
+       case HPD_PORT_D:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC1);
+       case HPD_PORT_E:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC2);
+       case HPD_PORT_F:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC3);
+       case HPD_PORT_G:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC4);
+       case HPD_PORT_H:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC5);
+       case HPD_PORT_I:
+               return val & GEN11_HOTPLUG_CTL_LONG_DETECT(PORT_TC6);
+       default:
+               return false;
+       }
+}
+
 static bool bxt_port_hotplug_long_detect(enum hpd_pin pin, u32 val)
 {
        switch (pin) {
@@ -2881,6 +2912,16 @@ static void gen11_hpd_irq_handler(struct drm_i915_private *dev_priv, u32 iir)
        u32 pin_mask = 0, long_mask = 0;
        u32 trigger_tc = iir & GEN11_DE_TC_HOTPLUG_MASK;
        u32 trigger_tbt = iir & GEN11_DE_TBT_HOTPLUG_MASK;
+       long_pulse_detect_func long_pulse_detect;
+       const u32 *hpd;
+
+       if (INTEL_GEN(dev_priv) >= 12) {
+               long_pulse_detect = gen12_port_hotplug_long_detect;
+               hpd = hpd_gen12;
+       } else {
+               long_pulse_detect = gen11_port_hotplug_long_detect;
+               hpd = hpd_gen11;
+       }
 
        if (trigger_tc) {
                u32 dig_hotplug_reg;
@@ -2889,8 +2930,7 @@ static void gen11_hpd_irq_handler(struct drm_i915_private *dev_priv, u32 iir)
                I915_WRITE(GEN11_TC_HOTPLUG_CTL, dig_hotplug_reg);
 
                intel_get_hpd_pins(dev_priv, &pin_mask, &long_mask, trigger_tc,
-                                  dig_hotplug_reg, hpd_gen11,
-                                  gen11_port_hotplug_long_detect);
+                                  dig_hotplug_reg, hpd, long_pulse_detect);
        }
 
        if (trigger_tbt) {
@@ -2900,8 +2940,7 @@ static void gen11_hpd_irq_handler(struct drm_i915_private *dev_priv, u32 iir)
                I915_WRITE(GEN11_TBT_HOTPLUG_CTL, dig_hotplug_reg);
 
                intel_get_hpd_pins(dev_priv, &pin_mask, &long_mask, trigger_tbt,
-                                  dig_hotplug_reg, hpd_gen11,
-                                  gen11_port_hotplug_long_detect);
+                                  dig_hotplug_reg, hpd, long_pulse_detect);
        }
 
        if (pin_mask)
@@ -3928,9 +3967,11 @@ static void gen11_hpd_detection_setup(struct drm_i915_private *dev_priv)
 static void gen11_hpd_irq_setup(struct drm_i915_private *dev_priv)
 {
        u32 hotplug_irqs, enabled_irqs;
+       const u32 *hpd;
        u32 val;
 
-       enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd_gen11);
+       hpd = INTEL_GEN(dev_priv) >= 12 ? hpd_gen12 : hpd_gen11;
+       enabled_irqs = intel_hpd_enabled_irqs(dev_priv, hpd);
        hotplug_irqs = GEN11_DE_TC_HOTPLUG_MASK | GEN11_DE_TBT_HOTPLUG_MASK;
 
        val = I915_READ(GEN11_DE_HPD_IMR);
index f55dea63e6e56028f8ae5c1c298d24ba8f954e8f..e2f6634b65f40595b543617c1dcb7c87032793f2 100644 (file)
@@ -7461,21 +7461,29 @@ enum {
 #define GEN11_DE_HPD_IMR               _MMIO(0x44474)
 #define GEN11_DE_HPD_IIR               _MMIO(0x44478)
 #define GEN11_DE_HPD_IER               _MMIO(0x4447c)
+#define  GEN12_TC6_HOTPLUG                     (1 << 21)
+#define  GEN12_TC5_HOTPLUG                     (1 << 20)
 #define  GEN11_TC4_HOTPLUG                     (1 << 19)
 #define  GEN11_TC3_HOTPLUG                     (1 << 18)
 #define  GEN11_TC2_HOTPLUG                     (1 << 17)
 #define  GEN11_TC1_HOTPLUG                     (1 << 16)
 #define  GEN11_TC_HOTPLUG(tc_port)             (1 << ((tc_port) + 16))
-#define  GEN11_DE_TC_HOTPLUG_MASK              (GEN11_TC4_HOTPLUG | \
+#define  GEN11_DE_TC_HOTPLUG_MASK              (GEN12_TC6_HOTPLUG | \
+                                                GEN12_TC5_HOTPLUG | \
+                                                GEN11_TC4_HOTPLUG | \
                                                 GEN11_TC3_HOTPLUG | \
                                                 GEN11_TC2_HOTPLUG | \
                                                 GEN11_TC1_HOTPLUG)
+#define  GEN12_TBT6_HOTPLUG                    (1 << 5)
+#define  GEN12_TBT5_HOTPLUG                    (1 << 4)
 #define  GEN11_TBT4_HOTPLUG                    (1 << 3)
 #define  GEN11_TBT3_HOTPLUG                    (1 << 2)
 #define  GEN11_TBT2_HOTPLUG                    (1 << 1)
 #define  GEN11_TBT1_HOTPLUG                    (1 << 0)
 #define  GEN11_TBT_HOTPLUG(tc_port)            (1 << (tc_port))
-#define  GEN11_DE_TBT_HOTPLUG_MASK             (GEN11_TBT4_HOTPLUG | \
+#define  GEN11_DE_TBT_HOTPLUG_MASK             (GEN12_TBT6_HOTPLUG | \
+                                                GEN12_TBT5_HOTPLUG | \
+                                                GEN11_TBT4_HOTPLUG | \
                                                 GEN11_TBT3_HOTPLUG | \
                                                 GEN11_TBT2_HOTPLUG | \
                                                 GEN11_TBT1_HOTPLUG)