devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: stm32: add Timers driver for stm32f746 MCU
@ 2017-10-04  9:27 Benjamin Gaignard
  2017-10-05 16:06 ` Alexandre Torgue
  0 siblings, 1 reply; 3+ messages in thread
From: Benjamin Gaignard @ 2017-10-04  9:27 UTC (permalink / raw)
  To: robh+dt, mark.rutland, linux, mcoquelin.stm32, alexandre.torgue
  Cc: devicetree, linux-arm-kernel, linux-kernel, Benjamin Gaignard,
	Benjamin Gaignard

Add Timers and it sub-nodes into DT for stm32f746 family.

Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
---
 arch/arm/boot/dts/stm32f746.dtsi | 270 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 270 insertions(+)

diff --git a/arch/arm/boot/dts/stm32f746.dtsi b/arch/arm/boot/dts/stm32f746.dtsi
index 4506eb9..ae84816 100644
--- a/arch/arm/boot/dts/stm32f746.dtsi
+++ b/arch/arm/boot/dts/stm32f746.dtsi
@@ -82,6 +82,27 @@
 			status = "disabled";
 		};
 
+		timers2: timers@40000000 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40000000 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM2)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@1 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <1>;
+				status = "disabled";
+			};
+		};
+
 		timer3: timer@40000400 {
 			compatible = "st,stm32-timer";
 			reg = <0x40000400 0x400>;
@@ -90,6 +111,27 @@
 			status = "disabled";
 		};
 
+		timers3: timers@40000400 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40000400 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM3)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@2 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <2>;
+				status = "disabled";
+			};
+		};
+
 		timer4: timer@40000800 {
 			compatible = "st,stm32-timer";
 			reg = <0x40000800 0x400>;
@@ -98,6 +140,27 @@
 			status = "disabled";
 		};
 
+		timers4: timers@40000800 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40000800 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM4)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@3 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <3>;
+				status = "disabled";
+			};
+		};
+
 		timer5: timer@40000c00 {
 			compatible = "st,stm32-timer";
 			reg = <0x40000c00 0x400>;
@@ -105,6 +168,27 @@
 			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
 		};
 
+		timers5: timers@40000c00 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40000C00 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@4 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <4>;
+				status = "disabled";
+			};
+		};
+
 		timer6: timer@40001000 {
 			compatible = "st,stm32-timer";
 			reg = <0x40001000 0x400>;
@@ -113,6 +197,22 @@
 			status = "disabled";
 		};
 
+		timers6: timers@40001000 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40001000 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM6)>;
+			clock-names = "int";
+			status = "disabled";
+
+			timer@5 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <5>;
+				status = "disabled";
+			};
+		};
+
 		timer7: timer@40001400 {
 			compatible = "st,stm32-timer";
 			reg = <0x40001400 0x400>;
@@ -121,6 +221,73 @@
 			status = "disabled";
 		};
 
+		timers7: timers@40001400 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40001400 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM7)>;
+			clock-names = "int";
+			status = "disabled";
+
+			timer@6 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <6>;
+				status = "disabled";
+			};
+		};
+
+		timers12: timers@40001800 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40001800 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM12)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@11 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <11>;
+				status = "disabled";
+			};
+		};
+
+		timers13: timers@40001c00 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40001C00 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM13)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+		};
+
+		timers14: timers@40002000 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40002000 0x400>;
+			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM14)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+		};
+
 		rtc: rtc@40002800 {
 			compatible = "st,stm32-rtc";
 			reg = <0x40002800 0x400>;
@@ -183,6 +350,48 @@
 			status = "disabled";
 		};
 
+		timers1: timers@40010000 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40010000 0x400>;
+			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM1)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@0 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <0>;
+				status = "disabled";
+			};
+		};
+
+		timers8: timers@40010400 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40010400 0x400>;
+			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM8)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@7 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <7>;
+				status = "disabled";
+			};
+		};
+
 		usart1: serial@40011000 {
 			compatible = "st,stm32f7-usart", "st,stm32f7-uart";
 			reg = <0x40011000 0x400>;
@@ -212,11 +421,72 @@
 			interrupts = <1>, <2>, <3>, <6>, <7>, <8>, <9>, <10>, <23>, <40>, <41>, <42>, <62>, <76>;
 		};
 
+		timers9: timers@40014000 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40014000 0x400>;
+			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM9)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+
+			timer@8 {
+				compatible = "st,stm32-timer-trigger";
+				reg = <8>;
+				status = "disabled";
+			};
+		};
+
+		timers10: timers@40014400 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40014400 0x400>;
+			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM10)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+		};
+
+		timers11: timers@40014800 {
+			#address-cells = <1>;
+			#size-cells = <0>;
+			compatible = "st,stm32-timers";
+			reg = <0x40014800 0x400>;
+			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM11)>;
+			clock-names = "int";
+			status = "disabled";
+
+			pwm {
+				compatible = "st,stm32-pwm";
+				status = "disabled";
+			};
+		};
+
 		pwrcfg: power-config@40007000 {
 			compatible = "syscon";
 			reg = <0x40007000 0x400>;
 		};
 
+		ltdc: display-controller@40016800 {
+			compatible = "st,stm32-ltdc";
+			reg = <0x40016800 0x200>;
+			interrupts = <88>, <89>;
+			resets = <&rcc STM32F7_APB2_RESET(LTDC)>;
+			clocks = <&rcc 1 CLK_LCD>;
+			clock-names = "lcd";
+			status = "disabled";
+		};
+
 		pin-controller {
 			#address-cells = <1>;
 			#size-cells = <1>;
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: stm32: add Timers driver for stm32f746 MCU
  2017-10-04  9:27 [PATCH] ARM: dts: stm32: add Timers driver for stm32f746 MCU Benjamin Gaignard
@ 2017-10-05 16:06 ` Alexandre Torgue
       [not found]   ` <589f9f9e-8daf-ea31-e331-2e531793e440-qxv4g6HH51o@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: Alexandre Torgue @ 2017-10-05 16:06 UTC (permalink / raw)
  To: Benjamin Gaignard, robh+dt, mark.rutland, linux, mcoquelin.stm32
  Cc: devicetree, linux-kernel, linux-arm-kernel, Benjamin Gaignard

Hi Benjamin,

On 10/04/2017 11:27 AM, Benjamin Gaignard wrote:
> Add Timers and it sub-nodes into DT for stm32f746 family.
> 
> Signed-off-by: Benjamin Gaignard <benjamin.gaignard@st.com>
> ---
>   arch/arm/boot/dts/stm32f746.dtsi | 270 +++++++++++++++++++++++++++++++++++++++
>   1 file changed, 270 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/stm32f746.dtsi b/arch/arm/boot/dts/stm32f746.dtsi
> index 4506eb9..ae84816 100644
> --- a/arch/arm/boot/dts/stm32f746.dtsi
> +++ b/arch/arm/boot/dts/stm32f746.dtsi
> @@ -82,6 +82,27 @@
>   			status = "disabled";
>   		};
>   
> +		timers2: timers@40000000 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40000000 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM2)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@1 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <1>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		timer3: timer@40000400 {
>   			compatible = "st,stm32-timer";
>   			reg = <0x40000400 0x400>;
> @@ -90,6 +111,27 @@
>   			status = "disabled";
>   		};
>   
> +		timers3: timers@40000400 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40000400 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM3)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@2 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <2>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		timer4: timer@40000800 {
>   			compatible = "st,stm32-timer";
>   			reg = <0x40000800 0x400>;
> @@ -98,6 +140,27 @@
>   			status = "disabled";
>   		};
>   
> +		timers4: timers@40000800 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40000800 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM4)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@3 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <3>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		timer5: timer@40000c00 {
>   			compatible = "st,stm32-timer";
>   			reg = <0x40000c00 0x400>;
> @@ -105,6 +168,27 @@
>   			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
>   		};
>   
> +		timers5: timers@40000c00 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40000C00 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@4 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <4>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		timer6: timer@40001000 {
>   			compatible = "st,stm32-timer";
>   			reg = <0x40001000 0x400>;
> @@ -113,6 +197,22 @@
>   			status = "disabled";
>   		};
>   
> +		timers6: timers@40001000 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40001000 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM6)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			timer@5 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <5>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		timer7: timer@40001400 {
>   			compatible = "st,stm32-timer";
>   			reg = <0x40001400 0x400>;
> @@ -121,6 +221,73 @@
>   			status = "disabled";
>   		};
>   
> +		timers7: timers@40001400 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40001400 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM7)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			timer@6 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <6>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers12: timers@40001800 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40001800 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM12)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@11 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <11>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers13: timers@40001c00 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40001C00 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM13)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers14: timers@40002000 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40002000 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM14)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +		};
> +
>   		rtc: rtc@40002800 {
>   			compatible = "st,stm32-rtc";
>   			reg = <0x40002800 0x400>;
> @@ -183,6 +350,48 @@
>   			status = "disabled";
>   		};
>   
> +		timers1: timers@40010000 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40010000 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM1)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@0 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <0>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers8: timers@40010400 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40010400 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM8)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@7 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <7>;
> +				status = "disabled";
> +			};
> +		};
> +
>   		usart1: serial@40011000 {
>   			compatible = "st,stm32f7-usart", "st,stm32f7-uart";
>   			reg = <0x40011000 0x400>;
> @@ -212,11 +421,72 @@
>   			interrupts = <1>, <2>, <3>, <6>, <7>, <8>, <9>, <10>, <23>, <40>, <41>, <42>, <62>, <76>;
>   		};
>   
> +		timers9: timers@40014000 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40014000 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM9)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +
> +			timer@8 {
> +				compatible = "st,stm32-timer-trigger";
> +				reg = <8>;
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers10: timers@40014400 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40014400 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM10)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +		};
> +
> +		timers11: timers@40014800 {
> +			#address-cells = <1>;
> +			#size-cells = <0>;
> +			compatible = "st,stm32-timers";
> +			reg = <0x40014800 0x400>;
> +			clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM11)>;
> +			clock-names = "int";
> +			status = "disabled";
> +
> +			pwm {
> +				compatible = "st,stm32-pwm";
> +				status = "disabled";
> +			};
> +		};
> +
>   		pwrcfg: power-config@40007000 {
>   			compatible = "syscon";
>   			reg = <0x40007000 0x400>;
>   		};
>   
> +		ltdc: display-controller@40016800 {
> +			compatible = "st,stm32-ltdc";
> +			reg = <0x40016800 0x200>;
> +			interrupts = <88>, <89>;
> +			resets = <&rcc STM32F7_APB2_RESET(LTDC)>;
> +			clocks = <&rcc 1 CLK_LCD>;
> +			clock-names = "lcd";
> +			status = "disabled";
> +		};
> +

Why do you add ltdc node in this patch ? Maybe a separate patch would be 
better.

>   		pin-controller {
>   			#address-cells = <1>;
>   			#size-cells = <1>;
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: stm32: add Timers driver for stm32f746 MCU
       [not found]   ` <589f9f9e-8daf-ea31-e331-2e531793e440-qxv4g6HH51o@public.gmane.org>
@ 2017-10-06  7:13     ` Benjamin Gaignard
  0 siblings, 0 replies; 3+ messages in thread
From: Benjamin Gaignard @ 2017-10-06  7:13 UTC (permalink / raw)
  To: Alexandre Torgue
  Cc: Rob Herring, Mark Rutland, Russell King - ARM Linux,
	Maxime Coquelin, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Linux Kernel Mailing List, Benjamin Gaignard

2017-10-05 18:06 GMT+02:00 Alexandre Torgue <alexandre.torgue-qxv4g6HH51o@public.gmane.org>:
> Hi Benjamin,
>
> On 10/04/2017 11:27 AM, Benjamin Gaignard wrote:
>>
>> Add Timers and it sub-nodes into DT for stm32f746 family.
>>
>> Signed-off-by: Benjamin Gaignard <benjamin.gaignard-qxv4g6HH51o@public.gmane.org>
>> ---
>>   arch/arm/boot/dts/stm32f746.dtsi | 270
>> +++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 270 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/stm32f746.dtsi
>> b/arch/arm/boot/dts/stm32f746.dtsi
>> index 4506eb9..ae84816 100644
>> --- a/arch/arm/boot/dts/stm32f746.dtsi
>> +++ b/arch/arm/boot/dts/stm32f746.dtsi
>> @@ -82,6 +82,27 @@
>>                         status = "disabled";
>>                 };
>>   +             timers2: timers@40000000 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40000000 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM2)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@1 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <1>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 timer3: timer@40000400 {
>>                         compatible = "st,stm32-timer";
>>                         reg = <0x40000400 0x400>;
>> @@ -90,6 +111,27 @@
>>                         status = "disabled";
>>                 };
>>   +             timers3: timers@40000400 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40000400 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM3)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@2 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <2>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 timer4: timer@40000800 {
>>                         compatible = "st,stm32-timer";
>>                         reg = <0x40000800 0x400>;
>> @@ -98,6 +140,27 @@
>>                         status = "disabled";
>>                 };
>>   +             timers4: timers@40000800 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40000800 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM4)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@3 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <3>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 timer5: timer@40000c00 {
>>                         compatible = "st,stm32-timer";
>>                         reg = <0x40000c00 0x400>;
>> @@ -105,6 +168,27 @@
>>                         clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
>>                 };
>>   +             timers5: timers@40000c00 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40000C00 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM5)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@4 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <4>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 timer6: timer@40001000 {
>>                         compatible = "st,stm32-timer";
>>                         reg = <0x40001000 0x400>;
>> @@ -113,6 +197,22 @@
>>                         status = "disabled";
>>                 };
>>   +             timers6: timers@40001000 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40001000 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM6)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       timer@5 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <5>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 timer7: timer@40001400 {
>>                         compatible = "st,stm32-timer";
>>                         reg = <0x40001400 0x400>;
>> @@ -121,6 +221,73 @@
>>                         status = "disabled";
>>                 };
>>   +             timers7: timers@40001400 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40001400 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM7)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       timer@6 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <6>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers12: timers@40001800 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40001800 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM12)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@11 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <11>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers13: timers@40001c00 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40001C00 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM13)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers14: timers@40002000 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40002000 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB1_CLOCK(TIM14)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 rtc: rtc@40002800 {
>>                         compatible = "st,stm32-rtc";
>>                         reg = <0x40002800 0x400>;
>> @@ -183,6 +350,48 @@
>>                         status = "disabled";
>>                 };
>>   +             timers1: timers@40010000 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40010000 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM1)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@0 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <0>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers8: timers@40010400 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40010400 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM8)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@7 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <7>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 usart1: serial@40011000 {
>>                         compatible = "st,stm32f7-usart",
>> "st,stm32f7-uart";
>>                         reg = <0x40011000 0x400>;
>> @@ -212,11 +421,72 @@
>>                         interrupts = <1>, <2>, <3>, <6>, <7>, <8>, <9>,
>> <10>, <23>, <40>, <41>, <42>, <62>, <76>;
>>                 };
>>   +             timers9: timers@40014000 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40014000 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM9)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +
>> +                       timer@8 {
>> +                               compatible = "st,stm32-timer-trigger";
>> +                               reg = <8>;
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers10: timers@40014400 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40014400 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM10)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>> +               timers11: timers@40014800 {
>> +                       #address-cells = <1>;
>> +                       #size-cells = <0>;
>> +                       compatible = "st,stm32-timers";
>> +                       reg = <0x40014800 0x400>;
>> +                       clocks = <&rcc 0 STM32F7_APB2_CLOCK(TIM11)>;
>> +                       clock-names = "int";
>> +                       status = "disabled";
>> +
>> +                       pwm {
>> +                               compatible = "st,stm32-pwm";
>> +                               status = "disabled";
>> +                       };
>> +               };
>> +
>>                 pwrcfg: power-config@40007000 {
>>                         compatible = "syscon";
>>                         reg = <0x40007000 0x400>;
>>                 };
>>   +             ltdc: display-controller@40016800 {
>> +                       compatible = "st,stm32-ltdc";
>> +                       reg = <0x40016800 0x200>;
>> +                       interrupts = <88>, <89>;
>> +                       resets = <&rcc STM32F7_APB2_RESET(LTDC)>;
>> +                       clocks = <&rcc 1 CLK_LCD>;
>> +                       clock-names = "lcd";
>> +                       status = "disabled";
>> +               };
>> +
>
>
> Why do you add ltdc node in this patch ? Maybe a separate patch would be
> better.

Sorry you got a teaser of the next patch I will send a v2 without it

>
>
>>                 pin-controller {
>>                         #address-cells = <1>;
>>                         #size-cells = <1>;
>>
>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-10-06  7:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-10-04  9:27 [PATCH] ARM: dts: stm32: add Timers driver for stm32f746 MCU Benjamin Gaignard
2017-10-05 16:06 ` Alexandre Torgue
     [not found]   ` <589f9f9e-8daf-ea31-e331-2e531793e440-qxv4g6HH51o@public.gmane.org>
2017-10-06  7:13     ` Benjamin Gaignard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).