arm64: allwinner: a64: Add A64-OLinuXino initial support

PATCH Superseded Jagan Teki 2 participants 6 messages Aug 12, 2017 – Aug 14, 2017

Subsystems (2)

Device tree bindings Arm/Allwinner sunXi SoC support

lore

Share & tools

lore

Jagan Teki

author

2017-08-12 17:54 UTC #0 lore raw

PATCH v2 1/1 arm64: allwinner: a64: Add A64-OLinuXino initial support

From: Jagan Teki <[email protected]>

OLimex A64-OLinuXino is an open-source hardware board using the Allwinner A64 SOC.

OLimex A64-OLinuXino has
- A64 Quad-core Cortex-A53 64bit
- 1GB or 2GB RAM DDR3L @ 672Mhz
- microSD slot and 4/8/16GB eMMC
- Debug TTL UART
- HDMI
- LCD
- IR receiver
- 5V DC power supply

Signed-off-by Jagan Teki <[email protected]>

---
Changes for v2:
- Tested on board
- Added AXP803 regulator

 arch/arm64/boot/dts/allwinner/Makefile             |   1 +
 .../boot/dts/allwinner/sun50i-a64-olinuxino.dts    | 221 +++++++++++++++++++++
 2 files changed, 222 insertions(+)
 create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts

arch/arm64/boot/dts/allwinner/Makefile +1 -0

index c997b5c..19c3fbd 100644
@@ -1,5 +1,6 @@
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-bananapi-m64.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-nanopi-a64.dtb
+dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-olinuxino.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-orangepi-win.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-pine64-plus.dtb sun50i-a64-pine64.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-sopine-baseboard.dtb

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +221 -0

221 lines not on this page · raw message

Signature · 1 line

2.7.4

--
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe at googlegroups.com. For more options, visit https://groups.google.com/d/optout.

Jagan Teki

author

2017-08-14 07:34 UTC #2 lore raw

Quoted 18 lines · Chen-Yu Tsai

Hi,
Quoted 5 lines · Jagan Teki
From: Jagan Teki <[email protected]>
OLimex A64-OLinuXino is an open-source hardware board using the Allwinner A64 SOC.
Quoted 9 lines
OLimex A64-OLinuXino has
- A64 Quad-core Cortex-A53 64bit
- 1GB or 2GB RAM DDR3L @ 672Mhz
- microSD slot and 4/8/16GB eMMC
- Debug TTL UART
- HDMI
- LCD
- IR receiver
- 5V DC power supply

Quoted 1 line

Have you tried out Ethernet?

Not, tested yet. Will send separate patch.

Quoted 147 lines

Quoted 92 lines

Signed-off-by Jagan Teki <[email protected]>

---
Changes for v2:
- Tested on board
- Added AXP803 regulator

 arch/arm64/boot/dts/allwinner/Makefile             |   1 +
 .../boot/dts/allwinner/sun50i-a64-olinuxino.dts    | 221 +++++++++++++++++++++
 2 files changed, 222 insertions(+)
 create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts

arch/arm64/boot/dts/allwinner/Makefile +1 -0

index c997b5c..19c3fbd 100644
@@ -1,5 +1,6 @@
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-bananapi-m64.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-nanopi-a64.dtb
+dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-olinuxino.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-orangepi-win.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-pine64-plus.dtb sun50i-a64-pine64.dtb
 dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-sopine-baseboard.dtb

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +62 -0

new file mode 100644
index 0000000..0b9e742
@@ -0,0 +1,221 @@
+/*
+ * Copyright (C) 2017 Jagan Teki <[email protected]>
+ *
+ * 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 library 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 library 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.
+ */
+
+/dts-v1/;
+
+#include "sun50i-a64.dtsi"
+
+#include <dt-bindings/gpio/gpio.h>
+
+/ {
+       model = "Olimex A64-Olinuxino";
+       compatible = "olimex,a64-olinuxino", "allwinner,sun50i-a64";
+
+       aliases {
+               serial0 = &uart0;
+       };
+
+       chosen {
+               stdout-path = "serial0:115200n8";
+       };
+
+       reg_vcc1v8: vcc1v8 {
+               compatible = "regulator-fixed";

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 72

Quoted 10 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +10 -0

+               regulator-name = "vcc1v8";
+               regulator-min-microvolt = <1800000>;
+               regulator-max-microvolt = <1800000>;
+       };
+};
+
+&i2c1 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&i2c1_pins>;
+       status = "okay";

Please add a note saying what this is for. There should be a good reason to enable a non or only semi discoverable bus.

Quoted 41 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +41 -0

+};
+
+&i2c1_pins {
+       bias-pull-up;
+};
+
+&mmc0 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&mmc0_pins>;
+       vmmc-supply = <&reg_dcdc1>;
+       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>;
+       cd-inverted;
+       disable-wp;
+       bus-width = <4>;
+       status = "okay";
+};
+
+&r_rsb {
+       status = "okay";
+
+       axp803: pmic at 3a3 {
+               compatible = "x-powers,axp803";
+               reg = <0x3a3>;
+               interrupt-parent = <&r_intc>;
+               interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
+       };
+};
+
+&uart0 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&uart0_pins_a>;
+       status = "okay";
+};
+
+#include "axp803.dtsi"
+
+&reg_aldo1 {
+       regulator-always-on;
+       regulator-min-microvolt = <2800000>;
+       regulator-max-microvolt = <2800000>;
+       regulator-name = "vcc-pe";

Quoted 8 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 119
Quoted 6 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +6 -0
+};
+
+&reg_aldo2 {
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
Schematic says 1.8V?

I saw ALD02 VCC-PL=3.3v on schematic.

Quoted 16 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +15 -0

+       regulator-name = "vcc-pl";
+};
+
+&reg_aldo3 {
+       regulator-always-on;
+       regulator-min-microvolt = <3000000>;
+       regulator-max-microvolt = <3000000>;
+       regulator-name = "vcc-pll-avcc";
+};
+
+&reg_dcdc1 {
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-3v3";

Quoted 8 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 140
Quoted 6 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +6 -0
+};
+
+&reg_dcdc2 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
The SoC has a wider range than a fixed voltage for the CPUs.

So, max 1.1v can't enough CPUs? thought similar behavior with pine64?

Quoted 12 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 149
Quoted 10 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +9 -0
+       regulator-name = "vddfb-cpux";
+};
+
+&reg_dcdc3 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
+       regulator-name = "vddfb-cpuxx";
+};
It looks like DCDC3 is tied to DCDC2 in poly-phase mode? Please check.

Yes, will chage

Quoted 32 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +31 -0

+
+&reg_dcdc5 {
+       regulator-always-on;
+       regulator-min-microvolt = <1500000>;
+       regulator-max-microvolt = <1500000>;
+       regulator-name = "vcc-ddr3";
+};
+
+&reg_dcdc6 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
+       regulator-name = "vdd-sys";
+};
+
+&reg_dldo1 {
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-hdmi";
+};
+
+&reg_dldo2 {
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-mipi";
+};
+
+&reg_dldo3 {
+       regulator-min-microvolt = <2800000>;
+       regulator-max-microvolt = <2800000>;
+       regulator-name = "vcc-avdd-csi";

Quoted 8 lines

arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 186
Quoted 6 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +6 -0
+};
+
+&reg_dldo4 {
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-wifiio";
"wifi-io" is more readable.

thanks!

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Chen-Yu Tsai

2017-08-14 07:56 UTC #3 lore raw

Quoted 19 lines · Jagan Teki

Quoted 18 lines · Chen-Yu Tsai
Hi,
Quoted 14 lines · Jagan Teki
From: Jagan Teki <[email protected]>
OLimex A64-OLinuXino is an open-source hardware board using the Allwinner A64 SOC.
OLimex A64-OLinuXino has
- A64 Quad-core Cortex-A53 64bit
- 1GB or 2GB RAM DDR3L @ 672Mhz
- microSD slot and 4/8/16GB eMMC
- Debug TTL UART
- HDMI
- LCD
- IR receiver
- 5V DC power supply

Quoted 3 lines

Quoted 1 line
Have you tried out Ethernet?
Not, tested yet. Will send separate patch.

OK.

[...]

Quoted 9 lines

Quoted 7 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 119
Quoted 5 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +5 -0
+
+&reg_aldo2 {
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
Schematic says 1.8V?
I saw ALD02 VCC-PL=3.3v on schematic.

Ah. I see they changed it in a rev. B. Lets use 3.3V then.

Quoted 25 lines

Quoted 16 lines
Quoted 16 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +15 -0
+       regulator-name = "vcc-pl";
+};
+
+&reg_aldo3 {
+       regulator-always-on;
+       regulator-min-microvolt = <3000000>;
+       regulator-max-microvolt = <3000000>;
+       regulator-name = "vcc-pll-avcc";
+};
+
+&reg_dcdc1 {
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-3v3";
Quoted 8 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 140
Quoted 6 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +6 -0
+};
+
+&reg_dcdc2 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
The SoC has a wider range than a fixed voltage for the CPUs.

Quoted 1 line

So, max 1.1v can't enough CPUs? thought similar behavior with pine64?

The recommended operating conditions says 1.04 ~ 1.3 V for CPUx, while the typical voltage is 1.1V. The regulator constraints should match the datasheet, especially for power rails that are involved in DVFS.

ChenYu

Jagan Teki

author

2017-08-14 09:09 UTC #4 lore raw

Quoted 65 lines · Chen-Yu Tsai

Quoted 19 lines · Jagan Teki
Quoted 18 lines · Chen-Yu Tsai
Hi,
Quoted 14 lines · Jagan Teki
From: Jagan Teki <[email protected]>
OLimex A64-OLinuXino is an open-source hardware board using the Allwinner A64 SOC.
OLimex A64-OLinuXino has
- A64 Quad-core Cortex-A53 64bit
- 1GB or 2GB RAM DDR3L @ 672Mhz
- microSD slot and 4/8/16GB eMMC
- Debug TTL UART
- HDMI
- LCD
- IR receiver
- 5V DC power supply
Quoted 3 lines
Quoted 1 line
Have you tried out Ethernet?
Not, tested yet. Will send separate patch.
OK.
[...]
Quoted 9 lines
Quoted 7 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 119
Quoted 5 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +5 -0
+
+&reg_aldo2 {
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
Schematic says 1.8V?
I saw ALD02 VCC-PL=3.3v on schematic.
Ah. I see they changed it in a rev. B. Lets use 3.3V then.
Quoted 25 lines
Quoted 24 lines
Quoted 12 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +11 -0
+       regulator-name = "vcc-pl";
+};
+
+&reg_aldo3 {
+       regulator-always-on;
+       regulator-min-microvolt = <3000000>;
+       regulator-max-microvolt = <3000000>;
+       regulator-name = "vcc-pll-avcc";
+};
+
+&reg_dcdc1 {
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 140
Quoted 10 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +10 -0
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-3v3";
+};
+
+&reg_dcdc2 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
The SoC has a wider range than a fixed voltage for the CPUs.

Quoted 6 lines

Quoted 1 line
So, max 1.1v can't enough CPUs? thought similar behavior with pine64?
The recommended operating conditions says 1.04 ~ 1.3 V for CPUx, while the typical voltage is 1.1V. The regulator constraints should match the datasheet, especially for power rails that are involved in DVFS.

power rails from datasheet[1] shows 0.5v ~ 1.3v am I not looking updated details?

[1] http://files.pine64.org/doc/datasheet/pine64/AXP803_Datasheet_V1.0.pdf

thanks!

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Chen-Yu Tsai

2017-08-14 09:13 UTC #5 lore raw

Quoted 6 lines · Jagan Teki

Quoted 5 lines · Chen-Yu Tsai
Quoted 4 lines · Jagan Teki
Quoted 3 lines · Chen-Yu Tsai
Hi,
On Sun, Aug 13, 2017 at 1:54 AM, Jagan Teki <[email protected]> wrote:

[...]

Quoted 16 lines

Quoted 16 lines
Quoted 15 lines
Quoted 5 lines
Quoted 5 lines
+&reg_dcdc1 {
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +4 -0
+       regulator-always-on;
+       regulator-min-microvolt = <3300000>;
+       regulator-max-microvolt = <3300000>;
+       regulator-name = "vcc-3v3";
Quoted 8 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts line 140
Quoted 6 lines
arch/arm64/boot/dts/allwinner/sun50i-a64-olinuxino.dts new file +6 -0
+};
+
+&reg_dcdc2 {
+       regulator-always-on;
+       regulator-min-microvolt = <1100000>;
+       regulator-max-microvolt = <1100000>;
The SoC has a wider range than a fixed voltage for the CPUs.
So, max 1.1v can't enough CPUs? thought similar behavior with pine64?

Quoted 7 lines

Quoted 4 lines
The recommended operating conditions says 1.04 ~ 1.3 V for CPUx, while the typical voltage is 1.1V. The regulator constraints should match the datasheet, especially for power rails that are involved in DVFS.
power rails from datasheet[1] shows 0.5v ~ 1.3v am I not looking updated details?

The constraints are there to prevent users from setting a voltage too high or too low for the connected devices.

You should be looking at the recommended operating conditions for the A64. The regulator constraints should match the constraints of the consumer, not the provider. If there are multiple consumers, it must match the narrowest range supported by _all_ consumers, i.e. the intersection of all consumer's operating ranges.

ChenYu

Quoted 9 lines

[1] http://files.pine64.org/doc/datasheet/pine64/AXP803_Datasheet_V1.0.pdf
thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.