Re: [PATCH 0/8] Add imx8mp video support

[Miquel Raynal <miquel.raynal@bootlin.com>]

Hi Michael,

> > > Thank you for adding those patches. We are working on mipi support
> > > here and some of the clock patches
> > > are there too. I will try to look and rebase our patchset
> > >
> > > https://patchwork.amarulasolutions.com/patch/3401/  
> >
> > Thanks for letting me know. Indeed there are a couple of conflicts.
> > - My patch 2 can easily be replaced by your approach.
> > - Regarding the power domain we had two different approaches, I
> >   didn't look in details, but again one or the other seems fine, I
> >   guess.
> > - The LDB driver in my series is new.
> > - Finally regarding the LCDIF changes, the approach taken on your
> >   side follows the somewhat too ad-hoc logic and would probably benefit
> >   from being migrated to the driver I propose, which does make use of
> >   the (DM) clock, power domain and DT API. I am really happy with the
> >   core helper I've added retrieving a device in front of a graph
> >   endpoint, which makes the whole "get panel/bridge" approach much more
> >   flexible and adapted to today's needs.
> >  
> 
> My idea is to work on top of your patches and try to migrate them and
> cover conflicts.
> What be helpful is to get some of our clock patches and include in
> your series and then I will try
> to move our work on it

Of course, no problem. I'll be ready to test them again on i.MX8MP.

Fabio reported several typos which are easy to fix, but for instance
the DM helper needs a test, so I 'll work on it and share it with you.

Do you have a rough timeline for the next iteration?

Thanks,
Miquèl