[v3,00/14] arm64: dts: imx8mm-kontron: DT updates

  From: Frieder Schrempf <frieder.schrempf@kontron.de>

This patchset contains several improvements and updates for the devicetrees
for the i.MX8MM modules and boards by Kontron Electronics GmbH.

* HDMI/LVDS support for the BL/DL i.MX8MM
* Misc cleanups and small fixes
* OSM-S i.MX8MM module refactoring and update to latest HW revision

Changes in v3:
* Address Shawn's review comments for patch 1, 2 and 14 (thanks!)

Changes in v2:
* Rework DSI mux GPIO logic to be compatible with overlay
* Switch from 4 to 2 DSI lanes for LVDS bridge to fix non-working display

Frieder Schrempf (14):
  arm64: dts: imx8mm-kontron: Add support for display bridges on BL
    i.MX8MM
  arm64: dts: imx8mm-kontron: Add DL (Display-Line) overlay with LVDS
    support
  arm64: dts: imx8mm-kontron: Disable pullups for I2C signals on OSM-S
    i.MX8MM
  arm64: dts: imx8mm-kontron: Disable pullups for I2C signals on SL/BL
    i.MX8MM
  arm64: dts: imx8mm-kontron: Disable pullups for onboard UART signals
    on BL OSM-S board
  arm64: dts: imx8mm-kontron: Disable pullups for onboard UART signals
    on BL board
  arm64: dts: imx8mm-kontron: Disable pull resistors for SD card signals
    on BL OSM-S board
  arm64: dts: imx8mm-kontron: Disable pull resistors for SD card signals
    on BL board
  arm64: dts: imx8mm-kontron: Fix interrupt for RTC on OSM-S i.MX8MM
    module
  arm64: dts: imx8mm-kontron: Fix OSM-S devicetrees to match latest
    hardware
  arm64: dts: imx8mm-kontron: Disable uneffective PUE bit in SDIO IOMUX
  arm64: dts: imx8mm-kontron: Remove useless trickle-diode-disable from
    RTC node
  arm64: dts: imx8mm-kontron: Add I2C EEPROM on OSM-S Kontron i.MX8MM
  arm64: dts: imx8mm-kontron: Refactor devicetree for OSM-S module and
    board

 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
  

On Wed, Dec 06, 2023 at 03:41:03PM +0100, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
...
>  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?

- 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

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'

Shawn
  

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@kontron.de>
> ...
>>  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.
  

On 19.12.23 16:14, Frieder Schrempf wrote:
> 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@kontron.de>
>> ...
>>>  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.

Until I find out how to deal with the DTC warnings above, I sent a v4
without the display support patches to unblock the merging of the other
patches.