[PATCH v4 2/3] ARM: dts: at91: sama5d2: add TCB node

[Clément Léger]

Le Fri, 4 Mar 2022 10:46:45 +0100,
Clément Léger <clement.leger at bootlin.com> a écrit :

> Le Fri,  4 Mar 2022 10:45:21 +0100,
> Clément Léger <clement.leger at bootlin.com> a écrit :
> 
> > Add the device-tree node to describe the TCB timer.
> > 
> > Signed-off-by: Clément Léger <clement.leger at bootlin.com>
> > ---
> >  arch/arm/dts/sama5d2.dtsi | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> > 
> > diff --git a/arch/arm/dts/sama5d2.dtsi b/arch/arm/dts/sama5d2.dtsi
> > index 038cd73c03..54f3dd4c62 100644
> > --- a/arch/arm/dts/sama5d2.dtsi
> > +++ b/arch/arm/dts/sama5d2.dtsi
> > @@ -700,6 +700,22 @@
> >  				clocks = <&h32ck>;
> >  			};
> >  
> > +			tcb0: timer at f800c000 {
> > +				compatible = "atmel,sama5d2-tcb", "simple-mfd",
> > +					     "syscon";
> > +				reg = <0xf800c000 0x100>;
> > +				interrupts = <35 IRQ_TYPE_LEVEL_HIGH 0>;  
> 
> And I forgot to include the file which defines IRQ_TYPE_LEVEL_HIGH. Will
> need a V5 I guess.

By the way, this will also yield the following warning:

arch/arm/dts/at91-sama5d2_xplained.dtb: Warning (interrupts_property):
  /ahb/apb/timer at f800c000: Missing interrupt-parent

Because indeed, there is no interrupt controller declared at all in
this device-tree. What can I do about that ?

> 
> > +				clocks = <&tcb0_clk>, <&tcb0_gclk>, <&clk32k>;
> > +				clock-names = "t0_clk", "gclk", "slow_clk";
> > +				#address-cells = <1>;
> > +				#size-cells = <0>;
> > +
> > +				timer0: timer at 0 {
> > +					compatible = "atmel,tcb-timer";
> > +					reg = <0>, <1>;
> > +				};
> > +			};
> > +
> >  			watchdog at f8048040 {
> >  				compatible = "atmel,sama5d4-wdt";
> >  				reg = <0xf8048040 0x10>;  
> 
> 
> 



-- 
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin
https://bootlin.com