Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-10 Thread Arnd Bergmann
On Monday 10 June 2013, Maxime Ripard wrote:
> I don't have the documentation at the moment, so I'm not exactly sure at
> which extent it's similar to the A13. Yet for the moment, the two dtsi
> are very different (different clock definitions, different muxings,
> etc.), so , I'm not quite sure about wether it would make stuff smaller.
> 
> Of course, if we find they actually share a lot, we can definitely make
> a common sun5i DTSI later.
> 
> But if you prefer to have it right now, I can always send a v2 :)

I thought it was actually the same silicon in a different package with
fewer pins, but if the files look very different right now, just leave
them separately.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-10 Thread Maxime Ripard
Hi Arnd,

On Sun, Jun 09, 2013 at 11:36:24PM +0200, Arnd Bergmann wrote:
> On Sunday 09 June 2013 18:36:05 Maxime Ripard wrote:
> > Signed-off-by: Maxime Ripard 
> > ---
> >  arch/arm/boot/dts/sun5i-a10s.dtsi | 286 
> > ++
> >  1 file changed, 286 insertions(+)
> >  create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi
> 
> Would it make sense to split out the parts that are common with A13 into
> a separate sun5i.dtsi file?

I don't have the documentation at the moment, so I'm not exactly sure at
which extent it's similar to the A13. Yet for the moment, the two dtsi
are very different (different clock definitions, different muxings,
etc.), so , I'm not quite sure about wether it would make stuff smaller.

Of course, if we find they actually share a lot, we can definitely make
a common sun5i DTSI later.

But if you prefer to have it right now, I can always send a v2 :)

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-10 Thread Maxime Ripard
Hi Arnd,

On Sun, Jun 09, 2013 at 11:36:24PM +0200, Arnd Bergmann wrote:
 On Sunday 09 June 2013 18:36:05 Maxime Ripard wrote:
  Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com
  ---
   arch/arm/boot/dts/sun5i-a10s.dtsi | 286 
  ++
   1 file changed, 286 insertions(+)
   create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi
 
 Would it make sense to split out the parts that are common with A13 into
 a separate sun5i.dtsi file?

I don't have the documentation at the moment, so I'm not exactly sure at
which extent it's similar to the A13. Yet for the moment, the two dtsi
are very different (different clock definitions, different muxings,
etc.), so , I'm not quite sure about wether it would make stuff smaller.

Of course, if we find they actually share a lot, we can definitely make
a common sun5i DTSI later.

But if you prefer to have it right now, I can always send a v2 :)

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-10 Thread Arnd Bergmann
On Monday 10 June 2013, Maxime Ripard wrote:
 I don't have the documentation at the moment, so I'm not exactly sure at
 which extent it's similar to the A13. Yet for the moment, the two dtsi
 are very different (different clock definitions, different muxings,
 etc.), so , I'm not quite sure about wether it would make stuff smaller.
 
 Of course, if we find they actually share a lot, we can definitely make
 a common sun5i DTSI later.
 
 But if you prefer to have it right now, I can always send a v2 :)

I thought it was actually the same silicon in a different package with
fewer pins, but if the files look very different right now, just leave
them separately.

Arnd
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-09 Thread Arnd Bergmann
On Sunday 09 June 2013 18:36:05 Maxime Ripard wrote:
> Signed-off-by: Maxime Ripard 
> ---
>  arch/arm/boot/dts/sun5i-a10s.dtsi | 286 
> ++
>  1 file changed, 286 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi

Would it make sense to split out the parts that are common with A13 into
a separate sun5i.dtsi file?

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-09 Thread Maxime Ripard
Signed-off-by: Maxime Ripard 
---
 arch/arm/boot/dts/sun5i-a10s.dtsi | 286 ++
 1 file changed, 286 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi

diff --git a/arch/arm/boot/dts/sun5i-a10s.dtsi 
b/arch/arm/boot/dts/sun5i-a10s.dtsi
new file mode 100644
index 000..2307ce8
--- /dev/null
+++ b/arch/arm/boot/dts/sun5i-a10s.dtsi
@@ -0,0 +1,286 @@
+/*
+ * Copyright 2013 Maxime Ripard
+ *
+ * Maxime Ripard 
+ *
+ * The code contained herein is licensed under the GNU General Public
+ * License. You may obtain a copy of the GNU General Public License
+ * Version 2 or later at the following locations:
+ *
+ * http://www.opensource.org/licenses/gpl-license.html
+ * http://www.gnu.org/copyleft/gpl.html
+ */
+
+/include/ "skeleton.dtsi"
+
+/ {
+   interrupt-parent = <>;
+
+   cpus {
+   cpu@0 {
+   compatible = "arm,cortex-a8";
+   };
+   };
+
+   memory {
+   reg = <0x4000 0x2000>;
+   };
+
+   clocks {
+   #address-cells = <1>;
+   #size-cells = <1>;
+   ranges;
+
+   /*
+* This is a dummy clock, to be used as placeholder on
+* other mux clocks when a specific parent clock is not
+* yet implemented. It should be dropped when the driver
+* is complete.
+*/
+   dummy: dummy {
+   #clock-cells = <0>;
+   compatible = "fixed-clock";
+   clock-frequency = <0>;
+   };
+
+   osc24M: osc24M@01c20050 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-osc-clk";
+   reg = <0x01c20050 0x4>;
+   clock-frequency = <2400>;
+   };
+
+   osc32k: osc32k {
+   #clock-cells = <0>;
+   compatible = "fixed-clock";
+   clock-frequency = <32768>;
+   };
+
+   pll1: pll1@01c2 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-pll1-clk";
+   reg = <0x01c2 0x4>;
+   clocks = <>;
+   };
+
+   /* dummy is 200M */
+   cpu: cpu@01c20054 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-cpu-clk";
+   reg = <0x01c20054 0x4>;
+   clocks = <>, <>, <>, <>;
+   };
+
+   axi: axi@01c20054 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-axi-clk";
+   reg = <0x01c20054 0x4>;
+   clocks = <>;
+   };
+
+   axi_gates: axi_gates@01c2005c {
+   #clock-cells = <1>;
+   compatible = "allwinner,sun4i-axi-gates-clk";
+   reg = <0x01c2005c 0x4>;
+   clocks = <>;
+   clock-output-names = "axi_dram";
+   };
+
+   ahb: ahb@01c20054 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-ahb-clk";
+   reg = <0x01c20054 0x4>;
+   clocks = <>;
+   };
+
+   ahb_gates: ahb_gates@01c20060 {
+   #clock-cells = <1>;
+   compatible = "allwinner,sun4i-ahb-gates-clk";
+   reg = <0x01c20060 0x8>;
+   clocks = <>;
+   clock-output-names = "ahb_usb0", "ahb_ehci0",
+   "ahb_ohci0", "ahb_ehci1", "ahb_ohci1", "ahb_ss",
+   "ahb_dma", "ahb_bist", "ahb_mmc0", "ahb_mmc1",
+   "ahb_mmc2", "ahb_mmc3", "ahb_ms", "ahb_nand",
+   "ahb_sdram", "ahb_ace", "ahb_emac", "ahb_ts",
+   "ahb_spi0", "ahb_spi1", "ahb_spi2", "ahb_spi3",
+   "ahb_pata", "ahb_sata", "ahb_gps", "ahb_ve",
+   "ahb_tvd", "ahb_tve0", "ahb_tve1", "ahb_lcd0",
+   "ahb_lcd1", "ahb_csi0", "ahb_csi1", "ahb_hdmi",
+   "ahb_de_be0", "ahb_de_be1", "ahb_de_fe0",
+   "ahb_de_fe1", "ahb_mp", "ahb_mali400";
+   };
+
+   apb0: apb0@01c20054 {
+   #clock-cells = <0>;
+   compatible = "allwinner,sun4i-apb0-clk";
+   reg = <0x01c20054 0x4>;
+   clocks = <>;
+   };
+
+   apb0_gates: apb0_gates@01c20068 {
+   #clock-cells = <1>;
+   compatible = 

[PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-09 Thread Maxime Ripard
Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com
---
 arch/arm/boot/dts/sun5i-a10s.dtsi | 286 ++
 1 file changed, 286 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi

diff --git a/arch/arm/boot/dts/sun5i-a10s.dtsi 
b/arch/arm/boot/dts/sun5i-a10s.dtsi
new file mode 100644
index 000..2307ce8
--- /dev/null
+++ b/arch/arm/boot/dts/sun5i-a10s.dtsi
@@ -0,0 +1,286 @@
+/*
+ * Copyright 2013 Maxime Ripard
+ *
+ * Maxime Ripard maxime.rip...@free-electrons.com
+ *
+ * The code contained herein is licensed under the GNU General Public
+ * License. You may obtain a copy of the GNU General Public License
+ * Version 2 or later at the following locations:
+ *
+ * http://www.opensource.org/licenses/gpl-license.html
+ * http://www.gnu.org/copyleft/gpl.html
+ */
+
+/include/ skeleton.dtsi
+
+/ {
+   interrupt-parent = intc;
+
+   cpus {
+   cpu@0 {
+   compatible = arm,cortex-a8;
+   };
+   };
+
+   memory {
+   reg = 0x4000 0x2000;
+   };
+
+   clocks {
+   #address-cells = 1;
+   #size-cells = 1;
+   ranges;
+
+   /*
+* This is a dummy clock, to be used as placeholder on
+* other mux clocks when a specific parent clock is not
+* yet implemented. It should be dropped when the driver
+* is complete.
+*/
+   dummy: dummy {
+   #clock-cells = 0;
+   compatible = fixed-clock;
+   clock-frequency = 0;
+   };
+
+   osc24M: osc24M@01c20050 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-osc-clk;
+   reg = 0x01c20050 0x4;
+   clock-frequency = 2400;
+   };
+
+   osc32k: osc32k {
+   #clock-cells = 0;
+   compatible = fixed-clock;
+   clock-frequency = 32768;
+   };
+
+   pll1: pll1@01c2 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-pll1-clk;
+   reg = 0x01c2 0x4;
+   clocks = osc24M;
+   };
+
+   /* dummy is 200M */
+   cpu: cpu@01c20054 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-cpu-clk;
+   reg = 0x01c20054 0x4;
+   clocks = osc32k, osc24M, pll1, dummy;
+   };
+
+   axi: axi@01c20054 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-axi-clk;
+   reg = 0x01c20054 0x4;
+   clocks = cpu;
+   };
+
+   axi_gates: axi_gates@01c2005c {
+   #clock-cells = 1;
+   compatible = allwinner,sun4i-axi-gates-clk;
+   reg = 0x01c2005c 0x4;
+   clocks = axi;
+   clock-output-names = axi_dram;
+   };
+
+   ahb: ahb@01c20054 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-ahb-clk;
+   reg = 0x01c20054 0x4;
+   clocks = axi;
+   };
+
+   ahb_gates: ahb_gates@01c20060 {
+   #clock-cells = 1;
+   compatible = allwinner,sun4i-ahb-gates-clk;
+   reg = 0x01c20060 0x8;
+   clocks = ahb;
+   clock-output-names = ahb_usb0, ahb_ehci0,
+   ahb_ohci0, ahb_ehci1, ahb_ohci1, ahb_ss,
+   ahb_dma, ahb_bist, ahb_mmc0, ahb_mmc1,
+   ahb_mmc2, ahb_mmc3, ahb_ms, ahb_nand,
+   ahb_sdram, ahb_ace, ahb_emac, ahb_ts,
+   ahb_spi0, ahb_spi1, ahb_spi2, ahb_spi3,
+   ahb_pata, ahb_sata, ahb_gps, ahb_ve,
+   ahb_tvd, ahb_tve0, ahb_tve1, ahb_lcd0,
+   ahb_lcd1, ahb_csi0, ahb_csi1, ahb_hdmi,
+   ahb_de_be0, ahb_de_be1, ahb_de_fe0,
+   ahb_de_fe1, ahb_mp, ahb_mali400;
+   };
+
+   apb0: apb0@01c20054 {
+   #clock-cells = 0;
+   compatible = allwinner,sun4i-apb0-clk;
+   reg = 0x01c20054 0x4;
+   clocks = ahb;
+   };
+
+   apb0_gates: apb0_gates@01c20068 {
+   #clock-cells = 1;
+   compatible = allwinner,sun4i-apb0-gates-clk;
+   reg = 0x01c20068 0x4;
+ 

Re: [PATCH 4/6] ARM: sunxi: dt: Add Allwinner A10s DTSI

2013-06-09 Thread Arnd Bergmann
On Sunday 09 June 2013 18:36:05 Maxime Ripard wrote:
 Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com
 ---
  arch/arm/boot/dts/sun5i-a10s.dtsi | 286 
 ++
  1 file changed, 286 insertions(+)
  create mode 100644 arch/arm/boot/dts/sun5i-a10s.dtsi

Would it make sense to split out the parts that are common with A13 into
a separate sun5i.dtsi file?

Arnd
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/