[AMD Official Use Only - General]
Thanks a lot Jerry! and Aurabindo!, much appreciate for your help in reviewing this patch.
-----Original Message-----
From: Zuo, Jerry <Jerry.Zuo@xxxxxxx>
Sent: Monday, April 24, 2023 10:52 PM
To: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM@xxxxxxx>; Pillai, Aurabindo <Aurabindo.Pillai@xxxxxxx>
Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Subject: RE: [PATCH] drm/amd/display: DSC passthrough is for DP-HDMI pcon (SST pcon).
[AMD Official Use Only - General]
Reviewed-by: Fangzhi Zuo <Jerry.Zuo@xxxxxxx>
> -----Original Message-----
> From: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM@xxxxxxx>
> Sent: Friday, April 21, 2023 9:36 PM
> To: Zuo, Jerry <Jerry.Zuo@xxxxxxx>; Pillai, Aurabindo
> <Aurabindo.Pillai@xxxxxxx>
> Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; SHANMUGAM, SRINIVASAN
> <SRINIVASAN.SHANMUGAM@xxxxxxx>; Zuo, Jerry <Jerry.Zuo@xxxxxxx>
> Subject: [PATCH] drm/amd/display: DSC passthrough is for DP-HDMI pcon
> (SST pcon).
>
> if check over DSC passthrough is removed, as this is not for MST use
> case. It is for DP-HDMI pcon use case. sst pcon is detected as sst not
> mst. In sst pcon dsc passthrough message will not get below log
> printed
>
> 'Fixes: 8dc5bfdab0ecf ("drm/amd/display: Check & log if receiver
> supports MST, DSC & FEC.")'
> Suggested-by: Fangzhi Zuo <Jerry.Zuo@xxxxxxx>
> Cc: Aurabindo Pillai <aurabindo.pillai@xxxxxxx>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@xxxxxxx>
> ---
> .../drm/amd/display/dc/link/protocols/link_dp_capability.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git
> a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> index 84265dc66bba..577f74cc50fb 100644
> ---
> a/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability.c
> +++ b/drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_capability
> +++ .c
> @@ -1820,9 +1820,8 @@ static bool retrieve_link_cap(struct dc_link *link)
> str_yes_no(is_fec_supported));
> DC_LOG_DC("%s: DSC_Basic_Sink_Support: %s\n", __func__,
> str_yes_no(is_dsc_basic_supported));
> - if (link->dpcd_caps.is_mst_capable)
> - DC_LOG_DC("%s:
> DSC_Passthrough_Sink_Support: %s\n", __func__,
> -
> str_yes_no(is_dsc_passthrough_supported));
> + DC_LOG_DC("%s:
> DSC_Passthrough_Sink_Support: %s\n", __func__,
> + str_yes_no(is_dsc_passthrough_supported));
> }
> if (link->dpcd_caps.dongle_type != DISPLAY_DONGLE_NONE) {
> status = core_link_read_dpcd(
> --
> 2.25.1
