From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.ripard@free-electrons.com (Maxime Ripard) Date: Fri, 17 May 2013 23:21:42 +0200 Subject: [PATCH 2/2] Add sunxi-sid to dts for sun4i and sun5i In-Reply-To: <1368797744-13737-3-git-send-email-oliver+list@schinagl.nl> References: <1368797744-13737-1-git-send-email-oliver+list@schinagl.nl> <1368797744-13737-3-git-send-email-oliver+list@schinagl.nl> Message-ID: <51969F66.7080205@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Oliver, Le 17/05/2013 15:35, Oliver Schinagl a ?crit : > From: Oliver Schinagl > > This should add support for the sunxi-sid driver to the device table for sun4i and sun5i And it actually does :) > > Signed-off-by: Oliver Schinagl > --- > arch/arm/boot/dts/sun4i-a10.dtsi | 5 +++++ > arch/arm/boot/dts/sun5i-a13.dtsi | 5 +++++ > 2 files changed, 10 insertions(+) > > diff --git a/arch/arm/boot/dts/sun4i-a10.dtsi b/arch/arm/boot/dts/sun4i-a10.dtsi > index e7ef619..1043db2 100644 > --- a/arch/arm/boot/dts/sun4i-a10.dtsi > +++ b/arch/arm/boot/dts/sun4i-a10.dtsi > @@ -163,6 +163,11 @@ > reg = <0x01c20000 0x300000>; > ranges; > > + sid: eeprom at 01c23800 { > + compatible = "allwinner,sun4i-sid"; > + reg = <0x01c23800 0x10>; > + }; > + I'd prefer to have the nodes sorted by base addresses. Also, the reserved address space for this IP is 1kB, please make it as such in the dt. > intc: interrupt-controller at 01c20400 { > compatible = "allwinner,sun4i-ic"; > reg = <0x01c20400 0x400>; > diff --git a/arch/arm/boot/dts/sun5i-a13.dtsi b/arch/arm/boot/dts/sun5i-a13.dtsi > index 8ba65c1..f715132 100644 > --- a/arch/arm/boot/dts/sun5i-a13.dtsi > +++ b/arch/arm/boot/dts/sun5i-a13.dtsi > @@ -153,6 +153,11 @@ > reg = <0x01c20000 0x300000>; > ranges; > > + sid: eeprom at 01c23800 { > + compatible = "allwinner,sun4i-sid"; > + reg = <0x01c23800 0x10>; > + }; Ditto, > + > intc: interrupt-controller at 01c20400 { > compatible = "allwinner,sun4i-ic"; > reg = <0x01c20400 0x400>; > Thanks, Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com