archive mirror
 help / color / mirror / Atom feed
From: Francesco Dolcini <>
To: Laurent Pinchart <>
Cc: Max Krummenacher <>,
	Max Krummenacher <>,
	Fabio Estevam <>,
	Krzysztof Kozlowski <>,
	Marcel Ziswiler <>,
	NXP Linux Team <>,
	Pengutronix Kernel Team <>,
	Philippe Schenker <>,
	Rob Herring <>,
	Sascha Hauer <>,
	Shawn Guo <>,,,
Subject: Re: [PATCH 1/2] arm64: dts: imx8mp-verdin: add dsi to hdmi functionality
Date: Fri, 2 Sep 2022 17:57:20 +0200	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

Hello Laurent,
answering here for both patches (1/2 and 2/2).

On Thu, Sep 01, 2022 at 09:07:49PM +0300, Laurent Pinchart wrote:
> On Thu, Sep 01, 2022 at 05:40:50PM +0200, Max Krummenacher wrote:
> > From: Max Krummenacher <>
> > 
> > Add the hdmi connector present on the dsi to hdmi adapter now
> > required by the upstream lontium bridge driver.
> > The dsi to hdmi adapter is enabled in an device tree overlay.
> Shouldn't the connector also be in the overlay ? There's certainly no
> physical HDMI connector on the i.MX8MP Verdin SoM :-)

Toradex DTS include and overlay files structure so far has been a little
bit different and not following the expectation you just stated here,
you can just check the current *toradex*dts* files and you'll see that there
is other stuff that is not strictly part of the module.

Copying from a previous email thread on a very similar discussion [0]
some of the reasons:

 - The SoM dtsi representing not only the functionality implemented into
   the SoM, but the whole connector pinout to the carrier makes very easy
   to just include a different som.dtsi in the carrier board dts and just
   switch SoM, for example from a colibri-imx6 to a colibri-imx7.
 - We avoid code duplication

This is working for us pretty well so far and the majority of the users
of ours modules rely on this structure, we would prefer not to change that.



linux-arm-kernel mailing list

  reply	other threads:[~2022-09-02 15:58 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-01 15:40 [PATCH 1/2] arm64: dts: imx8mp-verdin: add dsi to hdmi functionality Max Krummenacher
2022-09-01 15:40 ` [PATCH 2/2] arm64: dts: imx8mp-verdin: add dsi to lvds functionality Max Krummenacher
2022-09-01 18:10   ` Laurent Pinchart
2022-09-01 18:07 ` [PATCH 1/2] arm64: dts: imx8mp-verdin: add dsi to hdmi functionality Laurent Pinchart
2022-09-02 15:57   ` Francesco Dolcini [this message]
2022-09-03  0:24     ` Laurent Pinchart
2022-09-03 12:47       ` Francesco Dolcini
2022-09-05 19:26         ` Laurent Pinchart
2022-09-05 21:17           ` Francesco Dolcini
2022-09-05 22:03             ` Laurent Pinchart

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).