All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frank Li <Frank.li@oss.nxp.com>
To: Marco Felsch <m.felsch@pengutronix.de>
Cc: Rob Herring <robh@kernel.org>,
	devicetree@vger.kernel.org, Conor Dooley <conor+dt@kernel.org>,
	imx@lists.linux.dev, Sascha Hauer <s.hauer@pengutronix.de>,
	Frank Li <Frank.Li@nxp.com>,
	linux-kernel@vger.kernel.org,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Fabio Estevam <festevam@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2] ARM: dts: nxp/imx6: Drop unused .dtsi
Date: Thu, 5 Nov 2026 10:41:59 -0600	[thread overview]
Message-ID: <auyx155Caf3kwT4h@SMW015318> (raw)
In-Reply-To: <vvsd6rchs6wff2n7zxtcarxcvbnuxd7fx632xogm6hrcxxxqsl@dp5dvof3qa7c>

On Thu, Nov 05, 2026 at 04:35:11PM +0100, Marco Felsch wrote:
> On 26-11-05 16:09, Marco Felsch wrote:
> > On 26-11-05 09:06, Rob Herring wrote:
> > > On Thu, Nov 5, 2026 at 9:00 AM Marco Felsch <m.felsch@pengutronix.de> wrote:
> > > >
> > > > On 26-11-05 08:49, Rob Herring (Arm) wrote:
> > > > > These .dtsi files are not included anywhere in the tree and can't be
> > > > > tested.
> > > > >
> > > > > Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
> > > > > ---
> > > > > v2:
> > > > >  - Rebased
> > > > >
> > > > > It's been nearly a year and these are still orphans. There was a promise
> > > > > to upstream something for the kontron ones, but that hasn't happened.
> > > >
> > > > That's unfortunate, but these includes are used at least by the barebox
> > > > bootloader [1,2], dropping these would cause a DT regresssion.
> > > >
> > > > Would it help to upstream [1,2]?
> > >
> > > You said you were going to do that on v1:
> >
> > Unfortunate no Kontron dev was chiming in. Will do it now!
>
> Okay, after checking the current linux source tree I noticed that
> Kontron did upstream the carrier board .dts files but in a different
> manner:
>
>  * arch/arm/boot/dts/nxp/imx/imx6dl-kontron-samx6i-ads2.dts
>  * arch/arm/boot/dts/nxp/imx/imx6q-kontron-samx6i-ads2.dts
>
> This means that barebox would need to adapt the includes. Can you please
> mention that the .dtsi are not used by linux mainline and instead a
> different approach was used?

So, Is it okay to pick this patch?

Frank

>
> Regards,
>   Marco
>
>
> >
> > Regards,
> >   Marco
> >
> > >
> > > >> I will add the som.dts, please don't remove the kontron dtsi files yet.
> >
> >
>

  reply	other threads:[~2026-11-05 16:42 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-11-05 14:49 [PATCH v2] ARM: dts: nxp/imx6: Drop unused .dtsi Rob Herring (Arm)
2026-11-05 14:59 ` sashiko-bot
2026-11-05 15:04   ` Rob Herring
2026-11-05 15:00 ` Marco Felsch
2026-11-05 15:06   ` Rob Herring
2026-11-05 15:09     ` Marco Felsch
2026-11-05 15:35       ` Marco Felsch
2026-11-05 16:41         ` Frank Li [this message]
2026-11-05 17:32           ` Marco Felsch

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=auyx155Caf3kwT4h@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=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=m.felsch@pengutronix.de \
    --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.