On 13.12.23 10:31, Shawn Guo wrote: > On Wed, Dec 06, 2023 at 03:41:03PM +0100, Frieder Schrempf wrote: >> From: Frieder Schrempf <frieder.schrempf@xxxxxxxxxx> > ... >> arch/arm64/boot/dts/freescale/Makefile | 4 + >> .../dts/freescale/imx8mm-kontron-bl-osm-s.dts | 295 +++------ >> .../boot/dts/freescale/imx8mm-kontron-bl.dts | 198 +++++- >> .../boot/dts/freescale/imx8mm-kontron-dl.dtso | 201 +++++++ >> .../dts/freescale/imx8mm-kontron-osm-s.dtsi | 567 +++++++++++++++++- >> .../boot/dts/freescale/imx8mm-kontron-sl.dtsi | 4 +- >> 6 files changed, 1035 insertions(+), 234 deletions(-) >> create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso > > A couple of things: > > - I have to use 'git am -3' apply some of the patches. Could you rebase > the series on imx/dt64 branch? Ok, sure! > > - There are a few DTC warnings that need to be addressed. > > ../arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts:235.34-238.7: Warning (graph_endpoint): /soc@0/bus@30800000/i2c@30a40000/bridge@2c/ports/port@0/endpoint: graph connection to node '/soc@0/bus@32c00000/dsi@32e10000/ports/port@1/endpoint' is not bidirectional I'm not aware of a proper solution for this warning. The thing is that there is a DSI switch on the board that is able to route the DSI output of the SoC to one of two bridges (HDMI or LVDS). On the driver side there is no representation of the switch. Therefore the result is a three-way connection for the graph ports which is reported by the binding check, but works fine as we only enable one of the target port nodes at a time and use GPIO hogs to select the switch path. Rob, Krzystztof: Would it be acceptable to ignore this until there is a way to represent this in the DT? Are there alternative solutions? Is there anything to model a graph link switch? > arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtbo: Warning (graph_port): /fragment@3: graph port node name should be 'port' > ../arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso:61.17-63.3: Warning (graph_endpoint): /fragment@3/__overlay__: graph endpoint node name should be 'endpoint' For these warnings it seems like I could work around them by creating two separate port nodes in the board DT and then only enable the proper one in the overlay.
