Commit bb187e93 authored by James Ausmus's avatar James Ausmus Committed by Paulo Zanoni

drm/i915/icl: DP_AUX_E is valid on ICL+

Add support for DP_AUX_E. Here we also introduce the bits for the AUX
power well E, however ICL power well support is still not enabled yet,
so the power well is not used.

Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Reviewed-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: default avatarJames Ausmus <james.ausmus@intel.com>
Signed-off-by: default avatarPaulo Zanoni <paulo.r.zanoni@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180612002512.29783-2-paulo.r.zanoni@intel.com
parent af1f1b81
...@@ -1016,6 +1016,7 @@ enum modeset_restore { ...@@ -1016,6 +1016,7 @@ enum modeset_restore {
#define DP_AUX_B 0x10 #define DP_AUX_B 0x10
#define DP_AUX_C 0x20 #define DP_AUX_C 0x20
#define DP_AUX_D 0x30 #define DP_AUX_D 0x30
#define DP_AUX_E 0x50
#define DP_AUX_F 0x60 #define DP_AUX_F 0x60
#define DDC_PIN_B 0x05 #define DDC_PIN_B 0x05
......
...@@ -2640,6 +2640,9 @@ gen8_de_irq_handler(struct drm_i915_private *dev_priv, u32 master_ctl) ...@@ -2640,6 +2640,9 @@ gen8_de_irq_handler(struct drm_i915_private *dev_priv, u32 master_ctl)
GEN9_AUX_CHANNEL_C | GEN9_AUX_CHANNEL_C |
GEN9_AUX_CHANNEL_D; GEN9_AUX_CHANNEL_D;
if (INTEL_GEN(dev_priv) >= 11)
tmp_mask |= ICL_AUX_CHANNEL_E;
if (IS_CNL_WITH_PORT_F(dev_priv) || if (IS_CNL_WITH_PORT_F(dev_priv) ||
INTEL_GEN(dev_priv) >= 11) INTEL_GEN(dev_priv) >= 11)
tmp_mask |= CNL_AUX_CHANNEL_F; tmp_mask |= CNL_AUX_CHANNEL_F;
...@@ -3921,6 +3924,9 @@ static void gen8_de_irq_postinstall(struct drm_i915_private *dev_priv) ...@@ -3921,6 +3924,9 @@ static void gen8_de_irq_postinstall(struct drm_i915_private *dev_priv)
de_pipe_masked |= GEN8_DE_PIPE_IRQ_FAULT_ERRORS; de_pipe_masked |= GEN8_DE_PIPE_IRQ_FAULT_ERRORS;
} }
if (INTEL_GEN(dev_priv) >= 11)
de_port_masked |= ICL_AUX_CHANNEL_E;
if (IS_CNL_WITH_PORT_F(dev_priv) || INTEL_GEN(dev_priv) >= 11) if (IS_CNL_WITH_PORT_F(dev_priv) || INTEL_GEN(dev_priv) >= 11)
de_port_masked |= CNL_AUX_CHANNEL_F; de_port_masked |= CNL_AUX_CHANNEL_F;
......
...@@ -5315,6 +5315,13 @@ enum { ...@@ -5315,6 +5315,13 @@ enum {
#define _DPD_AUX_CH_DATA4 (dev_priv->info.display_mmio_offset + 0x64320) #define _DPD_AUX_CH_DATA4 (dev_priv->info.display_mmio_offset + 0x64320)
#define _DPD_AUX_CH_DATA5 (dev_priv->info.display_mmio_offset + 0x64324) #define _DPD_AUX_CH_DATA5 (dev_priv->info.display_mmio_offset + 0x64324)
#define _DPE_AUX_CH_CTL (dev_priv->info.display_mmio_offset + 0x64410)
#define _DPE_AUX_CH_DATA1 (dev_priv->info.display_mmio_offset + 0x64414)
#define _DPE_AUX_CH_DATA2 (dev_priv->info.display_mmio_offset + 0x64418)
#define _DPE_AUX_CH_DATA3 (dev_priv->info.display_mmio_offset + 0x6441c)
#define _DPE_AUX_CH_DATA4 (dev_priv->info.display_mmio_offset + 0x64420)
#define _DPE_AUX_CH_DATA5 (dev_priv->info.display_mmio_offset + 0x64424)
#define _DPF_AUX_CH_CTL (dev_priv->info.display_mmio_offset + 0x64510) #define _DPF_AUX_CH_CTL (dev_priv->info.display_mmio_offset + 0x64510)
#define _DPF_AUX_CH_DATA1 (dev_priv->info.display_mmio_offset + 0x64514) #define _DPF_AUX_CH_DATA1 (dev_priv->info.display_mmio_offset + 0x64514)
#define _DPF_AUX_CH_DATA2 (dev_priv->info.display_mmio_offset + 0x64518) #define _DPF_AUX_CH_DATA2 (dev_priv->info.display_mmio_offset + 0x64518)
...@@ -7019,6 +7026,7 @@ enum { ...@@ -7019,6 +7026,7 @@ enum {
#define GEN8_DE_PORT_IMR _MMIO(0x44444) #define GEN8_DE_PORT_IMR _MMIO(0x44444)
#define GEN8_DE_PORT_IIR _MMIO(0x44448) #define GEN8_DE_PORT_IIR _MMIO(0x44448)
#define GEN8_DE_PORT_IER _MMIO(0x4444c) #define GEN8_DE_PORT_IER _MMIO(0x4444c)
#define ICL_AUX_CHANNEL_E (1 << 29)
#define CNL_AUX_CHANNEL_F (1 << 28) #define CNL_AUX_CHANNEL_F (1 << 28)
#define GEN9_AUX_CHANNEL_D (1 << 27) #define GEN9_AUX_CHANNEL_D (1 << 27)
#define GEN9_AUX_CHANNEL_C (1 << 26) #define GEN9_AUX_CHANNEL_C (1 << 26)
......
...@@ -155,7 +155,7 @@ enum aux_ch { ...@@ -155,7 +155,7 @@ enum aux_ch {
AUX_CH_B, AUX_CH_B,
AUX_CH_C, AUX_CH_C,
AUX_CH_D, AUX_CH_D,
_AUX_CH_E, /* does not exist */ AUX_CH_E, /* ICL+ */
AUX_CH_F, AUX_CH_F,
}; };
...@@ -196,6 +196,7 @@ enum intel_display_power_domain { ...@@ -196,6 +196,7 @@ enum intel_display_power_domain {
POWER_DOMAIN_AUX_B, POWER_DOMAIN_AUX_B,
POWER_DOMAIN_AUX_C, POWER_DOMAIN_AUX_C,
POWER_DOMAIN_AUX_D, POWER_DOMAIN_AUX_D,
POWER_DOMAIN_AUX_E,
POWER_DOMAIN_AUX_F, POWER_DOMAIN_AUX_F,
POWER_DOMAIN_AUX_IO_A, POWER_DOMAIN_AUX_IO_A,
POWER_DOMAIN_GMBUS, POWER_DOMAIN_GMBUS,
......
...@@ -1347,6 +1347,9 @@ static enum aux_ch intel_aux_ch(struct intel_dp *intel_dp) ...@@ -1347,6 +1347,9 @@ static enum aux_ch intel_aux_ch(struct intel_dp *intel_dp)
case DP_AUX_D: case DP_AUX_D:
aux_ch = AUX_CH_D; aux_ch = AUX_CH_D;
break; break;
case DP_AUX_E:
aux_ch = AUX_CH_E;
break;
case DP_AUX_F: case DP_AUX_F:
aux_ch = AUX_CH_F; aux_ch = AUX_CH_F;
break; break;
...@@ -1374,6 +1377,8 @@ intel_aux_power_domain(struct intel_dp *intel_dp) ...@@ -1374,6 +1377,8 @@ intel_aux_power_domain(struct intel_dp *intel_dp)
return POWER_DOMAIN_AUX_C; return POWER_DOMAIN_AUX_C;
case AUX_CH_D: case AUX_CH_D:
return POWER_DOMAIN_AUX_D; return POWER_DOMAIN_AUX_D;
case AUX_CH_E:
return POWER_DOMAIN_AUX_E;
case AUX_CH_F: case AUX_CH_F:
return POWER_DOMAIN_AUX_F; return POWER_DOMAIN_AUX_F;
default: default:
...@@ -1460,6 +1465,7 @@ static i915_reg_t skl_aux_ctl_reg(struct intel_dp *intel_dp) ...@@ -1460,6 +1465,7 @@ static i915_reg_t skl_aux_ctl_reg(struct intel_dp *intel_dp)
case AUX_CH_B: case AUX_CH_B:
case AUX_CH_C: case AUX_CH_C:
case AUX_CH_D: case AUX_CH_D:
case AUX_CH_E:
case AUX_CH_F: case AUX_CH_F:
return DP_AUX_CH_CTL(aux_ch); return DP_AUX_CH_CTL(aux_ch);
default: default:
...@@ -1478,6 +1484,7 @@ static i915_reg_t skl_aux_data_reg(struct intel_dp *intel_dp, int index) ...@@ -1478,6 +1484,7 @@ static i915_reg_t skl_aux_data_reg(struct intel_dp *intel_dp, int index)
case AUX_CH_B: case AUX_CH_B:
case AUX_CH_C: case AUX_CH_C:
case AUX_CH_D: case AUX_CH_D:
case AUX_CH_E:
case AUX_CH_F: case AUX_CH_F:
return DP_AUX_CH_DATA(aux_ch, index); return DP_AUX_CH_DATA(aux_ch, index);
default: default:
......
...@@ -128,6 +128,8 @@ intel_display_power_domain_str(enum intel_display_power_domain domain) ...@@ -128,6 +128,8 @@ intel_display_power_domain_str(enum intel_display_power_domain domain)
return "AUX_C"; return "AUX_C";
case POWER_DOMAIN_AUX_D: case POWER_DOMAIN_AUX_D:
return "AUX_D"; return "AUX_D";
case POWER_DOMAIN_AUX_E:
return "AUX_E";
case POWER_DOMAIN_AUX_F: case POWER_DOMAIN_AUX_F:
return "AUX_F"; return "AUX_F";
case POWER_DOMAIN_AUX_IO_A: case POWER_DOMAIN_AUX_IO_A:
......
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