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.