mbox series

[v3,0/4] dt-bindings: power: reset: at91: convert to YAML

Message ID 20230616101646.879480-1-claudiu.beznea@microchip.com
Headers show
Series dt-bindings: power: reset: at91: convert to YAML | expand

Message

Claudiu Beznea June 16, 2023, 10:16 a.m. UTC
Hi,

Series convert AT91 shutdown controllers documentation to YAML.
Along with it device trees were updated and also entries to
MAINTAINERS with documentation files.

Thank you,
Claudiu

Changes in v3:
- do not define properties in allOf section
- used additionalProperties instead of unevaluatedProperties
- fixed compatible list of patch 3/4
- s/Atmel/Microchip AT91/g in title and description sections of YAML files
  and in commit description
- collected tags

Changes in v2:
- use minimum, maximum on uint32 enums instead of all possible values
- change commit description for patch 3/4 to explain why there is a
  syscon along with microchip,sama7g5-shdwc
- remove | near description entries

Claudiu Beznea (4):
  ARM: dts: at91: use generic name for shutdown controller
  dt-bindings: power: reset: atmel,at91sam9260-shdwc: convert to yaml
  dt-bindings: power: reset: atmel,sama5d2-shdwc: convert to yaml
  MAINTAINERS: add documentation file for Microchip SAMA5D2 shutdown
    controller

 .../devicetree/bindings/arm/atmel-sysregs.txt |  94 ---------------
 .../power/reset/atmel,at91sam9260-shdwc.yaml  |  82 +++++++++++++
 .../power/reset/atmel,sama5d2-shdwc.yaml      | 114 ++++++++++++++++++
 MAINTAINERS                                   |   1 +
 arch/arm/boot/dts/at91-qil_a9260.dts          |   2 +-
 arch/arm/boot/dts/at91-sama5d27_som1_ek.dts   |   2 +-
 arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts     |   2 +-
 arch/arm/boot/dts/at91-sama5d2_xplained.dts   |   2 +-
 arch/arm/boot/dts/at91sam9260.dtsi            |   2 +-
 arch/arm/boot/dts/at91sam9260ek.dts           |   2 +-
 arch/arm/boot/dts/at91sam9261.dtsi            |   2 +-
 arch/arm/boot/dts/at91sam9263.dtsi            |   2 +-
 arch/arm/boot/dts/at91sam9g20ek_common.dtsi   |   2 +-
 arch/arm/boot/dts/at91sam9g45.dtsi            |   2 +-
 arch/arm/boot/dts/at91sam9n12.dtsi            |   2 +-
 arch/arm/boot/dts/at91sam9rl.dtsi             |   2 +-
 arch/arm/boot/dts/at91sam9x5.dtsi             |   2 +-
 arch/arm/boot/dts/sam9x60.dtsi                |   2 +-
 arch/arm/boot/dts/sama5d2.dtsi                |   2 +-
 arch/arm/boot/dts/sama5d3.dtsi                |   2 +-
 arch/arm/boot/dts/sama5d4.dtsi                |   2 +-
 arch/arm/boot/dts/sama7g5.dtsi                |   2 +-
 arch/arm/boot/dts/usb_a9260.dts               |   2 +-
 arch/arm/boot/dts/usb_a9263.dts               |   2 +-
 24 files changed, 217 insertions(+), 114 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/power/reset/atmel,at91sam9260-shdwc.yaml
 create mode 100644 Documentation/devicetree/bindings/power/reset/atmel,sama5d2-shdwc.yaml

Comments

Sebastian Reichel June 19, 2023, 9:54 p.m. UTC | #1
Hi,

On Fri, Jun 16, 2023 at 01:16:46PM +0300, Claudiu Beznea wrote:
> Add documentation file for SAMA5D2 shutdown controller.
> 
> Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> ---

Thanks, queued.

-- Sebastian

>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 74a6b6b13d84..cdf4b41cf62b 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13811,6 +13811,7 @@ F:	include/dt-bindings/iio/adc/at91-sama5d2_adc.h
>  MICROCHIP SAMA5D2-COMPATIBLE SHUTDOWN CONTROLLER
>  M:	Claudiu Beznea <claudiu.beznea@microchip.com>
>  S:	Supported
> +F:	Documentation/devicetree/bindings/power/reset/atmel,sama5d2-shdwc.yaml
>  F:	drivers/power/reset/at91-sama5d2_shdwc.c
>  
>  MICROCHIP SPI DRIVER
> -- 
> 2.34.1
>
Claudiu Beznea June 21, 2023, 9:31 a.m. UTC | #2
On 16.06.2023 13:16, Claudiu Beznea wrote:
> Use poweroff generic name for shdwc node to cope with device tree
> specifications.
> 
> Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
> Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Applied to at91-dt, thanks!

> ---
>  arch/arm/boot/dts/at91-qil_a9260.dts        | 2 +-
>  arch/arm/boot/dts/at91-sama5d27_som1_ek.dts | 2 +-
>  arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts   | 2 +-
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts | 2 +-
>  arch/arm/boot/dts/at91sam9260.dtsi          | 2 +-
>  arch/arm/boot/dts/at91sam9260ek.dts         | 2 +-
>  arch/arm/boot/dts/at91sam9261.dtsi          | 2 +-
>  arch/arm/boot/dts/at91sam9263.dtsi          | 2 +-
>  arch/arm/boot/dts/at91sam9g20ek_common.dtsi | 2 +-
>  arch/arm/boot/dts/at91sam9g45.dtsi          | 2 +-
>  arch/arm/boot/dts/at91sam9n12.dtsi          | 2 +-
>  arch/arm/boot/dts/at91sam9rl.dtsi           | 2 +-
>  arch/arm/boot/dts/at91sam9x5.dtsi           | 2 +-
>  arch/arm/boot/dts/sam9x60.dtsi              | 2 +-
>  arch/arm/boot/dts/sama5d2.dtsi              | 2 +-
>  arch/arm/boot/dts/sama5d3.dtsi              | 2 +-
>  arch/arm/boot/dts/sama5d4.dtsi              | 2 +-
>  arch/arm/boot/dts/sama7g5.dtsi              | 2 +-
>  arch/arm/boot/dts/usb_a9260.dts             | 2 +-
>  arch/arm/boot/dts/usb_a9263.dts             | 2 +-
>  20 files changed, 20 insertions(+), 20 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/at91-qil_a9260.dts b/arch/arm/boot/dts/at91-qil_a9260.dts
> index 9d26f9996348..5ccb3c139592 100644
> --- a/arch/arm/boot/dts/at91-qil_a9260.dts
> +++ b/arch/arm/boot/dts/at91-qil_a9260.dts
> @@ -108,7 +108,7 @@ dbgu: serial@fffff200 {
>  				status = "okay";
>  			};
>  
> -			shdwc@fffffd10 {
> +			shdwc: poweroff@fffffd10 {
>  				atmel,wakeup-counter = <10>;
>  				atmel,wakeup-rtt-timer;
>  			};
> diff --git a/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> index 0dc6ca377b0c..cb6243cfe182 100644
> --- a/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> +++ b/arch/arm/boot/dts/at91-sama5d27_som1_ek.dts
> @@ -139,7 +139,7 @@ i2c3: i2c@600 {
>  				};
>  			};
>  
> -			shdwc@f8048010 {
> +			poweroff@f8048010 {
>  				debounce-delay-us = <976>;
>  				atmel,wakeup-rtc-timer;
>  
> diff --git a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> index 76a711b167b0..86b860d80726 100644
> --- a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> +++ b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> @@ -204,7 +204,7 @@ i2c2: i2c@600 {
>  				};
>  			};
>  
> -			shdwc@f8048010 {
> +			poweroff@f8048010 {
>  				debounce-delay-us = <976>;
>  
>  				input@0 {
> diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> index 85949c24b687..024a553842a9 100644
> --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> @@ -348,7 +348,7 @@ i2c2: i2c@600 {
>  				};
>  			};
>  
> -			shdwc@f8048010 {
> +			poweroff@f8048010 {
>  				debounce-delay-us = <976>;
>  				atmel,wakeup-rtc-timer;
>  
> diff --git a/arch/arm/boot/dts/at91sam9260.dtsi b/arch/arm/boot/dts/at91sam9260.dtsi
> index 16e3b24b4ddd..35a007365b6a 100644
> --- a/arch/arm/boot/dts/at91sam9260.dtsi
> +++ b/arch/arm/boot/dts/at91sam9260.dtsi
> @@ -130,7 +130,7 @@ reset-controller@fffffd00 {
>  				clocks = <&pmc PMC_TYPE_CORE PMC_SLOW>;
>  			};
>  
> -			shdwc@fffffd10 {
> +			shdwc: poweroff@fffffd10 {
>  				compatible = "atmel,at91sam9260-shdwc";
>  				reg = <0xfffffd10 0x10>;
>  				clocks = <&pmc PMC_TYPE_CORE PMC_SLOW>;
> diff --git a/arch/arm/boot/dts/at91sam9260ek.dts b/arch/arm/boot/dts/at91sam9260ek.dts
> index bb72f050a4fe..720c15472c4a 100644
> --- a/arch/arm/boot/dts/at91sam9260ek.dts
> +++ b/arch/arm/boot/dts/at91sam9260ek.dts
> @@ -112,7 +112,7 @@ pinctrl_board_mmc0_slot1: mmc0_slot1-board {
>  				};
>  			};
>  
> -			shdwc@fffffd10 {
> +			shdwc: poweroff@fffffd10 {
>  				atmel,wakeup-counter = <10>;
>  				atmel,wakeup-rtt-timer;
>  			};
> diff --git a/arch/arm/boot/dts/at91sam9261.dtsi b/arch/arm/boot/dts/at91sam9261.dtsi
> index fe9ead867e2a..528ffc6f6f96 100644
> --- a/arch/arm/boot/dts/at91sam9261.dtsi
> +++ b/arch/arm/boot/dts/at91sam9261.dtsi
> @@ -614,7 +614,7 @@ reset-controller@fffffd00 {
>  				clocks = <&slow_xtal>;
>  			};
>  
> -			shdwc@fffffd10 {
> +			poweroff@fffffd10 {
>  				compatible = "atmel,at91sam9260-shdwc";
>  				reg = <0xfffffd10 0x10>;
>  				clocks = <&slow_xtal>;
> diff --git a/arch/arm/boot/dts/at91sam9263.dtsi b/arch/arm/boot/dts/at91sam9263.dtsi
> index ee5e6ed44dd4..75d8ff2d12c8 100644
> --- a/arch/arm/boot/dts/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/at91sam9263.dtsi
> @@ -158,7 +158,7 @@ reset-controller@fffffd00 {
>  				clocks = <&slow_xtal>;
>  			};
>  
> -			shdwc@fffffd10 {
> +			poweroff@fffffd10 {
>  				compatible = "atmel,at91sam9260-shdwc";
>  				reg = <0xfffffd10 0x10>;
>  				clocks = <&slow_xtal>;
> diff --git a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi
> index 024af2db638e..565b99e79c52 100644
> --- a/arch/arm/boot/dts/at91sam9g20ek_common.dtsi
> +++ b/arch/arm/boot/dts/at91sam9g20ek_common.dtsi
> @@ -126,7 +126,7 @@ flash@1 {
>  				};
>  			};
>  
> -			shdwc@fffffd10 {
> +			shdwc: poweroff@fffffd10 {
>  				atmel,wakeup-counter = <10>;
>  				atmel,wakeup-rtt-timer;
>  			};
> diff --git a/arch/arm/boot/dts/at91sam9g45.dtsi b/arch/arm/boot/dts/at91sam9g45.dtsi
> index 498cb92b29f9..7cccc606e36c 100644
> --- a/arch/arm/boot/dts/at91sam9g45.dtsi
> +++ b/arch/arm/boot/dts/at91sam9g45.dtsi
> @@ -152,7 +152,7 @@ pit: timer@fffffd30 {
>  			};
>  
>  
> -			shdwc@fffffd10 {
> +			poweroff@fffffd10 {
>  				compatible = "atmel,at91sam9rl-shdwc";
>  				reg = <0xfffffd10 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/at91sam9n12.dtsi b/arch/arm/boot/dts/at91sam9n12.dtsi
> index 0e28101b26bf..8dc04e9031a6 100644
> --- a/arch/arm/boot/dts/at91sam9n12.dtsi
> +++ b/arch/arm/boot/dts/at91sam9n12.dtsi
> @@ -140,7 +140,7 @@ pit: timer@fffffe30 {
>  				clocks = <&pmc PMC_TYPE_CORE PMC_MCK>;
>  			};
>  
> -			shdwc@fffffe10 {
> +			poweroff@fffffe10 {
>  				compatible = "atmel,at91sam9x5-shdwc";
>  				reg = <0xfffffe10 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/at91sam9rl.dtsi b/arch/arm/boot/dts/at91sam9rl.dtsi
> index d7e8a115c916..3d089ffbe162 100644
> --- a/arch/arm/boot/dts/at91sam9rl.dtsi
> +++ b/arch/arm/boot/dts/at91sam9rl.dtsi
> @@ -778,7 +778,7 @@ reset-controller@fffffd00 {
>  				clocks = <&clk32k>;
>  			};
>  
> -			shdwc@fffffd10 {
> +			poweroff@fffffd10 {
>  				compatible = "atmel,at91sam9260-shdwc";
>  				reg = <0xfffffd10 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/at91sam9x5.dtsi b/arch/arm/boot/dts/at91sam9x5.dtsi
> index 0123ee47151c..a1fed912f2ee 100644
> --- a/arch/arm/boot/dts/at91sam9x5.dtsi
> +++ b/arch/arm/boot/dts/at91sam9x5.dtsi
> @@ -141,7 +141,7 @@ reset_controller: reset-controller@fffffe00 {
>  				clocks = <&clk32k>;
>  			};
>  
> -			shutdown_controller: shdwc@fffffe10 {
> +			shutdown_controller: poweroff@fffffe10 {
>  				compatible = "atmel,at91sam9x5-shdwc";
>  				reg = <0xfffffe10 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi
> index 933d73505a8b..47c4f267a45c 100644
> --- a/arch/arm/boot/dts/sam9x60.dtsi
> +++ b/arch/arm/boot/dts/sam9x60.dtsi
> @@ -675,7 +675,7 @@ reset_controller: reset-controller@fffffe00 {
>  				clocks = <&clk32k 0>;
>  			};
>  
> -			shutdown_controller: shdwc@fffffe10 {
> +			shutdown_controller: poweroff@fffffe10 {
>  				compatible = "microchip,sam9x60-shdwc";
>  				reg = <0xfffffe10 0x10>;
>  				clocks = <&clk32k 0>;
> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> index 7b6471655dee..daeeb24e5f4d 100644
> --- a/arch/arm/boot/dts/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> @@ -680,7 +680,7 @@ reset_controller: reset-controller@f8048000 {
>  				clocks = <&clk32k>;
>  			};
>  
> -			shutdown_controller: shdwc@f8048010 {
> +			shutdown_controller: poweroff@f8048010 {
>  				compatible = "atmel,sama5d2-shdwc";
>  				reg = <0xf8048010 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/sama5d3.dtsi b/arch/arm/boot/dts/sama5d3.dtsi
> index 0eebf6c760b3..d9e66700d1c2 100644
> --- a/arch/arm/boot/dts/sama5d3.dtsi
> +++ b/arch/arm/boot/dts/sama5d3.dtsi
> @@ -1016,7 +1016,7 @@ reset_controller: reset-controller@fffffe00 {
>  				clocks = <&clk32k>;
>  			};
>  
> -			shutdown_controller: shutdown-controller@fffffe10 {
> +			shutdown_controller: poweroff@fffffe10 {
>  				compatible = "atmel,at91sam9x5-shdwc";
>  				reg = <0xfffffe10 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi
> index de6c82969232..41284e013f53 100644
> --- a/arch/arm/boot/dts/sama5d4.dtsi
> +++ b/arch/arm/boot/dts/sama5d4.dtsi
> @@ -740,7 +740,7 @@ reset_controller: reset-controller@fc068600 {
>  				clocks = <&clk32k>;
>  			};
>  
> -			shutdown_controller: shdwc@fc068610 {
> +			shutdown_controller: poweroff@fc068610 {
>  				compatible = "atmel,at91sam9x5-shdwc";
>  				reg = <0xfc068610 0x10>;
>  				clocks = <&clk32k>;
> diff --git a/arch/arm/boot/dts/sama7g5.dtsi b/arch/arm/boot/dts/sama7g5.dtsi
> index f0478a43edc2..886b6209a71e 100644
> --- a/arch/arm/boot/dts/sama7g5.dtsi
> +++ b/arch/arm/boot/dts/sama7g5.dtsi
> @@ -257,7 +257,7 @@ reset_controller: reset-controller@e001d000 {
>  			clocks = <&clk32k 0>;
>  		};
>  
> -		shdwc: shdwc@e001d010 {
> +		shdwc: poweroff@e001d010 {
>  			compatible = "microchip,sama7g5-shdwc", "syscon";
>  			reg = <0xe001d010 0x10>;
>  			clocks = <&clk32k 0>;
> diff --git a/arch/arm/boot/dts/usb_a9260.dts b/arch/arm/boot/dts/usb_a9260.dts
> index 6cfa83921ac2..66f8da89007d 100644
> --- a/arch/arm/boot/dts/usb_a9260.dts
> +++ b/arch/arm/boot/dts/usb_a9260.dts
> @@ -22,7 +22,7 @@ memory@20000000 {
>  
>  	ahb {
>  		apb {
> -			shdwc@fffffd10 {
> +			shdwc: poweroff@fffffd10 {
>  				atmel,wakeup-counter = <10>;
>  				atmel,wakeup-rtt-timer;
>  			};
> diff --git a/arch/arm/boot/dts/usb_a9263.dts b/arch/arm/boot/dts/usb_a9263.dts
> index b6cb9cdf8197..45745915b2e1 100644
> --- a/arch/arm/boot/dts/usb_a9263.dts
> +++ b/arch/arm/boot/dts/usb_a9263.dts
> @@ -67,7 +67,7 @@ flash@0 {
>  				};
>  			};
>  
> -			shdwc@fffffd10 {
> +			poweroff@fffffd10 {
>  				atmel,wakeup-counter = <10>;
>  				atmel,wakeup-rtt-timer;
>  			};