mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Chiam, Evon" <evon.chiam@altera.com>
To: Dinh Nguyen <dinguyen@kernel.org>, 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 22:19:05 +0800	[thread overview]
Message-ID: <9c97aeac-566a-410b-aece-66d5b123d9d2@altera.com> (raw)
In-Reply-To: <7055515a-8892-4fc2-ab5c-81283e40e054@kernel.org>

Hi Dinh

On 11/9/2026 5:45 pm, Dinh Nguyen wrote:
> 
> 
> 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://nam10.safelinks.protection.outlook.com/? 
>> url=https%3A%2F%2Flore.kernel.org%2Fall%2Fcd16fb78-551e-4630-83d6- 
>> a44a68e2e411%40kernel.org%2F&data=05%7C02%7Cevon.chiam%40altera.com%7Cdbebcf1cfb6d48efd95808df0fe96622%7Cfbd72e03d4a54110adce614d51f2077a%7C0%7C0%7C639247167296589037%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=YZT4tZ2b6W3InI1ubN1thEAmAkSy%2BdJXpya1Nknlkjc%3D&reserved=0
>> 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://nam10.safelinks.protection.outlook.com/? 
>> url=https%3A%2F%2Flore.kernel.org%2Fall%2F802bf0d27fbff9b1ddb6b9fc323a1a9f8f050f74.1788512748.git.evon.chiam%40altera.com%2F&data=05%7C02%7Cevon.chiam%40altera.com%7Cdbebcf1cfb6d48efd95808df0fe96622%7Cfbd72e03d4a54110adce614d51f2077a%7C0%7C0%7C639247167296637660%7CUnknown%7CTWFpbGZsb3d8eyJFbXB0eU1hcGkiOnRydWUsIlYiOiIwLjAuMDAwMCIsIlAiOiJXaW4zMiIsIkFOIjoiTWFpbCIsIldUIjoyfQ%3D%3D%7C0%7C%7C%7C&sdata=vUgJ1yBU8%2FtFnCcXoP30XgamJOr1bhqrMNbqdt9BKTA%3D&reserved=0
>>
>>   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.
> 
I've split this change into a separate patch and sent it in v4:
https://lore.kernel.org/all/cover.1789123511.git.evon.chiam@altera.com/

> Dinh

Thanks and regards,
Evon

      reply	other threads:[~2026-09-11 14:19 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
2026-09-11 14:19   ` Chiam, Evon [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=9c97aeac-566a-410b-aece-66d5b123d9d2@altera.com \
    --to=evon.chiam@altera.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dinguyen@kernel.org \
    --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®