On Sat, Jan 21, 2023 at 12:29 AM Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> wrote: > > Hi Marek & Jagan > > On Fri, 20 Jan 2023 at 15:06, Marek Vasut <marex@xxxxxxx> wrote: > > > > On 1/20/23 15:41, Jagan Teki wrote: > > > Hi Fabio, > > > > Hello all, > > > > > On Fri, Jan 20, 2023 at 5:36 PM Fabio Estevam <festevam@xxxxxxxxx> wrote: > > >> > > >> Hi Jagan, > > >> > > >> On Thu, Jan 19, 2023 at 2:59 PM Jagan Teki <jagan@xxxxxxxxxxxxxxxxxxxx> wrote: > > >> > > >>> There are two patch series prior to this need to apply. > > >>> > > >>> https://patchwork.kernel.org/project/dri-devel/patch/20221212145745.15387-1-jagan@xxxxxxxxxxxxxxxxxxxx/ > > >>> https://patchwork.kernel.org/project/dri-devel/cover/20221212182923.29155-1-jagan@xxxxxxxxxxxxxxxxxxxx/ > > >> > > >> Would it make sense to re-submit these two patches as part of your series? > > > > > > The previous version's comment was to separate them from the DSIM series. > > > > Hmmmmm, seems like those first two patches got stuck. I fixed up the > > malformed Fixes: line (it was split across two lines and had angular > > brackets around it) and picked the first series via drm-misc-next . > > > > Can you send a subsequent patch to convert the DSIM_* macros to BIT() > > macro , since checkpatch --strict complains about it ? > > > > For the second series, you likely want a RB from Maxime Ripard and Dave > > Stevenson first about the probe order handling. > > Not sure what I can add for "Enable prepare_prev_first flag for > samsung-s6e panels" and "tc358764: Enable pre_enable_prev_first flag" > as I have no datasheet for those devices. > On the basis that they are wanting the DSI host to be in LP-11 before > prepare/pre_enable, then setting the flag is the right thing. More > than happy to say > Acked-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> > to those two (I can't find them quickly in my mail to respond directly). > > I have just sent a separate response on the third patch. Fyi: please check my answers on that thread. Thanks, Jagan.
