[PATCH 1/5] dt-bindings: add vendor prefix for Touchless Biometric Systems AG

Subsystems: open firmware and flattened device tree bindings, the rest

STALE3256d

17 messages, 2 authors, 2017-09-01 · open the first message on its own page

[PATCH 1/5] dt-bindings: add vendor prefix for Touchless Biometric Systems AG

From: Maxime Ripard <hidden>
Date: 2017-08-25 09:26:13

From: Tomas Novotny <redacted>

Signed-off-by: Tomas Novotny <redacted>
Signed-off-by: Maxime Ripard <redacted>
---
 Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
 1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/vendor-prefixes.txt b/Documentation/devicetree/bindings/vendor-prefixes.txt
index 37867c3cb94b..86803f592544 100644
--- a/Documentation/devicetree/bindings/vendor-prefixes.txt
+++ b/Documentation/devicetree/bindings/vendor-prefixes.txt
@@ -326,6 +326,7 @@ swir	Sierra Wireless
 syna	Synaptics Inc.
 synology	Synology, Inc.
 tbs	TBS Technologies
+tbs-biometrics	Touchless Biometric Systems AG
 tcg	Trusted Computing Group
 tcl	Toby Churchill Ltd.
 technexion	TechNexion
-- 
2.13.5

[PATCH 2/5] arm: dts: sun8i: a83t: Remove useless, empty nodes

From: Maxime Ripard <hidden>
Date: 2017-08-25 09:26:14

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 6 ------
 1 file changed, 6 deletions(-)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index f996bd343e50..8df34d8ac51d 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -54,12 +54,6 @@
 	#address-cells = <1>;
 	#size-cells = <1>;
 
-	aliases {
-	};
-
-	chosen {
-	};
-
 	cpus {
 		#address-cells = <1>;
 		#size-cells = <0>;
-- 
2.13.5

[PATCH 2/5] arm: dts: sun8i: a83t: Remove useless, empty nodes

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 06:46:02

On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
Signed-off-by: Maxime Ripard <redacted>
Acked-by: Chen-Yu Tsai <redacted>

[PATCH 2/5] arm: dts: sun8i: a83t: Remove useless, empty nodes

From: Maxime Ripard <hidden>
Date: 2017-09-01 13:14:24

On Thu, Aug 31, 2017 at 02:46:02PM +0800, Chen-Yu Tsai wrote:
On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted
Signed-off-by: Maxime Ripard <redacted>
Acked-by: Chen-Yu Tsai <redacted>
Queued for 4.15, 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/20170901/9e2fdadb/attachment.sig>

[PATCH 3/5] arm: dts: sun8i: a83t: Add MMC1 pins

From: Maxime Ripard <hidden>
Date: 2017-08-25 09:26:15

Add the pinctrl definitions for the A83t MMC1 controller.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 8df34d8ac51d..91dee798f3ca 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -342,6 +342,14 @@
 				bias-pull-up;
 			};
 
+			mmc1_pins: mmc1-pins {
+				pins = "PG0", "PG1", "PG2",
+				       "PG3", "PG4", "PG5";
+				function = "mmc1";
+				drive-strength = <30>;
+				bias-pull-up;
+			};
+
 			mmc2_8bit_emmc_pins: mmc2-8bit-emmc-pins {
 				pins = "PC5", "PC6", "PC8", "PC9",
 				       "PC10", "PC11", "PC12", "PC13",
-- 
2.13.5

[PATCH 3/5] arm: dts: sun8i: a83t: Add MMC1 pins

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 06:46:21

On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
Add the pinctrl definitions for the A83t MMC1 controller.

Signed-off-by: Maxime Ripard <redacted>
Acked-by: Chen-Yu Tsai <redacted>

[PATCH 3/5] arm: dts: sun8i: a83t: Add MMC1 pins

From: Maxime Ripard <hidden>
Date: 2017-09-01 13:14:34

On Thu, Aug 31, 2017 at 02:46:21PM +0800, Chen-Yu Tsai wrote:
On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted
Add the pinctrl definitions for the A83t MMC1 controller.

Signed-off-by: Maxime Ripard <redacted>
Acked-by: Chen-Yu Tsai <redacted>
Queued for 4.15, 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/20170901/38f537c7/attachment.sig>

[PATCH 4/5] arm: dts: sun8i: a83t: Add the UART1 controller

From: Maxime Ripard <hidden>
Date: 2017-08-25 09:26:16

The A83T has an UART1 controller, with the RTS and CTS pins routed so it
can be used for devices with hardware flow control, like a bluetooth chip.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 91dee798f3ca..f6aaaa9a1fe2 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -373,6 +373,16 @@
 				pins = "PF2", "PF4";
 				function = "uart0";
 			};
+
+			uart1_pg_pins: uart1-pg-pins {
+				pins = "PG6", "PG7";
+				function = "uart1";
+			};
+
+			uart1_rtscts_pins: uart1-rtscts-pins {
+				pins = "PG8", "PG9";
+				function = "uart1";
+			};
 		};
 
 		timer at 1c20c00 {
@@ -417,6 +427,17 @@
 			status = "disabled";
 		};
 
+		uart1: serial at 01c28400 {
+			compatible = "snps,dw-apb-uart";
+			reg = <0x01c28400 0x400>;
+			interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
+			reg-shift = <2>;
+			reg-io-width = <4>;
+			clocks = <&ccu CLK_BUS_UART1>;
+			resets = <&ccu RST_BUS_UART1>;
+			status = "disabled";
+		};
+
 		gic: interrupt-controller at 1c81000 {
 			compatible = "arm,cortex-a7-gic", "arm,cortex-a15-gic";
 			reg = <0x01c81000 0x1000>,
-- 
2.13.5

[PATCH 4/5] arm: dts: sun8i: a83t: Add the UART1 controller

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 06:48:20

On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted hunk
The A83T has an UART1 controller, with the RTS and CTS pins routed so it
can be used for devices with hardware flow control, like a bluetooth chip.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 91dee798f3ca..f6aaaa9a1fe2 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -373,6 +373,16 @@
                                pins = "PF2", "PF4";
                                function = "uart0";
                        };
+
+                       uart1_pg_pins: uart1-pg-pins {
Since uart1 is only available on the PG pin group, this name
doesn't provide much info. Instead lets name it uart1_rxtx_pins.
This matches the naming scheme for the rts/cts pingroup.

Otherwise,

Acked-by: Chen-Yu Tsai <redacted>
quoted hunk
+                               pins = "PG6", "PG7";
+                               function = "uart1";
+                       };
+
+                       uart1_rtscts_pins: uart1-rtscts-pins {
+                               pins = "PG8", "PG9";
+                               function = "uart1";
+                       };
                };

                timer at 1c20c00 {
@@ -417,6 +427,17 @@
                        status = "disabled";
                };

+               uart1: serial at 01c28400 {
+                       compatible = "snps,dw-apb-uart";
+                       reg = <0x01c28400 0x400>;
+                       interrupts = <GIC_SPI 1 IRQ_TYPE_LEVEL_HIGH>;
+                       reg-shift = <2>;
+                       reg-io-width = <4>;
+                       clocks = <&ccu CLK_BUS_UART1>;
+                       resets = <&ccu RST_BUS_UART1>;
+                       status = "disabled";
+               };
+
                gic: interrupt-controller at 1c81000 {
                        compatible = "arm,cortex-a7-gic", "arm,cortex-a15-gic";
                        reg = <0x01c81000 0x1000>,
--
2.13.5

[PATCH 4/5] arm: dts: sun8i: a83t: Add the UART1 controller

From: Maxime Ripard <hidden>
Date: 2017-09-01 13:16:24

On Thu, Aug 31, 2017 at 02:48:20PM +0800, Chen-Yu Tsai wrote:
On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted
The A83T has an UART1 controller, with the RTS and CTS pins routed so it
can be used for devices with hardware flow control, like a bluetooth chip.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/sun8i-a83t.dtsi | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)
diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi b/arch/arm/boot/dts/sun8i-a83t.dtsi
index 91dee798f3ca..f6aaaa9a1fe2 100644
--- a/arch/arm/boot/dts/sun8i-a83t.dtsi
+++ b/arch/arm/boot/dts/sun8i-a83t.dtsi
@@ -373,6 +373,16 @@
                                pins = "PF2", "PF4";
                                function = "uart0";
                        };
+
+                       uart1_pg_pins: uart1-pg-pins {
Since uart1 is only available on the PG pin group, this name
doesn't provide much info. Instead lets name it uart1_rxtx_pins.
This matches the naming scheme for the rts/cts pingroup.
I've fixed it 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/20170901/2d7c2229/attachment.sig>

[PATCH 5/5] arm: dts: sun8i: Add the TBS A711 tablet devicetree

From: Maxime Ripard <hidden>
Date: 2017-08-25 09:26:17

The TBS A711 is a tablet with an A83T, a modem, wifi/BT chip, an eMMC and a
1024x600 LVDS display.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/Makefile                |   1 +
 arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts | 195 ++++++++++++++++++++++++++++++
 2 files changed, 196 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 212589532b2e..8fa36746d738 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -914,6 +914,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
 	sun8i-a33-sinlinx-sina33.dtb \
 	sun8i-a83t-allwinner-h8homlet-v2.dtb \
 	sun8i-a83t-cubietruck-plus.dtb \
+	sun8i-a83t-tbs-a711.dtb \
 	sun8i-h2-plus-orangepi-zero.dtb \
 	sun8i-h3-bananapi-m2-plus.dtb \
 	sun8i-h3-beelink-x2.dtb \
diff --git a/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts b/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
new file mode 100644
index 000000000000..743509fdc8f9
--- /dev/null
+++ b/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
@@ -0,0 +1,195 @@
+/*
+ * Copyright (C) 2017 Touchless Biometric Systems AG
+ * Tomas Novotny <tomas@novotny.cz>
+ *
+ * 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-a83t.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+/ {
+	model = "TBS A711 Tablet";
+	compatible = "tbs-biometrics,a711", "allwinner,sun8i-a83t";
+
+	aliases {
+		serial0 = &uart0;
+		serial1 = &uart1;
+	};
+
+	chosen {
+		stdout-path = "serial0:115200n8";
+	};
+
+	reg_vbat: reg-vbat {
+		compatible = "regulator-fixed";
+		regulator-name = "vbat";
+		regulator-min-microvolt = <3700000>;
+		regulator-max-microvolt = <3700000>;
+	};
+
+	reg_vmain: reg-vmain {
+		compatible = "regulator-fixed";
+		regulator-name = "vmain";
+		regulator-min-microvolt = <5000000>;
+		regulator-max-microvolt = <5000000>;
+		gpio = <&r_pio 0 9 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+		vin-supply = <&reg_vbat>;
+	};
+
+	wifi_pwrseq: wifi_pwrseq {
+		compatible = "mmc-pwrseq-simple";
+		reset-gpios = <&r_pio 0 2 GPIO_ACTIVE_LOW>; /* PL2 WL-PMU-EN */
+
+		/*
+		 * This is actually Bluetooth's clock, but we have to
+		 * hook it up somewheere
+		 */
+		clocks = <&ac100_rtc 1>;
+		clock-names = "ext_clock";
+	};
+};
+
+/*
+ * An USB-2 hub is connected here, which also means we don't need to
+ * enable the OHCI controller.
+ */
+&ehci0 {
+	status = "okay";
+};
+
+/*
+ * There's a modem connected here that needs to be initialised before
+ * being able to be enumerated.
+ */
+&ehci1 {
+	status = "okay";
+};
+
+&mmc0 {
+	vmmc-supply = <&reg_vcc3v3>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc0_pins>;
+	cd-gpios = <&pio 5 6 GPIO_ACTIVE_LOW>;
+	status = "okay";
+};
+
+&mmc1 {
+	mmc-pwrseq = <&wifi_pwrseq>;
+	bus-width = <4>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&mmc1_pins>;
+	vmmc-supply = <&reg_vcc3v3>;
+	non-removable;
+	wakeup-source;
+	status = "okay";
+
+	brcmf: wifi at 1 {
+		reg = <1>;
+		compatible = "brcm,bcm4329-fmac";
+		interrupt-parent = <&r_pio>;
+		interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 WL_WAKE_UP */
+		interrupt-names = "host-wake";
+	};
+};
+
+&mmc2 {
+	pinctrl-0 = <&mmc2_8bit_emmc_pins>;
+	pinctrl-names = "default";
+	vmmc-supply = <&reg_vcc3v3>;
+	bus-width = <8>;
+	non-removable;
+	cap-mmc-hw-reset;
+	status = "okay";
+};
+
+&r_rsb {
+	status = "okay";
+
+	axp813: pmic at 3a3 {
+		compatible = "x-powers,axp813";
+		reg = <0x3a3>;
+		interrupt-parent = <&r_intc>;
+		interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
+	};
+
+	ac100: codec at e89 {
+		compatible = "x-powers,ac100";
+		reg = <0xe89>;
+
+		ac100_codec: codec {
+			compatible = "x-powers,ac100-codec";
+			interrupt-parent = <&r_pio>;
+			interrupts = <0 12 IRQ_TYPE_LEVEL_LOW>; /* PL12 */
+			#clock-cells = <0>;
+			clock-output-names = "4M_adda";
+		};
+
+		ac100_rtc: rtc {
+			compatible = "x-powers,ac100-rtc";
+			interrupt-parent = <&r_intc>;
+			interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
+			clocks = <&ac100_codec>;
+			#clock-cells = <1>;
+			clock-output-names = "cko1_rtc",
+					     "cko2_rtc",
+					     "cko3_rtc";
+		};
+	};
+
+};
+
+&uart0 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart0_pb_pins>;
+	status = "okay";
+};
+
+/* There's the BT part of the AP6210 connected to that UART */
+&uart1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&uart1_pg_pins &uart1_rtscts_pins>;
+	status = "okay";
+};
+
+&usbphy {
+	status = "okay";
+};
-- 
2.13.5

[PATCH 5/5] arm: dts: sun8i: Add the TBS A711 tablet devicetree

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 06:52:43

On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted hunk
The TBS A711 is a tablet with an A83T, a modem, wifi/BT chip, an eMMC and a
1024x600 LVDS display.

Signed-off-by: Maxime Ripard <redacted>
---
 arch/arm/boot/dts/Makefile                |   1 +
 arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts | 195 ++++++++++++++++++++++++++++++
 2 files changed, 196 insertions(+)
 create mode 100644 arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index 212589532b2e..8fa36746d738 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -914,6 +914,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \
        sun8i-a33-sinlinx-sina33.dtb \
        sun8i-a83t-allwinner-h8homlet-v2.dtb \
        sun8i-a83t-cubietruck-plus.dtb \
+       sun8i-a83t-tbs-a711.dtb \
        sun8i-h2-plus-orangepi-zero.dtb \
        sun8i-h3-bananapi-m2-plus.dtb \
        sun8i-h3-beelink-x2.dtb \
diff --git a/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts b/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
new file mode 100644
index 000000000000..743509fdc8f9
--- /dev/null
+++ b/arch/arm/boot/dts/sun8i-a83t-tbs-a711.dts
@@ -0,0 +1,195 @@
+/*
+ * Copyright (C) 2017 Touchless Biometric Systems AG
+ * Tomas Novotny <tomas@novotny.cz>
+ *
+ * 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-a83t.dtsi"
+#include "sunxi-common-regulators.dtsi"
+
+/ {
+       model = "TBS A711 Tablet";
+       compatible = "tbs-biometrics,a711", "allwinner,sun8i-a83t";
+
+       aliases {
+               serial0 = &uart0;
+               serial1 = &uart1;
+       };
+
+       chosen {
+               stdout-path = "serial0:115200n8";
+       };
+
+       reg_vbat: reg-vbat {
+               compatible = "regulator-fixed";
+               regulator-name = "vbat";
+               regulator-min-microvolt = <3700000>;
+               regulator-max-microvolt = <3700000>;
+       };
+
+       reg_vmain: reg-vmain {
+               compatible = "regulator-fixed";
+               regulator-name = "vmain";
+               regulator-min-microvolt = <5000000>;
+               regulator-max-microvolt = <5000000>;
+               gpio = <&r_pio 0 9 GPIO_ACTIVE_HIGH>;
+               enable-active-high;
+               vin-supply = <&reg_vbat>;
+       };
+
+       wifi_pwrseq: wifi_pwrseq {
+               compatible = "mmc-pwrseq-simple";
+               reset-gpios = <&r_pio 0 2 GPIO_ACTIVE_LOW>; /* PL2 WL-PMU-EN */
+
+               /*
+                * This is actually Bluetooth's clock, but we have to
+                * hook it up somewheere
+                */
+               clocks = <&ac100_rtc 1>;
+               clock-names = "ext_clock";
+       };
+};
+
+/*
+ * An USB-2 hub is connected here, which also means we don't need to
+ * enable the OHCI controller.
+ */
+&ehci0 {
+       status = "okay";
+};
+
+/*
+ * There's a modem connected here that needs to be initialised before
+ * being able to be enumerated.
+ */
+&ehci1 {
+       status = "okay";
+};
+
+&mmc0 {
+       vmmc-supply = <&reg_vcc3v3>;
+       pinctrl-names = "default";
+       pinctrl-0 = <&mmc0_pins>;
+       cd-gpios = <&pio 5 6 GPIO_ACTIVE_LOW>;
+       status = "okay";
+};
+
+&mmc1 {
+       mmc-pwrseq = <&wifi_pwrseq>;
+       bus-width = <4>;
+       pinctrl-names = "default";
+       pinctrl-0 = <&mmc1_pins>;
+       vmmc-supply = <&reg_vcc3v3>;
vqmmc-supply should be provided. See below.
+       non-removable;
+       wakeup-source;
+       status = "okay";
+
+       brcmf: wifi at 1 {
+               reg = <1>;
+               compatible = "brcm,bcm4329-fmac";
+               interrupt-parent = <&r_pio>;
+               interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 WL_WAKE_UP */
+               interrupt-names = "host-wake";
+       };
+};
+
+&mmc2 {
+       pinctrl-0 = <&mmc2_8bit_emmc_pins>;
+       pinctrl-names = "default";
+       vmmc-supply = <&reg_vcc3v3>;
emmc has separate vddq lines. vqmmc-supply should be provided even though it
may be the same as vmmc-supply. The two refer to different things at the board
level.
+       bus-width = <8>;
+       non-removable;
+       cap-mmc-hw-reset;
+       status = "okay";
+};
+
+&r_rsb {
+       status = "okay";
+
+       axp813: pmic at 3a3 {
+               compatible = "x-powers,axp813";
+               reg = <0x3a3>;
+               interrupt-parent = <&r_intc>;
+               interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
+       };
+
+       ac100: codec at e89 {
+               compatible = "x-powers,ac100";
+               reg = <0xe89>;
+
+               ac100_codec: codec {
+                       compatible = "x-powers,ac100-codec";
+                       interrupt-parent = <&r_pio>;
+                       interrupts = <0 12 IRQ_TYPE_LEVEL_LOW>; /* PL12 */
+                       #clock-cells = <0>;
+                       clock-output-names = "4M_adda";
+               };
+
+               ac100_rtc: rtc {
+                       compatible = "x-powers,ac100-rtc";
+                       interrupt-parent = <&r_intc>;
+                       interrupts = <0 IRQ_TYPE_LEVEL_LOW>;
+                       clocks = <&ac100_codec>;
+                       #clock-cells = <1>;
+                       clock-output-names = "cko1_rtc",
+                                            "cko2_rtc",
+                                            "cko3_rtc";
+               };
+       };
+
+};
+
+&uart0 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&uart0_pb_pins>;
+       status = "okay";
+};
+
+/* There's the BT part of the AP6210 connected to that UART */
+&uart1 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&uart1_pg_pins &uart1_rtscts_pins>;
Splitting it as separate phandles ( <&...> ) would make it more readable.
Also the rx/tx pingroup name should be changed
+       status = "okay";
+};
+
+&usbphy {
No VBUS?

Regards
ChenYu
+       status = "okay";
+};
--
2.13.5

[PATCH 5/5] arm: dts: sun8i: Add the TBS A711 tablet devicetree

From: Maxime Ripard <hidden>
Date: 2017-08-31 09:03:14

Hi,

On Thu, Aug 31, 2017 at 02:52:43PM +0800, Chen-Yu Tsai wrote:
quoted
+       non-removable;
+       wakeup-source;
+       status = "okay";
+
+       brcmf: wifi at 1 {
+               reg = <1>;
+               compatible = "brcm,bcm4329-fmac";
+               interrupt-parent = <&r_pio>;
+               interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 WL_WAKE_UP */
+               interrupt-names = "host-wake";
+       };
+};
+
+&mmc2 {
+       pinctrl-0 = <&mmc2_8bit_emmc_pins>;
+       pinctrl-names = "default";
+       vmmc-supply = <&reg_vcc3v3>;
emmc has separate vddq lines. vqmmc-supply should be provided even though it
may be the same as vmmc-supply. The two refer to different things at the board
level.
My understanding was that vqmmc wasn't needed if it was the same
regulator than vmmc.
quoted
+/* There's the BT part of the AP6210 connected to that UART */
+&uart1 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&uart1_pg_pins &uart1_rtscts_pins>;
Splitting it as separate phandles ( <&...> ) would make it more readable.
Also the rx/tx pingroup name should be changed
Ack.
quoted
+       status = "okay";
+};
+
+&usbphy {
No VBUS?
Argh, I forgot them. I'll add them, obviously.

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/20170831/9e7f4950/attachment-0001.sig>

[PATCH 5/5] arm: dts: sun8i: Add the TBS A711 tablet devicetree

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 09:14:14

On Thu, Aug 31, 2017 at 5:03 PM, Maxime Ripard
[off-list ref] wrote:
Hi,

On Thu, Aug 31, 2017 at 02:52:43PM +0800, Chen-Yu Tsai wrote:
quoted
quoted
+       non-removable;
+       wakeup-source;
+       status = "okay";
+
+       brcmf: wifi at 1 {
+               reg = <1>;
+               compatible = "brcm,bcm4329-fmac";
+               interrupt-parent = <&r_pio>;
+               interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 WL_WAKE_UP */
+               interrupt-names = "host-wake";
+       };
+};
+
+&mmc2 {
+       pinctrl-0 = <&mmc2_8bit_emmc_pins>;
+       pinctrl-names = "default";
+       vmmc-supply = <&reg_vcc3v3>;
emmc has separate vddq lines. vqmmc-supply should be provided even though it
may be the same as vmmc-supply. The two refer to different things at the board
level.
My understanding was that vqmmc wasn't needed if it was the same
regulator than vmmc.
It doesn't say so in the binding, so that's probably only true for Linux.

ChenYu
quoted
quoted
+/* There's the BT part of the AP6210 connected to that UART */
+&uart1 {
+       pinctrl-names = "default";
+       pinctrl-0 = <&uart1_pg_pins &uart1_rtscts_pins>;
Splitting it as separate phandles ( <&...> ) would make it more readable.
Also the rx/tx pingroup name should be changed
Ack.
quoted
quoted
+       status = "okay";
+};
+
+&usbphy {
No VBUS?
Argh, I forgot them. I'll add them, obviously.

Thanks!
Maxime

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

[PATCH 5/5] arm: dts: sun8i: Add the TBS A711 tablet devicetree

From: Maxime Ripard <hidden>
Date: 2017-09-01 13:11:00

On Thu, Aug 31, 2017 at 05:14:14PM +0800, Chen-Yu Tsai wrote:
On Thu, Aug 31, 2017 at 5:03 PM, Maxime Ripard
[off-list ref] wrote:
quoted
Hi,

On Thu, Aug 31, 2017 at 02:52:43PM +0800, Chen-Yu Tsai wrote:
quoted
quoted
+       non-removable;
+       wakeup-source;
+       status = "okay";
+
+       brcmf: wifi at 1 {
+               reg = <1>;
+               compatible = "brcm,bcm4329-fmac";
+               interrupt-parent = <&r_pio>;
+               interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 WL_WAKE_UP */
+               interrupt-names = "host-wake";
+       };
+};
+
+&mmc2 {
+       pinctrl-0 = <&mmc2_8bit_emmc_pins>;
+       pinctrl-names = "default";
+       vmmc-supply = <&reg_vcc3v3>;
emmc has separate vddq lines. vqmmc-supply should be provided even though it
may be the same as vmmc-supply. The two refer to different things at the board
level.
My understanding was that vqmmc wasn't needed if it was the same
regulator than vmmc.
It doesn't say so in the binding, so that's probably only true for Linux.
Ok... I've only said that to a couple of patches this week :)

I guess I'll have to ask to revert that change :/

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/20170901/1ca6224d/attachment.sig>

[PATCH 1/5] dt-bindings: add vendor prefix for Touchless Biometric Systems AG

From: Chen-Yu Tsai <hidden>
Date: 2017-08-31 06:45:28

On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
From: Tomas Novotny <redacted>

Signed-off-by: Tomas Novotny <redacted>
Signed-off-by: Maxime Ripard <redacted>
A description of the vendor would be nice (though the name
pretty much says it all).

Acked-by: Chen-Yu Tsai <redacted>

[PATCH 1/5] dt-bindings: add vendor prefix for Touchless Biometric Systems AG

From: Maxime Ripard <hidden>
Date: 2017-09-01 13:12:46

On Thu, Aug 31, 2017 at 02:45:28PM +0800, Chen-Yu Tsai wrote:
On Fri, Aug 25, 2017 at 5:26 PM, Maxime Ripard
[off-list ref] wrote:
quoted
From: Tomas Novotny <redacted>

Signed-off-by: Tomas Novotny <redacted>
Signed-off-by: Maxime Ripard <redacted>
A description of the vendor would be nice (though the name
pretty much says it all).
Applied with a word on the company, 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/20170901/b0abe827/attachment-0001.sig>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help