Re: [PATCH 3/3] arm64: dts: rockchip: Add the needed timer for rk3368 SoC

[Caesar Wang]

å 2015å09æ18æ 03:01, Heiko StÃbner åé:
> Am Donnerstag, 17. September 2015, 15:51:12 schrieb Caesar Wang:
> > There is a need of a broadcast timer in this case to ensure proper
> > wakeup when the cpus are in sleep mode and a timer expires.
> >
> > Signed-off-by: Caesar Wang <wxt@xxxxxxxxxxxxxx>
> > ---
> >
> >   arch/arm64/boot/dts/rockchip/rk3368.dtsi | 6 ++++++
> >   1 file changed, 6 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> > b/arch/arm64/boot/dts/rockchip/rk3368.dtsi index a712bea..99edbaa 100644
> > --- a/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> > +++ b/arch/arm64/boot/dts/rockchip/rk3368.dtsi
> > @@ -214,6 +214,12 @@
> >   			(GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_HIGH)>;
> >   	};
> >
> > +	timer@ff810000 {
> > +		compatible = "rockchip,rk3288-timer";
> you probably want a
> 		compatible = "rockchip,rk3368-timer", "rockchip,rk3288-timer";

Done.

More consideration for the future.
Although the "rk3288-timer" is working for 3368, agreed to add the 
'rk3368-timer' for the rk3368-spec  timer in the future.

> > +		reg = <0x0 0xff810000 0x0 0x20>;
> > +		interrupts = <GIC_SPI 66 IRQ_TYPE_LEVEL_HIGH>;
> > +	};
> > +
> >   	xin24m: oscillator {
> >   		compatible = "fixed-clock";
> >   		clock-frequency = <24000000>;
>
>
>
>
> --
> Thanks,
> Caesar
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/