[PATCH] drm/amd/display: DSC passthrough is for DP-HDMI pcon (SST pcon).

[SHANMUGAM, SRINIVASAN]

[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 at amd.com> 
Sent: Monday, April 24, 2023 10:52 PM
To: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>; Pillai, Aurabindo <Aurabindo.Pillai at amd.com>
Cc: amd-gfx at lists.freedesktop.org
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 at amd.com>

> -----Original Message-----
> From: SHANMUGAM, SRINIVASAN <SRINIVASAN.SHANMUGAM at amd.com>
> Sent: Friday, April 21, 2023 9:36 PM
> To: Zuo, Jerry <Jerry.Zuo at amd.com>; Pillai, Aurabindo 
> <Aurabindo.Pillai at amd.com>
> Cc: amd-gfx at lists.freedesktop.org; SHANMUGAM, SRINIVASAN 
> <SRINIVASAN.SHANMUGAM at amd.com>; Zuo, Jerry <Jerry.Zuo at amd.com>
> 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 at amd.com>
> Cc: Aurabindo Pillai <aurabindo.pillai at amd.com>
> Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam at amd.com>
> ---
>  .../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