mbox series

[v2,0/4] dt-bindings: timer: Microchip AT91 convert to YAML

Message ID 20230529062604.1498052-1-claudiu.beznea@microchip.com
Headers show
Series dt-bindings: timer: Microchip AT91 convert to YAML | expand

Message

Claudiu Beznea May 29, 2023, 6:26 a.m. UTC
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

Comments

Conor Dooley May 29, 2023, 12:12 p.m. UTC | #1
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 <claudiu.beznea@microchip.com>
> ---
>  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
>