On 03.11.2017 18:01, Andrey Gusakov wrote:
> Do not fail data rates higher than 2.7 and more than 2 lanes.
> Try to fall back to 2.7Gbps and 2 lanes.
> ---
> drivers/gpu/drm/bridge/tc358767.c | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/bridge/tc358767.c
> b/drivers/gpu/drm/bridge/tc358767.c
> index 8571cfd..d52cd41 100644
> --- a/drivers/gpu/drm/bridge/tc358767.c
> +++ b/drivers/gpu/drm/bridge/tc358767.c
> @@ -603,8 +603,15 @@ static int tc_get_display_props(struct tc_data *tc)
> ret = drm_dp_link_probe(&tc->aux, &tc->link.base);
> if (ret < 0)
> goto err_dpcd_read;
> - if ((tc->link.base.rate != 162000) && (tc->link.base.rate != 270000))
> - goto err_dpcd_inval;
> + if ((tc->link.base.rate != 162000) && (tc->link.base.rate != 270000)) {
> + dev_dbg(tc->dev, "Falling to 2.7 Gbps rate\n");
> + tc->link.base.rate = 270000;
> + }
> +
> + if (tc->link.base.num_lanes > 2) {
> + dev_dbg(tc->dev, "Falling to 2 lanes\n");
> + tc->link.base.num_lanes = 2;
> + }
>
> ret = drm_dp_dpcd_readb(&tc->aux, DP_MAX_DOWNSPREAD, tmp);
> if (ret < 0)
> @@ -637,9 +644,6 @@ static int tc_get_display_props(struct tc_data *tc)
> err_dpcd_read:
> dev_err(tc->dev, "failed to read DPCD: %d\n", ret);
> return ret;
> -err_dpcd_inval:
> - dev_err(tc->dev, "invalid DPCD\n");
> - return -EINVAL;
> }
>
> static int tc_set_video_mode(struct tc_data *tc, struct drm_display_mode
> *mode)
It looks like tc_get_display_props does more than its name suggests, ie
it gets display caps and matches with source caps, but this is little
bit different issue.
So:
Reviewed-by: Andrzej Hajda <a.ha...@samsung.com>
--
Regards
Andrzej
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
