Re: [PATCH v4 1/4] drm: bridge: dw_hdmi: Add flag to indicate output port is optional

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@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@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/

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.



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux