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

["Clément Léger" <clement.leger@bootlin.com>]

Le Fri, 4 Mar 2022 09:32:16 +0000,
<Eugen.Hristev@microchip.com> a écrit :

> On 3/4/22 11:12 AM, Clément Léger wrote:
> 
> > Add the device-tree node to describe the TCB timer.
> > 
> > Signed-off-by: Clément Léger <clement.leger@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..63588b27b8 100644
> > --- a/arch/arm/dts/sama5d2.dtsi
> > +++ b/arch/arm/dts/sama5d2.dtsi
> > @@ -700,6 +700,22 @@
> >                                  clocks = <&h32ck>;
> >                          };
> > 
> > +                       tcb0: timer@f800c000 {
> > +                               compatible = "atmel,sama5d2-tcb", "simple-mfd",
> > +                                            "syscon";
> > +                               reg = <0xf800c000 0x100>;
> > +
> > +                               clocks = <&tcb0_clk>, <&tcb0_gclk>, <&clk32k>;
> > +                               clock-names = "t0_clk", "gclk", "slow_clk";
> > +                               #address-cells = <1>;
> > +                               #size-cells = <0>;
> > +
> > +                               timer0: timer@0 {
> > +                                       compatible = "atmel,tcb-timer";
> > +                                       reg = <0>, <1>;
> > +                               };
> > +                       };  
> 
> Hello Clement,
> 
> As discussed in v2, interrupts property is mandatory.

Args, I added syscon but forgot interrupts... Will resubmit then

> 
> 
> Eugen
> > +
> >                          watchdog@f8048040 {
> >                                  compatible = "atmel,sama5d4-wdt";
> >                                  reg = <0xf8048040 0x10>;
> > --
> > 2.34.1
> >   
> 



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