mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Dinh Nguyen <dinguyen@kernel.org>
To: evon.chiam@altera.com, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3] arm64: dts: altera: Move all SoCFPGA files to dts/altera
Date: Fri, 11 Sep 2026 04:45:19 -0500	[thread overview]
Message-ID: <7055515a-8892-4fc2-ab5c-81283e40e054@kernel.org> (raw)
In-Reply-To: <47719884b157863b2253b415daa447b417cd3419.1789048358.git.evon.chiam@altera.com>



On 9/10/26 09:16, evon.chiam@altera.com wrote:
> From: "Chiam, Evon" <evon.chiam@altera.com>
> 
> The dts/intel/ directory contains two distinct families:
> - Altera SoCFPGA platforms (Stratix 10, Agilex, Agilex5, Agilex72,
>    Agilex7M, N5X)
> - Intel Keembay
> 
> All Altera SoCFPGA device trees already have a home in dts/altera/
> (Stratix 10 has been there since the directory was introduced). Move
> the remaining SoCFPGA DTS/DTSI files from dts/intel/ to dts/altera/
> to consolidate all Altera SoCFPGA device trees under a single
> directory. Only the Keembay files remain in dts/intel/.
> 
> Also drop the stale SoCFPGA MAINTAINERS path for
> arch/arm64/boot/dts/intel/. Keembay DT files there remain covered
> by ARM/INTEL KEEMBAY ARCHITECTURE
> 
> All inter-file includes are relative so no include paths need to
> change. The set of DTBs produced by dtbs_install is unchanged; they
> are simply built from dts/altera/ instead of dts/intel/.
> 
> No compatible strings or DTS/DTSI contents are modified; the device
> tree move is rename-only.
> 
> Link: https://lore.kernel.org/all/cd16fb78-551e-4630-83d6-a44a68e2e411@kernel.org/
> Signed-off-by: Chiam, Evon <evon.chiam@altera.com>
> ---
> Changes in v3:
> - Drop stale SoCFPGA MAINTAINERS F: entry for arch/arm64/boot/dts/intel/
> 
> Changes in v2:
> - Reworded commit subject to drop redundant "device trees" and phrase
>    the move as "to dts/altera"
> 
> v1 patch link:
> https://lore.kernel.org/all/802bf0d27fbff9b1ddb6b9fc323a1a9f8f050f74.1788512748.git.evon.chiam@altera.com/
> 
>   MAINTAINERS                                        |  1 -
>   arch/arm64/boot/dts/altera/Makefile                | 14 +++++++++++++-
>   .../boot/dts/{intel => altera}/socfpga_agilex.dtsi |  0
>   .../{intel => altera}/socfpga_agilex3_socdk.dts    |  0
>   .../dts/{intel => altera}/socfpga_agilex5.dtsi     |  0
>   .../{intel => altera}/socfpga_agilex5_socdk.dts    |  0
>   .../socfpga_agilex5_socdk_013b.dts                 |  0
>   .../socfpga_agilex5_socdk_modular.dts              |  0
>   .../socfpga_agilex5_socdk_nand.dts                 |  0
>   .../dts/{intel => altera}/socfpga_agilex72.dtsi    |  0
>   .../{intel => altera}/socfpga_agilex72_socdk.dts   |  0
>   .../{intel => altera}/socfpga_agilex7m_socdk.dts   |  0
>   .../dts/{intel => altera}/socfpga_agilex_n6000.dts |  0
>   .../dts/{intel => altera}/socfpga_agilex_socdk.dts |  0
>   .../socfpga_agilex_socdk_emmc.dts                  |  0
>   .../socfpga_agilex_socdk_nand.dts                  |  0
>   .../dts/{intel => altera}/socfpga_n5x_socdk.dts    |  0
>   arch/arm64/boot/dts/intel/Makefile                 | 12 ------------
>   18 files changed, 13 insertions(+), 14 deletions(-)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex.dtsi (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex3_socdk.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex5.dtsi (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex5_socdk.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex5_socdk_013b.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex5_socdk_modular.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex5_socdk_nand.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex72.dtsi (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex72_socdk.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex7m_socdk.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex_n6000.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex_socdk.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex_socdk_emmc.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_agilex_socdk_nand.dts (100%)
>   rename arch/arm64/boot/dts/{intel => altera}/socfpga_n5x_socdk.dts (100%)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 15011f5752a9..98f374ae7586 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -3590,7 +3590,6 @@ F:	arch/arm/boot/dts/intel/socfpga/
>   F:	arch/arm/configs/socfpga_defconfig
>   F:	arch/arm/mach-socfpga/
>   F:	arch/arm64/boot/dts/altera/
> -F:	arch/arm64/boot/dts/intel/
>   
Please separate this change into a separate patch. Otherwise, LGTM.

Dinh

  reply	other threads:[~2026-09-11  9:45 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-10 14:16 evon.chiam
2026-09-11  9:45 ` Dinh Nguyen [this message]
2026-09-11 14:19   ` Chiam, Evon

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=7055515a-8892-4fc2-ab5c-81283e40e054@kernel.org \
    --to=dinguyen@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=evon.chiam@altera.com \
    --cc=krzk+dt@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh@kernel.org \
    /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

all inboxes | Powered by JetHome®