<div dir="ltr"><div>Robert,</div><div>I do not have the hardware to test this feature. Sorry for the late response.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Nov 18, 2021 at 8:20 PM Robert Foss <<a href="mailto:robert.foss@linaro.org">robert.foss@linaro.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">+ Zhen & Vinay<br>
<br>
This patch looks good in itself, but I would like to see a tested by<br>
tag. At the very least testing for regression in single-link LVDS but<br>
ideally some third party verification of this patch.<br>
<br>
On Wed, 10 Nov 2021 at 23:01, Jiri Vanek <<a href="mailto:jirivanek1@gmail.com" target="_blank">jirivanek1@gmail.com</a>> wrote:<br>
><br>
> Fixed wrong register shift for single/dual link LVDS output.<br>
><br>
> Signed-off-by: Jiri Vanek <<a href="mailto:jirivanek1@gmail.com" target="_blank">jirivanek1@gmail.com</a>><br>
> ---<br>
>  drivers/gpu/drm/bridge/tc358775.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/gpu/drm/bridge/tc358775.c b/drivers/gpu/drm/bridge/tc358775.c<br>
> index 2272adcc5b4a..1d6ec1baeff2 100644<br>
> --- a/drivers/gpu/drm/bridge/tc358775.c<br>
> +++ b/drivers/gpu/drm/bridge/tc358775.c<br>
> @@ -241,7 +241,7 @@ static inline u32 TC358775_LVCFG_PCLKDIV(uint32_t val)<br>
>  }<br>
><br>
>  #define TC358775_LVCFG_LVDLINK__MASK                         0x00000002<br>
> -#define TC358775_LVCFG_LVDLINK__SHIFT                        0<br>
> +#define TC358775_LVCFG_LVDLINK__SHIFT                        1<br>
>  static inline u32 TC358775_LVCFG_LVDLINK(uint32_t val)<br>
>  {<br>
>         return ((val) << TC358775_LVCFG_LVDLINK__SHIFT) &<br>
> --<br>
> 2.30.2<br>
><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">regards,<br>vinaysimha</div>