All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/amdgpu/display: remove VEGAM config option
@ 2018-05-16 14:12 Alex Deucher
       [not found] ` <20180516141201.29338-1-alexander.deucher-5C7GfCeVMHo@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Alex Deucher @ 2018-05-16 14:12 UTC (permalink / raw)
  To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Alex Deucher

Leftover from bringup.  No need to keep it around for
upstream.

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
 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,
-- 
2.13.6

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

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/amdgpu/display: remove VEGAM config option
       [not found] ` <20180516141201.29338-1-alexander.deucher-5C7GfCeVMHo@public.gmane.org>
@ 2018-05-16 14:19   ` Michel Dänzer
  2018-05-16 21:24   ` Harry Wentland
  1 sibling, 0 replies; 3+ messages in thread
From: Michel Dänzer @ 2018-05-16 14:19 UTC (permalink / raw)
  To: Alex Deucher; +Cc: Alex Deucher, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

On 2018-05-16 04:12 PM, Alex Deucher wrote:
> Leftover from bringup.  No need to keep it around for
> upstream.
> 
> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

Nice, thanks.

Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] drm/amdgpu/display: remove VEGAM config option
       [not found] ` <20180516141201.29338-1-alexander.deucher-5C7GfCeVMHo@public.gmane.org>
  2018-05-16 14:19   ` Michel Dänzer
@ 2018-05-16 21:24   ` Harry Wentland
  1 sibling, 0 replies; 3+ messages in thread
From: Harry Wentland @ 2018-05-16 21:24 UTC (permalink / raw)
  To: Alex Deucher, amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW; +Cc: Alex Deucher

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@amd.com>

Reviewed-by: Harry Wentland <harry.wentland@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,
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2018-05-16 21:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-16 14:12 [PATCH] drm/amdgpu/display: remove VEGAM config option Alex Deucher
     [not found] ` <20180516141201.29338-1-alexander.deucher-5C7GfCeVMHo@public.gmane.org>
2018-05-16 14:19   ` Michel Dänzer
2018-05-16 21:24   ` Harry Wentland

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.