Re: [PATCH 3/3] drm/amd/display: Drop DCN1_01 guards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 5/14/19 1:49 PM, Harry Wentland wrote:
> 
> [WHY]
> These were only needed for bringup. They're not needed anymore.
> 
> Signed-off-by: Harry Wentland <harry.wentland@xxxxxxx>

Series is:

Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@xxxxxxx>

I think a lot of those DCN guards around checking ASIC revision aren't 
strictly necessary (like the one in the first patch) and it seems kind 
of inconsistent between the files for what gets checked or not.

This patch series seems fine to me though with the exception that 
there's not really a way for a user to get it back without patching 
their tree. Seems like that should almost be a configurable option for 
DC itself with the default to off for Raven.

Nicholas Kazlauskas

> ---
>   drivers/gpu/drm/amd/display/Kconfig           |  6 ------
>   .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  2 --
>   .../display/dc/bios/command_table_helper2.c   |  5 -----
>   .../gpu/drm/amd/display/dc/core/dc_resource.c |  4 ----
>   .../drm/amd/display/dc/dcn10/dcn10_hubbub.c   |  2 --
>   .../drm/amd/display/dc/dcn10/dcn10_resource.c | 19 -------------------
>   .../gpu/drm/amd/display/dc/gpio/hw_factory.c  |  4 ----
>   .../drm/amd/display/dc/gpio/hw_translate.c    |  5 -----
>   .../gpu/drm/amd/display/include/dal_asic_id.h |  4 ----
>   .../gpu/drm/amd/display/include/dal_types.h   |  2 --
>   10 files changed, 53 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 13a6ce9c8e94..ed654a76c76a 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -5,7 +5,6 @@ config DRM_AMD_DC
>          bool "AMD DC - Enable new display engine"
>          default y
>          select DRM_AMD_DC_DCN1_0 if X86 && !(KCOV_INSTRUMENT_ALL && KCOV_ENABLE_COMPARISONS)
> -       select DRM_AMD_DC_DCN1_01 if X86 && !(KCOV_INSTRUMENT_ALL && KCOV_ENABLE_COMPARISONS)
>          help
>            Choose this option if you want to use the new display engine
>            support for AMDGPU. This adds required support for Vega and
> @@ -16,11 +15,6 @@ config DRM_AMD_DC_DCN1_0
>          help
>            RV family support for display engine
> 
> -config DRM_AMD_DC_DCN1_01
> -       def_bool n
> -       help
> -         RV2 family for display engine
> -
>   config DEBUG_KERNEL_DC
>          bool "Enable kgdb break in DC"
>          depends on DRM_AMD_DC
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index 9f5e1e79ac8a..4a1755bce96c 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -666,10 +666,8 @@ static int load_dmcu_fw(struct amdgpu_device *adev)
>          case CHIP_RAVEN:
>                  if (ASICREV_IS_PICASSO(adev->external_rev_id))
>                          fw_name_dmcu = FIRMWARE_RAVEN_DMCU;
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>                  else if (ASICREV_IS_RAVEN2(adev->external_rev_id))
>                          fw_name_dmcu = FIRMWARE_RAVEN_DMCU;
> -#endif
>                  else
>                          return 0;
>                  break;
> diff --git a/drivers/gpu/drm/amd/display/dc/bios/command_table_helper2.c b/drivers/gpu/drm/amd/display/dc/bios/command_table_helper2.c
> index 8196f3bb10c7..53deba42007a 100644
> --- a/drivers/gpu/drm/amd/display/dc/bios/command_table_helper2.c
> +++ b/drivers/gpu/drm/amd/display/dc/bios/command_table_helper2.c
> @@ -57,11 +57,6 @@ bool dal_bios_parser_init_cmd_tbl_helper2(
>                  return true;
>   #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
>          case DCN_VERSION_1_0:
> -               *h = dal_cmd_tbl_helper_dce112_get_table2();
> -               return true;
> -#endif
> -
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          case DCN_VERSION_1_01:
>                  *h = dal_cmd_tbl_helper_dce112_get_table2();
>                  return true;
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> index 58ce7a6b914c..d9bfffb2c48d 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> @@ -93,10 +93,8 @@ enum dce_version resource_parse_asic_id(struct hw_asic_id asic_id)
>   #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
>          case FAMILY_RV:
>                  dc_version = DCN_VERSION_1_0;
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>                  if (ASICREV_IS_RAVEN2(asic_id.hw_internal_rev))
>                          dc_version = DCN_VERSION_1_01;
> -#endif
>                  break;
>   #endif
>          default:
> @@ -147,9 +145,7 @@ struct resource_pool *dc_create_resource_pool(struct dc  *dc,
> 
>   #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
>          case DCN_VERSION_1_0:
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          case DCN_VERSION_1_01:
> -#endif
>                  res_pool = dcn10_create_resource_pool(init_data, dc);
>                  break;
>   #endif
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> index 177247595974..bf978831bb0e 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c
> @@ -927,9 +927,7 @@ void hubbub1_construct(struct hubbub *hubbub,
>          hubbub1->masks = hubbub_mask;
> 
>          hubbub1->debug_test_index_pstate = 0x7;
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          if (ctx->dce_version == DCN_VERSION_1_01)
>                  hubbub1->debug_test_index_pstate = 0xB;
> -#endif
>   }
> 
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> index ddb020a53098..dc7cf3704252 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> @@ -152,9 +152,7 @@ enum dcn10_clk_src_array_id {
>          DCN10_CLK_SRC_PLL2,
>          DCN10_CLK_SRC_PLL3,
>          DCN10_CLK_SRC_TOTAL,
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          DCN101_CLK_SRC_TOTAL = DCN10_CLK_SRC_PLL3
> -#endif
>   };
> 
>   /* begin *********************
> @@ -522,7 +520,6 @@ static const struct resource_caps res_cap = {
>                  .num_ddc = 4,
>   };
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>   static const struct resource_caps rv2_res_cap = {
>                  .num_timing_generator = 3,
>                  .num_opp = 3,
> @@ -532,7 +529,6 @@ static const struct resource_caps rv2_res_cap = {
>                  .num_pll = 3,
>                  .num_ddc = 3,
>   };
> -#endif
> 
>   static const struct dc_plane_cap plane_cap = {
>          .type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> @@ -1270,11 +1266,9 @@ static bool construct(
> 
>          ctx->dc_bios->regs = &bios_regs;
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          if (ctx->dce_version == DCN_VERSION_1_01)
>                  pool->base.res_cap = &rv2_res_cap;
>          else
> -#endif
>                  pool->base.res_cap = &res_cap;
>          pool->base.funcs = &dcn10_res_pool_funcs;
> 
> @@ -1291,10 +1285,8 @@ static bool construct(
>          /* max pipe num for ASIC before check pipe fuses */
>          pool->base.pipe_count = pool->base.res_cap->num_timing_generator;
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          if (dc->ctx->dce_version == DCN_VERSION_1_01)
>                  pool->base.pipe_count = 3;
> -#endif
>          dc->caps.max_video_width = 3840;
>          dc->caps.max_downscale_ratio = 200;
>          dc->caps.i2c_speed_in_khz = 100;
> @@ -1327,26 +1319,17 @@ static bool construct(
>                                  CLOCK_SOURCE_COMBO_PHY_PLL2,
>                                  &clk_src_regs[2], false);
> 
> -#ifdef CONFIG_DRM_AMD_DC_DCN1_01
>          if (dc->ctx->dce_version == DCN_VERSION_1_0) {
>                  pool->base.clock_sources[DCN10_CLK_SRC_PLL3] =
>                                  dcn10_clock_source_create(ctx, ctx->dc_bios,
>                                          CLOCK_SOURCE_COMBO_PHY_PLL3,
>                                          &clk_src_regs[3], false);
>          }
> -#else
> -       pool->base.clock_sources[DCN10_CLK_SRC_PLL3] =
> -                       dcn10_clock_source_create(ctx, ctx->dc_bios,
> -                               CLOCK_SOURCE_COMBO_PHY_PLL3,
> -                               &clk_src_regs[3], false);
> -#endif
> 
>          pool->base.clk_src_count = DCN10_CLK_SRC_TOTAL;
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          if (dc->ctx->dce_version == DCN_VERSION_1_01)
>                  pool->base.clk_src_count = DCN101_CLK_SRC_TOTAL;
> -#endif
> 
>          pool->base.dp_clock_source =
>                          dcn10_clock_source_create(ctx, ctx->dc_bios,
> @@ -1386,7 +1369,6 @@ static bool construct(
>          memcpy(dc->dcn_ip, &dcn10_ip_defaults, sizeof(dcn10_ip_defaults));
>          memcpy(dc->dcn_soc, &dcn10_soc_defaults, sizeof(dcn10_soc_defaults));
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          if (dc->ctx->dce_version == DCN_VERSION_1_01) {
>                  struct dcn_soc_bounding_box *dcn_soc = dc->dcn_soc;
>                  struct dcn_ip_params *dcn_ip = dc->dcn_ip;
> @@ -1397,7 +1379,6 @@ static bool construct(
>                  dcn_soc->dram_clock_change_latency = 23;
>                  dcn_ip->max_num_dpp = 3;
>          }
> -#endif
>          if (ASICREV_IS_RV1_F0(dc->ctx->asic_id.hw_internal_rev)) {
>                  dc->dcn_soc->urgent_latency = 3;
>                  dc->debug.disable_dmcu = true;
> diff --git a/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c b/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c
> index c2028c4744a6..a610fae16280 100644
> --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c
> +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c
> @@ -84,10 +84,6 @@ bool dal_hw_factory_init(
>                  return true;
>   #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
>          case DCN_VERSION_1_0:
> -               dal_hw_factory_dcn10_init(factory);
> -               return true;
> -#endif
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          case DCN_VERSION_1_01:
>                  dal_hw_factory_dcn10_init(factory);
>                  return true;
> diff --git a/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c b/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c
> index 236ca28784a9..77615146b96e 100644
> --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c
> +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c
> @@ -84,11 +84,6 @@ bool dal_hw_translate_init(
>                  dal_hw_translate_dcn10_init(translate);
>                  return true;
>   #endif
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
> -       case DCN_VERSION_1_01:
> -               dal_hw_translate_dcn10_init(translate);
> -               return true;
> -#endif
> 
>          default:
>                  BREAK_TO_DEBUGGER();
> diff --git a/drivers/gpu/drm/amd/display/include/dal_asic_id.h b/drivers/gpu/drm/amd/display/include/dal_asic_id.h
> index 072d8d7debf5..63c3e77159d9 100644
> --- a/drivers/gpu/drm/amd/display/include/dal_asic_id.h
> +++ b/drivers/gpu/drm/amd/display/include/dal_asic_id.h
> @@ -131,11 +131,9 @@
>   #define INTERNAL_REV_RAVEN_A0             0x00    /* First spin of Raven */
>   #define RAVEN_A0 0x01
>   #define RAVEN_B0 0x21
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>   /* DCN1_01 */
>   #define PICASSO_A0 0x41
>   #define RAVEN2_A0 0x81
> -#endif
>   #define RAVEN1_F0 0xF0
>   #define RAVEN_UNKNOWN 0xFF
> 
> @@ -143,10 +141,8 @@
>   #define RAVEN1_F0 0xF0
>   #define ASICREV_IS_RV1_F0(eChipRev) ((eChipRev >= RAVEN1_F0) && (eChipRev < RAVEN_UNKNOWN))
> 
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>   #define ASICREV_IS_PICASSO(eChipRev) ((eChipRev >= PICASSO_A0) && (eChipRev < RAVEN2_A0))
>   #define ASICREV_IS_RAVEN2(eChipRev) ((eChipRev >= RAVEN2_A0) && (eChipRev < 0xF0))
> -#endif /* DCN1_01 */
> 
>   #define FAMILY_RV 142 /* DCN 1*/
> 
> diff --git a/drivers/gpu/drm/amd/display/include/dal_types.h b/drivers/gpu/drm/amd/display/include/dal_types.h
> index f5bd869d4320..dabdbc0999d4 100644
> --- a/drivers/gpu/drm/amd/display/include/dal_types.h
> +++ b/drivers/gpu/drm/amd/display/include/dal_types.h
> @@ -45,9 +45,7 @@ enum dce_version {
>          DCE_VERSION_12_1,
>          DCE_VERSION_MAX,
>          DCN_VERSION_1_0,
> -#if defined(CONFIG_DRM_AMD_DC_DCN1_01)
>          DCN_VERSION_1_01,
> -#endif /* DCN1_01 */
>          DCN_VERSION_MAX
>   };
> 
> --
> 2.21.0
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux