Hello Aradhya, On Fri, Apr 14, 2023 at 01:19:38PM +0530, Aradhya Bhatia wrote: > On 30-Mar-23 22:47, Francesco Dolcini wrote: > > 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. > > > Nikhil has moved on from TI. > > I will be taking up this patch series and implement the changes > requested. Great! What I was working on is really about having the media bus format taken from the closest bridge, this is really required for proper functionality when the display is connected through a bridge. This [1] is what I was working on before realizing about this specific patch here, in case you want to have a look. Please keep me in CC, I can test/review patches. Francesco [1] https://github.com/dolcini/linux/commit/150b1390bd4122a77c873d87bf506024f9775755
