All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-02 15:05 ` Icenowy Zheng
  0 siblings, 0 replies; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-02 15:05 UTC (permalink / raw)
  To: Maxime Ripard, Chen-Yu Tsai, Rob Herring, Russell King,
	Andre Przywara, Hans de Goede, Arnd Bergmann, Vishnu Patekar
  Cc: linux-doc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw, Icenowy Zheng

Allwinner H2+ is a quad-core Cortex-A7 SoC.

It is very like H3, that they share the same SoC ID (0x1680), and H3
memory maps as well as drivers works well on the SoC.

Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
---
Changes since v2:
- Changed compatible from allwinner,sun8i-h2plus to allwinner,sun8i-h2-plus.
 Documentation/arm/sunxi/README                  | 4 ++++
 Documentation/devicetree/bindings/arm/sunxi.txt | 1 +
 arch/arm/mach-sunxi/sunxi.c                     | 1 +
 3 files changed, 6 insertions(+)

diff --git a/Documentation/arm/sunxi/README b/Documentation/arm/sunxi/README
index cd02433..1fe4d99c 100644
--- a/Documentation/arm/sunxi/README
+++ b/Documentation/arm/sunxi/README
@@ -63,6 +63,10 @@ SunXi family
         + User Manual
           http://dl.linux-sunxi.org/A33/A33%20user%20manual%20release%201.1.pdf
 
+      - Allwinner H2+ (sun8i)
+        + No document available now, but is known to be working properly with
+          H3 drivers and memory map.
+
       - Allwinner H3 (sun8i)
         + Datasheet
           http://dl.linux-sunxi.org/H3/Allwinner_H3_Datasheet_V1.0.pdf
diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
index 4d6467c..59b143f 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.txt
+++ b/Documentation/devicetree/bindings/arm/sunxi.txt
@@ -13,6 +13,7 @@ using one of the following compatible strings:
   allwinner,sun8i-a33
   allwinner,sun8i-a83t
   allwinner,sun8i-h3
+  allwinner,sun8i-h2-plus
   allwinner,sun9i-a80
   allwinner,sun50i-a64
   nextthing,gr8
diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
index 2e2bde2..320d2af 100644
--- a/arch/arm/mach-sunxi/sunxi.c
+++ b/arch/arm/mach-sunxi/sunxi.c
@@ -63,6 +63,7 @@ static const char * const sun8i_board_dt_compat[] = {
 	"allwinner,sun8i-a23",
 	"allwinner,sun8i-a33",
 	"allwinner,sun8i-a83t",
+	"allwinner,sun8i-h2-plus",
 	"allwinner,sun8i-h3",
 	NULL,
 };
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-02 15:05 ` Icenowy Zheng
  0 siblings, 0 replies; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-02 15:05 UTC (permalink / raw)
  To: linux-arm-kernel

Allwinner H2+ is a quad-core Cortex-A7 SoC.

It is very like H3, that they share the same SoC ID (0x1680), and H3
memory maps as well as drivers works well on the SoC.

Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
---
Changes since v2:
- Changed compatible from allwinner,sun8i-h2plus to allwinner,sun8i-h2-plus.
 Documentation/arm/sunxi/README                  | 4 ++++
 Documentation/devicetree/bindings/arm/sunxi.txt | 1 +
 arch/arm/mach-sunxi/sunxi.c                     | 1 +
 3 files changed, 6 insertions(+)

diff --git a/Documentation/arm/sunxi/README b/Documentation/arm/sunxi/README
index cd02433..1fe4d99c 100644
--- a/Documentation/arm/sunxi/README
+++ b/Documentation/arm/sunxi/README
@@ -63,6 +63,10 @@ SunXi family
         + User Manual
           http://dl.linux-sunxi.org/A33/A33%20user%20manual%20release%201.1.pdf
 
+      - Allwinner H2+ (sun8i)
+        + No document available now, but is known to be working properly with
+          H3 drivers and memory map.
+
       - Allwinner H3 (sun8i)
         + Datasheet
           http://dl.linux-sunxi.org/H3/Allwinner_H3_Datasheet_V1.0.pdf
diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
index 4d6467c..59b143f 100644
--- a/Documentation/devicetree/bindings/arm/sunxi.txt
+++ b/Documentation/devicetree/bindings/arm/sunxi.txt
@@ -13,6 +13,7 @@ using one of the following compatible strings:
   allwinner,sun8i-a33
   allwinner,sun8i-a83t
   allwinner,sun8i-h3
+  allwinner,sun8i-h2-plus
   allwinner,sun9i-a80
   allwinner,sun50i-a64
   nextthing,gr8
diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
index 2e2bde2..320d2af 100644
--- a/arch/arm/mach-sunxi/sunxi.c
+++ b/arch/arm/mach-sunxi/sunxi.c
@@ -63,6 +63,7 @@ static const char * const sun8i_board_dt_compat[] = {
 	"allwinner,sun8i-a23",
 	"allwinner,sun8i-a33",
 	"allwinner,sun8i-a83t",
+	"allwinner,sun8i-h2-plus",
 	"allwinner,sun8i-h3",
 	NULL,
 };
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
  2016-12-02 15:05 ` Icenowy Zheng
@ 2016-12-02 15:05     ` Icenowy Zheng
  -1 siblings, 0 replies; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-02 15:05 UTC (permalink / raw)
  To: Maxime Ripard, Chen-Yu Tsai, Rob Herring, Russell King,
	Andre Przywara, Hans de Goede, Arnd Bergmann, Vishnu Patekar
  Cc: linux-doc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw, Icenowy Zheng

Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
SDIO Wi-Fi chip by Allwinner (XR819).

Add a device tree file for it.

Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
---
Changes since v2:
- Merged SDIO Wi-Fi patch into it.
- SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
- SDIO Wi-Fi: changed pinctrl binding to generic pinconf
- removed all gpio pinctrl nodes
- changed h2plus to h2-plus
Changes since v1:
- Convert to generic pinconf bindings.
- SDIO Wi-Fi: add patch.

Some notes:
- The uart1 and uart2 is available on the unsoldered gpio header.
- The onboard USB connector has its Vbus directly connected to DCIN-5V (the
  power jack)

 arch/arm/boot/dts/Makefile                        |   1 +
 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
 2 files changed, 160 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 6447abc..59f6e86 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
 	sun8i-a33-sinlinx-sina33.dtb \
 	sun8i-a83t-allwinner-h8homlet-v2.dtb \
 	sun8i-a83t-cubietruck-plus.dtb \
+	sun8i-h2-plus-orangepi-zero.dtb \
 	sun8i-h3-bananapi-m2-plus.dtb \
 	sun8i-h3-nanopi-neo.dtb \
 	sun8i-h3-orangepi-2.dtb \
diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
new file mode 100644
index 0000000..d18807f
--- /dev/null
+++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
@@ -0,0 +1,159 @@
+/*
+ * Copyright (C) 2016 Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
+ *
+ * Based on sun8i-h3-orangepi-one.dts, which is:
+ *   Copyright (C) 2016 Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
+ *
+ * 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.
+ */
+
+/dts-v1/;
+#include "sun8i-h3.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
+#include <dt-bindings/pinctrl/sun4i-a10.h>
+
+/ {
+	model = "Xunlong Orange Pi Zero";
+	compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
+
+	aliases {
+		serial0 = &uart0;
+		/* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
+		ethernet1 = &xr819;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	leds {
+		compatible = "gpio-leds";
+
+		pwr_led {
+			label = "orangepi:green:pwr";
+			gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
+			default-state = "on";
+		};
+
+		status_led {
+			label = "orangepi:red:status";
+			gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
+		};
+	};
+
+	reg_vcc_wifi: reg_vcc_wifi {
+		compatible = "regulator-fixed";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		regulator-name = "vcc-wifi";
+		enable-active-high;
+		gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
+	};
+
+	wifi_pwrseq: wifi_pwrseq {
+		compatible = "mmc-pwrseq-simple";
+		reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
+	};
+};
+
+&ehci1 {
+	status = "okay";
+};
+
+&mmc0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc0_pins_a>;
+	vmmc-supply = <&reg_vcc3v3>;
+	bus-width = <4>;
+	cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
+	cd-inverted;
+	status = "okay";
+};
+
+&mmc1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc1_pins_a>;
+	vmmc-supply = <&reg_vcc_wifi>;
+	mmc-pwrseq = <&wifi_pwrseq>;
+	bus-width = <4>;
+	non-removable;
+	status = "okay";
+
+	/*
+	 * Explicitly define the sdio device, so that we can add an ethernet
+	 * alias for it (which e.g. makes u-boot set a mac-address).
+	 */
+	xr819: sdio_wifi@1 {
+		reg = <1>;
+	};
+};
+
+&mmc1_pins_a {
+	bias-pull-up;
+};
+
+&ohci1 {
+	status = "okay";
+};
+
+&uart0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart0_pins_a>;
+	status = "okay";
+};
+
+&uart1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart1_pins>;
+	status = "disabled";
+};
+
+&uart2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart2_pins>;
+	status = "disabled";
+};
+
+&usbphy {
+	/* USB VBUS is always on */
+	status = "okay";
+};
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-02 15:05     ` Icenowy Zheng
  0 siblings, 0 replies; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-02 15:05 UTC (permalink / raw)
  To: linux-arm-kernel

Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
SDIO Wi-Fi chip by Allwinner (XR819).

Add a device tree file for it.

Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
---
Changes since v2:
- Merged SDIO Wi-Fi patch into it.
- SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
- SDIO Wi-Fi: changed pinctrl binding to generic pinconf
- removed all gpio pinctrl nodes
- changed h2plus to h2-plus
Changes since v1:
- Convert to generic pinconf bindings.
- SDIO Wi-Fi: add patch.

Some notes:
- The uart1 and uart2 is available on the unsoldered gpio header.
- The onboard USB connector has its Vbus directly connected to DCIN-5V (the
  power jack)

 arch/arm/boot/dts/Makefile                        |   1 +
 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
 2 files changed, 160 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 6447abc..59f6e86 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
 	sun8i-a33-sinlinx-sina33.dtb \
 	sun8i-a83t-allwinner-h8homlet-v2.dtb \
 	sun8i-a83t-cubietruck-plus.dtb \
+	sun8i-h2-plus-orangepi-zero.dtb \
 	sun8i-h3-bananapi-m2-plus.dtb \
 	sun8i-h3-nanopi-neo.dtb \
 	sun8i-h3-orangepi-2.dtb \
diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
new file mode 100644
index 0000000..d18807f
--- /dev/null
+++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
@@ -0,0 +1,159 @@
+/*
+ * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
+ *
+ * Based on sun8i-h3-orangepi-one.dts, which is:
+ *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
+ */
+
+/dts-v1/;
+#include "sun8i-h3.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
+#include <dt-bindings/pinctrl/sun4i-a10.h>
+
+/ {
+	model = "Xunlong Orange Pi Zero";
+	compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
+
+	aliases {
+		serial0 = &uart0;
+		/* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
+		ethernet1 = &xr819;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	leds {
+		compatible = "gpio-leds";
+
+		pwr_led {
+			label = "orangepi:green:pwr";
+			gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
+			default-state = "on";
+		};
+
+		status_led {
+			label = "orangepi:red:status";
+			gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
+		};
+	};
+
+	reg_vcc_wifi: reg_vcc_wifi {
+		compatible = "regulator-fixed";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		regulator-name = "vcc-wifi";
+		enable-active-high;
+		gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
+	};
+
+	wifi_pwrseq: wifi_pwrseq {
+		compatible = "mmc-pwrseq-simple";
+		reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
+	};
+};
+
+&ehci1 {
+	status = "okay";
+};
+
+&mmc0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc0_pins_a>;
+	vmmc-supply = <&reg_vcc3v3>;
+	bus-width = <4>;
+	cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
+	cd-inverted;
+	status = "okay";
+};
+
+&mmc1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc1_pins_a>;
+	vmmc-supply = <&reg_vcc_wifi>;
+	mmc-pwrseq = <&wifi_pwrseq>;
+	bus-width = <4>;
+	non-removable;
+	status = "okay";
+
+	/*
+	 * Explicitly define the sdio device, so that we can add an ethernet
+	 * alias for it (which e.g. makes u-boot set a mac-address).
+	 */
+	xr819: sdio_wifi at 1 {
+		reg = <1>;
+	};
+};
+
+&mmc1_pins_a {
+	bias-pull-up;
+};
+
+&ohci1 {
+	status = "okay";
+};
+
+&uart0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart0_pins_a>;
+	status = "okay";
+};
+
+&uart1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart1_pins>;
+	status = "disabled";
+};
+
+&uart2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart2_pins>;
+	status = "disabled";
+};
+
+&usbphy {
+	/* USB VBUS is always on */
+	status = "okay";
+};
-- 
2.10.2

^ permalink raw reply related	[flat|nested] 32+ messages in thread

* Re: [linux-sunxi] [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-02 16:41       ` Chen-Yu Tsai
  0 siblings, 0 replies; 32+ messages in thread
From: Chen-Yu Tsai @ 2016-12-02 16:41 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Maxime Ripard, Chen-Yu Tsai, Rob Herring, Russell King,
	Andre Przywara, Hans de Goede, Arnd Bergmann, Vishnu Patekar,
	linux-doc, linux-arm-kernel, linux-kernel, devicetree,
	linux-sunxi

Hi,

On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
>
> Add a device tree file for it.
>
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
>
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
>
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>         sun8i-a33-sinlinx-sina33.dtb \
>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>         sun8i-a83t-cubietruck-plus.dtb \
> +       sun8i-h2-plus-orangepi-zero.dtb \
>         sun8i-h3-bananapi-m2-plus.dtb \
>         sun8i-h3-nanopi-neo.dtb \
>         sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +       model = "Xunlong Orange Pi Zero";
> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +       aliases {
> +               serial0 = &uart0;
> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
> +               ethernet1 = &xr819;
> +       };
> +
> +       chosen {
> +               stdout-path = "serial0:115200n8";
> +       };
> +
> +       leds {
> +               compatible = "gpio-leds";
> +
> +               pwr_led {
> +                       label = "orangepi:green:pwr";
> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +                       default-state = "on";
> +               };
> +
> +               status_led {
> +                       label = "orangepi:red:status";
> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +               };
> +       };
> +
> +       reg_vcc_wifi: reg_vcc_wifi {
> +               compatible = "regulator-fixed";
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               regulator-name = "vcc-wifi";
> +               enable-active-high;
> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       wifi_pwrseq: wifi_pwrseq {
> +               compatible = "mmc-pwrseq-simple";
> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +       };
> +};
> +
> +&ehci1 {
> +       status = "okay";
> +};
> +
> +&mmc0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc0_pins_a>;
> +       vmmc-supply = <&reg_vcc3v3>;
> +       bus-width = <4>;
> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +       cd-inverted;
> +       status = "okay";
> +};
> +
> +&mmc1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc1_pins_a>;
> +       vmmc-supply = <&reg_vcc_wifi>;
> +       mmc-pwrseq = <&wifi_pwrseq>;
> +       bus-width = <4>;
> +       non-removable;
> +       status = "okay";
> +
> +       /*
> +        * Explicitly define the sdio device, so that we can add an ethernet
> +        * alias for it (which e.g. makes u-boot set a mac-address).
> +        */
> +       xr819: sdio_wifi@1 {
> +               reg = <1>;
> +       };
> +};
> +
> +&mmc1_pins_a {
> +       bias-pull-up;

This is already set in h3.dtsi

> +};
> +
> +&ohci1 {
> +       status = "okay";
> +};
> +
> +&uart0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart0_pins_a>;
> +       status = "okay";
> +};
> +
> +&uart1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart1_pins>;
> +       status = "disabled";
> +};
> +
> +&uart2 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart2_pins>;
> +       status = "disabled";
> +};
> +
> +&usbphy {
> +       /* USB VBUS is always on */

I think this comment could use a little work.

AFAIK this board doesn't have an actual USB port.
It's just the D+/D- pins on the pin header, along
with the board-wide 5V, also on the pin header.

ChenYu

> +       status = "okay";
> +};
> --
> 2.10.2
>
> --
> 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@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-02 16:41       ` Chen-Yu Tsai
  0 siblings, 0 replies; 32+ messages in thread
From: Chen-Yu Tsai @ 2016-12-02 16:41 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Maxime Ripard, Chen-Yu Tsai, Rob Herring, Russell King,
	Andre Przywara, Hans de Goede, Arnd Bergmann, Vishnu Patekar,
	linux-doc-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel, linux-kernel,
	devicetree, linux-sunxi

Hi,

On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
>
> Add a device tree file for it.
>
> Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
>
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
>
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>         sun8i-a33-sinlinx-sina33.dtb \
>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>         sun8i-a83t-cubietruck-plus.dtb \
> +       sun8i-h2-plus-orangepi-zero.dtb \
>         sun8i-h3-bananapi-m2-plus.dtb \
>         sun8i-h3-nanopi-neo.dtb \
>         sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
> + *
> + * 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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +       model = "Xunlong Orange Pi Zero";
> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +       aliases {
> +               serial0 = &uart0;
> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
> +               ethernet1 = &xr819;
> +       };
> +
> +       chosen {
> +               stdout-path = "serial0:115200n8";
> +       };
> +
> +       leds {
> +               compatible = "gpio-leds";
> +
> +               pwr_led {
> +                       label = "orangepi:green:pwr";
> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +                       default-state = "on";
> +               };
> +
> +               status_led {
> +                       label = "orangepi:red:status";
> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +               };
> +       };
> +
> +       reg_vcc_wifi: reg_vcc_wifi {
> +               compatible = "regulator-fixed";
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               regulator-name = "vcc-wifi";
> +               enable-active-high;
> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       wifi_pwrseq: wifi_pwrseq {
> +               compatible = "mmc-pwrseq-simple";
> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +       };
> +};
> +
> +&ehci1 {
> +       status = "okay";
> +};
> +
> +&mmc0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc0_pins_a>;
> +       vmmc-supply = <&reg_vcc3v3>;
> +       bus-width = <4>;
> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +       cd-inverted;
> +       status = "okay";
> +};
> +
> +&mmc1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc1_pins_a>;
> +       vmmc-supply = <&reg_vcc_wifi>;
> +       mmc-pwrseq = <&wifi_pwrseq>;
> +       bus-width = <4>;
> +       non-removable;
> +       status = "okay";
> +
> +       /*
> +        * Explicitly define the sdio device, so that we can add an ethernet
> +        * alias for it (which e.g. makes u-boot set a mac-address).
> +        */
> +       xr819: sdio_wifi@1 {
> +               reg = <1>;
> +       };
> +};
> +
> +&mmc1_pins_a {
> +       bias-pull-up;

This is already set in h3.dtsi

> +};
> +
> +&ohci1 {
> +       status = "okay";
> +};
> +
> +&uart0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart0_pins_a>;
> +       status = "okay";
> +};
> +
> +&uart1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart1_pins>;
> +       status = "disabled";
> +};
> +
> +&uart2 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart2_pins>;
> +       status = "disabled";
> +};
> +
> +&usbphy {
> +       /* USB VBUS is always on */

I think this comment could use a little work.

AFAIK this board doesn't have an actual USB port.
It's just the D+/D- pins on the pin header, along
with the board-wide 5V, also on the pin header.

ChenYu

> +       status = "okay";
> +};
> --
> 2.10.2
>
> --
> 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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
> For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [linux-sunxi] [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-02 16:41       ` Chen-Yu Tsai
  0 siblings, 0 replies; 32+ messages in thread
From: Chen-Yu Tsai @ 2016-12-02 16:41 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
>
> Add a device tree file for it.
>
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
>
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
>
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>         sun8i-a33-sinlinx-sina33.dtb \
>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>         sun8i-a83t-cubietruck-plus.dtb \
> +       sun8i-h2-plus-orangepi-zero.dtb \
>         sun8i-h3-bananapi-m2-plus.dtb \
>         sun8i-h3-nanopi-neo.dtb \
>         sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +       model = "Xunlong Orange Pi Zero";
> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +       aliases {
> +               serial0 = &uart0;
> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
> +               ethernet1 = &xr819;
> +       };
> +
> +       chosen {
> +               stdout-path = "serial0:115200n8";
> +       };
> +
> +       leds {
> +               compatible = "gpio-leds";
> +
> +               pwr_led {
> +                       label = "orangepi:green:pwr";
> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +                       default-state = "on";
> +               };
> +
> +               status_led {
> +                       label = "orangepi:red:status";
> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +               };
> +       };
> +
> +       reg_vcc_wifi: reg_vcc_wifi {
> +               compatible = "regulator-fixed";
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               regulator-name = "vcc-wifi";
> +               enable-active-high;
> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       wifi_pwrseq: wifi_pwrseq {
> +               compatible = "mmc-pwrseq-simple";
> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +       };
> +};
> +
> +&ehci1 {
> +       status = "okay";
> +};
> +
> +&mmc0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc0_pins_a>;
> +       vmmc-supply = <&reg_vcc3v3>;
> +       bus-width = <4>;
> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +       cd-inverted;
> +       status = "okay";
> +};
> +
> +&mmc1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&mmc1_pins_a>;
> +       vmmc-supply = <&reg_vcc_wifi>;
> +       mmc-pwrseq = <&wifi_pwrseq>;
> +       bus-width = <4>;
> +       non-removable;
> +       status = "okay";
> +
> +       /*
> +        * Explicitly define the sdio device, so that we can add an ethernet
> +        * alias for it (which e.g. makes u-boot set a mac-address).
> +        */
> +       xr819: sdio_wifi at 1 {
> +               reg = <1>;
> +       };
> +};
> +
> +&mmc1_pins_a {
> +       bias-pull-up;

This is already set in h3.dtsi

> +};
> +
> +&ohci1 {
> +       status = "okay";
> +};
> +
> +&uart0 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart0_pins_a>;
> +       status = "okay";
> +};
> +
> +&uart1 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart1_pins>;
> +       status = "disabled";
> +};
> +
> +&uart2 {
> +       pinctrl-names = "default";
> +       pinctrl-0 = <&uart2_pins>;
> +       status = "disabled";
> +};
> +
> +&usbphy {
> +       /* USB VBUS is always on */

I think this comment could use a little work.

AFAIK this board doesn't have an actual USB port.
It's just the D+/D- pins on the pin header, along
with the board-wide 5V, also on the pin header.

ChenYu

> +       status = "okay";
> +};
> --
> 2.10.2
>
> --
> 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.

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
       [not found]       ` <CAGb2v661hBsd7tBcYByeRKEZh+8YMB3ewbnkrT3rUvzP2f90yQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2016-12-03  9:43         ` Jernej Skrabec
  0 siblings, 0 replies; 32+ messages in thread
From: Jernej Skrabec @ 2016-12-03  9:43 UTC (permalink / raw)
  To: linux-sunxi
  Cc: icenowy-ymACFijhrKM,
	maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8, wens-jdAy2FN1RRM,
	robh+dt-DgEjT+Ai2ygdnm+yROfE0A, linux-I+IVW8TIWO2tmTQ+vhA3Yw,
	andre.przywara-5wv7dgnIgG8, hdegoede-H+wXaHxf7aLQT0dZR+AlfA,
	arnd-r2nGTMty4D4, vishnupatekar0510-Re5JQEeQqe8AvxtiuMwx3w,
	linux-doc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA


[-- Attachment #1.1: Type: text/plain, Size: 9524 bytes --]

Hi,

Dne petek, 02. december 2016 17.42.04 UTC+1 je oseba Chen-Yu Tsai napisala:
>
> Hi, 
>
> On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <ice...-ymACFijhrKM@public.gmane.org 
> <javascript:>> wrote: 
> > Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a 
> > SDIO Wi-Fi chip by Allwinner (XR819). 
> > 
> > Add a device tree file for it. 
> > 
> > Signed-off-by: Icenowy Zheng <ice...-ymACFijhrKM@public.gmane.org <javascript:>> 
> > --- 
> > Changes since v2: 
> > - Merged SDIO Wi-Fi patch into it. 
> > - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM. 
> > - SDIO Wi-Fi: changed pinctrl binding to generic pinconf 
> > - removed all gpio pinctrl nodes 
> > - changed h2plus to h2-plus 
> > Changes since v1: 
> > - Convert to generic pinconf bindings. 
> > - SDIO Wi-Fi: add patch. 
> > 
> > Some notes: 
> > - The uart1 and uart2 is available on the unsoldered gpio header. 
> > - The onboard USB connector has its Vbus directly connected to DCIN-5V 
> (the 
> >   power jack) 
> > 
> >  arch/arm/boot/dts/Makefile                        |   1 + 
> >  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 
> ++++++++++++++++++++++ 
> >  2 files changed, 160 insertions(+) 
> >  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile 
> > index 6447abc..59f6e86 100644 
> > --- a/arch/arm/boot/dts/Makefile 
> > +++ b/arch/arm/boot/dts/Makefile 
> > @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ 
> >         sun8i-a33-sinlinx-sina33.dtb \ 
> >         sun8i-a83t-allwinner-h8homlet-v2.dtb \ 
> >         sun8i-a83t-cubietruck-plus.dtb \ 
> > +       sun8i-h2-plus-orangepi-zero.dtb \ 
> >         sun8i-h3-bananapi-m2-plus.dtb \ 
> >         sun8i-h3-nanopi-neo.dtb \ 
> >         sun8i-h3-orangepi-2.dtb \ 
> > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > new file mode 100644 
> > index 0000000..d18807f 
> > --- /dev/null 
> > +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > @@ -0,0 +1,159 @@ 
> > +/* 
> > + * Copyright (C) 2016 Icenowy Zheng <ice...-ymACFijhrKM@public.gmane.org <javascript:>> 
> > + * 
> > + * Based on sun8i-h3-orangepi-one.dts, which is: 
> > + *   Copyright (C) 2016 Hans de Goede <hdeg...-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org <javascript:>> 
>
> > + * 
> > + * 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. 
> > + */ 
> > + 
> > +/dts-v1/; 
> > +#include "sun8i-h3.dtsi" 
> > +#include "sunxi-common-regulators.dtsi" 
> > + 
> > +#include <dt-bindings/gpio/gpio.h> 
> > +#include <dt-bindings/input/input.h> 
> > +#include <dt-bindings/pinctrl/sun4i-a10.h> 
> > + 
> > +/ { 
> > +       model = "Xunlong Orange Pi Zero"; 
> > +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus"; 
> > + 
> > +       aliases { 
> > +               serial0 = &uart0; 
> > +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */ 
> > +               ethernet1 = &xr819; 
> > +       }; 
> > + 
> > +       chosen { 
> > +               stdout-path = "serial0:115200n8"; 
> > +       }; 
> > + 
> > +       leds { 
> > +               compatible = "gpio-leds"; 
> > + 
> > +               pwr_led { 
> > +                       label = "orangepi:green:pwr"; 
> > +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>; 
> > +                       default-state = "on"; 
> > +               }; 
> > + 
> > +               status_led { 
> > +                       label = "orangepi:red:status"; 
> > +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>; 
> > +               }; 
> > +       }; 
> > + 
> > +       reg_vcc_wifi: reg_vcc_wifi { 
> > +               compatible = "regulator-fixed"; 
> > +               regulator-min-microvolt = <3300000>; 
> > +               regulator-max-microvolt = <3300000>; 
> > +               regulator-name = "vcc-wifi"; 
> > +               enable-active-high; 
> > +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>; 
> > +       }; 
> > + 
> > +       wifi_pwrseq: wifi_pwrseq { 
> > +               compatible = "mmc-pwrseq-simple"; 
> > +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; 
> > +       }; 
> > +}; 
> > + 
> > +&ehci1 { 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&mmc0 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&mmc0_pins_a>; 
> > +       vmmc-supply = <&reg_vcc3v3>; 
> > +       bus-width = <4>; 
> > +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */ 
> > +       cd-inverted; 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&mmc1 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&mmc1_pins_a>; 
> > +       vmmc-supply = <&reg_vcc_wifi>; 
> > +       mmc-pwrseq = <&wifi_pwrseq>; 
> > +       bus-width = <4>; 
> > +       non-removable; 
> > +       status = "okay"; 
> > + 
> > +       /* 
> > +        * Explicitly define the sdio device, so that we can add an 
> ethernet 
> > +        * alias for it (which e.g. makes u-boot set a mac-address). 
> > +        */ 
> > +       xr819: sdio_wifi@1 { 
> > +               reg = <1>; 
> > +       }; 
> > +}; 
> > + 
> > +&mmc1_pins_a { 
> > +       bias-pull-up; 
>
> This is already set in h3.dtsi 
>
> > +}; 
> > + 
> > +&ohci1 { 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&uart0 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart0_pins_a>; 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&uart1 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart1_pins>; 
> > +       status = "disabled"; 
> > +}; 
> > + 
> > +&uart2 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart2_pins>; 
> > +       status = "disabled"; 
> > +}; 
> > + 
> > +&usbphy { 
> > +       /* USB VBUS is always on */ 
>
> I think this comment could use a little work. 
>
> AFAIK this board doesn't have an actual USB port. 
> It's just the D+/D- pins on the pin header, along 
> with the board-wide 5V, also on the pin header. 
>

How is addon board handled then? It has two USBs,
IR, audio, mic and probably also TV out. I can't see
any obvious way how to detect that the addon board
is plugged in from the image.

https://www.aliexpress.com/item/New-Orange-Pi-Zreo-Expansion-board-Interface-board-Development-board-beyond-Raspberry-Pi/32770665186.html

Best regards,
Jernej Škrabec
 

>
> ChenYu 
>
> > +       status = "okay"; 
> > +}; 
> > -- 
> > 2.10.2 
> > 
> > -- 
> > 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...-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org <javascript:>. 
> > For more options, visit https://groups.google.com/d/optout. 
>

-- 
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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

[-- Attachment #1.2: Type: text/html, Size: 13038 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [linux-sunxi] [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
  2016-12-02 16:41       ` Chen-Yu Tsai
  (?)
@ 2016-12-04  8:12         ` Alexey Kardashevskiy
  -1 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-04  8:12 UTC (permalink / raw)
  To: Chen-Yu Tsai, Icenowy Zheng
  Cc: Maxime Ripard, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar, linux-doc,
	linux-arm-kernel, linux-kernel, devicetree, linux-sunxi

On 03/12/16 03:41, Chen-Yu Tsai wrote:
> Hi,
> 
> On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
>> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
>> SDIO Wi-Fi chip by Allwinner (XR819).
>>
>> Add a device tree file for it.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
>> ---
>> Changes since v2:
>> - Merged SDIO Wi-Fi patch into it.
>> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
>> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
>> - removed all gpio pinctrl nodes
>> - changed h2plus to h2-plus
>> Changes since v1:
>> - Convert to generic pinconf bindings.
>> - SDIO Wi-Fi: add patch.
>>
>> Some notes:
>> - The uart1 and uart2 is available on the unsoldered gpio header.
>> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>>   power jack)
>>
>>  arch/arm/boot/dts/Makefile                        |   1 +
>>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>>  2 files changed, 160 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 6447abc..59f6e86 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>>         sun8i-a33-sinlinx-sina33.dtb \
>>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>>         sun8i-a83t-cubietruck-plus.dtb \
>> +       sun8i-h2-plus-orangepi-zero.dtb \
>>         sun8i-h3-bananapi-m2-plus.dtb \
>>         sun8i-h3-nanopi-neo.dtb \
>>         sun8i-h3-orangepi-2.dtb \
>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> new file mode 100644
>> index 0000000..d18807f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> @@ -0,0 +1,159 @@
>> +/*
>> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
>> + *
>> + * Based on sun8i-h3-orangepi-one.dts, which is:
>> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
>> + */
>> +
>> +/dts-v1/;
>> +#include "sun8i-h3.dtsi"
>> +#include "sunxi-common-regulators.dtsi"
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/input/input.h>
>> +#include <dt-bindings/pinctrl/sun4i-a10.h>
>> +
>> +/ {
>> +       model = "Xunlong Orange Pi Zero";
>> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
>> +
>> +       aliases {
>> +               serial0 = &uart0;
>> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
>> +               ethernet1 = &xr819;
>> +       };
>> +
>> +       chosen {
>> +               stdout-path = "serial0:115200n8";
>> +       };
>> +
>> +       leds {
>> +               compatible = "gpio-leds";
>> +
>> +               pwr_led {
>> +                       label = "orangepi:green:pwr";
>> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
>> +                       default-state = "on";
>> +               };
>> +
>> +               status_led {
>> +                       label = "orangepi:red:status";
>> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
>> +               };
>> +       };
>> +
>> +       reg_vcc_wifi: reg_vcc_wifi {
>> +               compatible = "regulator-fixed";
>> +               regulator-min-microvolt = <3300000>;
>> +               regulator-max-microvolt = <3300000>;
>> +               regulator-name = "vcc-wifi";
>> +               enable-active-high;
>> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
>> +       };
>> +
>> +       wifi_pwrseq: wifi_pwrseq {
>> +               compatible = "mmc-pwrseq-simple";
>> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
>> +       };
>> +};
>> +
>> +&ehci1 {
>> +       status = "okay";
>> +};
>> +
>> +&mmc0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc0_pins_a>;
>> +       vmmc-supply = <&reg_vcc3v3>;
>> +       bus-width = <4>;
>> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
>> +       cd-inverted;
>> +       status = "okay";
>> +};
>> +
>> +&mmc1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc1_pins_a>;
>> +       vmmc-supply = <&reg_vcc_wifi>;
>> +       mmc-pwrseq = <&wifi_pwrseq>;
>> +       bus-width = <4>;
>> +       non-removable;
>> +       status = "okay";
>> +
>> +       /*
>> +        * Explicitly define the sdio device, so that we can add an ethernet
>> +        * alias for it (which e.g. makes u-boot set a mac-address).
>> +        */
>> +       xr819: sdio_wifi@1 {
>> +               reg = <1>;
>> +       };
>> +};
>> +
>> +&mmc1_pins_a {
>> +       bias-pull-up;
> 
> This is already set in h3.dtsi
> 
>> +};
>> +
>> +&ohci1 {
>> +       status = "okay";
>> +};
>> +
>> +&uart0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart0_pins_a>;
>> +       status = "okay";
>> +};
>> +
>> +&uart1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart1_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&uart2 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart2_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&usbphy {
>> +       /* USB VBUS is always on */
> 
> I think this comment could use a little work.
> 
> AFAIK this board doesn't have an actual USB port.


Mine does have one port.


> It's just the D+/D- pins on the pin header, along
> with the board-wide 5V, also on the pin header.
> 
> ChenYu
> 
>> +       status = "okay";
>> +};
>> --
>> 2.10.2
>>
>> --
>> 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@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [linux-sunxi] [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-04  8:12         ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-04  8:12 UTC (permalink / raw)
  To: Chen-Yu Tsai, Icenowy Zheng
  Cc: devicetree, Vishnu Patekar, Arnd Bergmann, linux-doc,
	Andre Przywara, Russell King, linux-kernel, Hans de Goede,
	linux-sunxi, Rob Herring, Maxime Ripard, linux-arm-kernel

On 03/12/16 03:41, Chen-Yu Tsai wrote:
> Hi,
> 
> On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
>> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
>> SDIO Wi-Fi chip by Allwinner (XR819).
>>
>> Add a device tree file for it.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
>> ---
>> Changes since v2:
>> - Merged SDIO Wi-Fi patch into it.
>> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
>> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
>> - removed all gpio pinctrl nodes
>> - changed h2plus to h2-plus
>> Changes since v1:
>> - Convert to generic pinconf bindings.
>> - SDIO Wi-Fi: add patch.
>>
>> Some notes:
>> - The uart1 and uart2 is available on the unsoldered gpio header.
>> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>>   power jack)
>>
>>  arch/arm/boot/dts/Makefile                        |   1 +
>>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>>  2 files changed, 160 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 6447abc..59f6e86 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>>         sun8i-a33-sinlinx-sina33.dtb \
>>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>>         sun8i-a83t-cubietruck-plus.dtb \
>> +       sun8i-h2-plus-orangepi-zero.dtb \
>>         sun8i-h3-bananapi-m2-plus.dtb \
>>         sun8i-h3-nanopi-neo.dtb \
>>         sun8i-h3-orangepi-2.dtb \
>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> new file mode 100644
>> index 0000000..d18807f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> @@ -0,0 +1,159 @@
>> +/*
>> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
>> + *
>> + * Based on sun8i-h3-orangepi-one.dts, which is:
>> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
>> + */
>> +
>> +/dts-v1/;
>> +#include "sun8i-h3.dtsi"
>> +#include "sunxi-common-regulators.dtsi"
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/input/input.h>
>> +#include <dt-bindings/pinctrl/sun4i-a10.h>
>> +
>> +/ {
>> +       model = "Xunlong Orange Pi Zero";
>> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
>> +
>> +       aliases {
>> +               serial0 = &uart0;
>> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
>> +               ethernet1 = &xr819;
>> +       };
>> +
>> +       chosen {
>> +               stdout-path = "serial0:115200n8";
>> +       };
>> +
>> +       leds {
>> +               compatible = "gpio-leds";
>> +
>> +               pwr_led {
>> +                       label = "orangepi:green:pwr";
>> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
>> +                       default-state = "on";
>> +               };
>> +
>> +               status_led {
>> +                       label = "orangepi:red:status";
>> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
>> +               };
>> +       };
>> +
>> +       reg_vcc_wifi: reg_vcc_wifi {
>> +               compatible = "regulator-fixed";
>> +               regulator-min-microvolt = <3300000>;
>> +               regulator-max-microvolt = <3300000>;
>> +               regulator-name = "vcc-wifi";
>> +               enable-active-high;
>> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
>> +       };
>> +
>> +       wifi_pwrseq: wifi_pwrseq {
>> +               compatible = "mmc-pwrseq-simple";
>> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
>> +       };
>> +};
>> +
>> +&ehci1 {
>> +       status = "okay";
>> +};
>> +
>> +&mmc0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc0_pins_a>;
>> +       vmmc-supply = <&reg_vcc3v3>;
>> +       bus-width = <4>;
>> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
>> +       cd-inverted;
>> +       status = "okay";
>> +};
>> +
>> +&mmc1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc1_pins_a>;
>> +       vmmc-supply = <&reg_vcc_wifi>;
>> +       mmc-pwrseq = <&wifi_pwrseq>;
>> +       bus-width = <4>;
>> +       non-removable;
>> +       status = "okay";
>> +
>> +       /*
>> +        * Explicitly define the sdio device, so that we can add an ethernet
>> +        * alias for it (which e.g. makes u-boot set a mac-address).
>> +        */
>> +       xr819: sdio_wifi@1 {
>> +               reg = <1>;
>> +       };
>> +};
>> +
>> +&mmc1_pins_a {
>> +       bias-pull-up;
> 
> This is already set in h3.dtsi
> 
>> +};
>> +
>> +&ohci1 {
>> +       status = "okay";
>> +};
>> +
>> +&uart0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart0_pins_a>;
>> +       status = "okay";
>> +};
>> +
>> +&uart1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart1_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&uart2 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart2_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&usbphy {
>> +       /* USB VBUS is always on */
> 
> I think this comment could use a little work.
> 
> AFAIK this board doesn't have an actual USB port.


Mine does have one port.


> It's just the D+/D- pins on the pin header, along
> with the board-wide 5V, also on the pin header.
> 
> ChenYu
> 
>> +       status = "okay";
>> +};
>> --
>> 2.10.2
>>
>> --
>> 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@googlegroups.com.
>> For more options, visit https://groups.google.com/d/optout.


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [linux-sunxi] [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-04  8:12         ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-04  8:12 UTC (permalink / raw)
  To: linux-arm-kernel

On 03/12/16 03:41, Chen-Yu Tsai wrote:
> Hi,
> 
> On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy@aosc.xyz> wrote:
>> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
>> SDIO Wi-Fi chip by Allwinner (XR819).
>>
>> Add a device tree file for it.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
>> ---
>> Changes since v2:
>> - Merged SDIO Wi-Fi patch into it.
>> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
>> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
>> - removed all gpio pinctrl nodes
>> - changed h2plus to h2-plus
>> Changes since v1:
>> - Convert to generic pinconf bindings.
>> - SDIO Wi-Fi: add patch.
>>
>> Some notes:
>> - The uart1 and uart2 is available on the unsoldered gpio header.
>> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>>   power jack)
>>
>>  arch/arm/boot/dts/Makefile                        |   1 +
>>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>>  2 files changed, 160 insertions(+)
>>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>>
>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
>> index 6447abc..59f6e86 100644
>> --- a/arch/arm/boot/dts/Makefile
>> +++ b/arch/arm/boot/dts/Makefile
>> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>>         sun8i-a33-sinlinx-sina33.dtb \
>>         sun8i-a83t-allwinner-h8homlet-v2.dtb \
>>         sun8i-a83t-cubietruck-plus.dtb \
>> +       sun8i-h2-plus-orangepi-zero.dtb \
>>         sun8i-h3-bananapi-m2-plus.dtb \
>>         sun8i-h3-nanopi-neo.dtb \
>>         sun8i-h3-orangepi-2.dtb \
>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> new file mode 100644
>> index 0000000..d18807f
>> --- /dev/null
>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
>> @@ -0,0 +1,159 @@
>> +/*
>> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
>> + *
>> + * Based on sun8i-h3-orangepi-one.dts, which is:
>> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
>> + */
>> +
>> +/dts-v1/;
>> +#include "sun8i-h3.dtsi"
>> +#include "sunxi-common-regulators.dtsi"
>> +
>> +#include <dt-bindings/gpio/gpio.h>
>> +#include <dt-bindings/input/input.h>
>> +#include <dt-bindings/pinctrl/sun4i-a10.h>
>> +
>> +/ {
>> +       model = "Xunlong Orange Pi Zero";
>> +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
>> +
>> +       aliases {
>> +               serial0 = &uart0;
>> +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */
>> +               ethernet1 = &xr819;
>> +       };
>> +
>> +       chosen {
>> +               stdout-path = "serial0:115200n8";
>> +       };
>> +
>> +       leds {
>> +               compatible = "gpio-leds";
>> +
>> +               pwr_led {
>> +                       label = "orangepi:green:pwr";
>> +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
>> +                       default-state = "on";
>> +               };
>> +
>> +               status_led {
>> +                       label = "orangepi:red:status";
>> +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
>> +               };
>> +       };
>> +
>> +       reg_vcc_wifi: reg_vcc_wifi {
>> +               compatible = "regulator-fixed";
>> +               regulator-min-microvolt = <3300000>;
>> +               regulator-max-microvolt = <3300000>;
>> +               regulator-name = "vcc-wifi";
>> +               enable-active-high;
>> +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
>> +       };
>> +
>> +       wifi_pwrseq: wifi_pwrseq {
>> +               compatible = "mmc-pwrseq-simple";
>> +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
>> +       };
>> +};
>> +
>> +&ehci1 {
>> +       status = "okay";
>> +};
>> +
>> +&mmc0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc0_pins_a>;
>> +       vmmc-supply = <&reg_vcc3v3>;
>> +       bus-width = <4>;
>> +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
>> +       cd-inverted;
>> +       status = "okay";
>> +};
>> +
>> +&mmc1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&mmc1_pins_a>;
>> +       vmmc-supply = <&reg_vcc_wifi>;
>> +       mmc-pwrseq = <&wifi_pwrseq>;
>> +       bus-width = <4>;
>> +       non-removable;
>> +       status = "okay";
>> +
>> +       /*
>> +        * Explicitly define the sdio device, so that we can add an ethernet
>> +        * alias for it (which e.g. makes u-boot set a mac-address).
>> +        */
>> +       xr819: sdio_wifi at 1 {
>> +               reg = <1>;
>> +       };
>> +};
>> +
>> +&mmc1_pins_a {
>> +       bias-pull-up;
> 
> This is already set in h3.dtsi
> 
>> +};
>> +
>> +&ohci1 {
>> +       status = "okay";
>> +};
>> +
>> +&uart0 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart0_pins_a>;
>> +       status = "okay";
>> +};
>> +
>> +&uart1 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart1_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&uart2 {
>> +       pinctrl-names = "default";
>> +       pinctrl-0 = <&uart2_pins>;
>> +       status = "disabled";
>> +};
>> +
>> +&usbphy {
>> +       /* USB VBUS is always on */
> 
> I think this comment could use a little work.
> 
> AFAIK this board doesn't have an actual USB port.


Mine does have one port.


> It's just the D+/D- pins on the pin header, along
> with the board-wide 5V, also on the pin header.
> 
> ChenYu
> 
>> +       status = "okay";
>> +};
>> --
>> 2.10.2
>>
>> --
>> 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.


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
  2016-12-02 15:05 ` Icenowy Zheng
  (?)
@ 2016-12-05  2:27   ` Alexey Kardashevskiy
  -1 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-05  2:27 UTC (permalink / raw)
  To: Icenowy Zheng, Maxime Ripard, Chen-Yu Tsai, Rob Herring,
	Russell King, Andre Przywara, Hans de Goede, Arnd Bergmann,
	Vishnu Patekar
  Cc: devicetree, linux-doc, linux-kernel, linux-sunxi, linux-arm-kernel

On 03/12/16 02:05, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.


What git tree is this made against of? Thanks.


> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Changed compatible from allwinner,sun8i-h2plus to allwinner,sun8i-h2-plus.
>  Documentation/arm/sunxi/README                  | 4 ++++
>  Documentation/devicetree/bindings/arm/sunxi.txt | 1 +
>  arch/arm/mach-sunxi/sunxi.c                     | 1 +
>  3 files changed, 6 insertions(+)
> 
> diff --git a/Documentation/arm/sunxi/README b/Documentation/arm/sunxi/README
> index cd02433..1fe4d99c 100644
> --- a/Documentation/arm/sunxi/README
> +++ b/Documentation/arm/sunxi/README
> @@ -63,6 +63,10 @@ SunXi family
>          + User Manual
>            http://dl.linux-sunxi.org/A33/A33%20user%20manual%20release%201.1.pdf
>  
> +      - Allwinner H2+ (sun8i)
> +        + No document available now, but is known to be working properly with
> +          H3 drivers and memory map.
> +
>        - Allwinner H3 (sun8i)
>          + Datasheet
>            http://dl.linux-sunxi.org/H3/Allwinner_H3_Datasheet_V1.0.pdf
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
> index 4d6467c..59b143f 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.txt
> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
> @@ -13,6 +13,7 @@ using one of the following compatible strings:
>    allwinner,sun8i-a33
>    allwinner,sun8i-a83t
>    allwinner,sun8i-h3
> +  allwinner,sun8i-h2-plus
>    allwinner,sun9i-a80
>    allwinner,sun50i-a64
>    nextthing,gr8
> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
> index 2e2bde2..320d2af 100644
> --- a/arch/arm/mach-sunxi/sunxi.c
> +++ b/arch/arm/mach-sunxi/sunxi.c
> @@ -63,6 +63,7 @@ static const char * const sun8i_board_dt_compat[] = {
>  	"allwinner,sun8i-a23",
>  	"allwinner,sun8i-a33",
>  	"allwinner,sun8i-a83t",
> +	"allwinner,sun8i-h2-plus",
>  	"allwinner,sun8i-h3",
>  	NULL,
>  };
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-05  2:27   ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-05  2:27 UTC (permalink / raw)
  To: Icenowy Zheng, Maxime Ripard, Chen-Yu Tsai, Rob Herring,
	Russell King, Andre Przywara, Hans de Goede, Arnd Bergmann,
	Vishnu Patekar
  Cc: devicetree, linux-sunxi, linux-kernel, linux-arm-kernel, linux-doc

On 03/12/16 02:05, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.


What git tree is this made against of? Thanks.


> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Changed compatible from allwinner,sun8i-h2plus to allwinner,sun8i-h2-plus.
>  Documentation/arm/sunxi/README                  | 4 ++++
>  Documentation/devicetree/bindings/arm/sunxi.txt | 1 +
>  arch/arm/mach-sunxi/sunxi.c                     | 1 +
>  3 files changed, 6 insertions(+)
> 
> diff --git a/Documentation/arm/sunxi/README b/Documentation/arm/sunxi/README
> index cd02433..1fe4d99c 100644
> --- a/Documentation/arm/sunxi/README
> +++ b/Documentation/arm/sunxi/README
> @@ -63,6 +63,10 @@ SunXi family
>          + User Manual
>            http://dl.linux-sunxi.org/A33/A33%20user%20manual%20release%201.1.pdf
>  
> +      - Allwinner H2+ (sun8i)
> +        + No document available now, but is known to be working properly with
> +          H3 drivers and memory map.
> +
>        - Allwinner H3 (sun8i)
>          + Datasheet
>            http://dl.linux-sunxi.org/H3/Allwinner_H3_Datasheet_V1.0.pdf
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
> index 4d6467c..59b143f 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.txt
> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
> @@ -13,6 +13,7 @@ using one of the following compatible strings:
>    allwinner,sun8i-a33
>    allwinner,sun8i-a83t
>    allwinner,sun8i-h3
> +  allwinner,sun8i-h2-plus
>    allwinner,sun9i-a80
>    allwinner,sun50i-a64
>    nextthing,gr8
> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
> index 2e2bde2..320d2af 100644
> --- a/arch/arm/mach-sunxi/sunxi.c
> +++ b/arch/arm/mach-sunxi/sunxi.c
> @@ -63,6 +63,7 @@ static const char * const sun8i_board_dt_compat[] = {
>  	"allwinner,sun8i-a23",
>  	"allwinner,sun8i-a33",
>  	"allwinner,sun8i-a83t",
> +	"allwinner,sun8i-h2-plus",
>  	"allwinner,sun8i-h3",
>  	NULL,
>  };
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-05  2:27   ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-05  2:27 UTC (permalink / raw)
  To: linux-arm-kernel

On 03/12/16 02:05, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.


What git tree is this made against of? Thanks.


> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Changed compatible from allwinner,sun8i-h2plus to allwinner,sun8i-h2-plus.
>  Documentation/arm/sunxi/README                  | 4 ++++
>  Documentation/devicetree/bindings/arm/sunxi.txt | 1 +
>  arch/arm/mach-sunxi/sunxi.c                     | 1 +
>  3 files changed, 6 insertions(+)
> 
> diff --git a/Documentation/arm/sunxi/README b/Documentation/arm/sunxi/README
> index cd02433..1fe4d99c 100644
> --- a/Documentation/arm/sunxi/README
> +++ b/Documentation/arm/sunxi/README
> @@ -63,6 +63,10 @@ SunXi family
>          + User Manual
>            http://dl.linux-sunxi.org/A33/A33%20user%20manual%20release%201.1.pdf
>  
> +      - Allwinner H2+ (sun8i)
> +        + No document available now, but is known to be working properly with
> +          H3 drivers and memory map.
> +
>        - Allwinner H3 (sun8i)
>          + Datasheet
>            http://dl.linux-sunxi.org/H3/Allwinner_H3_Datasheet_V1.0.pdf
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
> index 4d6467c..59b143f 100644
> --- a/Documentation/devicetree/bindings/arm/sunxi.txt
> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
> @@ -13,6 +13,7 @@ using one of the following compatible strings:
>    allwinner,sun8i-a33
>    allwinner,sun8i-a83t
>    allwinner,sun8i-h3
> +  allwinner,sun8i-h2-plus
>    allwinner,sun9i-a80
>    allwinner,sun50i-a64
>    nextthing,gr8
> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c
> index 2e2bde2..320d2af 100644
> --- a/arch/arm/mach-sunxi/sunxi.c
> +++ b/arch/arm/mach-sunxi/sunxi.c
> @@ -63,6 +63,7 @@ static const char * const sun8i_board_dt_compat[] = {
>  	"allwinner,sun8i-a23",
>  	"allwinner,sun8i-a33",
>  	"allwinner,sun8i-a83t",
> +	"allwinner,sun8i-h2-plus",
>  	"allwinner,sun8i-h3",
>  	NULL,
>  };
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
       [not found] ` <20161202150513.34691-1-icenowy-ymACFijhrKM@public.gmane.org>
  2016-12-02 15:05     ` Icenowy Zheng
@ 2016-12-05  9:19   ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-05  9:19 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar, linux-doc,
	linux-arm-kernel, linux-kernel, devicetree, linux-sunxi

[-- Attachment #1: Type: text/plain, Size: 488 bytes --]

On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

Fixed the alphabetical order in the bindings doc, and applied.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-05  9:19   ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-05  9:19 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar,
	linux-doc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw

[-- Attachment #1: Type: text/plain, Size: 490 bytes --]

On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.
> 
> Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>

Fixed the alphabetical order in the bindings doc, and applied.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-05  9:19   ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-05  9:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
> Allwinner H2+ is a quad-core Cortex-A7 SoC.
> 
> It is very like H3, that they share the same SoC ID (0x1680), and H3
> memory maps as well as drivers works well on the SoC.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

Fixed the alphabetical order in the bindings doc, and applied.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161205/364dbd55/attachment.sig>

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
  2016-12-02 15:05     ` Icenowy Zheng
  (?)
@ 2016-12-06  1:51       ` Alexey Kardashevskiy
  -1 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  1:51 UTC (permalink / raw)
  To: Icenowy Zheng, Maxime Ripard, Chen-Yu Tsai, Rob Herring,
	Russell King, Andre Przywara, Hans de Goede, Arnd Bergmann,
	Vishnu Patekar
  Cc: devicetree, linux-doc, linux-kernel, linux-sunxi, linux-arm-kernel

On 03/12/16 02:05, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
> 
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
> 
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>  	sun8i-a33-sinlinx-sina33.dtb \
>  	sun8i-a83t-allwinner-h8homlet-v2.dtb \
>  	sun8i-a83t-cubietruck-plus.dtb \
> +	sun8i-h2-plus-orangepi-zero.dtb \
>  	sun8i-h3-bananapi-m2-plus.dtb \
>  	sun8i-h3-nanopi-neo.dtb \
>  	sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +	model = "Xunlong Orange Pi Zero";
> +	compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +	aliases {
> +		serial0 = &uart0;
> +		/* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */


It is not defined there as for:

cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next
specific files for 20161205





> +		ethernet1 = &xr819;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		pwr_led {
> +			label = "orangepi:green:pwr";
> +			gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +			default-state = "on";
> +		};
> +
> +		status_led {
> +			label = "orangepi:red:status";
> +			gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +		};
> +	};
> +
> +	reg_vcc_wifi: reg_vcc_wifi {
> +		compatible = "regulator-fixed";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		regulator-name = "vcc-wifi";
> +		enable-active-high;
> +		gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	wifi_pwrseq: wifi_pwrseq {
> +		compatible = "mmc-pwrseq-simple";
> +		reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +	};
> +};
> +
> +&ehci1 {
> +	status = "okay";
> +};
> +
> +&mmc0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc0_pins_a>;
> +	vmmc-supply = <&reg_vcc3v3>;
> +	bus-width = <4>;
> +	cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +	cd-inverted;
> +	status = "okay";
> +};
> +
> +&mmc1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc1_pins_a>;
> +	vmmc-supply = <&reg_vcc_wifi>;
> +	mmc-pwrseq = <&wifi_pwrseq>;
> +	bus-width = <4>;
> +	non-removable;
> +	status = "okay";
> +
> +	/*
> +	 * Explicitly define the sdio device, so that we can add an ethernet
> +	 * alias for it (which e.g. makes u-boot set a mac-address).
> +	 */
> +	xr819: sdio_wifi@1 {
> +		reg = <1>;
> +	};
> +};
> +
> +&mmc1_pins_a {
> +	bias-pull-up;
> +};
> +
> +&ohci1 {
> +	status = "okay";
> +};
> +
> +&uart0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart0_pins_a>;
> +	status = "okay";
> +};
> +
> +&uart1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart1_pins>;
> +	status = "disabled";
> +};
> +
> +&uart2 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart2_pins>;
> +	status = "disabled";
> +};
> +
> +&usbphy {
> +	/* USB VBUS is always on */
> +	status = "okay";
> +};
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-06  1:51       ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  1:51 UTC (permalink / raw)
  To: Icenowy Zheng, Maxime Ripard, Chen-Yu Tsai, Rob Herring,
	Russell King, Andre Przywara, Hans de Goede, Arnd Bergmann,
	Vishnu Patekar
  Cc: devicetree, linux-sunxi, linux-kernel, linux-arm-kernel, linux-doc

On 03/12/16 02:05, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
> 
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
> 
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>  	sun8i-a33-sinlinx-sina33.dtb \
>  	sun8i-a83t-allwinner-h8homlet-v2.dtb \
>  	sun8i-a83t-cubietruck-plus.dtb \
> +	sun8i-h2-plus-orangepi-zero.dtb \
>  	sun8i-h3-bananapi-m2-plus.dtb \
>  	sun8i-h3-nanopi-neo.dtb \
>  	sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +	model = "Xunlong Orange Pi Zero";
> +	compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +	aliases {
> +		serial0 = &uart0;
> +		/* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */


It is not defined there as for:

cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next
specific files for 20161205





> +		ethernet1 = &xr819;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		pwr_led {
> +			label = "orangepi:green:pwr";
> +			gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +			default-state = "on";
> +		};
> +
> +		status_led {
> +			label = "orangepi:red:status";
> +			gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +		};
> +	};
> +
> +	reg_vcc_wifi: reg_vcc_wifi {
> +		compatible = "regulator-fixed";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		regulator-name = "vcc-wifi";
> +		enable-active-high;
> +		gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	wifi_pwrseq: wifi_pwrseq {
> +		compatible = "mmc-pwrseq-simple";
> +		reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +	};
> +};
> +
> +&ehci1 {
> +	status = "okay";
> +};
> +
> +&mmc0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc0_pins_a>;
> +	vmmc-supply = <&reg_vcc3v3>;
> +	bus-width = <4>;
> +	cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +	cd-inverted;
> +	status = "okay";
> +};
> +
> +&mmc1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc1_pins_a>;
> +	vmmc-supply = <&reg_vcc_wifi>;
> +	mmc-pwrseq = <&wifi_pwrseq>;
> +	bus-width = <4>;
> +	non-removable;
> +	status = "okay";
> +
> +	/*
> +	 * Explicitly define the sdio device, so that we can add an ethernet
> +	 * alias for it (which e.g. makes u-boot set a mac-address).
> +	 */
> +	xr819: sdio_wifi@1 {
> +		reg = <1>;
> +	};
> +};
> +
> +&mmc1_pins_a {
> +	bias-pull-up;
> +};
> +
> +&ohci1 {
> +	status = "okay";
> +};
> +
> +&uart0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart0_pins_a>;
> +	status = "okay";
> +};
> +
> +&uart1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart1_pins>;
> +	status = "disabled";
> +};
> +
> +&uart2 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart2_pins>;
> +	status = "disabled";
> +};
> +
> +&usbphy {
> +	/* USB VBUS is always on */
> +	status = "okay";
> +};
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-06  1:51       ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  1:51 UTC (permalink / raw)
  To: linux-arm-kernel

On 03/12/16 02:05, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> ---
> Changes since v2:
> - Merged SDIO Wi-Fi patch into it.
> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM.
> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf
> - removed all gpio pinctrl nodes
> - changed h2plus to h2-plus
> Changes since v1:
> - Convert to generic pinconf bindings.
> - SDIO Wi-Fi: add patch.
> 
> Some notes:
> - The uart1 and uart2 is available on the unsoldered gpio header.
> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the
>   power jack)
> 
>  arch/arm/boot/dts/Makefile                        |   1 +
>  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++
>  2 files changed, 160 insertions(+)
>  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> 
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index 6447abc..59f6e86 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
>  	sun8i-a33-sinlinx-sina33.dtb \
>  	sun8i-a83t-allwinner-h8homlet-v2.dtb \
>  	sun8i-a83t-cubietruck-plus.dtb \
> +	sun8i-h2-plus-orangepi-zero.dtb \
>  	sun8i-h3-bananapi-m2-plus.dtb \
>  	sun8i-h3-nanopi-neo.dtb \
>  	sun8i-h3-orangepi-2.dtb \
> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> new file mode 100644
> index 0000000..d18807f
> --- /dev/null
> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts
> @@ -0,0 +1,159 @@
> +/*
> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz>
> + *
> + * Based on sun8i-h3-orangepi-one.dts, which is:
> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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.
> + */
> +
> +/dts-v1/;
> +#include "sun8i-h3.dtsi"
> +#include "sunxi-common-regulators.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/input.h>
> +#include <dt-bindings/pinctrl/sun4i-a10.h>
> +
> +/ {
> +	model = "Xunlong Orange Pi Zero";
> +	compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus";
> +
> +	aliases {
> +		serial0 = &uart0;
> +		/* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */


It is not defined there as for:

cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next
specific files for 20161205





> +		ethernet1 = &xr819;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		pwr_led {
> +			label = "orangepi:green:pwr";
> +			gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>;
> +			default-state = "on";
> +		};
> +
> +		status_led {
> +			label = "orangepi:red:status";
> +			gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>;
> +		};
> +	};
> +
> +	reg_vcc_wifi: reg_vcc_wifi {
> +		compatible = "regulator-fixed";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		regulator-name = "vcc-wifi";
> +		enable-active-high;
> +		gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>;
> +	};
> +
> +	wifi_pwrseq: wifi_pwrseq {
> +		compatible = "mmc-pwrseq-simple";
> +		reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>;
> +	};
> +};
> +
> +&ehci1 {
> +	status = "okay";
> +};
> +
> +&mmc0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc0_pins_a>;
> +	vmmc-supply = <&reg_vcc3v3>;
> +	bus-width = <4>;
> +	cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */
> +	cd-inverted;
> +	status = "okay";
> +};
> +
> +&mmc1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&mmc1_pins_a>;
> +	vmmc-supply = <&reg_vcc_wifi>;
> +	mmc-pwrseq = <&wifi_pwrseq>;
> +	bus-width = <4>;
> +	non-removable;
> +	status = "okay";
> +
> +	/*
> +	 * Explicitly define the sdio device, so that we can add an ethernet
> +	 * alias for it (which e.g. makes u-boot set a mac-address).
> +	 */
> +	xr819: sdio_wifi at 1 {
> +		reg = <1>;
> +	};
> +};
> +
> +&mmc1_pins_a {
> +	bias-pull-up;
> +};
> +
> +&ohci1 {
> +	status = "okay";
> +};
> +
> +&uart0 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart0_pins_a>;
> +	status = "okay";
> +};
> +
> +&uart1 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart1_pins>;
> +	status = "disabled";
> +};
> +
> +&uart2 {
> +	pinctrl-names = "default";
> +	pinctrl-0 = <&uart2_pins>;
> +	status = "disabled";
> +};
> +
> +&usbphy {
> +	/* USB VBUS is always on */
> +	status = "okay";
> +};
> 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
  2016-12-05  9:19   ` Maxime Ripard
  (?)
@ 2016-12-06  2:01     ` Alexey Kardashevskiy
  -1 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  2:01 UTC (permalink / raw)
  To: Maxime Ripard, Icenowy Zheng
  Cc: Chen-Yu Tsai, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar, linux-doc,
	linux-arm-kernel, linux-kernel, devicetree, linux-sunxi


[-- Attachment #1.1: Type: text/plain, Size: 504 bytes --]

On 05/12/16 20:19, Maxime Ripard wrote:
> On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
>> Allwinner H2+ is a quad-core Cortex-A7 SoC.
>>
>> It is very like H3, that they share the same SoC ID (0x1680), and H3
>> memory maps as well as drivers works well on the SoC.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> 
> Fixed the alphabetical order in the bindings doc, and applied.

Where is that resulting tree, with these ones and EMAC? Thanks.



-- 
Alexey


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 839 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-06  2:01     ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  2:01 UTC (permalink / raw)
  To: Maxime Ripard, Icenowy Zheng
  Cc: devicetree, Vishnu Patekar, Arnd Bergmann, linux-doc,
	Andre Przywara, linux-sunxi, Russell King, linux-kernel,
	Hans de Goede, Chen-Yu Tsai, Rob Herring, linux-arm-kernel


[-- Attachment #1.1.1: Type: text/plain, Size: 504 bytes --]

On 05/12/16 20:19, Maxime Ripard wrote:
> On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
>> Allwinner H2+ is a quad-core Cortex-A7 SoC.
>>
>> It is very like H3, that they share the same SoC ID (0x1680), and H3
>> memory maps as well as drivers works well on the SoC.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> 
> Fixed the alphabetical order in the bindings doc, and applied.

Where is that resulting tree, with these ones and EMAC? Thanks.



-- 
Alexey


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 839 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC
@ 2016-12-06  2:01     ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06  2:01 UTC (permalink / raw)
  To: linux-arm-kernel

On 05/12/16 20:19, Maxime Ripard wrote:
> On Fri, Dec 02, 2016 at 11:05:12PM +0800, Icenowy Zheng wrote:
>> Allwinner H2+ is a quad-core Cortex-A7 SoC.
>>
>> It is very like H3, that they share the same SoC ID (0x1680), and H3
>> memory maps as well as drivers works well on the SoC.
>>
>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>
> 
> Fixed the alphabetical order in the bindings doc, and applied.

Where is that resulting tree, with these ones and EMAC? Thanks.



-- 
Alexey

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 839 bytes
Desc: OpenPGP digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161206/10a2c9c1/attachment.sig>

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-09  8:17       ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-09  8:17 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar, linux-doc,
	linux-arm-kernel, linux-kernel, devicetree, linux-sunxi

[-- Attachment #1: Type: text/plain, Size: 406 bytes --]

On Fri, Dec 02, 2016 at 11:05:13PM +0800, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

Applied, thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-09  8:17       ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-09  8:17 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Chen-Yu Tsai, Rob Herring, Russell King, Andre Przywara,
	Hans de Goede, Arnd Bergmann, Vishnu Patekar,
	linux-doc-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw

[-- Attachment #1: Type: text/plain, Size: 410 bytes --]

On Fri, Dec 02, 2016 at 11:05:13PM +0800, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org>

Applied, thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-09  8:17       ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-09  8:17 UTC (permalink / raw)
  To: linux-arm-kernel

On Fri, Dec 02, 2016 at 11:05:13PM +0800, Icenowy Zheng wrote:
> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a
> SDIO Wi-Fi chip by Allwinner (XR819).
> 
> Add a device tree file for it.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

Applied, thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20161209/2380b316/attachment-0001.sig>

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
  2016-12-06  7:43 Icenowy Zheng
@ 2016-12-06 21:52   ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06 21:52 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: devicetree, Vishnu Patekar, Arnd Bergmann, linux-doc,
	Andre Przywara, linux-kernel, Russell King, Hans de Goede,
	linux-sunxi, Rob Herring, Maxime Ripard, Chen-Yu Tsai,
	linux-arm-kernel

On 06/12/16 18:43, Icenowy Zheng wrote:
> 
> 2016年12月6日 09:51于 Alexey Kardashevskiy <aik@ozlabs.ru>写道:
>>
>> On 03/12/16 02:05, Icenowy Zheng wrote: 
>>> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a 
>>> SDIO Wi-Fi chip by Allwinner (XR819). 
>>>
>>> Add a device tree file for it. 
>>>
>>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz> 
>>> --- 
>>> Changes since v2: 
>>> - Merged SDIO Wi-Fi patch into it. 
>>> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM. 
>>> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf 
>>> - removed all gpio pinctrl nodes 
>>> - changed h2plus to h2-plus 
>>> Changes since v1: 
>>> - Convert to generic pinconf bindings. 
>>> - SDIO Wi-Fi: add patch. 
>>>
>>> Some notes: 
>>> - The uart1 and uart2 is available on the unsoldered gpio header. 
>>> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the 
>>>    power jack) 
>>>
>>>   arch/arm/boot/dts/Makefile                        |   1 + 
>>>   arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++ 
>>>   2 files changed, 160 insertions(+) 
>>>   create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>>
>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile 
>>> index 6447abc..59f6e86 100644 
>>> --- a/arch/arm/boot/dts/Makefile 
>>> +++ b/arch/arm/boot/dts/Makefile 
>>> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ 
>>>   sun8i-a33-sinlinx-sina33.dtb \ 
>>>   sun8i-a83t-allwinner-h8homlet-v2.dtb \ 
>>>   sun8i-a83t-cubietruck-plus.dtb \ 
>>> + sun8i-h2-plus-orangepi-zero.dtb \ 
>>>   sun8i-h3-bananapi-m2-plus.dtb \ 
>>>   sun8i-h3-nanopi-neo.dtb \ 
>>>   sun8i-h3-orangepi-2.dtb \ 
>>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>> new file mode 100644 
>>> index 0000000..d18807f 
>>> --- /dev/null 
>>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>> @@ -0,0 +1,159 @@ 
>>> +/* 
>>> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz> 
>>> + * 
>>> + * Based on sun8i-h3-orangepi-one.dts, which is: 
>>> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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. 
>>> + */ 
>>> + 
>>> +/dts-v1/; 
>>> +#include "sun8i-h3.dtsi" 
>>> +#include "sunxi-common-regulators.dtsi" 
>>> + 
>>> +#include <dt-bindings/gpio/gpio.h> 
>>> +#include <dt-bindings/input/input.h> 
>>> +#include <dt-bindings/pinctrl/sun4i-a10.h> 
>>> + 
>>> +/ { 
>>> + model = "Xunlong Orange Pi Zero"; 
>>> + compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus"; 
>>> + 
>>> + aliases { 
>>> + serial0 = &uart0; 
>>> + /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */ 
>>
>>
>> It is not defined there as for: 
>>
>> cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next 
>> specific files for 20161205 
> 
> The driver of H3's obfuscated DesignWare MAC is not yet mainlined, so there won't be one ethernet0 now.
> 
> But it's reserved for the onboard Ethernet.


Could you please elaborate how you tested this patch (ideally some tree
somewhere on github)? This patch added RX819, it assumes EMAC support is
there, neither is there nor there is a way to test this... Thanks.


> 
>>
>>
>>
>>
>>
>>> + ethernet1 = &xr819; 
>>> + }; 
>>> + 
>>> + chosen { 
>>> + stdout-path = "serial0:115200n8"; 
>>> + }; 
>>> + 
>>> + leds { 
>>> + compatible = "gpio-leds"; 
>>> + 
>>> + pwr_led { 
>>> + label = "orangepi:green:pwr"; 
>>> + gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>; 
>>> + default-state = "on"; 
>>> + }; 
>>> + 
>>> + status_led { 
>>> + label = "orangepi:red:status"; 
>>> + gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>; 
>>> + }; 
>>> + }; 
>>> + 
>>> + reg_vcc_wifi: reg_vcc_wifi { 
>>> + compatible = "regulator-fixed"; 
>>> + regulator-min-microvolt = <3300000>; 
>>> + regulator-max-microvolt = <3300000>; 
>>> + regulator-name = "vcc-wifi"; 
>>> + enable-active-high; 
>>> + gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>; 
>>> + }; 
>>> + 
>>> + wifi_pwrseq: wifi_pwrseq { 
>>> + compatible = "mmc-pwrseq-simple"; 
>>> + reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; 
>>> + }; 
>>> +}; 
>>> + 
>>> +&ehci1 { 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&mmc0 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&mmc0_pins_a>; 
>>> + vmmc-supply = <&reg_vcc3v3>; 
>>> + bus-width = <4>; 
>>> + cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */ 
>>> + cd-inverted; 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&mmc1 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&mmc1_pins_a>; 
>>> + vmmc-supply = <&reg_vcc_wifi>; 
>>> + mmc-pwrseq = <&wifi_pwrseq>; 
>>> + bus-width = <4>; 
>>> + non-removable; 
>>> + status = "okay"; 
>>> + 
>>> + /* 
>>> + * Explicitly define the sdio device, so that we can add an ethernet 
>>> + * alias for it (which e.g. makes u-boot set a mac-address). 
>>> + */ 
>>> + xr819: sdio_wifi@1 { 
>>> + reg = <1>; 
>>> + }; 
>>> +}; 
>>> + 
>>> +&mmc1_pins_a { 
>>> + bias-pull-up; 
>>> +}; 
>>> + 
>>> +&ohci1 { 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&uart0 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart0_pins_a>; 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&uart1 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart1_pins>; 
>>> + status = "disabled"; 
>>> +}; 
>>> + 
>>> +&uart2 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart2_pins>; 
>>> + status = "disabled"; 
>>> +}; 
>>> + 
>>> +&usbphy { 
>>> + /* USB VBUS is always on */ 
>>> + status = "okay"; 
>>> +}; 
>>>
>>
>>
>> -- 
>> Alexey 


-- 
Alexey

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 32+ messages in thread

* [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-06 21:52   ` Alexey Kardashevskiy
  0 siblings, 0 replies; 32+ messages in thread
From: Alexey Kardashevskiy @ 2016-12-06 21:52 UTC (permalink / raw)
  To: linux-arm-kernel

On 06/12/16 18:43, Icenowy Zheng wrote:
> 
> 2016?12?6? 09:51? Alexey Kardashevskiy <aik@ozlabs.ru>???
>>
>> On 03/12/16 02:05, Icenowy Zheng wrote: 
>>> Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a 
>>> SDIO Wi-Fi chip by Allwinner (XR819). 
>>>
>>> Add a device tree file for it. 
>>>
>>> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz> 
>>> --- 
>>> Changes since v2: 
>>> - Merged SDIO Wi-Fi patch into it. 
>>> - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM. 
>>> - SDIO Wi-Fi: changed pinctrl binding to generic pinconf 
>>> - removed all gpio pinctrl nodes 
>>> - changed h2plus to h2-plus 
>>> Changes since v1: 
>>> - Convert to generic pinconf bindings. 
>>> - SDIO Wi-Fi: add patch. 
>>>
>>> Some notes: 
>>> - The uart1 and uart2 is available on the unsoldered gpio header. 
>>> - The onboard USB connector has its Vbus directly connected to DCIN-5V (the 
>>>    power jack) 
>>>
>>>   arch/arm/boot/dts/Makefile                        |   1 + 
>>>   arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++ 
>>>   2 files changed, 160 insertions(+) 
>>>   create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>>
>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile 
>>> index 6447abc..59f6e86 100644 
>>> --- a/arch/arm/boot/dts/Makefile 
>>> +++ b/arch/arm/boot/dts/Makefile 
>>> @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ 
>>>   sun8i-a33-sinlinx-sina33.dtb \ 
>>>   sun8i-a83t-allwinner-h8homlet-v2.dtb \ 
>>>   sun8i-a83t-cubietruck-plus.dtb \ 
>>> + sun8i-h2-plus-orangepi-zero.dtb \ 
>>>   sun8i-h3-bananapi-m2-plus.dtb \ 
>>>   sun8i-h3-nanopi-neo.dtb \ 
>>>   sun8i-h3-orangepi-2.dtb \ 
>>> diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>> new file mode 100644 
>>> index 0000000..d18807f 
>>> --- /dev/null 
>>> +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
>>> @@ -0,0 +1,159 @@ 
>>> +/* 
>>> + * Copyright (C) 2016 Icenowy Zheng <icenowy@aosc.xyz> 
>>> + * 
>>> + * Based on sun8i-h3-orangepi-one.dts, which is: 
>>> + *   Copyright (C) 2016 Hans de Goede <hdegoede@redhat.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. 
>>> + */ 
>>> + 
>>> +/dts-v1/; 
>>> +#include "sun8i-h3.dtsi" 
>>> +#include "sunxi-common-regulators.dtsi" 
>>> + 
>>> +#include <dt-bindings/gpio/gpio.h> 
>>> +#include <dt-bindings/input/input.h> 
>>> +#include <dt-bindings/pinctrl/sun4i-a10.h> 
>>> + 
>>> +/ { 
>>> + model = "Xunlong Orange Pi Zero"; 
>>> + compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus"; 
>>> + 
>>> + aliases { 
>>> + serial0 = &uart0; 
>>> + /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */ 
>>
>>
>> It is not defined there as for: 
>>
>> cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next 
>> specific files for 20161205 
> 
> The driver of H3's obfuscated DesignWare MAC is not yet mainlined, so there won't be one ethernet0 now.
> 
> But it's reserved for the onboard Ethernet.


Could you please elaborate how you tested this patch (ideally some tree
somewhere on github)? This patch added RX819, it assumes EMAC support is
there, neither is there nor there is a way to test this... Thanks.


> 
>>
>>
>>
>>
>>
>>> + ethernet1 = &xr819; 
>>> + }; 
>>> + 
>>> + chosen { 
>>> + stdout-path = "serial0:115200n8"; 
>>> + }; 
>>> + 
>>> + leds { 
>>> + compatible = "gpio-leds"; 
>>> + 
>>> + pwr_led { 
>>> + label = "orangepi:green:pwr"; 
>>> + gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>; 
>>> + default-state = "on"; 
>>> + }; 
>>> + 
>>> + status_led { 
>>> + label = "orangepi:red:status"; 
>>> + gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>; 
>>> + }; 
>>> + }; 
>>> + 
>>> + reg_vcc_wifi: reg_vcc_wifi { 
>>> + compatible = "regulator-fixed"; 
>>> + regulator-min-microvolt = <3300000>; 
>>> + regulator-max-microvolt = <3300000>; 
>>> + regulator-name = "vcc-wifi"; 
>>> + enable-active-high; 
>>> + gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>; 
>>> + }; 
>>> + 
>>> + wifi_pwrseq: wifi_pwrseq { 
>>> + compatible = "mmc-pwrseq-simple"; 
>>> + reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; 
>>> + }; 
>>> +}; 
>>> + 
>>> +&ehci1 { 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&mmc0 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&mmc0_pins_a>; 
>>> + vmmc-supply = <&reg_vcc3v3>; 
>>> + bus-width = <4>; 
>>> + cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */ 
>>> + cd-inverted; 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&mmc1 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&mmc1_pins_a>; 
>>> + vmmc-supply = <&reg_vcc_wifi>; 
>>> + mmc-pwrseq = <&wifi_pwrseq>; 
>>> + bus-width = <4>; 
>>> + non-removable; 
>>> + status = "okay"; 
>>> + 
>>> + /* 
>>> + * Explicitly define the sdio device, so that we can add an ethernet 
>>> + * alias for it (which e.g. makes u-boot set a mac-address). 
>>> + */ 
>>> + xr819: sdio_wifi at 1 { 
>>> + reg = <1>; 
>>> + }; 
>>> +}; 
>>> + 
>>> +&mmc1_pins_a { 
>>> + bias-pull-up; 
>>> +}; 
>>> + 
>>> +&ohci1 { 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&uart0 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart0_pins_a>; 
>>> + status = "okay"; 
>>> +}; 
>>> + 
>>> +&uart1 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart1_pins>; 
>>> + status = "disabled"; 
>>> +}; 
>>> + 
>>> +&uart2 { 
>>> + pinctrl-names = "default"; 
>>> + pinctrl-0 = <&uart2_pins>; 
>>> + status = "disabled"; 
>>> +}; 
>>> + 
>>> +&usbphy { 
>>> + /* USB VBUS is always on */ 
>>> + status = "okay"; 
>>> +}; 
>>>
>>
>>
>> -- 
>> Alexey 


-- 
Alexey

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-06  7:43 Icenowy Zheng
  2016-12-06 21:52   ` Alexey Kardashevskiy
  0 siblings, 1 reply; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-06  7:43 UTC (permalink / raw)
  To: Alexey Kardashevskiy
  Cc: Vishnu Patekar, Rob Herring, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	Andre Przywara, devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-doc-u79uwXL29TY76Z2rM5mHXA, Hans de Goede, Arnd Bergmann,
	Russell King, Maxime Ripard,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw, Chen-Yu Tsai


2016年12月6日 09:51于 Alexey Kardashevskiy <aik@ozlabs.ru>写道:
>
> On 03/12/16 02:05, Icenowy Zheng wrote: 
> > Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a 
> > SDIO Wi-Fi chip by Allwinner (XR819). 
> > 
> > Add a device tree file for it. 
> > 
> > Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> 
> > --- 
> > Changes since v2: 
> > - Merged SDIO Wi-Fi patch into it. 
> > - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM. 
> > - SDIO Wi-Fi: changed pinctrl binding to generic pinconf 
> > - removed all gpio pinctrl nodes 
> > - changed h2plus to h2-plus 
> > Changes since v1: 
> > - Convert to generic pinconf bindings. 
> > - SDIO Wi-Fi: add patch. 
> > 
> > Some notes: 
> > - The uart1 and uart2 is available on the unsoldered gpio header. 
> > - The onboard USB connector has its Vbus directly connected to DCIN-5V (the 
> >   power jack) 
> > 
> >  arch/arm/boot/dts/Makefile                        |   1 + 
> >  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++ 
> >  2 files changed, 160 insertions(+) 
> >  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile 
> > index 6447abc..59f6e86 100644 
> > --- a/arch/arm/boot/dts/Makefile 
> > +++ b/arch/arm/boot/dts/Makefile 
> > @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ 
> >  sun8i-a33-sinlinx-sina33.dtb \ 
> >  sun8i-a83t-allwinner-h8homlet-v2.dtb \ 
> >  sun8i-a83t-cubietruck-plus.dtb \ 
> > + sun8i-h2-plus-orangepi-zero.dtb \ 
> >  sun8i-h3-bananapi-m2-plus.dtb \ 
> >  sun8i-h3-nanopi-neo.dtb \ 
> >  sun8i-h3-orangepi-2.dtb \ 
> > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > new file mode 100644 
> > index 0000000..d18807f 
> > --- /dev/null 
> > +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > @@ -0,0 +1,159 @@ 
> > +/* 
> > + * Copyright (C) 2016 Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> 
> > + * 
> > + * Based on sun8i-h3-orangepi-one.dts, which is: 
> > + *   Copyright (C) 2016 Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 
> > + * 
> > + * 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. 
> > + */ 
> > + 
> > +/dts-v1/; 
> > +#include "sun8i-h3.dtsi" 
> > +#include "sunxi-common-regulators.dtsi" 
> > + 
> > +#include <dt-bindings/gpio/gpio.h> 
> > +#include <dt-bindings/input/input.h> 
> > +#include <dt-bindings/pinctrl/sun4i-a10.h> 
> > + 
> > +/ { 
> > + model = "Xunlong Orange Pi Zero"; 
> > + compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus"; 
> > + 
> > + aliases { 
> > + serial0 = &uart0; 
> > + /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */ 
>
>
> It is not defined there as for: 
>
> cef87e9 (tag: next-20161205) 20 hours ago Stephen Rothwell Add linux-next 
> specific files for 20161205 

The driver of H3's obfuscated DesignWare MAC is not yet mainlined, so there won't be one ethernet0 now.

But it's reserved for the onboard Ethernet.

>
>
>
>
>
> > + ethernet1 = &xr819; 
> > + }; 
> > + 
> > + chosen { 
> > + stdout-path = "serial0:115200n8"; 
> > + }; 
> > + 
> > + leds { 
> > + compatible = "gpio-leds"; 
> > + 
> > + pwr_led { 
> > + label = "orangepi:green:pwr"; 
> > + gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>; 
> > + default-state = "on"; 
> > + }; 
> > + 
> > + status_led { 
> > + label = "orangepi:red:status"; 
> > + gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>; 
> > + }; 
> > + }; 
> > + 
> > + reg_vcc_wifi: reg_vcc_wifi { 
> > + compatible = "regulator-fixed"; 
> > + regulator-min-microvolt = <3300000>; 
> > + regulator-max-microvolt = <3300000>; 
> > + regulator-name = "vcc-wifi"; 
> > + enable-active-high; 
> > + gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>; 
> > + }; 
> > + 
> > + wifi_pwrseq: wifi_pwrseq { 
> > + compatible = "mmc-pwrseq-simple"; 
> > + reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; 
> > + }; 
> > +}; 
> > + 
> > +&ehci1 { 
> > + status = "okay"; 
> > +}; 
> > + 
> > +&mmc0 { 
> > + pinctrl-names = "default"; 
> > + pinctrl-0 = <&mmc0_pins_a>; 
> > + vmmc-supply = <&reg_vcc3v3>; 
> > + bus-width = <4>; 
> > + cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */ 
> > + cd-inverted; 
> > + status = "okay"; 
> > +}; 
> > + 
> > +&mmc1 { 
> > + pinctrl-names = "default"; 
> > + pinctrl-0 = <&mmc1_pins_a>; 
> > + vmmc-supply = <&reg_vcc_wifi>; 
> > + mmc-pwrseq = <&wifi_pwrseq>; 
> > + bus-width = <4>; 
> > + non-removable; 
> > + status = "okay"; 
> > + 
> > + /* 
> > + * Explicitly define the sdio device, so that we can add an ethernet 
> > + * alias for it (which e.g. makes u-boot set a mac-address). 
> > + */ 
> > + xr819: sdio_wifi@1 { 
> > + reg = <1>; 
> > + }; 
> > +}; 
> > + 
> > +&mmc1_pins_a { 
> > + bias-pull-up; 
> > +}; 
> > + 
> > +&ohci1 { 
> > + status = "okay"; 
> > +}; 
> > + 
> > +&uart0 { 
> > + pinctrl-names = "default"; 
> > + pinctrl-0 = <&uart0_pins_a>; 
> > + status = "okay"; 
> > +}; 
> > + 
> > +&uart1 { 
> > + pinctrl-names = "default"; 
> > + pinctrl-0 = <&uart1_pins>; 
> > + status = "disabled"; 
> > +}; 
> > + 
> > +&uart2 { 
> > + pinctrl-names = "default"; 
> > + pinctrl-0 = <&uart2_pins>; 
> > + status = "disabled"; 
> > +}; 
> > + 
> > +&usbphy { 
> > + /* USB VBUS is always on */ 
> > + status = "okay"; 
> > +}; 
> > 
>
>
> -- 
> Alexey 

-- 
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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
       [not found] ` <20161203162455.OlDm27ch-jDEamKawf7I0PDqKvflMoHmW9unr2Ajn@public.gmane.org>
@ 2016-12-05  9:12   ` Maxime Ripard
  0 siblings, 0 replies; 32+ messages in thread
From: Maxime Ripard @ 2016-12-05  9:12 UTC (permalink / raw)
  To: Icenowy Zheng
  Cc: Jernej Skrabec, linux-sunxi, arnd-r2nGTMty4D4,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-doc-u79uwXL29TY76Z2rM5mHXA,
	vishnupatekar0510-Re5JQEeQqe8AvxtiuMwx3w,
	andre.przywara-5wv7dgnIgG8,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-I+IVW8TIWO2tmTQ+vhA3Yw, hdegoede-H+wXaHxf7aLQT0dZR+AlfA,
	wens-jdAy2FN1RRM, robh+dt-DgEjT+Ai2ygdnm+yROfE0A

[-- Attachment #1: Type: text/plain, Size: 1099 bytes --]

Hi Icenowy,

On Sat, Dec 03, 2016 at 09:24:19PM +0800, Icenowy Zheng wrote:
> <p dir="ltr"><br>
> 2016&#24180;12&#26376;3&#26085; &#19979;&#21320;5:43&#20110; Jernej Skrabec &lt;jernej.skrabec-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org&gt;&#20889;&#36947;&#65306;<br>
> &gt;<br>
> &gt; Hi,<br>
> &gt;<br>
> &gt; Dne petek, 02. december 2016 17.42.04 UTC+1 je oseba Chen-Yu Tsai napisala:<br>
> &gt;&gt;<br>
> &gt;&gt; Hi, <br>
> &gt;&gt;<br>
> &gt;&gt; On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng &lt;ice...-ymACFijhrKM@public.gmane.org&gt; wrote: <br>
> &gt;&gt; &gt; Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a <br>
> &gt;&gt; &gt; SDIO Wi-Fi chip by Allwinner (XR819). <br>
> &gt;&gt; &gt; <br>
> &gt;&gt; &gt; Add a device tree file for it. <br>
> &gt;&gt; &gt; <br>
> &gt;&gt; &gt; Signed-off-by: Icenowy Zheng &lt;ice...-ymACFijhrKM@public.gmane.org&gt; <br>

Please make sure to disable the HTML replies, this is what your mail
looks like on a !HTML MUA :)

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-03 13:24 Icenowy Zheng
       [not found] ` <20161203162455.OlDm27ch-jDEamKawf7I0PDqKvflMoHmW9unr2Ajn@public.gmane.org>
  0 siblings, 1 reply; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-03 13:24 UTC (permalink / raw)
  To: Jernej Skrabec
  Cc: linux-sunxi, arnd-r2nGTMty4D4,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-doc-u79uwXL29TY76Z2rM5mHXA,
	vishnupatekar0510-Re5JQEeQqe8AvxtiuMwx3w,
	andre.przywara-5wv7dgnIgG8,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	linux-I+IVW8TIWO2tmTQ+vhA3Yw, hdegoede-H+wXaHxf7aLQT0dZR+AlfA,
	wens-jdAy2FN1RRM, robh+dt-DgEjT+Ai2ygdnm+yROfE0A,
	maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8

[-- Attachment #1: Type: text/html, Size: 15303 bytes --]

^ permalink raw reply	[flat|nested] 32+ messages in thread

* Re: [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board
@ 2016-12-02 18:52 Icenowy Zheng
  0 siblings, 0 replies; 32+ messages in thread
From: Icenowy Zheng @ 2016-12-02 18:52 UTC (permalink / raw)
  To: Chen-Yu Tsai
  Cc: linux-arm-kernel, linux-sunxi, Vishnu Patekar, Rob Herring,
	linux-kernel, Andre Przywara,
		linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Hans de Goede, Arnd Bergmann, devicetree, Russell King,
	Maxime Ripard


2016年12月3日 上午12:41于 Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>写道:
>
> Hi, 
>
> On Fri, Dec 2, 2016 at 11:05 PM, Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> wrote: 
> > Orange Pi Zero is a board that came with the new Allwinner H2+ SoC and a 
> > SDIO Wi-Fi chip by Allwinner (XR819). 
> > 
> > Add a device tree file for it. 
> > 
> > Signed-off-by: Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> 
> > --- 
> > Changes since v2: 
> > - Merged SDIO Wi-Fi patch into it. 
> > - SDIO Wi-Fi: add a ethernet1 alias to it, as it has no internal NVRAM. 
> > - SDIO Wi-Fi: changed pinctrl binding to generic pinconf 
> > - removed all gpio pinctrl nodes 
> > - changed h2plus to h2-plus 
> > Changes since v1: 
> > - Convert to generic pinconf bindings. 
> > - SDIO Wi-Fi: add patch. 
> > 
> > Some notes: 
> > - The uart1 and uart2 is available on the unsoldered gpio header. 
> > - The onboard USB connector has its Vbus directly connected to DCIN-5V (the 
> >   power jack) 
> > 
> >  arch/arm/boot/dts/Makefile                        |   1 + 
> >  arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts | 159 ++++++++++++++++++++++ 
> >  2 files changed, 160 insertions(+) 
> >  create mode 100644 arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > 
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile 
> > index 6447abc..59f6e86 100644 
> > --- a/arch/arm/boot/dts/Makefile 
> > +++ b/arch/arm/boot/dts/Makefile 
> > @@ -844,6 +844,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ 
> >         sun8i-a33-sinlinx-sina33.dtb \ 
> >         sun8i-a83t-allwinner-h8homlet-v2.dtb \ 
> >         sun8i-a83t-cubietruck-plus.dtb \ 
> > +       sun8i-h2-plus-orangepi-zero.dtb \ 
> >         sun8i-h3-bananapi-m2-plus.dtb \ 
> >         sun8i-h3-nanopi-neo.dtb \ 
> >         sun8i-h3-orangepi-2.dtb \ 
> > diff --git a/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > new file mode 100644 
> > index 0000000..d18807f 
> > --- /dev/null 
> > +++ b/arch/arm/boot/dts/sun8i-h2-plus-orangepi-zero.dts 
> > @@ -0,0 +1,159 @@ 
> > +/* 
> > + * Copyright (C) 2016 Icenowy Zheng <icenowy-ymACFijhrKM@public.gmane.org> 
> > + * 
> > + * Based on sun8i-h3-orangepi-one.dts, which is: 
> > + *   Copyright (C) 2016 Hans de Goede <hdegoede-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 
> > + * 
> > + * 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. 
> > + */ 
> > + 
> > +/dts-v1/; 
> > +#include "sun8i-h3.dtsi" 
> > +#include "sunxi-common-regulators.dtsi" 
> > + 
> > +#include <dt-bindings/gpio/gpio.h> 
> > +#include <dt-bindings/input/input.h> 
> > +#include <dt-bindings/pinctrl/sun4i-a10.h> 
> > + 
> > +/ { 
> > +       model = "Xunlong Orange Pi Zero"; 
> > +       compatible = "xunlong,orangepi-zero", "allwinner,sun8i-h2-plus"; 
> > + 
> > +       aliases { 
> > +               serial0 = &uart0; 
> > +               /* ethernet0 is the H3 emac, defined in sun8i-h3.dtsi */ 
> > +               ethernet1 = &xr819; 
> > +       }; 
> > + 
> > +       chosen { 
> > +               stdout-path = "serial0:115200n8"; 
> > +       }; 
> > + 
> > +       leds { 
> > +               compatible = "gpio-leds"; 
> > + 
> > +               pwr_led { 
> > +                       label = "orangepi:green:pwr"; 
> > +                       gpios = <&r_pio 0 10 GPIO_ACTIVE_HIGH>; 
> > +                       default-state = "on"; 
> > +               }; 
> > + 
> > +               status_led { 
> > +                       label = "orangepi:red:status"; 
> > +                       gpios = <&pio 0 17 GPIO_ACTIVE_HIGH>; 
> > +               }; 
> > +       }; 
> > + 
> > +       reg_vcc_wifi: reg_vcc_wifi { 
> > +               compatible = "regulator-fixed"; 
> > +               regulator-min-microvolt = <3300000>; 
> > +               regulator-max-microvolt = <3300000>; 
> > +               regulator-name = "vcc-wifi"; 
> > +               enable-active-high; 
> > +               gpio = <&pio 0 20 GPIO_ACTIVE_HIGH>; 
> > +       }; 
> > + 
> > +       wifi_pwrseq: wifi_pwrseq { 
> > +               compatible = "mmc-pwrseq-simple"; 
> > +               reset-gpios = <&r_pio 0 7 GPIO_ACTIVE_LOW>; 
> > +       }; 
> > +}; 
> > + 
> > +&ehci1 { 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&mmc0 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&mmc0_pins_a>; 
> > +       vmmc-supply = <&reg_vcc3v3>; 
> > +       bus-width = <4>; 
> > +       cd-gpios = <&pio 5 6 GPIO_ACTIVE_HIGH>; /* PF6 */ 
> > +       cd-inverted; 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&mmc1 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&mmc1_pins_a>; 
> > +       vmmc-supply = <&reg_vcc_wifi>; 
> > +       mmc-pwrseq = <&wifi_pwrseq>; 
> > +       bus-width = <4>; 
> > +       non-removable; 
> > +       status = "okay"; 
> > + 
> > +       /* 
> > +        * Explicitly define the sdio device, so that we can add an ethernet 
> > +        * alias for it (which e.g. makes u-boot set a mac-address). 
> > +        */ 
> > +       xr819: sdio_wifi@1 { 
> > +               reg = <1>; 
> > +       }; 
> > +}; 
> > + 
> > +&mmc1_pins_a { 
> > +       bias-pull-up; 
>
> This is already set in h3.dtsi 

Okay...

>
> > +}; 
> > + 
> > +&ohci1 { 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&uart0 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart0_pins_a>; 
> > +       status = "okay"; 
> > +}; 
> > + 
> > +&uart1 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart1_pins>; 
> > +       status = "disabled"; 
> > +}; 
> > + 
> > +&uart2 { 
> > +       pinctrl-names = "default"; 
> > +       pinctrl-0 = <&uart2_pins>; 
> > +       status = "disabled"; 
> > +}; 
> > + 
> > +&usbphy { 
> > +       /* USB VBUS is always on */ 
>
> I think this comment could use a little work. 
>
> AFAIK this board doesn't have an actual USB port. 
> It's just the D+/D- pins on the pin header, along 
> with the board-wide 5V, also on the pin header. 

There is a actual port. Additionally there are two pin pairs.

But there's also no regulator for the actual port.

>
> ChenYu 
>
> > +       status = "okay"; 
> > +}; 
> > -- 
> > 2.10.2 
> > 
> > -- 
> > 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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org 
> > For more options, visit https://groups.google.com/d/optout. 
>
> -- 
> 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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org 
> For more options, visit https://groups.google.com/d/optout. 

-- 
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-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 32+ messages in thread

end of thread, other threads:[~2016-12-09  8:18 UTC | newest]

Thread overview: 32+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-12-02 15:05 [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC Icenowy Zheng
2016-12-02 15:05 ` Icenowy Zheng
     [not found] ` <20161202150513.34691-1-icenowy-ymACFijhrKM@public.gmane.org>
2016-12-02 15:05   ` [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board Icenowy Zheng
2016-12-02 15:05     ` Icenowy Zheng
2016-12-02 16:41     ` [linux-sunxi] " Chen-Yu Tsai
2016-12-02 16:41       ` Chen-Yu Tsai
2016-12-02 16:41       ` Chen-Yu Tsai
     [not found]       ` <CAGb2v661hBsd7tBcYByeRKEZh+8YMB3ewbnkrT3rUvzP2f90yQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-12-03  9:43         ` Jernej Skrabec
2016-12-04  8:12       ` [linux-sunxi] " Alexey Kardashevskiy
2016-12-04  8:12         ` Alexey Kardashevskiy
2016-12-04  8:12         ` Alexey Kardashevskiy
2016-12-06  1:51     ` Alexey Kardashevskiy
2016-12-06  1:51       ` Alexey Kardashevskiy
2016-12-06  1:51       ` Alexey Kardashevskiy
2016-12-09  8:17     ` Maxime Ripard
2016-12-09  8:17       ` Maxime Ripard
2016-12-09  8:17       ` Maxime Ripard
2016-12-05  2:27 ` [PATCH v3 -next 1/2] ARM: sunxi: add support for H2+ SoC Alexey Kardashevskiy
2016-12-05  2:27   ` Alexey Kardashevskiy
2016-12-05  2:27   ` Alexey Kardashevskiy
2016-12-05  9:19 ` Maxime Ripard
2016-12-05  9:19   ` Maxime Ripard
2016-12-05  9:19   ` Maxime Ripard
2016-12-06  2:01   ` Alexey Kardashevskiy
2016-12-06  2:01     ` Alexey Kardashevskiy
2016-12-06  2:01     ` Alexey Kardashevskiy
2016-12-02 18:52 [PATCH v3 -next 2/2] ARM: dts: sunxi: add support for Orange Pi Zero board Icenowy Zheng
2016-12-03 13:24 Icenowy Zheng
     [not found] ` <20161203162455.OlDm27ch-jDEamKawf7I0PDqKvflMoHmW9unr2Ajn@public.gmane.org>
2016-12-05  9:12   ` Maxime Ripard
2016-12-06  7:43 Icenowy Zheng
2016-12-06 21:52 ` Alexey Kardashevskiy
2016-12-06 21:52   ` Alexey Kardashevskiy

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.