OSDN Git Service

drm/bridge: ti-sn65dsi86: Fix off-by-one error in clock choice
authorDouglas Anderson <dianders@chromium.org>
Tue, 5 May 2020 04:32:29 +0000 (21:32 -0700)
committerSam Ravnborg <sam@ravnborg.org>
Mon, 18 May 2020 18:29:57 +0000 (20:29 +0200)
If the rate in our table is _equal_ to the rate we want then it's OK
to pick it.  It doesn't need to be greater than the one we want.

Fixes: a095f15c00e2 ("drm/bridge: add support for sn65dsi86 bridge driver")
Signed-off-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Rob Clark <robdclark@gmail.com>
Signed-off-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20200504213225.1.I21646c7c37ff63f52ae6cdccc9bc829fbc3d9424@changeid
drivers/gpu/drm/bridge/ti-sn65dsi86.c

index 03a29c7..1855fb9 100644 (file)
@@ -517,7 +517,7 @@ static int ti_sn_bridge_calc_min_dp_rate_idx(struct ti_sn_bridge *pdata)
                                   1000 * pdata->dp_lanes * DP_CLK_FUDGE_DEN);
 
        for (i = 1; i < ARRAY_SIZE(ti_sn_bridge_dp_rate_lut) - 1; i++)
-               if (ti_sn_bridge_dp_rate_lut[i] > dp_rate_mhz)
+               if (ti_sn_bridge_dp_rate_lut[i] >= dp_rate_mhz)
                        break;
 
        return i;