Re: [PATCH 26/27] Revert "drm/amd/display: Fix error in dmesg at boot"

[Paul Menzel <pmenzel@xxxxxxxxxxxxx>]

Dear Augustin,


Am 15.10.21 um 20:43 schrieb Agustin Gutierrez:
> This reverts commit 4e605d4b6a510f751b22df4d13829aefb8a0ccec.

Why?

(Do revert commits need a Signed-off-by line?)


Kind regards,

Paul


> ---
>   drivers/gpu/drm/amd/display/dc/core/dc_link.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_link.c b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> index 6db611f9f554..c01309a1cbf2 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_link.c
> @@ -1999,7 +1999,7 @@ void blank_all_dp_displays(struct dc *dc, bool hw_init)
>   
>   		if ((signal == SIGNAL_TYPE_EDP) ||
>   			(signal == SIGNAL_TYPE_DISPLAY_PORT)) {
> -			if (hw_init && signal != SIGNAL_TYPE_EDP && dc->links[i]->priv != NULL) {
> +			if (hw_init && signal != SIGNAL_TYPE_EDP) {
>   				/* DP 2.0 spec requires that we read LTTPR caps first */
>   				dp_retrieve_lttpr_cap(dc->links[i]);
>   				/* if any of the displays are lit up turn them off */
> @@ -2025,7 +2025,7 @@ void blank_all_dp_displays(struct dc *dc, bool hw_init)
>   				}
>   
>   				if (!dc->links[i]->wa_flags.dp_keep_receiver_powered ||
> -					(hw_init && signal != SIGNAL_TYPE_EDP && dc->links[i]->priv != NULL))
> +					(hw_init && signal != SIGNAL_TYPE_EDP))
>   					dp_receiver_power_ctrl(dc->links[i], false);
>   			}
>   		}