Devicetree
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@oss.nxp.com>
To: Francesco Dolcini <francesco@dolcini.it>
Cc: Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>, Frank Li <Frank.Li@nxp.com>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Francesco Dolcini <francesco.dolcini@toradex.com>,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 1/2] arm64: dts: freescale: imx95-toradex-osm-dev: Remove ETH_A PHY interrupt
Date: Thu, 15 Oct 2026 11:26:09 -0500	[thread overview]
Message-ID: <atD-oWQMKKTmXXwL@SMW015318> (raw)
In-Reply-To: <20261015073719.GA58156@francesco-nb>

On Thu, Oct 15, 2026 at 09:37:19AM +0200, Francesco Dolcini wrote:
> Hello Frank,
>
> On Wed, Oct 14, 2026 at 09:20:23AM -0500, Frank Li wrote:
> > On Wed, Oct 14, 2026 at 09:58:09AM +0200, Francesco Dolcini wrote:
> > > From: Francesco Dolcini <francesco.dolcini@toradex.com>
> > >
> > > GPIO_A7 is a boot strap on the OSM iMX95 module. The pull-up on the OSM
> > > Development Board Ethernet PHY interrupt line causes the module to enter
> > > USB loader mode instead of booting normally when the reset button is
> > > held for more than 0.5 seconds.
> > >
> > > The PHY interrupt connection has been removed from the carrier board,
> >
> > Look like it is hardware modification, can declear the version number here
> >
> > Since V_x.y, the PHY interrupt connection has been removed...
>
> Valid point. It deserves a better explanation.
>
>  1. There is no change in the actual board revision.
>  2. Moving from interrupt to polling works always, so it does not
>     require any change on the board.
>  3. On our board is possible to disconnect the interrupt just removing a
>     jumper. The actual board revision stay the same. This is a new
>     product, that is going to be generally available in a few days. We
>     are going to ship boards with such jumper removed to
>     solve the i.MX95 strap option issue on reset.

Please add this information at commit message, otherwise, it will be strange,
why wire trace suddently route to HEADER from irq line.

Frank

>  4. Future board revisions might completely disconnect the ethernet
>     interrupt, removing the actual connection from the schematics. Given
>     we work in polling (after this patch is merged), this will be a
>     transparent change that will not create any kind of software
>     incompatibility.
>
>
> Should I incorporate this detailed information in the commit message and
> send a v2 or the current commit message is fine ?
>
> Francesco
>

  reply	other threads:[~2026-10-15 16:26 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-14  7:58 [PATCH v1 0/2] arm64: dts: freescale: Remove Toradex OSM Development Board ETH_A PHY interrupt Francesco Dolcini
2026-10-14  7:58 ` [PATCH v1 1/2] arm64: dts: freescale: imx95-toradex-osm-dev: Remove " Francesco Dolcini
2026-10-14 14:20   ` Frank Li
2026-10-15  7:37     ` Francesco Dolcini
2026-10-15 16:26       ` Frank Li [this message]
2026-10-14  7:58 ` [PATCH v1 2/2] arm64: dts: freescale: imx93-imx91-toradex-osm-dev: " Francesco Dolcini

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:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

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

  git send-email \
    --in-reply-to=atD-oWQMKKTmXXwL@SMW015318 \
    --to=frank.li@oss.nxp.com \
    --cc=Frank.Li@nxp.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=francesco.dolcini@toradex.com \
    --cc=francesco@dolcini.it \
    --cc=imx@lists.linux.dev \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=s.hauer@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* 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