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

> Le Fri,  4 Mar 2022 10:45:21 +0100,
> Clément Léger <clement.le...@bootlin.com> a écrit :
> 
> > Add the device-tree node to describe the TCB timer.
> > 
> > Signed-off-by: Clément Léger <clement.le...@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@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@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@0 {
> > +                                   compatible = "atmel,tcb-timer";
> > +                                   reg = <0>, <1>;
> > +                           };
> > +                   };
> > +
> >                     watchdog@f8048040 {
> >                             compatible = "atmel,sama5d4-wdt";
> >                             reg = <0xf8048040 0x10>;  
> 
> 
> 



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

Reply via email to