From: <Manikandan.M@microchip.com>
To: <sumit.garg@kernel.org>
Cc: <eugen.hristev@linaro.org>, <lukma@denx.de>, <seanga2@gmail.com>,
<sumit.garg@linaro.org>, <kever.yang@rock-chips.com>,
<marek.vasut+renesas@mailbox.org>, <jonas@kwiboo.se>,
<festevam@denx.de>, <Oliver.Gaskell@analog.com>,
<aford173@gmail.com>, <prasad.kummari@amd.com>,
<caleb.connolly@linaro.org>, <andre.przywara@arm.com>,
<neil.armstrong@linaro.org>, <mibodhi@gmail.com>,
<Nicolas.Ferre@microchip.com>, <Mihai.Sain@microchip.com>,
<sjg@chromium.org>, <u-boot@lists.denx.de>
Subject: Re: [PATCH v3 1/8] dt-bindings: clock: rename at91.h to at91-pmc-status.h
Date: Thu, 6 Mar 2025 10:39:28 +0000 [thread overview]
Message-ID: <02132f8e-79b2-4b47-b4d4-60bb9c9a793f@microchip.com> (raw)
In-Reply-To: <Z8lH93V9DUfdoiLd@sumit-X1>
Hi Sumit,
On 06/03/25 12:30 pm, Sumit Garg wrote:
> [Some people who received this message don't often get email from sumit.garg@kernel.org. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
>
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
>
> Hi Manikandan,
>
> On Wed, Mar 05, 2025 at 11:37:04AM +0530, Manikandan Muralidharan wrote:
>> Rename the include at91.h to at91-pmc-status.h to avoid conflicts with
>> the upstream bindings that has the same file and update the
>> relevant legacy SoC Device Trees to reflect this change.
>> This is useful while compiling the DT and driver of the new SoC files with
>> OF_UPSTREAM enabled.
>>
>> Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com>
>> ---
>> MAINTAINERS | 2 +-
>> arch/arm/dts/at91-sama7g5ek-u-boot.dtsi | 2 +-
>> arch/arm/dts/at91sam9260.dtsi | 2 +-
>> arch/arm/dts/at91sam9261.dtsi | 2 +-
>> arch/arm/dts/at91sam9263.dtsi | 2 +-
>> arch/arm/dts/at91sam9g45.dtsi | 2 +-
>> arch/arm/dts/at91sam9n12.dtsi | 2 +-
>> arch/arm/dts/at91sam9rl.dtsi | 2 +-
>> arch/arm/dts/at91sam9x5.dtsi | 2 +-
>> arch/arm/dts/sama5d3.dtsi | 2 +-
>> arch/arm/dts/sama5d3_mci2.dtsi | 2 +-
>> arch/arm/dts/sama5d3_tcb1.dtsi | 2 +-
>> arch/arm/dts/sama5d3_uart.dtsi | 2 +-
>> arch/arm/dts/sama5d4.dtsi | 2 +-
>> include/dt-bindings/clock/{at91.h => at91-pmc-status.h} | 0
>> 15 files changed, 14 insertions(+), 14 deletions(-)
>> rename include/dt-bindings/clock/{at91.h => at91-pmc-status.h} (100%)
>>
>
> AFAICS, include/dt-bindings/clock/at91.h is just a subset of
> dts/upstream/include/dt-bindings/clock/at91.h. If that's true then we
> should just be able to drop local include/dt-bindings/clock/at91.h from
> U-Boot tree. Won't that just work fine?
Yes, you're right. It's a subset of the upstream at91.h. However, we
must retain it due to dependencies with legacy SoC files that haven't
yet been migrated to OF_UPSTREAM. I believe migrating those files now
would introduce a significant number of changes w.r.t Device Tree files."
>
> -Sumit
>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 10f7f1fd180..d50841a4a41 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -464,7 +464,7 @@ F: drivers/memory/atmel-ebi.c
>> F: drivers/misc/microchip_flexcom.c
>> F: drivers/timer/atmel_tcb_timer.c
>> F: include/dt-bindings/clk/at91.h
>> -F: include/dt-bindings/clock/at91.h
>> +F: include/dt-bindings/clock/at91-pmc-status.h
>> F: include/dt-bindings/dma/at91.h
>> F: include/dt-bindings/mfd/at91-usart.h
>> F: include/dt-bindings/mfd/atmel-flexcom.h
>> diff --git a/arch/arm/dts/at91-sama7g5ek-u-boot.dtsi b/arch/arm/dts/at91-sama7g5ek-u-boot.dtsi
>> index 8b2e990de72..6e5ca0fb4ce 100644
>> --- a/arch/arm/dts/at91-sama7g5ek-u-boot.dtsi
>> +++ b/arch/arm/dts/at91-sama7g5ek-u-boot.dtsi
>> @@ -12,7 +12,7 @@
>>
>> #include "sama7g5-pinfunc.h"
>> #include <dt-bindings/reset/sama7g5-reset.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> chosen {
>> diff --git a/arch/arm/dts/at91sam9260.dtsi b/arch/arm/dts/at91sam9260.dtsi
>> index 8d496205523..4ccfe2b9e4b 100644
>> --- a/arch/arm/dts/at91sam9260.dtsi
>> +++ b/arch/arm/dts/at91sam9260.dtsi
>> @@ -12,7 +12,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9260 family SoC";
>> diff --git a/arch/arm/dts/at91sam9261.dtsi b/arch/arm/dts/at91sam9261.dtsi
>> index 65e0e4f0de0..0c7485bff98 100644
>> --- a/arch/arm/dts/at91sam9261.dtsi
>> +++ b/arch/arm/dts/at91sam9261.dtsi
>> @@ -10,7 +10,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9261 family SoC";
>> diff --git a/arch/arm/dts/at91sam9263.dtsi b/arch/arm/dts/at91sam9263.dtsi
>> index 55b79667564..e5d1cc04f5c 100644
>> --- a/arch/arm/dts/at91sam9263.dtsi
>> +++ b/arch/arm/dts/at91sam9263.dtsi
>> @@ -10,7 +10,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9263 family SoC";
>> diff --git a/arch/arm/dts/at91sam9g45.dtsi b/arch/arm/dts/at91sam9g45.dtsi
>> index 63a061354e4..5c79f33f48d 100644
>> --- a/arch/arm/dts/at91sam9g45.dtsi
>> +++ b/arch/arm/dts/at91sam9g45.dtsi
>> @@ -14,7 +14,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9G45 family SoC";
>> diff --git a/arch/arm/dts/at91sam9n12.dtsi b/arch/arm/dts/at91sam9n12.dtsi
>> index 84089837013..5034db0b617 100644
>> --- a/arch/arm/dts/at91sam9n12.dtsi
>> +++ b/arch/arm/dts/at91sam9n12.dtsi
>> @@ -12,7 +12,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9N12 SoC";
>> diff --git a/arch/arm/dts/at91sam9rl.dtsi b/arch/arm/dts/at91sam9rl.dtsi
>> index 3b99de21058..8fa8067709c 100644
>> --- a/arch/arm/dts/at91sam9rl.dtsi
>> +++ b/arch/arm/dts/at91sam9rl.dtsi
>> @@ -8,7 +8,7 @@
>>
>> #include "skeleton.dtsi"
>> #include <dt-bindings/pinctrl/at91.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> #include <dt-bindings/pwm/pwm.h>
>> diff --git a/arch/arm/dts/at91sam9x5.dtsi b/arch/arm/dts/at91sam9x5.dtsi
>> index 4c6d8b9bb65..1922c5f03e0 100644
>> --- a/arch/arm/dts/at91sam9x5.dtsi
>> +++ b/arch/arm/dts/at91sam9x5.dtsi
>> @@ -14,7 +14,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel AT91SAM9x5 family SoC";
>> diff --git a/arch/arm/dts/sama5d3.dtsi b/arch/arm/dts/sama5d3.dtsi
>> index 10d6e74586d..bb9290fba57 100644
>> --- a/arch/arm/dts/sama5d3.dtsi
>> +++ b/arch/arm/dts/sama5d3.dtsi
>> @@ -13,7 +13,7 @@
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> #include <dt-bindings/gpio/gpio.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> model = "Atmel SAMA5D3 family SoC";
>> diff --git a/arch/arm/dts/sama5d3_mci2.dtsi b/arch/arm/dts/sama5d3_mci2.dtsi
>> index 2d289ed7be8..05753edeb0f 100644
>> --- a/arch/arm/dts/sama5d3_mci2.dtsi
>> +++ b/arch/arm/dts/sama5d3_mci2.dtsi
>> @@ -9,7 +9,7 @@
>>
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> ahb {
>> diff --git a/arch/arm/dts/sama5d3_tcb1.dtsi b/arch/arm/dts/sama5d3_tcb1.dtsi
>> index 5be60b4f8b1..0d96fe77723 100644
>> --- a/arch/arm/dts/sama5d3_tcb1.dtsi
>> +++ b/arch/arm/dts/sama5d3_tcb1.dtsi
>> @@ -9,7 +9,7 @@
>>
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> aliases {
>> diff --git a/arch/arm/dts/sama5d3_uart.dtsi b/arch/arm/dts/sama5d3_uart.dtsi
>> index 6a547fdadcc..10f36eb3ac2 100644
>> --- a/arch/arm/dts/sama5d3_uart.dtsi
>> +++ b/arch/arm/dts/sama5d3_uart.dtsi
>> @@ -9,7 +9,7 @@
>>
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>>
>> / {
>> aliases {
>> diff --git a/arch/arm/dts/sama5d4.dtsi b/arch/arm/dts/sama5d4.dtsi
>> index 482cf03e61b..af5e9bd6dba 100644
>> --- a/arch/arm/dts/sama5d4.dtsi
>> +++ b/arch/arm/dts/sama5d4.dtsi
>> @@ -44,7 +44,7 @@
>> */
>>
>> #include "skeleton.dtsi"
>> -#include <dt-bindings/clock/at91.h>
>> +#include <dt-bindings/clock/at91-pmc-status.h>
>> #include <dt-bindings/dma/at91.h>
>> #include <dt-bindings/pinctrl/at91.h>
>> #include <dt-bindings/interrupt-controller/irq.h>
>> diff --git a/include/dt-bindings/clock/at91.h b/include/dt-bindings/clock/at91-pmc-status.h
>> similarity index 100%
>> rename from include/dt-bindings/clock/at91.h
>> rename to include/dt-bindings/clock/at91-pmc-status.h
>> --
>> 2.25.1
>>
--
Thanks and Regards,
Manikandan M.
next prev parent reply other threads:[~2025-03-06 10:39 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-05 6:07 [PATCH v3 0/8] Add support for sam9x7 SoC and SAM9X75 Curiosity board Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 1/8] dt-bindings: clock: rename at91.h to at91-pmc-status.h Manikandan Muralidharan
2025-03-06 7:00 ` Sumit Garg
2025-03-06 10:39 ` Manikandan.M [this message]
2025-03-07 6:16 ` Sumit Garg
2025-03-17 5:40 ` Manikandan.M
2025-03-07 13:19 ` Eugen Hristev
2025-03-05 6:07 ` [PATCH v3 2/8] clk: at91: sam9x60-pll: add support for core clock frequency inputs Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 3/8] clk: at91: sam9x60-pll: add support for HW PLL freq dividers Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 4/8] clk: at91: sam9x7: add pmc driver for sam9x7 SoC family Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 5/8] ARM: at91: Add sam9x7 soc Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 6/8] ARM: dts: at91: sam9x75_curiosity: add tweaks for sam9x75 curiosity board Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 7/8] board: sam9x75_curiosity: Add support for sam9x75 curiosity Manikandan Muralidharan
2025-03-05 6:07 ` [PATCH v3 8/8] configs: sam9x75_curiosity: Add initial mmc default config Manikandan Muralidharan
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=02132f8e-79b2-4b47-b4d4-60bb9c9a793f@microchip.com \
--to=manikandan.m@microchip.com \
--cc=Mihai.Sain@microchip.com \
--cc=Nicolas.Ferre@microchip.com \
--cc=Oliver.Gaskell@analog.com \
--cc=aford173@gmail.com \
--cc=andre.przywara@arm.com \
--cc=caleb.connolly@linaro.org \
--cc=eugen.hristev@linaro.org \
--cc=festevam@denx.de \
--cc=jonas@kwiboo.se \
--cc=kever.yang@rock-chips.com \
--cc=lukma@denx.de \
--cc=marek.vasut+renesas@mailbox.org \
--cc=mibodhi@gmail.com \
--cc=neil.armstrong@linaro.org \
--cc=prasad.kummari@amd.com \
--cc=seanga2@gmail.com \
--cc=sjg@chromium.org \
--cc=sumit.garg@kernel.org \
--cc=sumit.garg@linaro.org \
--cc=u-boot@lists.denx.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