Received: by 2002:ac0:a581:0:0:0:0:0 with SMTP id m1-v6csp1053282imm; Fri, 22 Jun 2018 09:32:02 -0700 (PDT) X-Google-Smtp-Source: ADUXVKIk94azN6xOLKRLL1W8mONUDnlBJqIq1UcE3tpR76Cf8xAWDatEWXgY4j8nF8CHbtDny5yR X-Received: by 2002:a63:af49:: with SMTP id s9-v6mr2096590pgo.49.1529685122077; Fri, 22 Jun 2018 09:32:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1529685122; cv=none; d=google.com; s=arc-20160816; b=VSIBLYCImWJWjzKxXl8qzDGi86ORUm+I9MyR3mSaKOeiccr2EBvfdjBKHlm46HgaZc zF8cLSdl5E1I3qfsBqNpl5nQFPYRsf533SUCTlm+a1lQQp3Q+LSCyLXHm9onuY877pzR aLsAp5WvbkZxa2kX6sVtDKNxyOrj37Qs5X5snlcuL2mPqysC6CqULJuoyG/vab/MhpZN VI05sNA9JNuavs1NC9zHsBJLAD/8xEI9ls2uSUGOTkPXPVIkagC589Yu3Kuzw2PCgKuU 2nKKc/TRgdfXfeMPXOV3GvVXcQF1L66WPQbA0NPBDv3QveIH37/D0GdeiRC1OExJvXFY x2LQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :organization:references:in-reply-to:date:cc:to:from:subject :message-id:arc-authentication-results; bh=S1Qt2wZs+7LmFXZl3QiLu6uHz5ieuZolB9HshMdNEUo=; b=WmcF+mpbqO4sTbkOtPc6Uke07yZCICMYx79w1tOXsplqignIK+6tMxCF7us593Mekn D87LMw85RuS4gpYjcIyT8j7y5sA22XsyPVvokuEtSpQa4B6iotR+Yovr+Hmp+G6XfSjq SodgfvTiN+OzPICjvsN8a7aDmxFSNDCi23SPh9wvQKqzwo1h/dM/sJRvm/5xEFWmHyYC 1U5zB/PE/cX65cqLlSz9LwkZZaH2lE+FZDd3aBDZJhtjM6hReMOFSctm5MSB8nMZEfvD nP3UQzhbWZrfoItN2N6+DEE/2mD6jvAyO3jHoLTduEbCb/48HpRVQQF4K+Z/zZ8mawJ4 v5VA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id o3-v6si6661798pgc.381.2018.06.22.09.31.36; Fri, 22 Jun 2018 09:32:02 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934416AbeFVQaI (ORCPT + 99 others); Fri, 22 Jun 2018 12:30:08 -0400 Received: from hermes.aosc.io ([199.195.250.187]:35373 "EHLO hermes.aosc.io" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934203AbeFVQaG (ORCPT ); Fri, 22 Jun 2018 12:30:06 -0400 Received: from localhost (localhost [127.0.0.1]) (Authenticated sender: icenowy@aosc.io) by hermes.aosc.io (Postfix) with ESMTPSA id 8A9E8630E0; Fri, 22 Jun 2018 16:29:37 +0000 (UTC) Message-ID: Subject: Re: [PATCHv2 5/5] arm64: allwinner: a64: Add support for TERES-I laptop From: Icenowy Zheng To: Harald Geyer , Maxime Ripard , Chen-Yu Tsai , Kalle Valo Cc: Rob Herring , Mark Rutland , Catalin Marinas , Will Deacon , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Andre Przywara , info@olimex.com, linux-wireless@vger.kernel.org Date: Sat, 23 Jun 2018 00:27:21 +0800 In-Reply-To: <20180315162510.11669-6-harald@ccbib.org> References: <20180315162510.11669-1-harald@ccbib.org> <20180315162510.11669-6-harald@ccbib.org> Organization: Anthon Open-Source Community Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 在 2018-03-15四的 16:25 +0000,Harald Geyer写道: > The TERES-I is an open hardware laptop built by Olimex using the > Allwinner A64 SoC. > > Add the board specific .dts file, which includes the A64 .dtsi and > enables the peripherals that we support so far. > > Signed-off-by: Harald Geyer > --- > changes since v1: > * use SPDX header instead of license text > * change model string to match compatible string > * removed node labes from leds > * added label properties ot led nodes > * add a comment about the purpose of i2c0 > > arch/arm64/boot/dts/allwinner/Makefile | 1 + > .../boot/dts/allwinner/sun50i-a64-teres-i.dts | 279 > +++++++++++++++++++++ > 2 files changed, 280 insertions(+) > create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a64-teres- > i.dts > > diff --git a/arch/arm64/boot/dts/allwinner/Makefile > b/arch/arm64/boot/dts/allwinner/Makefile > index f505227b0250..5f073f7423b7 100644 > --- a/arch/arm64/boot/dts/allwinner/Makefile > +++ b/arch/arm64/boot/dts/allwinner/Makefile > @@ -5,6 +5,7 @@ dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-olinuxino.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-orangepi-win.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-pine64-plus.dtb sun50i-a64- > pine64.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-sopine-baseboard.dtb > +dtb-$(CONFIG_ARCH_SUNXI) += sun50i-a64-teres-i.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-h5-orangepi-pc2.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-h5-orangepi-prime.dtb > dtb-$(CONFIG_ARCH_SUNXI) += sun50i-h5-orangepi-zero-plus2.dtb > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts > b/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts > new file mode 100644 > index 000000000000..b105430e0368 > --- /dev/null > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64-teres-i.dts > @@ -0,0 +1,279 @@ > +/* > + * Copyright (C) Harald Geyer > + * based on sun50i-a64-olinuxino.dts by Jagan Teki om> > + * > + * SPDX-License-Identifier: (GPL-2.0 OR MIT) > + */ > + > +/dts-v1/; > + > +#include "sun50i-a64.dtsi" > + > +#include > +#include > +#include > + > +/ { > + model = "Olimex A64 Teres-I"; > + compatible = "olimex,a64-teres-i", "allwinner,sun50i-a64"; > + > + aliases { > + serial0 = &uart0; > + }; > + > + backlight: backlight { > + compatible = "pwm-backlight"; > + pwms = <&pwm 0 50000 0>; > + brightness-levels = <0 10 20 30 40 50 60 70 100>; > + default-brightness-level = <3>; > + enable-gpios = <&pio 3 23 GPIO_ACTIVE_HIGH>; /* PD23 > */ > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + > + framebuffer-lcd { > + eDP25-supply = <®_dldo2>; > + eDP12-supply = <®_dldo3>; > + }; > + }; > + > + gpio-keys { > + compatible = "gpio-keys"; > + > + lid-switch { > + label = "Lid Switch"; > + gpios = <&r_pio 0 8 GPIO_ACTIVE_LOW>; /* PL8 > */ > + linux,input-type = ; > + linux,code = ; > + }; > + }; > + > + leds { > + compatible = "gpio-leds"; > + > + capslock { > + label = "leds:green:capslock"; > + gpios = <&pio 2 7 GPIO_ACTIVE_HIGH>; /* PC7 > */ > + }; > + > + numlock { > + label = "leds:green:numlock"; > + gpios = <&pio 2 4 GPIO_ACTIVE_HIGH>; /* PC4 > */ > + }; > + }; > + > + reg_usb1_vbus: usb1-vbus { > + compatible = "regulator-fixed"; > + regulator-name = "usb1-vbus"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + enable-active-high; > + gpio = <&r_pio 0 7 GPIO_ACTIVE_HIGH>; /* PL7 */ > + status = "okay"; > + }; > + > + wifi_pwrseq: wifi_pwrseq { > + compatible = "mmc-pwrseq-simple"; > + reset-gpios = <&r_pio 0 2 GPIO_ACTIVE_LOW>; /* PL2 > */ > + }; > +}; > + > +&ehci1 { > + status = "okay"; > +}; > + > + > +/* The ANX6345 eDP-bridge is on i2c0. There is no linux (mainline) > + * driver for this chip at the moment, the bootloader initializes > it. > + * However it can be accessed with the i2c-dev driver from user > space. > + */ > +&i2c0 { > + clock-frequency = <100000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c0_pins>; > + status = "okay"; > +}; > + > +&mmc0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_pins>; > + vmmc-supply = <®_dcdc1>; > + cd-gpios = <&pio 5 6 GPIO_ACTIVE_LOW>; > + disable-wp; > + bus-width = <4>; > + status = "okay"; > +}; > + > +&mmc1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc1_pins>; > + vmmc-supply = <®_aldo2>; > + vqmmc-supply = <®_dldo4>; > + mmc-pwrseq = <&wifi_pwrseq>; > + bus-width = <4>; > + non-removable; > + status = "okay"; > + > + rtl8723bs: wifi@1 { > + reg = <1>; > + interrupt-parent = <&r_pio>; > + interrupts = <0 3 IRQ_TYPE_LEVEL_LOW>; /* PL3 */ > + interrupt-names = "host-wake"; > + }; I think this node has some problem: - This device node has no binding. The "host-wake" interrupt is part of Broadcom SDIO Wi-Fi binding, rather than a generic one. - Without the interrupt this device node isn't needed, as RTL8723BS has MAC eFUSE and doesn't need a MAC in device tree. In order to solve the problems. I suggest either drop this device node or make a generic "sdio-wifi" device tree binding. Personally I prefer the latter, as it's more accurate device representation. If such a device tree binding is added, I think it should contain the "host-wake" interrupt and a "local-mac-address" property. Both can be ignored by the driver. (This interrupt can be needed if a more card- ventor-neutral name is found.) > +}; > + > +&mmc2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc2_pins>; > + vmmc-supply = <®_dcdc1>; > + vqmmc-supply = <®_dcdc1>; > + bus-width = <8>; > + non-removable; > + cap-mmc-hw-reset; > + status = "okay"; > +}; > + > +&pwm { > + pinctrl-names = "default"; > + pinctrl-0 = <&pwm_pin>; > + status = "okay"; > +}; > + > +&ohci1 { > + status = "okay"; > +}; > + > +&r_rsb { > + status = "okay"; > + > + axp803: pmic@3a3 { > + compatible = "x-powers,axp803"; > + reg = <0x3a3>; > + interrupt-parent = <&r_intc>; > + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > + wakeup-source; > + }; > +}; > + > +#include "axp803.dtsi" > + > +®_aldo1 { > + regulator-always-on; > + regulator-min-microvolt = <2800000>; > + regulator-max-microvolt = <2800000>; > + regulator-name = "vcc-pe"; > +}; > + > +®_aldo2 { > + regulator-always-on; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-pl"; > +}; > + > +®_aldo3 { > + regulator-always-on; > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "vcc-pll-avcc"; > +}; > + > +®_dcdc1 { > + regulator-always-on; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-3v3"; > +}; > + > +®_dcdc2 { > + regulator-always-on; > + regulator-min-microvolt = <1040000>; > + regulator-max-microvolt = <1300000>; > + regulator-name = "vdd-cpux"; > +}; > + > +/* DCDC3 is polyphased with DCDC2 */ > + > +®_dcdc5 { > + regulator-always-on; > + regulator-min-microvolt = <1500000>; > + regulator-max-microvolt = <1500000>; > + regulator-name = "vcc-ddr3"; > +}; > + > +®_dcdc6 { > + regulator-always-on; > + regulator-min-microvolt = <1100000>; > + regulator-max-microvolt = <1100000>; > + regulator-name = "vdd-sys"; > +}; > + > +®_dldo1 { > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-hdmi"; > +}; > + > +®_dldo2 { > + regulator-min-microvolt = <2500000>; > + regulator-max-microvolt = <2500000>; > + regulator-name = "vcc-pd"; > +}; > + > +®_dldo3 { > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-name = "eDP12"; > +}; > + > +®_dldo4 { > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-wifi-io"; > +}; > + > +®_eldo1 { > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-name = "cpvdd"; > +}; > + > +®_eldo2 { > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-name = "vcc-dvdd-csi"; > +}; > + > +®_fldo1 { > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-name = "vcc-1v2-hsic"; > +}; > + > +/* > + * The A64 chip cannot work without this regulator off, although > + * it seems to be only driving the AR100 core. > + * Maybe we don't still know well about CPUs domain. > + */ > +®_fldo2 { > + regulator-always-on; > + regulator-min-microvolt = <1100000>; > + regulator-max-microvolt = <1100000>; > + regulator-name = "vdd-cpus"; > +}; > + > +®_rtc_ldo { > + regulator-name = "vcc-rtc"; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart0_pins_a>; > + status = "okay"; > +}; > + > +&usbphy { > + usb1_vbus-supply = <®_usb1_vbus>; > + status = "okay"; > +};