[PATCH] drm/amdgpu/display: remove VEGAM config option
Harry Wentland
harry.wentland at amd.com
Wed May 16 21:24:31 UTC 2018
On 2018-05-16 10:12 AM, Alex Deucher wrote:
> Leftover from bringup. No need to keep it around for
> upstream.
>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
Reviewed-by: Harry Wentland <harry.wentland at amd.com>
Harry
> ---
> drivers/gpu/drm/amd/display/Kconfig | 7 -------
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 ----
> drivers/gpu/drm/amd/display/dc/bios/command_table_helper.c | 2 --
> drivers/gpu/drm/amd/display/dc/bios/command_table_helper2.c | 2 --
> drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c | 4 ----
> drivers/gpu/drm/amd/display/dc/core/dc_resource.c | 4 ----
> drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c | 4 ----
> drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c | 2 --
> drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c | 2 --
> drivers/gpu/drm/amd/display/dc/i2caux/i2caux.c | 2 --
> drivers/gpu/drm/amd/display/dc/inc/dce_calcs.h | 2 --
> drivers/gpu/drm/amd/display/include/dal_asic_id.h | 6 +-----
> drivers/gpu/drm/amd/display/include/dal_types.h | 2 --
> 13 files changed, 1 insertion(+), 42 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 6dcec9c9126b..a0eef59e65ba 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -34,13 +34,6 @@ config DEBUG_KERNEL_DC
> if you want to hit
> kdgb_break in assert.
>
> -config DRM_AMD_DC_VEGAM
> - bool "VEGAM support"
> - depends on DRM_AMD_DC
> - help
> - Choose this option if you want to have
> - VEGAM support for display engine
> -
> config DRM_AMD_DC_VG20
> bool "Vega20 support"
> 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 6f5cb26b243c..6d0dc1fecb39 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -1514,9 +1514,7 @@ static int amdgpu_dm_initialize_drm_device(struct amdgpu_device *adev)
> case CHIP_POLARIS11:
> case CHIP_POLARIS10:
> case CHIP_POLARIS12:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case CHIP_VEGAM:
> -#endif
> case CHIP_VEGA10:
> case CHIP_VEGA12:
> case CHIP_VEGA20:
> @@ -1710,9 +1708,7 @@ static int dm_early_init(void *handle)
> adev->mode_info.plane_type = dm_plane_type_default;
> break;
> case CHIP_POLARIS10:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case CHIP_VEGAM:
> -#endif
> adev->mode_info.num_crtc = 6;
> adev->mode_info.num_hpd = 6;
> adev->mode_info.num_dig = 6;
> diff --git a/drivers/gpu/drm/amd/display/dc/bios/command_table_helper.c b/drivers/gpu/drm/amd/display/dc/bios/command_table_helper.c
> index be066c49b984..253bbb1eea60 100644
> --- a/drivers/gpu/drm/amd/display/dc/bios/command_table_helper.c
> +++ b/drivers/gpu/drm/amd/display/dc/bios/command_table_helper.c
> @@ -51,9 +51,7 @@ bool dal_bios_parser_init_cmd_tbl_helper(
> return true;
>
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> *h = dal_cmd_tbl_helper_dce112_get_table();
> return true;
>
> 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 9b9e06995805..bbbcef566c55 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
> @@ -52,9 +52,7 @@ bool dal_bios_parser_init_cmd_tbl_helper2(
> return true;
>
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> *h = dal_cmd_tbl_helper_dce112_get_table2();
> return true;
> #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
> diff --git a/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c b/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c
> index 4ee3c26f7c13..2c4e8f0cb2dc 100644
> --- a/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c
> +++ b/drivers/gpu/drm/amd/display/dc/calcs/dce_calcs.c
> @@ -59,10 +59,8 @@ static enum bw_calcs_version bw_calcs_version_from_asic_id(struct hw_asic_id asi
> return BW_CALCS_VERSION_POLARIS10;
> if (ASIC_REV_IS_POLARIS11_M(asic_id.hw_internal_rev))
> return BW_CALCS_VERSION_POLARIS11;
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> if (ASIC_REV_IS_VEGAM(asic_id.hw_internal_rev))
> return BW_CALCS_VERSION_VEGAM;
> -#endif
> return BW_CALCS_VERSION_INVALID;
>
> case FAMILY_AI:
> @@ -2151,11 +2149,9 @@ void bw_calcs_init(struct bw_calcs_dceip *bw_dceip,
> dceip.mcifwr_all_surfaces_burst_time = bw_int_to_fixed(0); /* todo: this is a bug*/
> break;
> case BW_CALCS_VERSION_POLARIS10:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> /* TODO: Treat VEGAM the same as P10 for now
> * Need to tune the para for VEGAM if needed */
> case BW_CALCS_VERSION_VEGAM:
> -#endif
> vbios.memory_type = bw_def_gddr5;
> vbios.dram_channel_width_in_bits = 32;
> vbios.number_of_dram_channels = asic_id.vram_width / vbios.dram_channel_width_in_bits;
> 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 9eb731fb5251..345835ff58d1 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> @@ -79,10 +79,8 @@ enum dce_version resource_parse_asic_id(struct hw_asic_id asic_id)
> ASIC_REV_IS_POLARIS12_V(asic_id.hw_internal_rev)) {
> dc_version = DCE_VERSION_11_2;
> }
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> if (ASIC_REV_IS_VEGAM(asic_id.hw_internal_rev))
> dc_version = DCE_VERSION_11_22;
> -#endif
> break;
> case FAMILY_AI:
> dc_version = DCE_VERSION_12_0;
> @@ -129,9 +127,7 @@ struct resource_pool *dc_create_resource_pool(
> num_virtual_links, dc, asic_id);
> break;
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> res_pool = dce112_create_resource_pool(
> num_virtual_links, dc);
> break;
> diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c b/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> index 223db98a568a..0570e7e4d0a0 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce/dce_clock_source.c
> @@ -590,9 +590,7 @@ static uint32_t dce110_get_pix_clk_dividers(
> pll_settings, pix_clk_params);
> break;
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> case DCE_VERSION_12_0:
> #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
> case DCN_VERSION_1_0:
> @@ -982,9 +980,7 @@ static bool dce110_program_pix_clk(
>
> break;
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> case DCE_VERSION_12_0:
> #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
> case DCN_VERSION_1_0:
> 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 61fe484da1a0..0caee3523017 100644
> --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c
> +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_factory.c
> @@ -75,9 +75,7 @@ bool dal_hw_factory_init(
> return true;
> case DCE_VERSION_11_0:
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> dal_hw_factory_dce110_init(factory);
> return true;
> case DCE_VERSION_12_0:
> 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 910ae2b7bf64..55c707488541 100644
> --- a/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c
> +++ b/drivers/gpu/drm/amd/display/dc/gpio/hw_translate.c
> @@ -72,9 +72,7 @@ bool dal_hw_translate_init(
> case DCE_VERSION_10_0:
> case DCE_VERSION_11_0:
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> dal_hw_translate_dce110_init(translate);
> return true;
> case DCE_VERSION_12_0:
> diff --git a/drivers/gpu/drm/amd/display/dc/i2caux/i2caux.c b/drivers/gpu/drm/amd/display/dc/i2caux/i2caux.c
> index c3d7c320fdba..14dc8c94d862 100644
> --- a/drivers/gpu/drm/amd/display/dc/i2caux/i2caux.c
> +++ b/drivers/gpu/drm/amd/display/dc/i2caux/i2caux.c
> @@ -83,9 +83,7 @@ struct i2caux *dal_i2caux_create(
> case DCE_VERSION_8_3:
> return dal_i2caux_dce80_create(ctx);
> case DCE_VERSION_11_2:
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> case DCE_VERSION_11_22:
> -#endif
> return dal_i2caux_dce112_create(ctx);
> case DCE_VERSION_11_0:
> return dal_i2caux_dce110_create(ctx);
> diff --git a/drivers/gpu/drm/amd/display/dc/inc/dce_calcs.h b/drivers/gpu/drm/amd/display/dc/inc/dce_calcs.h
> index 933ea7a1e18b..eece165206f9 100644
> --- a/drivers/gpu/drm/amd/display/dc/inc/dce_calcs.h
> +++ b/drivers/gpu/drm/amd/display/dc/inc/dce_calcs.h
> @@ -43,9 +43,7 @@ enum bw_calcs_version {
> BW_CALCS_VERSION_POLARIS10,
> BW_CALCS_VERSION_POLARIS11,
> BW_CALCS_VERSION_POLARIS12,
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> BW_CALCS_VERSION_VEGAM,
> -#endif
> BW_CALCS_VERSION_STONEY,
> BW_CALCS_VERSION_VEGA10
> };
> 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 77d2856be9f6..6aeb5a2902c3 100644
> --- a/drivers/gpu/drm/amd/display/include/dal_asic_id.h
> +++ b/drivers/gpu/drm/amd/display/include/dal_asic_id.h
> @@ -86,6 +86,7 @@
> #define VI_POLARIS10_P_A0 80
> #define VI_POLARIS11_M_A0 90
> #define VI_POLARIS12_V_A0 100
> +#define VI_VEGAM_A0 110
>
> #define VI_UNKNOWN 0xFF
>
> @@ -98,14 +99,9 @@
> (eChipRev < VI_POLARIS11_M_A0))
> #define ASIC_REV_IS_POLARIS11_M(eChipRev) ((eChipRev >= VI_POLARIS11_M_A0) && \
> (eChipRev < VI_POLARIS12_V_A0))
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> -#define VI_VEGAM_A0 110
> #define ASIC_REV_IS_POLARIS12_V(eChipRev) ((eChipRev >= VI_POLARIS12_V_A0) && \
> (eChipRev < VI_VEGAM_A0))
> #define ASIC_REV_IS_VEGAM(eChipRev) (eChipRev >= VI_VEGAM_A0)
> -#else
> -#define ASIC_REV_IS_POLARIS12_V(eChipRev) (eChipRev >= VI_POLARIS12_V_A0)
> -#endif
>
> /* DCE11 */
> #define CZ_CARRIZO_A0 0x01
> diff --git a/drivers/gpu/drm/amd/display/include/dal_types.h b/drivers/gpu/drm/amd/display/include/dal_types.h
> index 5b1f8cef0c22..840142b65f8b 100644
> --- a/drivers/gpu/drm/amd/display/include/dal_types.h
> +++ b/drivers/gpu/drm/amd/display/include/dal_types.h
> @@ -40,9 +40,7 @@ enum dce_version {
> DCE_VERSION_10_0,
> DCE_VERSION_11_0,
> DCE_VERSION_11_2,
> -#if defined(CONFIG_DRM_AMD_DC_VEGAM)
> DCE_VERSION_11_22,
> -#endif
> DCE_VERSION_12_0,
> DCE_VERSION_MAX,
> DCN_VERSION_1_0,
>
More information about the amd-gfx
mailing list