drm/amd/display: Enable dpia in dmub only for DCN31 B0
authorJude Shih <shenshih@amd.com>
Thu, 21 Oct 2021 14:00:13 +0000 (22:00 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 28 Oct 2021 18:26:59 +0000 (14:26 -0400)
[Why]
DMUB binary is common for both A0 and B0. Hence, driver should
notify FW about the support for DPIA in B0.

[How]
Added dpia_supported bit in dmub_fw_boot_options and will be set
only for B0.

Assign dpia_supported to true before dm_dmub_hw_init
in B0 case.

v2: fix build without CONFIG_DRM_AMD_DC_DCN (Alex)

Signed-off-by: Jude Shih <shenshih@amd.com>
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
drivers/gpu/drm/amd/display/dmub/dmub_srv.h
drivers/gpu/drm/amd/display/dmub/src/dmub_dcn31.c

index 6f980938fae9ac972d3ead038483589555a4d274..43e983e42c0f5b709b08e1f6b5dd9d14714056f6 100644 (file)
@@ -1017,6 +1017,7 @@ static int dm_dmub_hw_init(struct amdgpu_device *adev)
        const unsigned char *fw_inst_const, *fw_bss_data;
        uint32_t i, fw_inst_const_size, fw_bss_data_size;
        bool has_hw_support;
+       struct dc *dc = adev->dm.dc;
 
        if (!dmub_srv)
                /* DMUB isn't supported on the ASIC. */
@@ -1103,6 +1104,19 @@ 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;
+#if defined(CONFIG_DRM_AMD_DC_DCN)
+                       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);
index 6c4f0ada163f5cb80d60db1850cccaff3b402eef..717c0e572d2feb2b582f17afcb514b4ad8bd573c 100644 (file)
@@ -238,6 +238,7 @@ struct dmub_srv_hw_params {
        bool load_inst_const;
        bool skip_panel_power_sequence;
        bool disable_z10;
+       bool dpia_supported;
        bool disable_dpia;
 };
 
index 5df990277dd41ffd66a77e874db0812b6021e723..10ebf20eaa4159e687af78c1292d63a384271836 100644 (file)
@@ -338,6 +338,7 @@ void dmub_dcn31_enable_dmub_boot_options(struct dmub_srv *dmub, const struct dmu
        union dmub_fw_boot_options boot_options = {0};
 
        boot_options.bits.z10_disable = params->disable_z10;
+       boot_options.bits.dpia_supported = params->dpia_supported;
        boot_options.bits.enable_dpia = params->disable_dpia ? 0 : 1;
 
        boot_options.bits.sel_mux_phy_c_d_phy_f_g = (dmub->asic == DMUB_ASIC_DCN31B) ? 1 : 0;