[PATCH 3/3] drm/amd/display: convert to DCE IP version checking

Liu, Aaron Aaron.Liu at amd.com
Mon Jan 24 06:14:53 UTC 2022


[AMD Official Use Only]


Tim,

Please remove TAB in the beginning of an empty line and the SPACE in the end of a line.

With this fixed, series is:
Reviewed-by: Aaron Liu aaron.liu at amd.com<mailto:aaron.liu at amd.com>



--

Best Regards

Aaron Liu



> -----Original Message-----

> From: Huang, Tim <Tim.Huang at amd.com>

> Sent: Monday, January 24, 2022 11:21 AM

> To: amd-gfx at lists.freedesktop.org

> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; Huang, Ray

> <Ray.Huang at amd.com>; Liu, Aaron <Aaron.Liu at amd.com>; Huang, Tim

> <Tim.Huang at amd.com>

> Subject: [PATCH 3/3] drm/amd/display: convert to DCE IP version checking

>

> Use IP versions rather than asic_type to differentiate IP version specific

> features.

>

> Signed-off-by: Tim Huang <xiaohu.huang at amd.com<mailto:xiaohu.huang at amd.com>>

> ---

>  drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 12 +++++-------

>  1 file changed, 5 insertions(+), 7 deletions(-)

>

> 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 87299e62fe12..c967e1e344e4 100644

> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

> @@ -1119,19 +1119,17 @@ static int dm_dmub_hw_init(struct

> amdgpu_device *adev)

>          for (i = 0; i < fb_info->num_fb; ++i)

>                      hw_params.fb[i] = &fb_info->fb[i];

>

> -        switch (adev->asic_type) {

> -        case CHIP_YELLOW_CARP:

> -                    if (dc->ctx->asic_id.hw_internal_rev != YELLOW_CARP_A0) {

> -                                hw_params.dpia_supported = true;

> +       switch (adev->ip_versions[DCE_HWIP][0]) {

> +       case IP_VERSION(3, 1, 3): /* Only for this asic hw internal rev B0 */

> +                   hw_params.dpia_supported = true;

>  #if defined(CONFIG_DRM_AMD_DC_DCN)

> -                                hw_params.disable_dpia = dc-

> >debug.dpia_debug.bits.disable_dpia;

> +                   hw_params.disable_dpia = dc-

> >debug.dpia_debug.bits.disable_dpia;

>  #endif

> -                    }

>                      break;

>          default:

>                      break;

>          }

> -

> +

>          status = dmub_srv_hw_init(dmub_srv, &hw_params);

>          if (status != DMUB_STATUS_OK) {

>                      DRM_ERROR("Error initializing DMUB HW: %d\n", status);

> --

> 2.25.1


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20220124/e5d9c3a9/attachment-0001.htm>


More information about the amd-gfx mailing list