Re: [PATCH v4 5/7] drm/tidss: Set bus_format correctly from bridge/connector

From: Francesco Dolcini <francesco@dolcini.it>
To: Nikhil Devshatwar <nikhil.nd@ti.com>
Cc: Nishanth Menon <nm@ti.com>, Jayesh Choudhary <j-choudhary@ti.com>,
	Aradhya Bhatia <a-bhatia1@ti.com>,
	Yuti Amonkar <yamonkar@cadence.com>,
	Swapnil Jakhade <sjakhade@cadence.com>,
	Sekhar Nori <nsekhar@ti.com>,
	dri-devel@lists.freedesktop.org,
	Tomi Valkeinen <tomi.valkeinen@ti.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Boris Brezillon <boris.brezillon@collabora.com>,
	Devarsh Thakkar <devarsht@ti.com>, Rahul T R <r-ravikumar@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>
Subject: Re: [PATCH v4 5/7] drm/tidss: Set bus_format correctly from bridge/connector
Date: Thu, 30 Mar 2023 19:17:40 +0200	[thread overview]
Message-ID: <ZCXENFdTY6g2rb0i@francesco-nb.int.toradex.com> (raw)
In-Reply-To: <20201204115030.04509092@collabora.com>

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@ti.com/

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@ti.com> 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