On Mon Nov 10, 2025 at 9:58 AM CET, Damon Ding wrote:
> Use existing DP helper API instead of analogix_dp_clock_recovery_ok()
> with the same function.
>
> Signed-off-by: Damon Ding <[email protected]>
> Tested-by: Marek Szyprowski <[email protected]>
> Reviewed-by: Dmitry Baryshkov <[email protected]>

> @@ -398,7 +385,7 @@ static int analogix_dp_process_clock_recovery(struct 
> analogix_dp_device *dp)
>       if (retval < 0)
>               return retval;
>
> -     if (analogix_dp_clock_recovery_ok(link_status, lane_count) == 0) {
> +     if (drm_dp_clock_recovery_ok(link_status, lane_count)) {

It took a while to realize that drm_dp_clock_recovery_ok() really does the
same thing as analogix_dp_clock_recovery_ok(), but it looks fine.

Reviewed-by: Luca Ceresoli <[email protected]>

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Reply via email to