2023-05-29 06:52:13

by Claudiu Beznea

[permalink] [raw]
Subject: [PATCH v2 0/4] dt-bindings: timer: Microchip AT91 convert to YAML

Hi,

Series converts Microchip AT91 timers DT bindings to YAML. Along with it the
atmel,at91rm9200-wdt has been converted and introduced in this series
as it is referenced by atmel,at91rm9200-st.

Thank you,
Claudiu Beznea

Changes in v2:
- fixed compilation errors (sorry for that in v1)
- moved DTS patch as the 1st patch in series
- squash PIT and PIT64B into the same documentation file
- use Microchip in titles, descriptions and commit message
- in patch 4/4 remove oneOf from compatible section

Claudiu Beznea (4):
ARM: dts: at91: sama7g5: keep only microchip,sam9x60-pit64b compatible
for pit64b
dt-bindings: timer: atmel,at91sam9260-pit: convert to yaml
dt-bindings: watchdog: atmel,at91rm9200-wdt: convert to yaml
dt-bindings: timer: atmel,at91rm9200-st: convert to yaml

.../devicetree/bindings/arm/atmel-sysregs.txt | 21 ----
.../bindings/timer/atmel,at91rm9200-st.yaml | 65 +++++++++++++
.../bindings/timer/atmel,at91sam9260-pit.yaml | 96 +++++++++++++++++++
.../watchdog/atmel,at91rm9200-wdt.yaml | 29 ++++++
.../watchdog/atmel-at91rm9200-wdt.txt | 9 --
arch/arm/boot/dts/sama7g5.dtsi | 4 +-
6 files changed, 192 insertions(+), 32 deletions(-)
create mode 100644 Documentation/devicetree/bindings/timer/atmel,at91rm9200-st.yaml
create mode 100644 Documentation/devicetree/bindings/timer/atmel,at91sam9260-pit.yaml
create mode 100644 Documentation/devicetree/bindings/watchdog/atmel,at91rm9200-wdt.yaml
delete mode 100644 Documentation/devicetree/bindings/watchdog/atmel-at91rm9200-wdt.txt

--
2.34.1



2023-05-29 06:53:45

by Claudiu Beznea

[permalink] [raw]
Subject: [PATCH v2 1/4] ARM: dts: at91: sama7g5: keep only microchip,sam9x60-pit64b compatible for pit64b

SAM9X60 and SAMA7G5 PIT64Bs are 1 to 1 compatible. Thus, there is no
need for specific compatible string on SAMA7G5.

Signed-off-by: Claudiu Beznea <[email protected]>
---
arch/arm/boot/dts/sama7g5.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/boot/dts/sama7g5.dtsi b/arch/arm/boot/dts/sama7g5.dtsi
index 886b6209a71e..500b00508621 100644
--- a/arch/arm/boot/dts/sama7g5.dtsi
+++ b/arch/arm/boot/dts/sama7g5.dtsi
@@ -610,7 +610,7 @@ eic: interrupt-controller@e1628000 {
};

pit64b0: timer@e1800000 {
- compatible = "microchip,sama7g5-pit64b", "microchip,sam9x60-pit64b";
+ compatible = "microchip,sam9x60-pit64b";
reg = <0xe1800000 0x4000>;
interrupts = <GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pmc PMC_TYPE_PERIPHERAL 70>, <&pmc PMC_TYPE_GCK 70>;
@@ -618,7 +618,7 @@ pit64b0: timer@e1800000 {
};

pit64b1: timer@e1804000 {
- compatible = "microchip,sama7g5-pit64b", "microchip,sam9x60-pit64b";
+ compatible = "microchip,sam9x60-pit64b";
reg = <0xe1804000 0x4000>;
interrupts = <GIC_SPI 71 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&pmc PMC_TYPE_PERIPHERAL 71>, <&pmc PMC_TYPE_GCK 71>;
--
2.34.1


2023-05-29 12:16:11

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v2 1/4] ARM: dts: at91: sama7g5: keep only microchip,sam9x60-pit64b compatible for pit64b

Hey Claudiu,

On Mon, May 29, 2023 at 09:26:01AM +0300, Claudiu Beznea wrote:
> SAM9X60 and SAMA7G5 PIT64Bs are 1 to 1 compatible. Thus, there is no
> need for specific compatible string on SAMA7G5.

Generally, soc-specific compatibles are something that we want people to
add, not remove!

Cheers,
Conor.

>
> Signed-off-by: Claudiu Beznea <[email protected]>
> ---
> arch/arm/boot/dts/sama7g5.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/sama7g5.dtsi b/arch/arm/boot/dts/sama7g5.dtsi
> index 886b6209a71e..500b00508621 100644
> --- a/arch/arm/boot/dts/sama7g5.dtsi
> +++ b/arch/arm/boot/dts/sama7g5.dtsi
> @@ -610,7 +610,7 @@ eic: interrupt-controller@e1628000 {
> };
>
> pit64b0: timer@e1800000 {
> - compatible = "microchip,sama7g5-pit64b", "microchip,sam9x60-pit64b";
> + compatible = "microchip,sam9x60-pit64b";
> reg = <0xe1800000 0x4000>;
> interrupts = <GIC_SPI 70 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&pmc PMC_TYPE_PERIPHERAL 70>, <&pmc PMC_TYPE_GCK 70>;
> @@ -618,7 +618,7 @@ pit64b0: timer@e1800000 {
> };
>
> pit64b1: timer@e1804000 {
> - compatible = "microchip,sama7g5-pit64b", "microchip,sam9x60-pit64b";
> + compatible = "microchip,sam9x60-pit64b";
> reg = <0xe1804000 0x4000>;
> interrupts = <GIC_SPI 71 IRQ_TYPE_LEVEL_HIGH>;
> clocks = <&pmc PMC_TYPE_PERIPHERAL 71>, <&pmc PMC_TYPE_GCK 71>;
> --
> 2.34.1
>


Attachments:
(No filename) (1.45 kB)
signature.asc (235.00 B)
Download all attachments