Hi, On Tue, Jul 26, 2016 at 5:13 AM, Caesar Wang <wxt@xxxxxxxxxxxxxx> wrote: > This patch adds saradc needed information on rk3399 SoCs. > > Signed-off-by: Caesar Wang <wxt@xxxxxxxxxxxxxx> > --- > > Changes in v2: None > > arch/arm64/boot/dts/rockchip/rk3399.dtsi | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/arch/arm64/boot/dts/rockchip/rk3399.dtsi b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > index 4c84229..b81f84b 100644 > --- a/arch/arm64/boot/dts/rockchip/rk3399.dtsi > +++ b/arch/arm64/boot/dts/rockchip/rk3399.dtsi > @@ -299,6 +299,18 @@ > }; > }; > > + saradc: saradc@ff100000 { > + compatible = "rockchip,rk3399-saradc"; > + reg = <0x0 0xff100000 0x0 0x100>; > + interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>; > + #io-channel-cells = <1>; > + clocks = <&cru SCLK_SARADC>, <&cru PCLK_SARADC>; > + clock-names = "saradc", "apb_pclk"; > + resets = <&cru SRST_P_SARADC>; > + reset-names = "saradc-apb"; > + status = "disabled"; > + }; > + Seems reasonable to me. I'll let Heiko comment if he cares about the sort ordering. I can never quite figure out what it should be so I've sorta given up on it... ;) Reviewed-by: Douglas Anderson <dianders@xxxxxxxxxxxx> -Doug -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html