[Intel-gfx] [PATCH] drm/i915/tgl+: Sanitize the DDI LANES/IO and AUX power domain names
Souza, Jose
jose.souza at intel.com
Tue Feb 23 13:56:12 UTC 2021
On Mon, 2021-02-22 at 23:04 +0200, Imre Deak wrote:
> In Bspec the TGL TypeC ports are TC1-6, the AUX power well request flags
> are USBC1-6/TBT1-6, so for clarity use these names in the port power
> domain names instead of the D-I terminology (which Bspec uses only for
> the ICL TypeC ports).
>
> No functional change.
>
> Cc: Souza Jose <jose.souza at intel.com>
> Signed-off-by: Imre Deak <imre.deak at intel.com>
> ---
> .../drm/i915/display/intel_display_power.c | 212 ++++++++----------
> .../drm/i915/display/intel_display_power.h | 32 +++
> 2 files changed, 130 insertions(+), 114 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.c b/drivers/gpu/drm/i915/display/intel_display_power.c
> index f00c1750febd..7e0eaa872350 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.c
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.c
> @@ -2886,24 +2886,24 @@ intel_display_power_put_mask_in_set(struct drm_i915_private *i915,
> BIT_ULL(POWER_DOMAIN_PIPE_B) | \
> BIT_ULL(POWER_DOMAIN_TRANSCODER_B) | \
> BIT_ULL(POWER_DOMAIN_PIPE_B_PANEL_FITTER) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_D_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_E_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_F_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_G_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_H_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_I_LANES) | \
> - BIT_ULL(POWER_DOMAIN_AUX_D) | \
> - BIT_ULL(POWER_DOMAIN_AUX_E) | \
> - BIT_ULL(POWER_DOMAIN_AUX_F) | \
> - BIT_ULL(POWER_DOMAIN_AUX_G) | \
> - BIT_ULL(POWER_DOMAIN_AUX_H) | \
> - BIT_ULL(POWER_DOMAIN_AUX_I) | \
> - BIT_ULL(POWER_DOMAIN_AUX_D_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_E_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_F_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_G_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_H_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_I_TBT) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC1) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC2) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC3) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC4) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC5) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC6) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC1) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC2) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC3) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC4) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC5) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC6) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT1) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT2) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT3) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT4) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT5) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT6) | \
> BIT_ULL(POWER_DOMAIN_VGA) | \
> BIT_ULL(POWER_DOMAIN_AUDIO) | \
> BIT_ULL(POWER_DOMAIN_INIT))
> @@ -2921,18 +2921,12 @@ intel_display_power_put_mask_in_set(struct drm_i915_private *i915,
> BIT_ULL(POWER_DOMAIN_AUX_C) | \
> BIT_ULL(POWER_DOMAIN_INIT))
>
>
> -#define TGL_DDI_IO_D_TC1_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_D_IO))
> -#define TGL_DDI_IO_E_TC2_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_E_IO))
> -#define TGL_DDI_IO_F_TC3_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_F_IO))
> -#define TGL_DDI_IO_G_TC4_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_G_IO))
> -#define TGL_DDI_IO_H_TC5_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_H_IO))
> -#define TGL_DDI_IO_I_TC6_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_I_IO))
> +#define TGL_DDI_IO_TC1_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC1)
> +#define TGL_DDI_IO_TC2_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC2)
> +#define TGL_DDI_IO_TC3_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC3)
> +#define TGL_DDI_IO_TC4_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC4)
> +#define TGL_DDI_IO_TC5_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC5)
> +#define TGL_DDI_IO_TC6_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_PORT_DDI_IO_TC6)
>
>
> #define TGL_AUX_A_IO_POWER_DOMAINS ( \
> BIT_ULL(POWER_DOMAIN_AUX_IO_A) | \
> @@ -2941,44 +2935,34 @@ intel_display_power_put_mask_in_set(struct drm_i915_private *i915,
> BIT_ULL(POWER_DOMAIN_AUX_B))
> #define TGL_AUX_C_IO_POWER_DOMAINS ( \
> BIT_ULL(POWER_DOMAIN_AUX_C))
> -#define TGL_AUX_D_TC1_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_D))
> -#define TGL_AUX_E_TC2_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_E))
> -#define TGL_AUX_F_TC3_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_F))
> -#define TGL_AUX_G_TC4_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_G))
> -#define TGL_AUX_H_TC5_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_H))
> -#define TGL_AUX_I_TC6_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_I))
> -#define TGL_AUX_D_TBT1_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_D_TBT))
> -#define TGL_AUX_E_TBT2_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_E_TBT))
> -#define TGL_AUX_F_TBT3_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_F_TBT))
> -#define TGL_AUX_G_TBT4_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_G_TBT))
> -#define TGL_AUX_H_TBT5_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_H_TBT))
> -#define TGL_AUX_I_TBT6_IO_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_I_TBT))
> +
> +#define TGL_AUX_IO_USBC1_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC1)
> +#define TGL_AUX_IO_USBC2_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC2)
> +#define TGL_AUX_IO_USBC3_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC3)
> +#define TGL_AUX_IO_USBC4_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC4)
> +#define TGL_AUX_IO_USBC5_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC5)
> +#define TGL_AUX_IO_USBC6_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_USBC6)
> +
> +#define TGL_AUX_IO_TBT1_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT1)
> +#define TGL_AUX_IO_TBT2_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT2)
> +#define TGL_AUX_IO_TBT3_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT3)
> +#define TGL_AUX_IO_TBT4_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT4)
> +#define TGL_AUX_IO_TBT5_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT5)
> +#define TGL_AUX_IO_TBT6_POWER_DOMAINS BIT_ULL(POWER_DOMAIN_AUX_TBT6)
>
>
> #define TGL_TC_COLD_OFF_POWER_DOMAINS ( \
> - BIT_ULL(POWER_DOMAIN_AUX_D) | \
> - BIT_ULL(POWER_DOMAIN_AUX_E) | \
> - BIT_ULL(POWER_DOMAIN_AUX_F) | \
> - BIT_ULL(POWER_DOMAIN_AUX_G) | \
> - BIT_ULL(POWER_DOMAIN_AUX_H) | \
> - BIT_ULL(POWER_DOMAIN_AUX_I) | \
> - BIT_ULL(POWER_DOMAIN_AUX_D_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_E_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_F_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_G_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_H_TBT) | \
> - BIT_ULL(POWER_DOMAIN_AUX_I_TBT) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC1) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC2) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC3) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC4) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC5) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC6) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT1) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT2) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT3) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT4) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT5) | \
> + BIT_ULL(POWER_DOMAIN_AUX_TBT6) | \
> BIT_ULL(POWER_DOMAIN_TC_COLD_OFF))
>
>
> #define RKL_PW_4_POWER_DOMAINS ( \
> @@ -2994,10 +2978,10 @@ intel_display_power_put_mask_in_set(struct drm_i915_private *i915,
> BIT_ULL(POWER_DOMAIN_AUDIO) | \
> BIT_ULL(POWER_DOMAIN_VGA) | \
> BIT_ULL(POWER_DOMAIN_TRANSCODER_B) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_D_LANES) | \
> - BIT_ULL(POWER_DOMAIN_PORT_DDI_E_LANES) | \
> - BIT_ULL(POWER_DOMAIN_AUX_D) | \
> - BIT_ULL(POWER_DOMAIN_AUX_E) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC1) | \
> + BIT_ULL(POWER_DOMAIN_PORT_DDI_LANES_TC2) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC1) | \
> + BIT_ULL(POWER_DOMAIN_AUX_USBC2) | \
> BIT_ULL(POWER_DOMAIN_INIT))
>
>
> /*
> @@ -4145,8 +4129,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> }
> },
> {
> - .name = "DDI D TC1 IO",
> - .domains = TGL_DDI_IO_D_TC1_POWER_DOMAINS,
> + .name = "DDI IO TC1",
> + .domains = TGL_DDI_IO_TC1_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4155,8 +4139,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "DDI E TC2 IO",
> - .domains = TGL_DDI_IO_E_TC2_POWER_DOMAINS,
> + .name = "DDI IO TC2",
> + .domains = TGL_DDI_IO_TC2_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4165,8 +4149,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "DDI F TC3 IO",
> - .domains = TGL_DDI_IO_F_TC3_POWER_DOMAINS,
> + .name = "DDI IO TC3",
> + .domains = TGL_DDI_IO_TC3_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4175,8 +4159,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "DDI G TC4 IO",
> - .domains = TGL_DDI_IO_G_TC4_POWER_DOMAINS,
> + .name = "DDI IO TC4",
> + .domains = TGL_DDI_IO_TC4_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4185,8 +4169,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "DDI H TC5 IO",
> - .domains = TGL_DDI_IO_H_TC5_POWER_DOMAINS,
> + .name = "DDI IO TC5",
> + .domains = TGL_DDI_IO_TC5_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4195,8 +4179,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "DDI I TC6 IO",
> - .domains = TGL_DDI_IO_I_TC6_POWER_DOMAINS,
> + .name = "DDI IO TC6",
> + .domains = TGL_DDI_IO_TC6_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4241,8 +4225,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX D TC1",
> - .domains = TGL_AUX_D_TC1_IO_POWER_DOMAINS,
> + .name = "AUX USBC1",
> + .domains = TGL_AUX_IO_USBC1_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4252,8 +4236,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX E TC2",
> - .domains = TGL_AUX_E_TC2_IO_POWER_DOMAINS,
> + .name = "AUX USBC2",
> + .domains = TGL_AUX_IO_USBC2_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4263,8 +4247,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX F TC3",
> - .domains = TGL_AUX_F_TC3_IO_POWER_DOMAINS,
> + .name = "AUX USBC3",
> + .domains = TGL_AUX_IO_USBC3_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4274,8 +4258,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX G TC4",
> - .domains = TGL_AUX_G_TC4_IO_POWER_DOMAINS,
> + .name = "AUX USBC4",
> + .domains = TGL_AUX_IO_USBC4_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4285,8 +4269,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX H TC5",
> - .domains = TGL_AUX_H_TC5_IO_POWER_DOMAINS,
> + .name = "AUX USBC5",
> + .domains = TGL_AUX_IO_USBC5_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4296,8 +4280,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX I TC6",
> - .domains = TGL_AUX_I_TC6_IO_POWER_DOMAINS,
> + .name = "AUX USBC6",
> + .domains = TGL_AUX_IO_USBC6_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4307,8 +4291,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX D TBT1",
> - .domains = TGL_AUX_D_TBT1_IO_POWER_DOMAINS,
> + .name = "AUX TBT1",
> + .domains = TGL_AUX_IO_TBT1_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4318,8 +4302,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX E TBT2",
> - .domains = TGL_AUX_E_TBT2_IO_POWER_DOMAINS,
> + .name = "AUX TBT2",
> + .domains = TGL_AUX_IO_TBT2_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4329,8 +4313,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX F TBT3",
> - .domains = TGL_AUX_F_TBT3_IO_POWER_DOMAINS,
> + .name = "AUX TBT3",
> + .domains = TGL_AUX_IO_TBT3_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4340,8 +4324,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX G TBT4",
> - .domains = TGL_AUX_G_TBT4_IO_POWER_DOMAINS,
> + .name = "AUX TBT4",
> + .domains = TGL_AUX_IO_TBT4_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4351,8 +4335,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX H TBT5",
> - .domains = TGL_AUX_H_TBT5_IO_POWER_DOMAINS,
> + .name = "AUX TBT5",
> + .domains = TGL_AUX_IO_TBT5_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4362,8 +4346,8 @@ static const struct i915_power_well_desc tgl_power_wells[] = {
> },
> },
> {
> - .name = "AUX I TBT6",
> - .domains = TGL_AUX_I_TBT6_IO_POWER_DOMAINS,
> + .name = "AUX TBT6",
> + .domains = TGL_AUX_IO_TBT6_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4471,8 +4455,8 @@ static const struct i915_power_well_desc rkl_power_wells[] = {
> }
> },
> {
> - .name = "DDI D TC1 IO",
> - .domains = TGL_DDI_IO_D_TC1_POWER_DOMAINS,
> + .name = "DDI IO TC1",
> + .domains = TGL_DDI_IO_TC1_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4481,8 +4465,8 @@ static const struct i915_power_well_desc rkl_power_wells[] = {
> },
> },
> {
> - .name = "DDI E TC2 IO",
> - .domains = TGL_DDI_IO_E_TC2_POWER_DOMAINS,
> + .name = "DDI IO TC2",
> + .domains = TGL_DDI_IO_TC2_POWER_DOMAINS,
> .ops = &hsw_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4511,8 +4495,8 @@ static const struct i915_power_well_desc rkl_power_wells[] = {
> },
> },
> {
> - .name = "AUX D TC1",
> - .domains = TGL_AUX_D_TC1_IO_POWER_DOMAINS,
> + .name = "AUX USBC1",
> + .domains = TGL_AUX_IO_USBC1_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> @@ -4521,8 +4505,8 @@ static const struct i915_power_well_desc rkl_power_wells[] = {
> },
> },
> {
> - .name = "AUX E TC2",
> - .domains = TGL_AUX_E_TC2_IO_POWER_DOMAINS,
> + .name = "AUX USBC2",
> + .domains = TGL_AUX_IO_USBC2_POWER_DOMAINS,
> .ops = &icl_aux_power_well_ops,
> .id = DISP_PW_ID_NONE,
> {
> diff --git a/drivers/gpu/drm/i915/display/intel_display_power.h b/drivers/gpu/drm/i915/display/intel_display_power.h
> index bc30c479be53..f3ca5d5c9778 100644
> --- a/drivers/gpu/drm/i915/display/intel_display_power.h
> +++ b/drivers/gpu/drm/i915/display/intel_display_power.h
> @@ -41,6 +41,14 @@ enum intel_display_power_domain {
> POWER_DOMAIN_PORT_DDI_G_LANES,
> POWER_DOMAIN_PORT_DDI_H_LANES,
> POWER_DOMAIN_PORT_DDI_I_LANES,
> +
> + POWER_DOMAIN_PORT_DDI_LANES_TC1 = POWER_DOMAIN_PORT_DDI_D_LANES, /* tgl+ */
To follow others it should be POWER_DOMAIN_PORT_DDI_TC1_LANES
> + POWER_DOMAIN_PORT_DDI_LANES_TC2,
> + POWER_DOMAIN_PORT_DDI_LANES_TC3,
> + POWER_DOMAIN_PORT_DDI_LANES_TC4,
> + POWER_DOMAIN_PORT_DDI_LANES_TC5,
> + POWER_DOMAIN_PORT_DDI_LANES_TC6,
> +
> POWER_DOMAIN_PORT_DDI_A_IO,
> POWER_DOMAIN_PORT_DDI_B_IO,
> POWER_DOMAIN_PORT_DDI_C_IO,
> @@ -50,6 +58,14 @@ enum intel_display_power_domain {
> POWER_DOMAIN_PORT_DDI_G_IO,
> POWER_DOMAIN_PORT_DDI_H_IO,
> POWER_DOMAIN_PORT_DDI_I_IO,
> +
> + POWER_DOMAIN_PORT_DDI_IO_TC1 = POWER_DOMAIN_PORT_DDI_D_IO, /* tgl+ */
Same as comment above: POWER_DOMAIN_PORT_DDI_TC1_IO
With this 2 renames, looks good to me.
Reviewed-by: José Roberto de Souza <jose.souza at intel.com>
> + POWER_DOMAIN_PORT_DDI_IO_TC2,
> + POWER_DOMAIN_PORT_DDI_IO_TC3,
> + POWER_DOMAIN_PORT_DDI_IO_TC4,
> + POWER_DOMAIN_PORT_DDI_IO_TC5,
> + POWER_DOMAIN_PORT_DDI_IO_TC6,
> +
> POWER_DOMAIN_PORT_DSI,
> POWER_DOMAIN_PORT_CRT,
> POWER_DOMAIN_PORT_OTHER,
> @@ -64,6 +80,14 @@ enum intel_display_power_domain {
> POWER_DOMAIN_AUX_G,
> POWER_DOMAIN_AUX_H,
> POWER_DOMAIN_AUX_I,
> +
> + POWER_DOMAIN_AUX_USBC1 = POWER_DOMAIN_AUX_D, /* tgl+ */
> + POWER_DOMAIN_AUX_USBC2,
> + POWER_DOMAIN_AUX_USBC3,
> + POWER_DOMAIN_AUX_USBC4,
> + POWER_DOMAIN_AUX_USBC5,
> + POWER_DOMAIN_AUX_USBC6,
> +
> POWER_DOMAIN_AUX_IO_A,
> POWER_DOMAIN_AUX_C_TBT,
> POWER_DOMAIN_AUX_D_TBT,
> @@ -72,6 +96,14 @@ enum intel_display_power_domain {
> POWER_DOMAIN_AUX_G_TBT,
> POWER_DOMAIN_AUX_H_TBT,
> POWER_DOMAIN_AUX_I_TBT,
> +
> + POWER_DOMAIN_AUX_TBT1 = POWER_DOMAIN_AUX_D_TBT, /* tgl+ */
> + POWER_DOMAIN_AUX_TBT2,
> + POWER_DOMAIN_AUX_TBT3,
> + POWER_DOMAIN_AUX_TBT4,
> + POWER_DOMAIN_AUX_TBT5,
> + POWER_DOMAIN_AUX_TBT6,
> +
> POWER_DOMAIN_GMBUS,
> POWER_DOMAIN_MODESET,
> POWER_DOMAIN_GT_IRQ,
More information about the Intel-gfx
mailing list