[PATCH] drm/amd/display: Disable PSR-SU on Parade 0803 TCON again

[Mario Limonciello]

On 12/9/2023 14:08, Mario Limonciello wrote:
> When screen brightness is rapidly changed and PSR-SU is enabled the
> display hangs on panels with this TCON even on the latest DCN 3.1.4
> microcode (0x8002a81 at this time).
> 
> This was disabled previously as commit 072030b17830 ("drm/amd: Disable
> PSR-SU on Parade 0803 TCON") but reverted as commit 1e66a17ce546 ("Revert
> "drm/amd: Disable PSR-SU on Parade 0803 TCON"") in favor of testing for
> a new enough microcode (commit cd2e31a9ab93 ("drm/amd/display: Set minimum
> requirement for using PSR-SU on Phoenix")).
> 
> As hangs are still happening specifically with this TCON, disable PSR-SU
> again for it until it can be root caused.
> 
> Cc: stable at vger.kernel.org
> Cc: aaron.ma at canonical.com
> Cc: binli at gnome.org
> Cc: Marc Rossi <Marc.Rossi at amd.com>
> Cc: Hamza Mahfooz <Hamza.Mahfooz at amd.com>
> Signed-off-by: Mario Limonciello <mario.limonciello at amd.com>

Here is a bug associated with this to tag as well.

Link: https://bugs.launchpad.net/ubuntu/+source/linux/+bug/2046131

> ---
>   drivers/gpu/drm/amd/display/modules/power/power_helpers.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/display/modules/power/power_helpers.c b/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
> index a522a7c02911..1675314a3ff2 100644
> --- a/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
> +++ b/drivers/gpu/drm/amd/display/modules/power/power_helpers.c
> @@ -839,6 +839,8 @@ bool is_psr_su_specific_panel(struct dc_link *link)
>   				((dpcd_caps->sink_dev_id_str[1] == 0x08 && dpcd_caps->sink_dev_id_str[0] == 0x08) ||
>   				(dpcd_caps->sink_dev_id_str[1] == 0x08 && dpcd_caps->sink_dev_id_str[0] == 0x07)))
>   				isPSRSUSupported = false;
> +			else if (dpcd_caps->sink_dev_id_str[1] == 0x08 && dpcd_caps->sink_dev_id_str[0] == 0x03)
> +				isPSRSUSupported = false;
>   			else if (dpcd_caps->psr_info.force_psrsu_cap == 0x1)
>   				isPSRSUSupported = true;
>   		}