[PATCH] drm/amdgpu/display: remove VEGAM config option

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

 



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,
> 


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

  Powered by Linux