<div dir="ltr"><div>Sam,</div><div><br></div><div>I will look into this and send the patch ASAP.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Aug 12, 2020 at 12:47 AM Sam Ravnborg <<a href="mailto:sam@ravnborg.org">sam@ravnborg.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">Hi Vinay.<br>
<br>
> > <br>
> > If Laurent or others identify further things to improve we can take<br>
> > it in-tree.<br>
> <br>
> Just one thing, please see below.<br>
> <br>
> > > > >> +       d2l_write(tc->i2c, VTIM1, vtime1);<br>
> > > > >> +       d2l_write(tc->i2c, HTIM2, htime2);<br>
> > > > >> +       d2l_write(tc->i2c, VTIM2, vtime2);<br>
> > > > >> +<br>
> > > > >> +       d2l_write(tc->i2c, VFUEN, VFUEN_EN);<br>
> > > > >> +       d2l_write(tc->i2c, SYSRST, SYS_RST_LCD);<br>
> > > > >> +       d2l_write(tc->i2c, LVPHY0, LV_PHY0_PRBS_ON(4) | LV_PHY0_ND(6));<br>
> > > > >> +<br>
> > > > >> +       dev_dbg(tc->dev, "bus_formats %04x bpc %d\n",<br>
> > > > >> +               connector->display_info.bus_formats[0],<br>
> > > > >> +               tc->bpc);<br>
> > > > >> +       /*<br>
> > > > >> +        * Default hardware register settings of tc358775 configured<br>
> > > > >> +        * with MEDIA_BUS_FMT_RGB888_1X7X4_JEIDA jeida-24 format<br>
> > > > >> +        */<br>
> > > > >> +       if (connector->display_info.bus_formats[0] ==<br>
> > > > >> +               MEDIA_BUS_FMT_RGB888_1X7X4_SPWG) {<br>
> <br>
> This shouldn't come from the connector, but from the<br>
> drm_bridge_state.output_bus_cfg.format. The drm_bridge_funcs<br>
> .atomic_get_input_bus_fmts() operation likely needs to be implemented.<br>
<br>
I trust you will look into this and submit a patch on top of<br>
drm-misc-next.<br>
Please add a proper "Fixes:" tag identifying the commit that introduced<br>
this bug - in this case the commit introducing the driver.<br>
<br>
Do not hesitate to reply to all if you have any questions.<br>
We will help you if we can.<br>
<br>
        Sam<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">regards,<br>vinaysimha</div>