Hello, chiming in in this *old* email thread. Adding in copy a few more *@ti.com people recently involved in other tidss changes [1] [1] https://lore.kernel.org/all/20230125113529.13952-1-a-bhatia1@xxxxxx/ On Fri, Dec 04, 2020 at 11:50:30AM +0100, Boris Brezillon wrote: > On Tue, 1 Dec 2020 17:48:28 +0530 > Nikhil Devshatwar <nikhil.nd@xxxxxx> wrote: > > > Remove the old code to iterate over the bridge chain, as this is > > already done by the framework. > > The bridge state should have the negotiated bus format and flags. > > Use these from the bridge's state. > > If the bridge does not support format negotiation, error out > > and fail. > > That'd be even better if you implement the bridge interface instead of > the encoder one so we can get rid of the encoder_{helper}_funcs and use > drm_simple_encoder_init(). Did anything happened on this old discussion? I was working on a very similar change and after a while I realized about this thread. Thanks, Francesco