Thread (19 messages) 19 messages, 4 authors, 2015-10-02

[PATCH 3/4] ARM: dts: sun8i: Add Allwinner A83T dtsi

From: Chen-Yu Tsai <hidden>
Date: 2015-09-22 16:15:54
Also in: linux-devicetree, linux-gpio, lkml

On Tue, Sep 22, 2015 at 11:54 PM, Maxime Ripard
[off-list ref] wrote:
On Tue, Sep 22, 2015 at 11:38:56PM +0800, Vishnu Patekar wrote:
quoted
Allwinner A83T is new octa-core cortex-a7 SOC.
This adds the basic dtsi, the clocks differs from
earlier sun8i SOCs.

Signed-off-by: Vishnu Patekar <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 243 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 243 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun8i-a83t.dtsi
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
new file mode 100644
index 0000000..f6ddd9c
--- /dev/null
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -0,0 +1,243 @@
+/*
+ * Copyright 2015 Vishnu Patekar
+ *
+ * Vishnu Patekar <vishnupatekar0510@gmail.com>
+ *
+ * This file is dual-licensed: you can use it either under the terms
+ * of the GPL or the X11 license, at your option. Note that this dual
+ * licensing only applies to this file, and not this project as a
+ * whole.
+ *
+ *  a) This file is free software; you can redistribute it and/or
+ *     modify it under the terms of the GNU General Public License as
+ *     published by the Free Software Foundation; either version 2 of the
+ *     License, or (at your option) any later version.
+ *
+ *     This file is distributed in the hope that it will be useful,
+ *     but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *     GNU General Public License for more details.
+ *
+ * Or, alternatively,
+ *
+ *  b) Permission is hereby granted, free of charge, to any person
+ *     obtaining a copy of this software and associated documentation
+ *     files (the "Software"), to deal in the Software without
+ *     restriction, including without limitation the rights to use,
+ *     copy, modify, merge, publish, distribute, sublicense, and/or
+ *     sell copies of the Software, and to permit persons to whom the
+ *     Software is furnished to do so, subject to the following
+ *     conditions:
+ *
+ *     The above copyright notice and this permission notice shall be
+ *     included in all copies or substantial portions of the Software.
+ *
+ *     THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+ *     EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES
+ *     OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+ *     NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
+ *     HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY,
+ *     WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ *     FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
+ *     OTHER DEALINGS IN THE SOFTWARE.
+
+ */
+
+#include "skeleton.dtsi"
+
+#include <dt-bindings/interrupt-controller/arm-gic.h>
+
+#include <dt-bindings/pinctrl/sun4i-a10.h>
+
+/ {
+     interrupt-parent = <&gic>;
+
+     chosen {
+             #address-cells = <1>;
+             #size-cells = <1>;
+             ranges;
+     };
+
+     cpus {
+             #address-cells = <1>;
+             #size-cells = <0>;
+
+             cpu at 0 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <0>;
+             };
+
+             cpu at 1 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <1>;
+             };
+
+             cpu at 2 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <2>;
+             };
+
+             cpu at 3 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <3>;
+             };
+             cpu at 4 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <4>;
These don't look right. See Documentation/devicetree/bindings/arm/cpus.txt
Cluster 1 would be 0x100 ~ 0x103.
quoted
+             };
+
+             cpu at 5 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <5>;
+             };
+             cpu at 6 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <6>;
+             };
+
+             cpu at 7 {
+                     compatible = "arm,cortex-a7";
+                     device_type = "cpu";
+                     reg = <7>;
+             };
+     };
+
+     memory {
+             reg = <0x40000000 0x80000000>;
+     };
+
+     timer {
+             compatible = "arm,armv7-timer";
+             interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
+                          <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
+                          <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,
+                          <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>;
+             clock-frequency = <24000000>;
+             arm,cpu-registers-not-fw-configured;
+     };
+
+     clocks {
+             #address-cells = <1>;
+             #size-cells = <1>;
+             ranges;
+
+             osc24M: osc24M_clk {
+                     #clock-cells = <0>;
+                     compatible = "fixed-clock";
+                     clock-frequency = <24000000>;
+                     clock-output-names = "osc24M";
+             };
+
+             osc32k: osc32k_clk {
+                     #clock-cells = <0>;
+                     compatible = "fixed-clock";
+                     clock-frequency = <32768>;
+                     clock-output-names = "osc32k";
+             };
+     };
+
+     soc at 01c00000 {
+             compatible = "simple-bus";
+             #address-cells = <1>;
+             #size-cells = <1>;
+             ranges;
+
+             pio: pinctrl at 01c20800 {
+                     compatible = "allwinner,sun8i-a83t-pinctrl";
+                     interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>,
+                     <GIC_SPI 17 IRQ_TYPE_LEVEL_HIGH>,
+                     <GIC_SPI 19 IRQ_TYPE_LEVEL_HIGH>;
+                     /* compatible gets set in SoC specific dtsi file */
Which DTSI?
quoted
+                     reg = <0x01c20800 0x400>;
+                     /* interrupts get set in SoC specific dtsi file */
Ditto
quoted
+                     clocks = <&osc24M>;
+                     gpio-controller;
+                     interrupt-controller;
+                     #interrupt-cells = <3>;
+                     #gpio-cells = <3>;
+
+                     uart0_pins_a: uart0 at 0 {
+                             allwinner,pins = "PF2", "PF4";
+                             allwinner,function = "uart0";
+                             allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
Could you use PB9 & PB10 for uart0 pins, and not the PF ones?

http://linux-sunxi.org/User:Simos/H8HomletProtoV20_A83T shows
they are available on the headers, and I tested them before.

Thanks!
ChenYu
quoted
+
+                     mmc0_pins_a: mmc0 at 0 {
+                             allwinner,pins = "PF0", "PF1", "PF2",
+                                              "PF3", "PF4", "PF5";
+                             allwinner,function = "mmc0";
+                             allwinner,drive = <SUN4I_PINCTRL_30_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
+
+                     mmc1_pins_a: mmc1 at 0 {
+                             allwinner,pins = "PG0", "PG1", "PG2",
+                                              "PG3", "PG4", "PG5";
+                             allwinner,function = "mmc1";
+                             allwinner,drive = <SUN4I_PINCTRL_30_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
+
+                     mmc2_8bit_pins: mmc2_8bit {
+                             allwinner,pins = "PC5", "PC6", "PC8",
+                                              "PC9", "PC10", "PC11",
+                                              "PC12", "PC13", "PC14",
+                                              "PC15";
Would we also want PC16 (SDC2_RST)?
quoted
+                             allwinner,function = "mmc2";
+                             allwinner,drive = <SUN4I_PINCTRL_30_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
+
+                     i2c0_pins_a: i2c0 at 0 {
+                             allwinner,pins = "PH0", "PH1";
+                             allwinner,function = "i2c0";
+                             allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
+
+                     i2c1_pins_a: i2c1 at 0 {
+                             allwinner,pins = "PH2", "PH3";
+                             allwinner,function = "i2c1";
+                             allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
+
+                     i2c2_pins_a: i2c2 at 0 {
+                             allwinner,pins = "PH4", "PH5";
+                             allwinner,function = "i2c2";
+                             allwinner,drive = <SUN4I_PINCTRL_10_MA>;
+                             allwinner,pull = <SUN4I_PINCTRL_NO_PULL>;
+                     };
Please order those by alphabetical order.
quoted
+             };
+
+             uart0: serial at 01c28000 {
+                     compatible = "snps,dw-apb-uart";
+                     reg = <0x01c28000 0x400>;
+                     interrupts = <GIC_SPI 0 IRQ_TYPE_LEVEL_HIGH>;
+                     reg-shift = <2>;
+                     reg-io-width = <4>;
+                     clocks = <&osc24M>;
+                     status = "disabled";
+             };
+
+             gic: interrupt-controller at 01c81000 {
+                     compatible = "arm,cortex-a7-gic", "arm,cortex-a15-gic";
+                     reg = <0x01c81000 0x1000>,
+                           <0x01c82000 0x1000>,
+                           <0x01c84000 0x2000>,
+                           <0x01c86000 0x2000>;
+                     interrupt-controller;
+                     #interrupt-cells = <3>;
+                     interrupts = <GIC_PPI 9 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>;
+             };
+
+     };
+};
--
1.9.1
Thanks!
Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help