[PATCH v4 1/4] drm: bridge: dw_hdmi: Add flag to indicate output port is optional
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Mon Jan 6 07:05:14 UTC 2025
On Mon, Jan 06, 2025 at 03:48:52AM +0100, Marek Vasut wrote:
> On 1/6/25 12:22 AM, Laurent Pinchart wrote:
> > Hi Marek,
>
> Hi,
>
> > Thank you for the patch.
> >
> > On Sun, Jan 05, 2025 at 08:06:03PM +0100, Marek Vasut wrote:
> >> Add a flag meant purely to work around broken i.MX8MP DTs which enable
> >> HDMI but do not contain the HDMI connector node. This flag allows such
> >> DTs to work by creating the connector in the HDMI bridge driver. Do not
> >> use this flag, do not proliferate this flag, please fix your DTs and add
> >> the connector node this way:
> >>
> >> ```
> >> / {
> >> hdmi-connector {
> >> compatible = "hdmi-connector";
> >> label = "FIXME-Board-Specific-Connector-Label"; // Modify this
> >> type = "a";
> >>
> >> port {
> >> hdmi_connector_in: endpoint {
> >> remote-endpoint = <&hdmi_tx_out>;
> >> };
> >> };
> >> };
> >> };
> >>
> >> &hdmi_tx {
> >> ...
> >>
> >> ports {
> >> port at 1 {
> >> hdmi_tx_out: endpoint {
> >> remote-endpoint = <&hdmi_connector_in>;
> >> };
> >> };
> >> };
> >> };
> >> ```
> >
> > Are there any in-tree DT sources that use the old bindings ?
>
> See
> https://lore.kernel.org/dri-devel/AM7PR04MB704688150ACD5D209290246A98092@AM7PR04MB7046.eurprd04.prod.outlook.com/
Maybe I'm missing something obvious, but where is the patch series that
moves the DT sources mentioned in that mail thread to the new bindings ?
> The rest is fixed, thanks.
--
Regards,
Laurent Pinchart
More information about the dri-devel
mailing list