[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 15:56:53 UTC 2025
On Mon, Jan 06, 2025 at 04:36:26PM +0100, Marek Vasut wrote:
> On 1/6/25 8:05 AM, Laurent Pinchart wrote:
> > 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 ?
>
> Since this optional flag is added, that DT update series can be done
> separately.
It can, but I'd like to see it merged in the same time frame as this
series, so it should be posted (and reviewed and tested).
--
Regards,
Laurent Pinchart
More information about the dri-devel
mailing list