All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shawn Guo <shawnguo@kernel.org>
To: Marek Vasut <marek.vasut@mailbox.org>
Cc: "Rob Herring (Arm)" <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	imx@lists.linux.dev, Conor Dooley <conor+dt@kernel.org>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Pengutronix Kernel Team <kernel@pengutronix.de>
Subject: Re: [PATCH] arm64: dts: imx8mp: Update Data Modul i.MX8M Plus eDM SBC DT to rev.903
Date: Mon, 29 Dec 2025 11:45:21 +0800	[thread overview]
Message-ID: <aVH5Ua2g8WbpW3Ge@dragon> (raw)
In-Reply-To: <b61fc304-26e0-422f-bb31-303e7bcbb824@mailbox.org>

On Mon, Dec 29, 2025 at 03:53:53AM +0100, Marek Vasut wrote:
> On 12/29/25 3:42 AM, Shawn Guo wrote:
> > On Fri, Dec 26, 2025 at 11:24:29AM +0100, Marek Vasut wrote:
> > > On 12/27/24 4:18 PM, Rob Herring (Arm) wrote:
> > > 
> > > Hello Shawn,
> > > 
> > > > On Thu, 26 Dec 2024 18:13:27 +0100, Marek Vasut wrote:
> > > > > Update the DT to match newest Data Modul i.MX8M Plus eDM SBC rev.903
> > > > > board which implements significant changes. Keep some of the rev.900
> > > > > and rev.902 nodes in the DT so that a DTO can be used to support old
> > > > > rev.900 and rev.902 boards easily.
> > > 
> > > [...]
> > > 
> > > > arch/arm64/boot/dts/freescale/imx8mp-data-modul-edm-sbc.dtb: tpm@2e: compatible: 'oneOf' conditional failed, one must be fixed:
> > > > 	['st,st33tphf2ei2c', 'tcg,tpm-tis-i2c'] is too long
> > > > 	'st,st33tphf2ei2c' is not one of ['infineon,slb9673', 'nuvoton,npct75x', 'st,st33ktpm2xi2c']
> > > 
> > > This is addressed in bindings patch [1] . Can this patch and the bindings
> > > patch be applied ?
> > > 
> > > https://lore.kernel.org/all/20241226171124.83735-1-marex@denx.de/
> > 
> > Could you resend them?
> Why ? The emails are in lore/patchwork and the patches still apply just
> fine, don't they ?

A couple of reasons:

- The patches were sent one year ago.
- They should be in a single series, as binding change is a prerequisite of DTS one.

Shawn

      reply	other threads:[~2025-12-29  3:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-26 17:13 [PATCH] arm64: dts: imx8mp: Update Data Modul i.MX8M Plus eDM SBC DT to rev.903 Marek Vasut
2024-12-27 15:18 ` Rob Herring (Arm)
2025-12-26 10:24   ` Marek Vasut
2025-12-29  2:42     ` Shawn Guo
2025-12-29  2:53       ` Marek Vasut
2025-12-29  3:45         ` Shawn Guo [this message]

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=aVH5Ua2g8WbpW3Ge@dragon \
    --to=shawnguo@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=imx@lists.linux.dev \
    --cc=kernel@pengutronix.de \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=marek.vasut@mailbox.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.