Hi Boris, On Wed, Dec 04, 2019 at 10:09:57AM +0100, Boris Brezillon wrote: > On Tue, 3 Dec 2019 20:19:24 +0200 Laurent Pinchart wrote: > > On Tue, Dec 03, 2019 at 03:15:04PM +0100, Boris Brezillon wrote: > > > This patch series aims at adding support for runtime bus-format > > > negotiation between all elements of the > > > 'encoder -> bridges -> connector/display' section of the pipeline. > > > > > > In order to support that, we need drm bridges to fully take part in the > > > atomic state validation process, which requires adding a > > > drm_bridge_state and a new drm_bridge_funcs.atomic_check() hook. > > > Once those basic building blocks are in place, we can add new hooks to > > > allow bus format negotiation (those are called just before > > > ->atomic_check()). The bus format selection is done at runtime by > > > testing all possible combinations across the whole bridge chain until > > > one is reported to work. > > > > > > No Major changes in this v4. I think I addressed all comments I got > > > from Neil and Laurent (thanks for the detailed reviews BTW). Note that > > > this version only contains core changes. Once those changes are merged > > > I'll send the imx/panel/lvds-codec specific bits. > > > > I think it would make sense to fast-track patches 01 to 07 (a bit > > selfishly as I would have a need for them in omapdrm :-)) > > Will do that end of this week. Thank you. > >, but starting > > from 08 I wonder if it wouldn't make more sense to merge them with the > > imx/panel/lvds-codec bits to have a user. > > I'd really like to see that happen early in this release cycle so we > can catch bugs before those commits reach Linus' tree, I have no issue merging all your patches early in the release cycle, including the imx/panel/lvds-codec bits. Given the rebase required for lvds-codec it may take a bit of time though, so I'm OK if you prefer merging the core first, but I'm not sure it will help catching bugs if it's unused :-) > and the > lvds-codec changes depend on your PR. Can we make sure this one is > merged just after the MW has closed and the drm-next -> drm-misc-next > back merge done just after that? I've sent the pull request already, as soon as Dave or Daniel handles it, you'll have the necessary dependencies. -- Regards, Laurent Pinchart
