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. Dave
