* [PATCH] ARM: sun8i: Add Parrot Board DTS @ 2016-06-13 10:15 Quentin Schulz 2016-06-13 10:15 ` Quentin Schulz 0 siblings, 1 reply; 12+ messages in thread From: Quentin Schulz @ 2016-06-13 10:15 UTC (permalink / raw) To: robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw, maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8, wens-jdAy2FN1RRM Cc: Quentin Schulz, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-kernel-u79uwXL29TY76Z2rM5mHXA, thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8 The Parrot Board is an evaluation board with an Allwinner R16 (assumed to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 controllable buttons, an LVDS port with separated backlight and capacitive touch panel ports, an audio/microphone jack, a camera CSI port, 2 sets of 22 GPIOs and an accelerometer. This patch requires patches named "[v3] regulator: axp20x: Add support for the (external) drivebus regulator" and "[PATCH 3/4] ARM: dts: axp22x.dtsi: Add reg_drivebus node" from Hans de Goede or the micro USB port would not be powered. Quentin Schulz (1): ARM: sun8i: Add Parrot Board DTS arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ 2 files changed, 334 insertions(+) create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-13 10:15 [PATCH] ARM: sun8i: Add Parrot Board DTS Quentin Schulz @ 2016-06-13 10:15 ` Quentin Schulz [not found] ` <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> 0 siblings, 1 reply; 12+ messages in thread From: Quentin Schulz @ 2016-06-13 10:15 UTC (permalink / raw) To: robh+dt, pawel.moll, mark.rutland, ijc+devicetree, galak, linux, maxime.ripard, wens Cc: Quentin Schulz, devicetree, linux-arm-kernel, linux-kernel, thomas.petazzoni The Parrot Board is an evaluation board with an Allwinner R16 (assumed to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 controllable buttons, an LVDS port with separated backlight and capacitive touch panel ports, an audio/microphone jack, a camera CSI port, 2 sets of 22 GPIOs and an accelerometer. Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ 2 files changed, 334 insertions(+) create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index 06b6c2d..1149512 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ sun8i-a33-ippo-q8h-v1.2.dtb \ sun8i-a33-q8-tablet.dtb \ sun8i-a33-sinlinx-sina33.dtb \ + sun8i-r16-parrot.dtb \ sun8i-a83t-allwinner-h8homlet-v2.dtb \ sun8i-a83t-cubietruck-plus.dtb \ sun8i-h3-orangepi-2.dtb \ diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts new file mode 100644 index 0000000..75e2420 --- /dev/null +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts @@ -0,0 +1,333 @@ +/* + * Copyright 2015 Quentin Schulz + * + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" +#include "sunxi-common-regulators.dtsi" + +#include <dt-bindings/gpio/gpio.h> +#include <dt-bindings/input/input.h> + +/ { + model = "Allwinner Parrot EVB R16"; + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; + + aliases { + serial0 = &uart0; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; + + leds { + compatible = "gpio-leds"; + pinctrl-names = "default"; + pinctrl-0 = <&led_pins_r16>; + + led1 { + label = "r16:led1:usr"; + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ + }; + + led2 { + label = "r16:led2:usr"; + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ + }; + }; + + wifi_pwrseq: wifi_pwrseq { + compatible = "mmc-pwrseq-simple"; + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ + }; + +}; + +&ehci0 { + status = "okay"; +}; + +&i2c1 { + pinctrl-names = "default"; + pinctrl-0 = <&i2c1_pins_a>; + status = "okay"; +}; + +&lradc { + vref-supply = <®_aldo3>; + status = "okay"; + + button@0 { + label = "V+"; + linux,code = <KEY_VOLUMEUP>; + channel = <0>; + voltage = <190000>; + }; + + button@1 { + label = "V-"; + linux,code = <KEY_VOLUMEDOWN>; + channel = <0>; + voltage = <390000>; + }; + +}; + +&mmc0 { + pinctrl-names = "default"; + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; + vmmc-supply = <®_dcdc1>; + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ + bus-width = <4>; + status = "okay"; +}; + +&mmc1 { + pinctrl-names = "default"; + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; + vmmc-supply = <®_aldo1>; + mmc-pwrseq = <&wifi_pwrseq>; + bus-width = <4>; + non-removable; + status = "okay"; +}; + +&mmc2 { + pinctrl-names = "default"; + pinctrl-0 = <&mmc2_8bit_pins>; + vmmc-supply = <®_dcdc1>; + bus-width = <8>; + non-removable; + cap-mmc-hw-reset; + status = "okay"; +}; + +&mmc2_8bit_pins { + allwinner,drive = <SUN4I_PINCTRL_40_MA>; + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; +}; + +&ohci0 { + status = "okay"; +}; + +&pio { + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { + allwinner,pins = "PD14"; + allwinner,function = "gpio_in"; + allwinner,drive = <SUN4I_PINCTRL_10_MA>; + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; + }; + + led_pins_r16: led_pins@0 { + allwinner,pins = "PE16", "PE17"; + allwinner,function = "gpio_out"; + allwinner,drive = <SUN4I_PINCTRL_10_MA>; + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; + }; + + usb0_id_det: usb0_id_detect_pin@0 { + allwinner,pins = "PD10"; + allwinner,function = "gpio_in"; + allwinner,drive = <SUN4I_PINCTRL_10_MA>; + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; + }; + + usb1_vbus_pin_r16: usb1_vbus_pin@0 { + allwinner,pins = "PD12"; + allwinner,function = "gpio_out"; + allwinner,drive = <SUN4I_PINCTRL_10_MA>; + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; + }; +}; + +&r_pio { + wifi_reset_pin_r16: wifi_reset_pin@3 { + allwinner,pins = "PL6"; + allwinner,function = "gpio_out"; + allwinner,drive = <SUN4I_PINCTRL_10_MA>; + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; + }; +}; + +&r_rsb { + status = "okay"; + + axp22x: pmic@3a3 { + compatible = "x-powers,axp223"; + reg = <0x3a3>; + interrupt-parent = <&nmi_intc>; + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; + eldoin-supply = <®_dcdc1>; + x-powers,drive-vbus-en; + }; +}; + +#include "axp22x.dtsi" + +®_aldo1 { + regulator-always-on; + regulator-min-microvolt = <3000000>; + regulator-max-microvolt = <3000000>; + regulator-name = "aldo1"; +}; + +®_aldo2 { + regulator-always-on; + regulator-min-microvolt = <2350000>; + regulator-max-microvolt = <2650000>; + regulator-name = "vdd-dll"; +}; + +®_aldo3 { + regulator-always-on; + regulator-min-microvolt = <2700000>; + regulator-max-microvolt = <3300000>; + regulator-name = "vcc-pll-avcc"; +}; + +®_dc5ldo { + regulator-always-on; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <1400000>; + regulator-name = "vdd-cpus"; +}; + +®_dcdc1 { + regulator-always-on; + regulator-min-microvolt = <3000000>; + regulator-max-microvolt = <3000000>; + regulator-name = "vcc-3v0"; +}; + +®_dcdc2 { + regulator-always-on; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <1400000>; + regulator-name = "vdd-sys"; +}; + +®_dcdc3 { + regulator-always-on; + regulator-min-microvolt = <900000>; + regulator-max-microvolt = <1400000>; + regulator-name = "vdd-cpu"; +}; + +®_dcdc5 { + regulator-always-on; + regulator-min-microvolt = <1500000>; + regulator-max-microvolt = <1500000>; + regulator-name = "vcc-dram"; +}; + +®_dldo1 { + regulator-always-on; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-name = "vcc-wifi"; +}; + +®_dldo2 { + regulator-always-on; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-name = "vcc-wifi1"; +}; + +®_dldo3 { + regulator-min-microvolt = <3000000>; + regulator-max-microvolt = <3000000>; + regulator-name = "vcc-3v0-csi"; +}; + +®_drivevbus { + regulator-name = "usb0-vbus"; + status = "okay"; +}; + +®_eldo1 { + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1200000>; + regulator-name = "vcc-1v2-hsic"; +}; + +®_eldo2 { + regulator-min-microvolt = <3000000>; + regulator-max-microvolt = <3000000>; + regulator-name = "dsp-vcc"; +}; + +®_eldo3 { + regulator-min-microvolt = <3000000>; + regulator-max-microvolt = <3000000>; + regulator-name = "eldo3"; +}; + +®_usb1_vbus { + pinctrl-names = "default"; + pinctrl-0 = <&usb1_vbus_pin_r16>; + gpio = <&pio 3 12 GPIO_ACTIVE_HIGH>; /* PD12 */ + status = "okay"; +}; + +&uart0 { + pinctrl-names = "default"; + pinctrl-0 = <&uart0_pins_b>; + status = "okay"; +}; + +&usb_otg { + dr_mode = "otg"; + status = "okay"; +}; + +&usbphy { + status = "okay"; + pinctrl-names = "default"; + pinctrl-0 = <&usb0_id_det>; + usb0_vbus-supply = <®_drivevbus>; + usb0_id_det-gpios = <&pio 3 10 GPIO_ACTIVE_HIGH>; /* PD10 */ + usb1_vbus-supply = <®_usb1_vbus>; /* USB1 VBUS is always on */ +}; -- 2.5.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
[parent not found: <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>]
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS [not found] ` <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> @ 2016-06-13 12:43 ` kbuild test robot 2016-06-13 13:04 ` Chen-Yu Tsai 1 sibling, 0 replies; 12+ messages in thread From: kbuild test robot @ 2016-06-13 12:43 UTC (permalink / raw) Cc: kbuild-all-JC7UmRfGjtg, robh+dt-DgEjT+Ai2ygdnm+yROfE0A, pawel.moll-5wv7dgnIgG8, mark.rutland-5wv7dgnIgG8, ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg, galak-sgV2jX0FEOL9JmXXK+q4OQ, linux-I+IVW8TIWO2tmTQ+vhA3Yw, maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8, wens-jdAy2FN1RRM, Quentin Schulz, devicetree-u79uwXL29TY76Z2rM5mHXA, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, linux-kernel-u79uwXL29TY76Z2rM5mHXA, thomas.petazzoni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8 [-- Attachment #1: Type: text/plain, Size: 1090 bytes --] Hi, [auto build test ERROR on robh/for-next] [also build test ERROR on v4.7-rc3 next-20160609] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Quentin-Schulz/ARM-sun8i-Add-Parrot-Board-DTS/20160613-181947 base: https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux for-next config: arm-sunxi_defconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (Debian 5.3.1-8) 5.3.1 20160205 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross ARCH=arm All errors (new ones prefixed by >>): >> Error: arch/arm/boot/dts/sun8i-r16-parrot.dts:285.1-15 Label or path reg_drivevbus not found >> FATAL ERROR: Syntax error parsing input tree --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/octet-stream, Size: 19879 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS [not found] ` <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> 2016-06-13 12:43 ` kbuild test robot @ 2016-06-13 13:04 ` Chen-Yu Tsai 2016-06-14 12:59 ` Quentin Schulz 1 sibling, 1 reply; 12+ messages in thread From: Chen-Yu Tsai @ 2016-06-13 13:04 UTC (permalink / raw) To: Quentin Schulz Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux-I+IVW8TIWO2tmTQ+vhA3Yw, Maxime Ripard, Chen-Yu Tsai, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni Hi, On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz <quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> wrote: > The Parrot Board is an evaluation board with an Allwinner R16 (assumed > to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host You say NAND here, but you enable mmc2 for eMMC below. Please correct it. > and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 > controllable buttons, an LVDS port with separated backlight and > capacitive touch panel ports, an audio/microphone jack, a camera CSI > port, 2 sets of 22 GPIOs and an accelerometer. I assume the board is this one: https://world.taobao.com/item/530374411673.htm > Signed-off-by: Quentin Schulz <quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org> > --- > arch/arm/boot/dts/Makefile | 1 + > arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ > 2 files changed, 334 insertions(+) > create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 06b6c2d..1149512 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ > sun8i-a33-ippo-q8h-v1.2.dtb \ > sun8i-a33-q8-tablet.dtb \ > sun8i-a33-sinlinx-sina33.dtb \ > + sun8i-r16-parrot.dtb \ > sun8i-a83t-allwinner-h8homlet-v2.dtb \ > sun8i-a83t-cubietruck-plus.dtb \ > sun8i-h3-orangepi-2.dtb \ > diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts > new file mode 100644 > index 0000000..75e2420 > --- /dev/null > +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts > @@ -0,0 +1,333 @@ > +/* > + * Copyright 2015 Quentin Schulz > + * > + * Quentin Schulz <quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@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-a33.dtsi" > +#include "sunxi-common-regulators.dtsi" > + > +#include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/input/input.h> > + > +/ { > + model = "Allwinner Parrot EVB R16"; > + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; > + > + aliases { > + serial0 = &uart0; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > + > + leds { > + compatible = "gpio-leds"; > + pinctrl-names = "default"; > + pinctrl-0 = <&led_pins_r16>; IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. Same goes for all the other r16 identifier names. > + > + led1 { > + label = "r16:led1:usr"; > + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ > + }; > + > + led2 { > + label = "r16:led2:usr"; > + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ > + }; > + }; > + > + wifi_pwrseq: wifi_pwrseq { > + compatible = "mmc-pwrseq-simple"; > + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ > + }; > + > +}; > + > +&ehci0 { > + status = "okay"; > +}; > + > +&i2c1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&i2c1_pins_a>; > + status = "okay"; Nothing connected? A comment mentioning which connector this is on if it's not directly connecting something on the board would be nice. > +}; > + > +&lradc { > + vref-supply = <®_aldo3>; > + status = "okay"; > + > + button@0 { > + label = "V+"; > + linux,code = <KEY_VOLUMEUP>; > + channel = <0>; > + voltage = <190000>; > + }; > + > + button@1 { > + label = "V-"; > + linux,code = <KEY_VOLUMEDOWN>; > + channel = <0>; > + voltage = <390000>; > + }; > + > +}; > + > +&mmc0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; > + vmmc-supply = <®_dcdc1>; > + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ > + bus-width = <4>; > + status = "okay"; > +}; > + > +&mmc1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; > + vmmc-supply = <®_aldo1>; This looks fishy. See below. > + mmc-pwrseq = <&wifi_pwrseq>; > + bus-width = <4>; > + non-removable; > + status = "okay"; > +}; > + > +&mmc2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc2_8bit_pins>; > + vmmc-supply = <®_dcdc1>; > + bus-width = <8>; > + non-removable; > + cap-mmc-hw-reset; > + status = "okay"; > +}; > + > +&mmc2_8bit_pins { > + allwinner,drive = <SUN4I_PINCTRL_40_MA>; > + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > +}; > + > +&ohci0 { > + status = "okay"; > +}; > + > +&pio { > + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { _parrot suffix works as well. > + allwinner,pins = "PD14"; > + allwinner,function = "gpio_in"; > + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > + }; > + > + led_pins_r16: led_pins@0 { > + allwinner,pins = "PE16", "PE17"; > + allwinner,function = "gpio_out"; > + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > + }; > + > + usb0_id_det: usb0_id_detect_pin@0 { > + allwinner,pins = "PD10"; > + allwinner,function = "gpio_in"; > + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > + }; > + > + usb1_vbus_pin_r16: usb1_vbus_pin@0 { > + allwinner,pins = "PD12"; > + allwinner,function = "gpio_out"; > + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > + }; > +}; > + > +&r_pio { > + wifi_reset_pin_r16: wifi_reset_pin@3 { Why @3? > + allwinner,pins = "PL6"; > + allwinner,function = "gpio_out"; > + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > + }; > +}; > + > +&r_rsb { > + status = "okay"; > + > + axp22x: pmic@3a3 { > + compatible = "x-powers,axp223"; > + reg = <0x3a3>; > + interrupt-parent = <&nmi_intc>; > + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > + eldoin-supply = <®_dcdc1>; A drivevbus-supply referencing reg_vcc5v0 here would be better. > + x-powers,drive-vbus-en; > + }; > +}; > + > +#include "axp22x.dtsi" > + > +®_aldo1 { > + regulator-always-on; > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "aldo1"; What is this for exactly? Would turning it off render the system inoperable? How was it referenced in the fex file? If this is for WiFi I/O VCC, then you should specify it in mmc1 with vqmmc-supply. > +}; > + > +®_aldo2 { > + regulator-always-on; > + regulator-min-microvolt = <2350000>; > + regulator-max-microvolt = <2650000>; > + regulator-name = "vdd-dll"; > +}; > + > +®_aldo3 { > + regulator-always-on; > + regulator-min-microvolt = <2700000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-pll-avcc"; > +}; > + > +®_dc5ldo { > + regulator-always-on; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <1400000>; > + regulator-name = "vdd-cpus"; > +}; > + > +®_dcdc1 { > + regulator-always-on; > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "vcc-3v0"; > +}; > + > +®_dcdc2 { > + regulator-always-on; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <1400000>; > + regulator-name = "vdd-sys"; > +}; > + > +®_dcdc3 { > + regulator-always-on; > + regulator-min-microvolt = <900000>; > + regulator-max-microvolt = <1400000>; > + regulator-name = "vdd-cpu"; > +}; > + > +®_dcdc5 { > + regulator-always-on; > + regulator-min-microvolt = <1500000>; > + regulator-max-microvolt = <1500000>; > + regulator-name = "vcc-dram"; > +}; > + > +®_dldo1 { > + regulator-always-on; A comment saying why this is always on would be nice. I assume this is and dldo2 are waiting on regulator supply list support. > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-wifi"; > +}; > + > +®_dldo2 { > + regulator-always-on; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "vcc-wifi1"; > +}; > + > +®_dldo3 { > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "vcc-3v0-csi"; > +}; > + > +®_drivevbus { > + regulator-name = "usb0-vbus"; > + status = "okay"; > +}; > + > +®_eldo1 { > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-name = "vcc-1v2-hsic"; > +}; > + > +®_eldo2 { > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "dsp-vcc"; > +}; > + > +®_eldo3 { > + regulator-min-microvolt = <3000000>; > + regulator-max-microvolt = <3000000>; > + regulator-name = "eldo3"; Is this connected or used? If not you could just omit it. > +}; > + > +®_usb1_vbus { > + pinctrl-names = "default"; > + pinctrl-0 = <&usb1_vbus_pin_r16>; > + gpio = <&pio 3 12 GPIO_ACTIVE_HIGH>; /* PD12 */ > + status = "okay"; > +}; > + > +&uart0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&uart0_pins_b>; > + status = "okay"; > +}; > + > +&usb_otg { > + dr_mode = "otg"; > + status = "okay"; > +}; > + > +&usbphy { > + status = "okay"; > + pinctrl-names = "default"; > + pinctrl-0 = <&usb0_id_det>; > + usb0_vbus-supply = <®_drivevbus>; > + usb0_id_det-gpios = <&pio 3 10 GPIO_ACTIVE_HIGH>; /* PD10 */ No vbus detect or vbus-power-supply? Though IIRC it still works, just slower. Regards, ChenYu > + usb1_vbus-supply = <®_usb1_vbus>; /* USB1 VBUS is always on */ > +}; > -- > 2.5.0 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-13 13:04 ` Chen-Yu Tsai @ 2016-06-14 12:59 ` Quentin Schulz 2016-06-14 13:19 ` Chen-Yu Tsai 0 siblings, 1 reply; 12+ messages in thread From: Quentin Schulz @ 2016-06-14 12:59 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux, Maxime Ripard, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni Hi, On 13/06/2016 15:04, Chen-Yu Tsai wrote: > Hi, > > On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz > <quentin.schulz@free-electrons.com> wrote: >> The Parrot Board is an evaluation board with an Allwinner R16 (assumed >> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host > > You say NAND here, but you enable mmc2 for eMMC below. Please correct it. > ACK. >> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 >> controllable buttons, an LVDS port with separated backlight and >> capacitive touch panel ports, an audio/microphone jack, a camera CSI >> port, 2 sets of 22 GPIOs and an accelerometer. > > I assume the board is this one: > > https://world.taobao.com/item/530374411673.htm > Definitely looks like it. >> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >> --- >> arch/arm/boot/dts/Makefile | 1 + >> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ >> 2 files changed, 334 insertions(+) >> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts >> >> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >> index 06b6c2d..1149512 100644 >> --- a/arch/arm/boot/dts/Makefile >> +++ b/arch/arm/boot/dts/Makefile >> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ >> sun8i-a33-ippo-q8h-v1.2.dtb \ >> sun8i-a33-q8-tablet.dtb \ >> sun8i-a33-sinlinx-sina33.dtb \ >> + sun8i-r16-parrot.dtb \ >> sun8i-a83t-allwinner-h8homlet-v2.dtb \ >> sun8i-a83t-cubietruck-plus.dtb \ >> sun8i-h3-orangepi-2.dtb \ >> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts >> new file mode 100644 >> index 0000000..75e2420 >> --- /dev/null >> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts >> @@ -0,0 +1,333 @@ >> +/* >> + * Copyright 2015 Quentin Schulz >> + * >> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" >> +#include "sunxi-common-regulators.dtsi" >> + >> +#include <dt-bindings/gpio/gpio.h> >> +#include <dt-bindings/input/input.h> >> + >> +/ { >> + model = "Allwinner Parrot EVB R16"; >> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; >> + >> + aliases { >> + serial0 = &uart0; >> + }; >> + >> + chosen { >> + stdout-path = "serial0:115200n8"; >> + }; >> + >> + leds { >> + compatible = "gpio-leds"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&led_pins_r16>; > > IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. > Same goes for all the other r16 identifier names. > ACK. >> + >> + led1 { >> + label = "r16:led1:usr"; >> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ >> + }; >> + >> + led2 { >> + label = "r16:led2:usr"; >> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ >> + }; >> + }; >> + >> + wifi_pwrseq: wifi_pwrseq { >> + compatible = "mmc-pwrseq-simple"; >> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ >> + }; >> + >> +}; >> + >> +&ehci0 { >> + status = "okay"; >> +}; >> + >> +&i2c1 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&i2c1_pins_a>; >> + status = "okay"; > > Nothing connected? A comment mentioning which connector this is on > if it's not directly connecting something on the board would be nice. > An accelerometer is connected to this i2c, but: 1) The given address of the i2c device given by i2cdetect is not the same as specified in both fex and schematics. 2) The accelerometer has a "product reference" on the schematics for a Broadcom BMA250 but the associated driver does not work with it. So there is an accelerometer connected to this i2c but I've not found yet what can drive it. I could add a comment specifying the accelerometer is attached to this i2c or remove the node? >> +}; >> + >> +&lradc { >> + vref-supply = <®_aldo3>; >> + status = "okay"; >> + >> + button@0 { >> + label = "V+"; >> + linux,code = <KEY_VOLUMEUP>; >> + channel = <0>; >> + voltage = <190000>; >> + }; >> + >> + button@1 { >> + label = "V-"; >> + linux,code = <KEY_VOLUMEDOWN>; >> + channel = <0>; >> + voltage = <390000>; >> + }; >> + >> +}; >> + >> +&mmc0 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; >> + vmmc-supply = <®_dcdc1>; >> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ >> + bus-width = <4>; >> + status = "okay"; >> +}; >> + >> +&mmc1 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; >> + vmmc-supply = <®_aldo1>; > > This looks fishy. See below. > >> + mmc-pwrseq = <&wifi_pwrseq>; >> + bus-width = <4>; >> + non-removable; >> + status = "okay"; >> +}; >> + >> +&mmc2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&mmc2_8bit_pins>; >> + vmmc-supply = <®_dcdc1>; >> + bus-width = <8>; >> + non-removable; >> + cap-mmc-hw-reset; >> + status = "okay"; >> +}; >> + >> +&mmc2_8bit_pins { >> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> +}; >> + >> +&ohci0 { >> + status = "okay"; >> +}; >> + >> +&pio { >> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { > > _parrot suffix works as well. > >> + allwinner,pins = "PD14"; >> + allwinner,function = "gpio_in"; >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> + }; >> + >> + led_pins_r16: led_pins@0 { >> + allwinner,pins = "PE16", "PE17"; >> + allwinner,function = "gpio_out"; >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> + }; >> + >> + usb0_id_det: usb0_id_detect_pin@0 { >> + allwinner,pins = "PD10"; >> + allwinner,function = "gpio_in"; >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> + }; >> + >> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { >> + allwinner,pins = "PD12"; >> + allwinner,function = "gpio_out"; >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> + }; >> +}; >> + >> +&r_pio { >> + wifi_reset_pin_r16: wifi_reset_pin@3 { > > Why @3? > This is a typo, I'll correct it. >> + allwinner,pins = "PL6"; >> + allwinner,function = "gpio_out"; >> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> + }; >> +}; >> + >> +&r_rsb { >> + status = "okay"; >> + >> + axp22x: pmic@3a3 { >> + compatible = "x-powers,axp223"; >> + reg = <0x3a3>; >> + interrupt-parent = <&nmi_intc>; >> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; >> + eldoin-supply = <®_dcdc1>; > > A drivevbus-supply referencing reg_vcc5v0 here would be better. > ACK. >> + x-powers,drive-vbus-en; >> + }; >> +}; >> + >> +#include "axp22x.dtsi" >> + >> +®_aldo1 { >> + regulator-always-on; >> + regulator-min-microvolt = <3000000>; >> + regulator-max-microvolt = <3000000>; >> + regulator-name = "aldo1"; > > What is this for exactly? Would turning it off render the system inoperable? > How was it referenced in the fex file? > > If this is for WiFi I/O VCC, then you should specify it in mmc1 with > vqmmc-supply. > In the fex, aldo1 is one of the three power inputs for the WiFi (the others being dldo1 and dldo2) and in the schematics it is linked to both VCC-USB and VCC-IO-WIFI. I tried to turn it off and, indeed, the system becomes inoperable. I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am wondering what to put in vmmc-supply for mmc1 since the WiFi module has three power inputs: dldo1, dldo2 and aldo1. In the fex, they are referenced as, respectively, module_power1, module_power2 and module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI while aldo1 is used for VCC-IO-WIFI (if it can help in any way). VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. VCC-IO-WIFI is connected to pin VDDIO of the chip. >> +}; >> + >> +®_aldo2 { >> + regulator-always-on; >> + regulator-min-microvolt = <2350000>; >> + regulator-max-microvolt = <2650000>; >> + regulator-name = "vdd-dll"; >> +}; >> + >> +®_aldo3 { >> + regulator-always-on; >> + regulator-min-microvolt = <2700000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-name = "vcc-pll-avcc"; >> +}; >> + >> +®_dc5ldo { >> + regulator-always-on; >> + regulator-min-microvolt = <900000>; >> + regulator-max-microvolt = <1400000>; >> + regulator-name = "vdd-cpus"; >> +}; >> + >> +®_dcdc1 { >> + regulator-always-on; >> + regulator-min-microvolt = <3000000>; >> + regulator-max-microvolt = <3000000>; >> + regulator-name = "vcc-3v0"; >> +}; >> + >> +®_dcdc2 { >> + regulator-always-on; >> + regulator-min-microvolt = <900000>; >> + regulator-max-microvolt = <1400000>; >> + regulator-name = "vdd-sys"; >> +}; >> + >> +®_dcdc3 { >> + regulator-always-on; >> + regulator-min-microvolt = <900000>; >> + regulator-max-microvolt = <1400000>; >> + regulator-name = "vdd-cpu"; >> +}; >> + >> +®_dcdc5 { >> + regulator-always-on; >> + regulator-min-microvolt = <1500000>; >> + regulator-max-microvolt = <1500000>; >> + regulator-name = "vcc-dram"; >> +}; >> + >> +®_dldo1 { >> + regulator-always-on; > > A comment saying why this is always on would be nice. > I assume this is and dldo2 are waiting on regulator supply list support. > dldo1 and dldo2 are not always on. It is a mistake on my side. Certainly left after quick debugging sessions for the WiFi. >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-name = "vcc-wifi"; >> +}; >> + >> +®_dldo2 { >> + regulator-always-on; >> + regulator-min-microvolt = <3300000>; >> + regulator-max-microvolt = <3300000>; >> + regulator-name = "vcc-wifi1"; >> +}; >> + >> +®_dldo3 { >> + regulator-min-microvolt = <3000000>; >> + regulator-max-microvolt = <3000000>; >> + regulator-name = "vcc-3v0-csi"; >> +}; >> + >> +®_drivevbus { >> + regulator-name = "usb0-vbus"; >> + status = "okay"; >> +}; >> + >> +®_eldo1 { >> + regulator-min-microvolt = <1200000>; >> + regulator-max-microvolt = <1200000>; >> + regulator-name = "vcc-1v2-hsic"; >> +}; >> + >> +®_eldo2 { >> + regulator-min-microvolt = <3000000>; >> + regulator-max-microvolt = <3000000>; >> + regulator-name = "dsp-vcc"; >> +}; >> + >> +®_eldo3 { >> + regulator-min-microvolt = <3000000>; >> + regulator-max-microvolt = <3000000>; >> + regulator-name = "eldo3"; > > Is this connected or used? If not you could just omit it. > eldo3 is connected to a single GPIO. >> +}; >> + >> +®_usb1_vbus { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&usb1_vbus_pin_r16>; >> + gpio = <&pio 3 12 GPIO_ACTIVE_HIGH>; /* PD12 */ >> + status = "okay"; >> +}; >> + >> +&uart0 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&uart0_pins_b>; >> + status = "okay"; >> +}; >> + >> +&usb_otg { >> + dr_mode = "otg"; >> + status = "okay"; >> +}; >> + >> +&usbphy { >> + status = "okay"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&usb0_id_det>; >> + usb0_vbus-supply = <®_drivevbus>; >> + usb0_id_det-gpios = <&pio 3 10 GPIO_ACTIVE_HIGH>; /* PD10 */ > > No vbus detect or vbus-power-supply? Though IIRC it still works, just slower. > Adding "usb0_vbus_power-supply = <&usb_power_suply>;" (and setting status of usb_power_supply to okay) makes the micro USB port not detecting USB cable plugged in (in host or peripheral mode). In the fex, the vbus_det-gpio is "apx_ctrl", I guess this means we don't have a GPIO for vbus detection? Thanks! Regards, Quentin > Regards, > ChenYu > >> + usb1_vbus-supply = <®_usb1_vbus>; /* USB1 VBUS is always on */ >> +}; >> -- >> 2.5.0 >> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-14 12:59 ` Quentin Schulz @ 2016-06-14 13:19 ` Chen-Yu Tsai 2016-06-15 7:52 ` Hans de Goede 2016-06-20 15:44 ` Maxime Ripard 0 siblings, 2 replies; 12+ messages in thread From: Chen-Yu Tsai @ 2016-06-14 13:19 UTC (permalink / raw) To: Quentin Schulz Cc: Chen-Yu Tsai, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux, Maxime Ripard, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni, Hans De Goede On Tue, Jun 14, 2016 at 8:59 PM, Quentin Schulz <quentin.schulz@free-electrons.com> wrote: > Hi, > > On 13/06/2016 15:04, Chen-Yu Tsai wrote: >> Hi, >> >> On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz >> <quentin.schulz@free-electrons.com> wrote: >>> The Parrot Board is an evaluation board with an Allwinner R16 (assumed >>> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host >> >> You say NAND here, but you enable mmc2 for eMMC below. Please correct it. >> > > ACK. > >>> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 >>> controllable buttons, an LVDS port with separated backlight and >>> capacitive touch panel ports, an audio/microphone jack, a camera CSI >>> port, 2 sets of 22 GPIOs and an accelerometer. >> >> I assume the board is this one: >> >> https://world.taobao.com/item/530374411673.htm >> > > Definitely looks like it. > >>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >>> --- >>> arch/arm/boot/dts/Makefile | 1 + >>> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ >>> 2 files changed, 334 insertions(+) >>> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts >>> >>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >>> index 06b6c2d..1149512 100644 >>> --- a/arch/arm/boot/dts/Makefile >>> +++ b/arch/arm/boot/dts/Makefile >>> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ >>> sun8i-a33-ippo-q8h-v1.2.dtb \ >>> sun8i-a33-q8-tablet.dtb \ >>> sun8i-a33-sinlinx-sina33.dtb \ >>> + sun8i-r16-parrot.dtb \ >>> sun8i-a83t-allwinner-h8homlet-v2.dtb \ >>> sun8i-a83t-cubietruck-plus.dtb \ >>> sun8i-h3-orangepi-2.dtb \ >>> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>> new file mode 100644 >>> index 0000000..75e2420 >>> --- /dev/null >>> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>> @@ -0,0 +1,333 @@ >>> +/* >>> + * Copyright 2015 Quentin Schulz >>> + * >>> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" >>> +#include "sunxi-common-regulators.dtsi" >>> + >>> +#include <dt-bindings/gpio/gpio.h> >>> +#include <dt-bindings/input/input.h> >>> + >>> +/ { >>> + model = "Allwinner Parrot EVB R16"; >>> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; >>> + >>> + aliases { >>> + serial0 = &uart0; >>> + }; >>> + >>> + chosen { >>> + stdout-path = "serial0:115200n8"; >>> + }; >>> + >>> + leds { >>> + compatible = "gpio-leds"; >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&led_pins_r16>; >> >> IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. >> Same goes for all the other r16 identifier names. >> > > ACK. > >>> + >>> + led1 { >>> + label = "r16:led1:usr"; >>> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ >>> + }; >>> + >>> + led2 { >>> + label = "r16:led2:usr"; >>> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ >>> + }; >>> + }; >>> + >>> + wifi_pwrseq: wifi_pwrseq { >>> + compatible = "mmc-pwrseq-simple"; >>> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ >>> + }; >>> + >>> +}; >>> + >>> +&ehci0 { >>> + status = "okay"; >>> +}; >>> + >>> +&i2c1 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&i2c1_pins_a>; >>> + status = "okay"; >> >> Nothing connected? A comment mentioning which connector this is on >> if it's not directly connecting something on the board would be nice. >> > > An accelerometer is connected to this i2c, but: > 1) The given address of the i2c device given by i2cdetect is not the > same as specified in both fex and schematics. > 2) The accelerometer has a "product reference" on the schematics for a > Broadcom BMA250 but the associated driver does not work with it. > > So there is an accelerometer connected to this i2c but I've not found > yet what can drive it. I could add a comment specifying the > accelerometer is attached to this i2c or remove the node? A comment will suffice, until you figure out what exactly is on there. > >>> +}; >>> + >>> +&lradc { >>> + vref-supply = <®_aldo3>; >>> + status = "okay"; >>> + >>> + button@0 { >>> + label = "V+"; >>> + linux,code = <KEY_VOLUMEUP>; >>> + channel = <0>; >>> + voltage = <190000>; >>> + }; >>> + >>> + button@1 { >>> + label = "V-"; >>> + linux,code = <KEY_VOLUMEDOWN>; >>> + channel = <0>; >>> + voltage = <390000>; >>> + }; >>> + >>> +}; >>> + >>> +&mmc0 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; >>> + vmmc-supply = <®_dcdc1>; >>> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ >>> + bus-width = <4>; >>> + status = "okay"; >>> +}; >>> + >>> +&mmc1 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; >>> + vmmc-supply = <®_aldo1>; >> >> This looks fishy. See below. >> >>> + mmc-pwrseq = <&wifi_pwrseq>; >>> + bus-width = <4>; >>> + non-removable; >>> + status = "okay"; >>> +}; >>> + >>> +&mmc2 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&mmc2_8bit_pins>; >>> + vmmc-supply = <®_dcdc1>; >>> + bus-width = <8>; >>> + non-removable; >>> + cap-mmc-hw-reset; >>> + status = "okay"; >>> +}; >>> + >>> +&mmc2_8bit_pins { >>> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>> +}; >>> + >>> +&ohci0 { >>> + status = "okay"; >>> +}; >>> + >>> +&pio { >>> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { >> >> _parrot suffix works as well. >> >>> + allwinner,pins = "PD14"; >>> + allwinner,function = "gpio_in"; >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>> + }; >>> + >>> + led_pins_r16: led_pins@0 { >>> + allwinner,pins = "PE16", "PE17"; >>> + allwinner,function = "gpio_out"; >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>> + }; >>> + >>> + usb0_id_det: usb0_id_detect_pin@0 { >>> + allwinner,pins = "PD10"; >>> + allwinner,function = "gpio_in"; >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>> + }; >>> + >>> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { >>> + allwinner,pins = "PD12"; >>> + allwinner,function = "gpio_out"; >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>> + }; >>> +}; >>> + >>> +&r_pio { >>> + wifi_reset_pin_r16: wifi_reset_pin@3 { >> >> Why @3? >> > > This is a typo, I'll correct it. > >>> + allwinner,pins = "PL6"; >>> + allwinner,function = "gpio_out"; >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>> + }; >>> +}; >>> + >>> +&r_rsb { >>> + status = "okay"; >>> + >>> + axp22x: pmic@3a3 { >>> + compatible = "x-powers,axp223"; >>> + reg = <0x3a3>; >>> + interrupt-parent = <&nmi_intc>; >>> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; >>> + eldoin-supply = <®_dcdc1>; >> >> A drivevbus-supply referencing reg_vcc5v0 here would be better. >> > > ACK. > >>> + x-powers,drive-vbus-en; >>> + }; >>> +}; >>> + >>> +#include "axp22x.dtsi" >>> + >>> +®_aldo1 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <3000000>; >>> + regulator-max-microvolt = <3000000>; >>> + regulator-name = "aldo1"; >> >> What is this for exactly? Would turning it off render the system inoperable? >> How was it referenced in the fex file? >> >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with >> vqmmc-supply. >> > > In the fex, aldo1 is one of the three power inputs for the WiFi (the > others being dldo1 and dldo2) and in the schematics it is linked to > both VCC-USB and VCC-IO-WIFI. > > I tried to turn it off and, indeed, the system becomes inoperable. > > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am > wondering what to put in vmmc-supply for mmc1 since the WiFi module has > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are > referenced as, respectively, module_power1, module_power2 and > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI > while aldo1 is used for VCC-IO-WIFI (if it can help in any way). > > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. > VCC-IO-WIFI is connected to pin VDDIO of the chip. VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. About having 2 regulators, Maxime is working on a solution. In the meantime, you could have dldo1/dldo2 always-on, and put a TODO comment on them. Not sure about it also linked to VCC-USB though. Is VCC-USB connected to the SoC or just a trace on the board? If it's the latter, it might also be designed to work with a USB based WiFi module. > >>> +}; >>> + >>> +®_aldo2 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <2350000>; >>> + regulator-max-microvolt = <2650000>; >>> + regulator-name = "vdd-dll"; >>> +}; >>> + >>> +®_aldo3 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <2700000>; >>> + regulator-max-microvolt = <3300000>; >>> + regulator-name = "vcc-pll-avcc"; >>> +}; >>> + >>> +®_dc5ldo { >>> + regulator-always-on; >>> + regulator-min-microvolt = <900000>; >>> + regulator-max-microvolt = <1400000>; >>> + regulator-name = "vdd-cpus"; >>> +}; >>> + >>> +®_dcdc1 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <3000000>; >>> + regulator-max-microvolt = <3000000>; >>> + regulator-name = "vcc-3v0"; >>> +}; >>> + >>> +®_dcdc2 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <900000>; >>> + regulator-max-microvolt = <1400000>; >>> + regulator-name = "vdd-sys"; >>> +}; >>> + >>> +®_dcdc3 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <900000>; >>> + regulator-max-microvolt = <1400000>; >>> + regulator-name = "vdd-cpu"; >>> +}; >>> + >>> +®_dcdc5 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <1500000>; >>> + regulator-max-microvolt = <1500000>; >>> + regulator-name = "vcc-dram"; >>> +}; >>> + >>> +®_dldo1 { >>> + regulator-always-on; >> >> A comment saying why this is always on would be nice. >> I assume this is and dldo2 are waiting on regulator supply list support. >> > > dldo1 and dldo2 are not always on. It is a mistake on my side. Certainly > left after quick debugging sessions for the WiFi. > >>> + regulator-min-microvolt = <3300000>; >>> + regulator-max-microvolt = <3300000>; >>> + regulator-name = "vcc-wifi"; >>> +}; >>> + >>> +®_dldo2 { >>> + regulator-always-on; >>> + regulator-min-microvolt = <3300000>; >>> + regulator-max-microvolt = <3300000>; >>> + regulator-name = "vcc-wifi1"; >>> +}; >>> + >>> +®_dldo3 { >>> + regulator-min-microvolt = <3000000>; >>> + regulator-max-microvolt = <3000000>; >>> + regulator-name = "vcc-3v0-csi"; >>> +}; >>> + >>> +®_drivevbus { >>> + regulator-name = "usb0-vbus"; >>> + status = "okay"; >>> +}; >>> + >>> +®_eldo1 { >>> + regulator-min-microvolt = <1200000>; >>> + regulator-max-microvolt = <1200000>; >>> + regulator-name = "vcc-1v2-hsic"; >>> +}; >>> + >>> +®_eldo2 { >>> + regulator-min-microvolt = <3000000>; >>> + regulator-max-microvolt = <3000000>; >>> + regulator-name = "dsp-vcc"; >>> +}; >>> + >>> +®_eldo3 { >>> + regulator-min-microvolt = <3000000>; >>> + regulator-max-microvolt = <3000000>; >>> + regulator-name = "eldo3"; >> >> Is this connected or used? If not you could just omit it. >> > > eldo3 is connected to a single GPIO. Any idea what it's used for? > >>> +}; >>> + >>> +®_usb1_vbus { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&usb1_vbus_pin_r16>; >>> + gpio = <&pio 3 12 GPIO_ACTIVE_HIGH>; /* PD12 */ >>> + status = "okay"; >>> +}; >>> + >>> +&uart0 { >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&uart0_pins_b>; >>> + status = "okay"; >>> +}; >>> + >>> +&usb_otg { >>> + dr_mode = "otg"; >>> + status = "okay"; >>> +}; >>> + >>> +&usbphy { >>> + status = "okay"; >>> + pinctrl-names = "default"; >>> + pinctrl-0 = <&usb0_id_det>; >>> + usb0_vbus-supply = <®_drivevbus>; >>> + usb0_id_det-gpios = <&pio 3 10 GPIO_ACTIVE_HIGH>; /* PD10 */ >> >> No vbus detect or vbus-power-supply? Though IIRC it still works, just slower. >> > > Adding "usb0_vbus_power-supply = <&usb_power_suply>;" (and setting > status of usb_power_supply to okay) makes the micro USB port not > detecting USB cable plugged in (in host or peripheral mode). (CC-ing Hans) There are some fixes for this, though they might not have landed. > > In the fex, the vbus_det-gpio is "apx_ctrl", I guess this means we don't > have a GPIO for vbus detection? Yeah. This means we're using the PMIC's VBUS sensing function. Regards ChenYu > > Thanks! > > Regards, > Quentin > >> Regards, >> ChenYu >> >>> + usb1_vbus-supply = <®_usb1_vbus>; /* USB1 VBUS is always on */ >>> +}; >>> -- >>> 2.5.0 >>> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-14 13:19 ` Chen-Yu Tsai @ 2016-06-15 7:52 ` Hans de Goede 2016-06-20 15:44 ` Maxime Ripard 1 sibling, 0 replies; 12+ messages in thread From: Hans de Goede @ 2016-06-15 7:52 UTC (permalink / raw) To: Chen-Yu Tsai, Quentin Schulz Cc: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux, Maxime Ripard, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni Hi, On 06/14/2016 03:19 PM, Chen-Yu Tsai wrote: > On Tue, Jun 14, 2016 at 8:59 PM, Quentin Schulz > <quentin.schulz@free-electrons.com> wrote: >> Hi, >> >> On 13/06/2016 15:04, Chen-Yu Tsai wrote: >>> Hi, >>> >>> On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz >>> <quentin.schulz@free-electrons.com> wrote: >>>> The Parrot Board is an evaluation board with an Allwinner R16 (assumed >>>> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host >>> >>> You say NAND here, but you enable mmc2 for eMMC below. Please correct it. >>> >> >> ACK. >> >>>> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 >>>> controllable buttons, an LVDS port with separated backlight and >>>> capacitive touch panel ports, an audio/microphone jack, a camera CSI >>>> port, 2 sets of 22 GPIOs and an accelerometer. >>> >>> I assume the board is this one: >>> >>> https://world.taobao.com/item/530374411673.htm >>> >> >> Definitely looks like it. >> >>>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >>>> --- >>>> arch/arm/boot/dts/Makefile | 1 + >>>> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ >>>> 2 files changed, 334 insertions(+) >>>> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts >>>> >>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >>>> index 06b6c2d..1149512 100644 >>>> --- a/arch/arm/boot/dts/Makefile >>>> +++ b/arch/arm/boot/dts/Makefile >>>> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ >>>> sun8i-a33-ippo-q8h-v1.2.dtb \ >>>> sun8i-a33-q8-tablet.dtb \ >>>> sun8i-a33-sinlinx-sina33.dtb \ >>>> + sun8i-r16-parrot.dtb \ >>>> sun8i-a83t-allwinner-h8homlet-v2.dtb \ >>>> sun8i-a83t-cubietruck-plus.dtb \ >>>> sun8i-h3-orangepi-2.dtb \ >>>> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>>> new file mode 100644 >>>> index 0000000..75e2420 >>>> --- /dev/null >>>> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>>> @@ -0,0 +1,333 @@ >>>> +/* >>>> + * Copyright 2015 Quentin Schulz >>>> + * >>>> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" >>>> +#include "sunxi-common-regulators.dtsi" >>>> + >>>> +#include <dt-bindings/gpio/gpio.h> >>>> +#include <dt-bindings/input/input.h> >>>> + >>>> +/ { >>>> + model = "Allwinner Parrot EVB R16"; >>>> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; >>>> + >>>> + aliases { >>>> + serial0 = &uart0; >>>> + }; >>>> + >>>> + chosen { >>>> + stdout-path = "serial0:115200n8"; >>>> + }; >>>> + >>>> + leds { >>>> + compatible = "gpio-leds"; >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&led_pins_r16>; >>> >>> IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. >>> Same goes for all the other r16 identifier names. >>> >> >> ACK. >> >>>> + >>>> + led1 { >>>> + label = "r16:led1:usr"; >>>> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ >>>> + }; >>>> + >>>> + led2 { >>>> + label = "r16:led2:usr"; >>>> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ >>>> + }; >>>> + }; >>>> + >>>> + wifi_pwrseq: wifi_pwrseq { >>>> + compatible = "mmc-pwrseq-simple"; >>>> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ >>>> + }; >>>> + >>>> +}; >>>> + >>>> +&ehci0 { >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&i2c1 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&i2c1_pins_a>; >>>> + status = "okay"; >>> >>> Nothing connected? A comment mentioning which connector this is on >>> if it's not directly connecting something on the board would be nice. >>> >> >> An accelerometer is connected to this i2c, but: >> 1) The given address of the i2c device given by i2cdetect is not the >> same as specified in both fex and schematics. >> 2) The accelerometer has a "product reference" on the schematics for a >> Broadcom BMA250 but the associated driver does not work with it. >> >> So there is an accelerometer connected to this i2c but I've not found >> yet what can drive it. I could add a comment specifying the >> accelerometer is attached to this i2c or remove the node? > > A comment will suffice, until you figure out what exactly is on there. > >> >>>> +}; >>>> + >>>> +&lradc { >>>> + vref-supply = <®_aldo3>; >>>> + status = "okay"; >>>> + >>>> + button@0 { >>>> + label = "V+"; >>>> + linux,code = <KEY_VOLUMEUP>; >>>> + channel = <0>; >>>> + voltage = <190000>; >>>> + }; >>>> + >>>> + button@1 { >>>> + label = "V-"; >>>> + linux,code = <KEY_VOLUMEDOWN>; >>>> + channel = <0>; >>>> + voltage = <390000>; >>>> + }; >>>> + >>>> +}; >>>> + >>>> +&mmc0 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; >>>> + vmmc-supply = <®_dcdc1>; >>>> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ >>>> + bus-width = <4>; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&mmc1 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; >>>> + vmmc-supply = <®_aldo1>; >>> >>> This looks fishy. See below. >>> >>>> + mmc-pwrseq = <&wifi_pwrseq>; >>>> + bus-width = <4>; >>>> + non-removable; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&mmc2 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&mmc2_8bit_pins>; >>>> + vmmc-supply = <®_dcdc1>; >>>> + bus-width = <8>; >>>> + non-removable; >>>> + cap-mmc-hw-reset; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&mmc2_8bit_pins { >>>> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>> +}; >>>> + >>>> +&ohci0 { >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&pio { >>>> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { >>> >>> _parrot suffix works as well. >>> >>>> + allwinner,pins = "PD14"; >>>> + allwinner,function = "gpio_in"; >>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>> + }; >>>> + >>>> + led_pins_r16: led_pins@0 { >>>> + allwinner,pins = "PE16", "PE17"; >>>> + allwinner,function = "gpio_out"; >>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>> + }; >>>> + >>>> + usb0_id_det: usb0_id_detect_pin@0 { >>>> + allwinner,pins = "PD10"; >>>> + allwinner,function = "gpio_in"; >>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>> + }; >>>> + >>>> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { >>>> + allwinner,pins = "PD12"; >>>> + allwinner,function = "gpio_out"; >>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>> + }; >>>> +}; >>>> + >>>> +&r_pio { >>>> + wifi_reset_pin_r16: wifi_reset_pin@3 { >>> >>> Why @3? >>> >> >> This is a typo, I'll correct it. >> >>>> + allwinner,pins = "PL6"; >>>> + allwinner,function = "gpio_out"; >>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>> + }; >>>> +}; >>>> + >>>> +&r_rsb { >>>> + status = "okay"; >>>> + >>>> + axp22x: pmic@3a3 { >>>> + compatible = "x-powers,axp223"; >>>> + reg = <0x3a3>; >>>> + interrupt-parent = <&nmi_intc>; >>>> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; >>>> + eldoin-supply = <®_dcdc1>; >>> >>> A drivevbus-supply referencing reg_vcc5v0 here would be better. >>> >> >> ACK. >> >>>> + x-powers,drive-vbus-en; >>>> + }; >>>> +}; >>>> + >>>> +#include "axp22x.dtsi" >>>> + >>>> +®_aldo1 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <3000000>; >>>> + regulator-max-microvolt = <3000000>; >>>> + regulator-name = "aldo1"; >>> >>> What is this for exactly? Would turning it off render the system inoperable? >>> How was it referenced in the fex file? >>> >>> If this is for WiFi I/O VCC, then you should specify it in mmc1 with >>> vqmmc-supply. >>> >> >> In the fex, aldo1 is one of the three power inputs for the WiFi (the >> others being dldo1 and dldo2) and in the schematics it is linked to >> both VCC-USB and VCC-IO-WIFI. >> >> I tried to turn it off and, indeed, the system becomes inoperable. >> >> I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am >> wondering what to put in vmmc-supply for mmc1 since the WiFi module has >> three power inputs: dldo1, dldo2 and aldo1. In the fex, they are >> referenced as, respectively, module_power1, module_power2 and >> module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI >> while aldo1 is used for VCC-IO-WIFI (if it can help in any way). >> >> VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. >> VCC-IO-WIFI is connected to pin VDDIO of the chip. > > VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. > > About having 2 regulators, Maxime is working on a solution. In the > meantime, you could have dldo1/dldo2 always-on, and put a TODO > comment on them. > > Not sure about it also linked to VCC-USB though. Is VCC-USB connected > to the SoC or just a trace on the board? If it's the latter, it might > also be designed to work with a USB based WiFi module. > >> >>>> +}; >>>> + >>>> +®_aldo2 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <2350000>; >>>> + regulator-max-microvolt = <2650000>; >>>> + regulator-name = "vdd-dll"; >>>> +}; >>>> + >>>> +®_aldo3 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <2700000>; >>>> + regulator-max-microvolt = <3300000>; >>>> + regulator-name = "vcc-pll-avcc"; >>>> +}; >>>> + >>>> +®_dc5ldo { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <900000>; >>>> + regulator-max-microvolt = <1400000>; >>>> + regulator-name = "vdd-cpus"; >>>> +}; >>>> + >>>> +®_dcdc1 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <3000000>; >>>> + regulator-max-microvolt = <3000000>; >>>> + regulator-name = "vcc-3v0"; >>>> +}; >>>> + >>>> +®_dcdc2 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <900000>; >>>> + regulator-max-microvolt = <1400000>; >>>> + regulator-name = "vdd-sys"; >>>> +}; >>>> + >>>> +®_dcdc3 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <900000>; >>>> + regulator-max-microvolt = <1400000>; >>>> + regulator-name = "vdd-cpu"; >>>> +}; >>>> + >>>> +®_dcdc5 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <1500000>; >>>> + regulator-max-microvolt = <1500000>; >>>> + regulator-name = "vcc-dram"; >>>> +}; >>>> + >>>> +®_dldo1 { >>>> + regulator-always-on; >>> >>> A comment saying why this is always on would be nice. >>> I assume this is and dldo2 are waiting on regulator supply list support. >>> >> >> dldo1 and dldo2 are not always on. It is a mistake on my side. Certainly >> left after quick debugging sessions for the WiFi. >> >>>> + regulator-min-microvolt = <3300000>; >>>> + regulator-max-microvolt = <3300000>; >>>> + regulator-name = "vcc-wifi"; >>>> +}; >>>> + >>>> +®_dldo2 { >>>> + regulator-always-on; >>>> + regulator-min-microvolt = <3300000>; >>>> + regulator-max-microvolt = <3300000>; >>>> + regulator-name = "vcc-wifi1"; >>>> +}; >>>> + >>>> +®_dldo3 { >>>> + regulator-min-microvolt = <3000000>; >>>> + regulator-max-microvolt = <3000000>; >>>> + regulator-name = "vcc-3v0-csi"; >>>> +}; >>>> + >>>> +®_drivevbus { >>>> + regulator-name = "usb0-vbus"; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +®_eldo1 { >>>> + regulator-min-microvolt = <1200000>; >>>> + regulator-max-microvolt = <1200000>; >>>> + regulator-name = "vcc-1v2-hsic"; >>>> +}; >>>> + >>>> +®_eldo2 { >>>> + regulator-min-microvolt = <3000000>; >>>> + regulator-max-microvolt = <3000000>; >>>> + regulator-name = "dsp-vcc"; >>>> +}; >>>> + >>>> +®_eldo3 { >>>> + regulator-min-microvolt = <3000000>; >>>> + regulator-max-microvolt = <3000000>; >>>> + regulator-name = "eldo3"; >>> >>> Is this connected or used? If not you could just omit it. >>> >> >> eldo3 is connected to a single GPIO. > > Any idea what it's used for? > >> >>>> +}; >>>> + >>>> +®_usb1_vbus { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&usb1_vbus_pin_r16>; >>>> + gpio = <&pio 3 12 GPIO_ACTIVE_HIGH>; /* PD12 */ >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&uart0 { >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&uart0_pins_b>; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&usb_otg { >>>> + dr_mode = "otg"; >>>> + status = "okay"; >>>> +}; >>>> + >>>> +&usbphy { >>>> + status = "okay"; >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&usb0_id_det>; >>>> + usb0_vbus-supply = <®_drivevbus>; >>>> + usb0_id_det-gpios = <&pio 3 10 GPIO_ACTIVE_HIGH>; /* PD10 */ >>> >>> No vbus detect or vbus-power-supply? Though IIRC it still works, just slower. >>> >> >> Adding "usb0_vbus_power-supply = <&usb_power_suply>;" (and setting >> status of usb_power_supply to okay) makes the micro USB port not >> detecting USB cable plugged in (in host or peripheral mode). > > (CC-ing Hans) > > There are some fixes for this, though they might not have landed. Right, I do not know which tree you are using, but the usb_power_supply support for the axp221 / axp223 has not landed upstream yet, nor is it in linux-sunxi/sunxi-next. The patches have been accepted by lee, but he has not yet pushed them to next. For now you can try: https://github.com/jwrdegoede/linux-sunxi/commits/sunxi-wip Which has all the necessary commits. With that kernel tree the "usb0_vbus_power-supply = <&usb_power_suply>;" should work, and all necessary bits are on their way to next, there just not all there yet. Regards, Hans > >> >> In the fex, the vbus_det-gpio is "apx_ctrl", I guess this means we don't >> have a GPIO for vbus detection? > > Yeah. This means we're using the PMIC's VBUS sensing function. > > Regards > ChenYu > >> >> Thanks! >> >> Regards, >> Quentin >> >>> Regards, >>> ChenYu >>> >>>> + usb1_vbus-supply = <®_usb1_vbus>; /* USB1 VBUS is always on */ >>>> +}; >>>> -- >>>> 2.5.0 >>>> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-14 13:19 ` Chen-Yu Tsai 2016-06-15 7:52 ` Hans de Goede @ 2016-06-20 15:44 ` Maxime Ripard 2016-06-20 16:30 ` Chen-Yu Tsai 1 sibling, 1 reply; 12+ messages in thread From: Maxime Ripard @ 2016-06-20 15:44 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Mark Rutland, devicetree, Hans De Goede, Pawel Moll, Ian Campbell, linux, linux-kernel, Quentin Schulz, Rob Herring, Kumar Gala, Thomas Petazzoni, linux-arm-kernel [-- Attachment #1.1: Type: text/plain, Size: 13334 bytes --] Hi, On Tue, Jun 14, 2016 at 09:19:58PM +0800, Chen-Yu Tsai wrote: > On Tue, Jun 14, 2016 at 8:59 PM, Quentin Schulz > <quentin.schulz@free-electrons.com> wrote: > > Hi, > > > > On 13/06/2016 15:04, Chen-Yu Tsai wrote: > >> Hi, > >> > >> On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz > >> <quentin.schulz@free-electrons.com> wrote: > >>> The Parrot Board is an evaluation board with an Allwinner R16 (assumed > >>> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host > >> > >> You say NAND here, but you enable mmc2 for eMMC below. Please correct it. > >> > > > > ACK. > > > >>> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 > >>> controllable buttons, an LVDS port with separated backlight and > >>> capacitive touch panel ports, an audio/microphone jack, a camera CSI > >>> port, 2 sets of 22 GPIOs and an accelerometer. > >> > >> I assume the board is this one: > >> > >> https://world.taobao.com/item/530374411673.htm > >> > > > > Definitely looks like it. > > > >>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> > >>> --- > >>> arch/arm/boot/dts/Makefile | 1 + > >>> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ > >>> 2 files changed, 334 insertions(+) > >>> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts > >>> > >>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > >>> index 06b6c2d..1149512 100644 > >>> --- a/arch/arm/boot/dts/Makefile > >>> +++ b/arch/arm/boot/dts/Makefile > >>> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ > >>> sun8i-a33-ippo-q8h-v1.2.dtb \ > >>> sun8i-a33-q8-tablet.dtb \ > >>> sun8i-a33-sinlinx-sina33.dtb \ > >>> + sun8i-r16-parrot.dtb \ > >>> sun8i-a83t-allwinner-h8homlet-v2.dtb \ > >>> sun8i-a83t-cubietruck-plus.dtb \ > >>> sun8i-h3-orangepi-2.dtb \ > >>> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts > >>> new file mode 100644 > >>> index 0000000..75e2420 > >>> --- /dev/null > >>> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts > >>> @@ -0,0 +1,333 @@ > >>> +/* > >>> + * Copyright 2015 Quentin Schulz > >>> + * > >>> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" > >>> +#include "sunxi-common-regulators.dtsi" > >>> + > >>> +#include <dt-bindings/gpio/gpio.h> > >>> +#include <dt-bindings/input/input.h> > >>> + > >>> +/ { > >>> + model = "Allwinner Parrot EVB R16"; > >>> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; > >>> + > >>> + aliases { > >>> + serial0 = &uart0; > >>> + }; > >>> + > >>> + chosen { > >>> + stdout-path = "serial0:115200n8"; > >>> + }; > >>> + > >>> + leds { > >>> + compatible = "gpio-leds"; > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&led_pins_r16>; > >> > >> IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. > >> Same goes for all the other r16 identifier names. > >> > > > > ACK. > > > >>> + > >>> + led1 { > >>> + label = "r16:led1:usr"; > >>> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ > >>> + }; > >>> + > >>> + led2 { > >>> + label = "r16:led2:usr"; > >>> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ > >>> + }; > >>> + }; > >>> + > >>> + wifi_pwrseq: wifi_pwrseq { > >>> + compatible = "mmc-pwrseq-simple"; > >>> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ > >>> + }; > >>> + > >>> +}; > >>> + > >>> +&ehci0 { > >>> + status = "okay"; > >>> +}; > >>> + > >>> +&i2c1 { > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&i2c1_pins_a>; > >>> + status = "okay"; > >> > >> Nothing connected? A comment mentioning which connector this is on > >> if it's not directly connecting something on the board would be nice. > >> > > > > An accelerometer is connected to this i2c, but: > > 1) The given address of the i2c device given by i2cdetect is not the > > same as specified in both fex and schematics. > > 2) The accelerometer has a "product reference" on the schematics for a > > Broadcom BMA250 but the associated driver does not work with it. > > > > So there is an accelerometer connected to this i2c but I've not found > > yet what can drive it. I could add a comment specifying the > > accelerometer is attached to this i2c or remove the node? > > A comment will suffice, until you figure out what exactly is on there. > > > > >>> +}; > >>> + > >>> +&lradc { > >>> + vref-supply = <®_aldo3>; > >>> + status = "okay"; > >>> + > >>> + button@0 { > >>> + label = "V+"; > >>> + linux,code = <KEY_VOLUMEUP>; > >>> + channel = <0>; > >>> + voltage = <190000>; > >>> + }; > >>> + > >>> + button@1 { > >>> + label = "V-"; > >>> + linux,code = <KEY_VOLUMEDOWN>; > >>> + channel = <0>; > >>> + voltage = <390000>; > >>> + }; > >>> + > >>> +}; > >>> + > >>> +&mmc0 { > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; > >>> + vmmc-supply = <®_dcdc1>; > >>> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ > >>> + bus-width = <4>; > >>> + status = "okay"; > >>> +}; > >>> + > >>> +&mmc1 { > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; > >>> + vmmc-supply = <®_aldo1>; > >> > >> This looks fishy. See below. > >> > >>> + mmc-pwrseq = <&wifi_pwrseq>; > >>> + bus-width = <4>; > >>> + non-removable; > >>> + status = "okay"; > >>> +}; > >>> + > >>> +&mmc2 { > >>> + pinctrl-names = "default"; > >>> + pinctrl-0 = <&mmc2_8bit_pins>; > >>> + vmmc-supply = <®_dcdc1>; > >>> + bus-width = <8>; > >>> + non-removable; > >>> + cap-mmc-hw-reset; > >>> + status = "okay"; > >>> +}; > >>> + > >>> +&mmc2_8bit_pins { > >>> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > >>> +}; > >>> + > >>> +&ohci0 { > >>> + status = "okay"; > >>> +}; > >>> + > >>> +&pio { > >>> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { > >> > >> _parrot suffix works as well. > >> > >>> + allwinner,pins = "PD14"; > >>> + allwinner,function = "gpio_in"; > >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > >>> + }; > >>> + > >>> + led_pins_r16: led_pins@0 { > >>> + allwinner,pins = "PE16", "PE17"; > >>> + allwinner,function = "gpio_out"; > >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > >>> + }; > >>> + > >>> + usb0_id_det: usb0_id_detect_pin@0 { > >>> + allwinner,pins = "PD10"; > >>> + allwinner,function = "gpio_in"; > >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; > >>> + }; > >>> + > >>> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { > >>> + allwinner,pins = "PD12"; > >>> + allwinner,function = "gpio_out"; > >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > >>> + }; > >>> +}; > >>> + > >>> +&r_pio { > >>> + wifi_reset_pin_r16: wifi_reset_pin@3 { > >> > >> Why @3? > >> > > > > This is a typo, I'll correct it. > > > >>> + allwinner,pins = "PL6"; > >>> + allwinner,function = "gpio_out"; > >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; > >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; > >>> + }; > >>> +}; > >>> + > >>> +&r_rsb { > >>> + status = "okay"; > >>> + > >>> + axp22x: pmic@3a3 { > >>> + compatible = "x-powers,axp223"; > >>> + reg = <0x3a3>; > >>> + interrupt-parent = <&nmi_intc>; > >>> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > >>> + eldoin-supply = <®_dcdc1>; > >> > >> A drivevbus-supply referencing reg_vcc5v0 here would be better. > >> > > > > ACK. > > > >>> + x-powers,drive-vbus-en; > >>> + }; > >>> +}; > >>> + > >>> +#include "axp22x.dtsi" > >>> + > >>> +®_aldo1 { > >>> + regulator-always-on; > >>> + regulator-min-microvolt = <3000000>; > >>> + regulator-max-microvolt = <3000000>; > >>> + regulator-name = "aldo1"; > >> > >> What is this for exactly? Would turning it off render the system inoperable? > >> How was it referenced in the fex file? > >> > >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with > >> vqmmc-supply. > >> > > > > In the fex, aldo1 is one of the three power inputs for the WiFi (the > > others being dldo1 and dldo2) and in the schematics it is linked to > > both VCC-USB and VCC-IO-WIFI. > > > > I tried to turn it off and, indeed, the system becomes inoperable. > > > > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am > > wondering what to put in vmmc-supply for mmc1 since the WiFi module has > > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are > > referenced as, respectively, module_power1, module_power2 and > > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI > > while aldo1 is used for VCC-IO-WIFI (if it can help in any way). > > > > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. > > VCC-IO-WIFI is connected to pin VDDIO of the chip. > > VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one used to drop to 1.8V with the UHS cards? In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the wifi chip power rail, to be handled by a power sequence. > About having 2 regulators, Maxime is working on a solution. In the > meantime, you could have dldo1/dldo2 always-on, and put a TODO > comment on them. Yep. > Not sure about it also linked to VCC-USB though. Is VCC-USB connected > to the SoC or just a trace on the board? If it's the latter, it might > also be designed to work with a USB based WiFi module. It seems to be one of the inputs of the SoC. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 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] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-20 15:44 ` Maxime Ripard @ 2016-06-20 16:30 ` Chen-Yu Tsai 2016-06-20 16:55 ` Hans de Goede 2016-06-20 18:35 ` Maxime Ripard 0 siblings, 2 replies; 12+ messages in thread From: Chen-Yu Tsai @ 2016-06-20 16:30 UTC (permalink / raw) To: Maxime Ripard Cc: Mark Rutland, devicetree, Hans De Goede, Pawel Moll, Ian Campbell, linux, linux-kernel, Quentin Schulz, Chen-Yu Tsai, Rob Herring, Kumar Gala, Thomas Petazzoni, linux-arm-kernel Hi, On Mon, Jun 20, 2016 at 11:44 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Hi, > > On Tue, Jun 14, 2016 at 09:19:58PM +0800, Chen-Yu Tsai wrote: >> On Tue, Jun 14, 2016 at 8:59 PM, Quentin Schulz >> <quentin.schulz@free-electrons.com> wrote: >> > Hi, >> > >> > On 13/06/2016 15:04, Chen-Yu Tsai wrote: >> >> Hi, >> >> >> >> On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz >> >> <quentin.schulz@free-electrons.com> wrote: >> >>> The Parrot Board is an evaluation board with an Allwinner R16 (assumed >> >>> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host >> >> >> >> You say NAND here, but you enable mmc2 for eMMC below. Please correct it. >> >> >> > >> > ACK. >> > >> >>> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 >> >>> controllable buttons, an LVDS port with separated backlight and >> >>> capacitive touch panel ports, an audio/microphone jack, a camera CSI >> >>> port, 2 sets of 22 GPIOs and an accelerometer. >> >> >> >> I assume the board is this one: >> >> >> >> https://world.taobao.com/item/530374411673.htm >> >> >> > >> > Definitely looks like it. >> > >> >>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >> >>> --- >> >>> arch/arm/boot/dts/Makefile | 1 + >> >>> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ >> >>> 2 files changed, 334 insertions(+) >> >>> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts >> >>> >> >>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >> >>> index 06b6c2d..1149512 100644 >> >>> --- a/arch/arm/boot/dts/Makefile >> >>> +++ b/arch/arm/boot/dts/Makefile >> >>> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ >> >>> sun8i-a33-ippo-q8h-v1.2.dtb \ >> >>> sun8i-a33-q8-tablet.dtb \ >> >>> sun8i-a33-sinlinx-sina33.dtb \ >> >>> + sun8i-r16-parrot.dtb \ >> >>> sun8i-a83t-allwinner-h8homlet-v2.dtb \ >> >>> sun8i-a83t-cubietruck-plus.dtb \ >> >>> sun8i-h3-orangepi-2.dtb \ >> >>> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts >> >>> new file mode 100644 >> >>> index 0000000..75e2420 >> >>> --- /dev/null >> >>> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts >> >>> @@ -0,0 +1,333 @@ >> >>> +/* >> >>> + * Copyright 2015 Quentin Schulz >> >>> + * >> >>> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" >> >>> +#include "sunxi-common-regulators.dtsi" >> >>> + >> >>> +#include <dt-bindings/gpio/gpio.h> >> >>> +#include <dt-bindings/input/input.h> >> >>> + >> >>> +/ { >> >>> + model = "Allwinner Parrot EVB R16"; >> >>> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; >> >>> + >> >>> + aliases { >> >>> + serial0 = &uart0; >> >>> + }; >> >>> + >> >>> + chosen { >> >>> + stdout-path = "serial0:115200n8"; >> >>> + }; >> >>> + >> >>> + leds { >> >>> + compatible = "gpio-leds"; >> >>> + pinctrl-names = "default"; >> >>> + pinctrl-0 = <&led_pins_r16>; >> >> >> >> IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. >> >> Same goes for all the other r16 identifier names. >> >> >> > >> > ACK. >> > >> >>> + >> >>> + led1 { >> >>> + label = "r16:led1:usr"; >> >>> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ >> >>> + }; >> >>> + >> >>> + led2 { >> >>> + label = "r16:led2:usr"; >> >>> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ >> >>> + }; >> >>> + }; >> >>> + >> >>> + wifi_pwrseq: wifi_pwrseq { >> >>> + compatible = "mmc-pwrseq-simple"; >> >>> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ >> >>> + }; >> >>> + >> >>> +}; >> >>> + >> >>> +&ehci0 { >> >>> + status = "okay"; >> >>> +}; >> >>> + >> >>> +&i2c1 { >> >>> + pinctrl-names = "default"; >> >>> + pinctrl-0 = <&i2c1_pins_a>; >> >>> + status = "okay"; >> >> >> >> Nothing connected? A comment mentioning which connector this is on >> >> if it's not directly connecting something on the board would be nice. >> >> >> > >> > An accelerometer is connected to this i2c, but: >> > 1) The given address of the i2c device given by i2cdetect is not the >> > same as specified in both fex and schematics. >> > 2) The accelerometer has a "product reference" on the schematics for a >> > Broadcom BMA250 but the associated driver does not work with it. >> > >> > So there is an accelerometer connected to this i2c but I've not found >> > yet what can drive it. I could add a comment specifying the >> > accelerometer is attached to this i2c or remove the node? >> >> A comment will suffice, until you figure out what exactly is on there. >> >> > >> >>> +}; >> >>> + >> >>> +&lradc { >> >>> + vref-supply = <®_aldo3>; >> >>> + status = "okay"; >> >>> + >> >>> + button@0 { >> >>> + label = "V+"; >> >>> + linux,code = <KEY_VOLUMEUP>; >> >>> + channel = <0>; >> >>> + voltage = <190000>; >> >>> + }; >> >>> + >> >>> + button@1 { >> >>> + label = "V-"; >> >>> + linux,code = <KEY_VOLUMEDOWN>; >> >>> + channel = <0>; >> >>> + voltage = <390000>; >> >>> + }; >> >>> + >> >>> +}; >> >>> + >> >>> +&mmc0 { >> >>> + pinctrl-names = "default"; >> >>> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; >> >>> + vmmc-supply = <®_dcdc1>; >> >>> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ >> >>> + bus-width = <4>; >> >>> + status = "okay"; >> >>> +}; >> >>> + >> >>> +&mmc1 { >> >>> + pinctrl-names = "default"; >> >>> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; >> >>> + vmmc-supply = <®_aldo1>; >> >> >> >> This looks fishy. See below. >> >> >> >>> + mmc-pwrseq = <&wifi_pwrseq>; >> >>> + bus-width = <4>; >> >>> + non-removable; >> >>> + status = "okay"; >> >>> +}; >> >>> + >> >>> +&mmc2 { >> >>> + pinctrl-names = "default"; >> >>> + pinctrl-0 = <&mmc2_8bit_pins>; >> >>> + vmmc-supply = <®_dcdc1>; >> >>> + bus-width = <8>; >> >>> + non-removable; >> >>> + cap-mmc-hw-reset; >> >>> + status = "okay"; >> >>> +}; >> >>> + >> >>> +&mmc2_8bit_pins { >> >>> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> >>> +}; >> >>> + >> >>> +&ohci0 { >> >>> + status = "okay"; >> >>> +}; >> >>> + >> >>> +&pio { >> >>> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { >> >> >> >> _parrot suffix works as well. >> >> >> >>> + allwinner,pins = "PD14"; >> >>> + allwinner,function = "gpio_in"; >> >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> >>> + }; >> >>> + >> >>> + led_pins_r16: led_pins@0 { >> >>> + allwinner,pins = "PE16", "PE17"; >> >>> + allwinner,function = "gpio_out"; >> >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> >>> + }; >> >>> + >> >>> + usb0_id_det: usb0_id_detect_pin@0 { >> >>> + allwinner,pins = "PD10"; >> >>> + allwinner,function = "gpio_in"; >> >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >> >>> + }; >> >>> + >> >>> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { >> >>> + allwinner,pins = "PD12"; >> >>> + allwinner,function = "gpio_out"; >> >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> >>> + }; >> >>> +}; >> >>> + >> >>> +&r_pio { >> >>> + wifi_reset_pin_r16: wifi_reset_pin@3 { >> >> >> >> Why @3? >> >> >> > >> > This is a typo, I'll correct it. >> > >> >>> + allwinner,pins = "PL6"; >> >>> + allwinner,function = "gpio_out"; >> >>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >> >>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >> >>> + }; >> >>> +}; >> >>> + >> >>> +&r_rsb { >> >>> + status = "okay"; >> >>> + >> >>> + axp22x: pmic@3a3 { >> >>> + compatible = "x-powers,axp223"; >> >>> + reg = <0x3a3>; >> >>> + interrupt-parent = <&nmi_intc>; >> >>> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; >> >>> + eldoin-supply = <®_dcdc1>; >> >> >> >> A drivevbus-supply referencing reg_vcc5v0 here would be better. >> >> >> > >> > ACK. >> > >> >>> + x-powers,drive-vbus-en; >> >>> + }; >> >>> +}; >> >>> + >> >>> +#include "axp22x.dtsi" >> >>> + >> >>> +®_aldo1 { >> >>> + regulator-always-on; >> >>> + regulator-min-microvolt = <3000000>; >> >>> + regulator-max-microvolt = <3000000>; >> >>> + regulator-name = "aldo1"; >> >> >> >> What is this for exactly? Would turning it off render the system inoperable? >> >> How was it referenced in the fex file? >> >> >> >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with >> >> vqmmc-supply. >> >> >> > >> > In the fex, aldo1 is one of the three power inputs for the WiFi (the >> > others being dldo1 and dldo2) and in the schematics it is linked to >> > both VCC-USB and VCC-IO-WIFI. >> > >> > I tried to turn it off and, indeed, the system becomes inoperable. >> > >> > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am >> > wondering what to put in vmmc-supply for mmc1 since the WiFi module has >> > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are >> > referenced as, respectively, module_power1, module_power2 and >> > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI >> > while aldo1 is used for VCC-IO-WIFI (if it can help in any way). >> > >> > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. >> > VCC-IO-WIFI is connected to pin VDDIO of the chip. >> >> VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. > > Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one > used to drop to 1.8V with the UHS cards? For eMMC, vmmc and vqmmc correspond to the pins on the eMMC chip. vmmc provides power to the internals, while vqmmc provides power to the I/O buffers on both sides. With SD this is a bit less clear. IIUC the SD card itself supplies I/O voltage, dropping it to 1.8V if necessary, from VDD (vmmc). vqmmc only affects the host side, and whatever external pull-ups that might exist. > In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the > wifi chip power rail, to be handled by a power sequence. For the WiFi chip, there is a separate IO-VCC rail, which is VCC-IO-WIFI. This, from what I understand of the datasheet, is for I/O signaling, and thus vqmmc. VCC-WIFI is what powers the chip, which is what vmmc is. >> About having 2 regulators, Maxime is working on a solution. In the >> meantime, you could have dldo1/dldo2 always-on, and put a TODO >> comment on them. > > Yep. > >> Not sure about it also linked to VCC-USB though. Is VCC-USB connected >> to the SoC or just a trace on the board? If it's the latter, it might >> also be designed to work with a USB based WiFi module. > > It seems to be one of the inputs of the SoC. A33 datasheet has a "VCC-USB" power pin, which it says is "Power supply for the USB PHY". This should probably be tied to the USB PHY node. The PHY framework has a common "phy-supply" property, though I don't know if it works with sunxi's multiple PHY per node approach. Regards ChenYu ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-20 16:30 ` Chen-Yu Tsai @ 2016-06-20 16:55 ` Hans de Goede 2016-06-20 18:35 ` Maxime Ripard 1 sibling, 0 replies; 12+ messages in thread From: Hans de Goede @ 2016-06-20 16:55 UTC (permalink / raw) To: Chen-Yu Tsai, Maxime Ripard Cc: Mark Rutland, devicetree, Pawel Moll, Ian Campbell, linux, linux-kernel, Quentin Schulz, Rob Herring, Kumar Gala, Thomas Petazzoni, linux-arm-kernel Hi, On 20-06-16 18:30, Chen-Yu Tsai wrote: > Hi, > > On Mon, Jun 20, 2016 at 11:44 PM, Maxime Ripard > <maxime.ripard@free-electrons.com> wrote: >> Hi, >> >> On Tue, Jun 14, 2016 at 09:19:58PM +0800, Chen-Yu Tsai wrote: >>> On Tue, Jun 14, 2016 at 8:59 PM, Quentin Schulz >>> <quentin.schulz@free-electrons.com> wrote: >>>> Hi, >>>> >>>> On 13/06/2016 15:04, Chen-Yu Tsai wrote: >>>>> Hi, >>>>> >>>>> On Mon, Jun 13, 2016 at 6:15 PM, Quentin Schulz >>>>> <quentin.schulz@free-electrons.com> wrote: >>>>>> The Parrot Board is an evaluation board with an Allwinner R16 (assumed >>>>>> to be close to an Allwinner A33), 4GB of NAND, 512MB of RAM, USB host >>>>> >>>>> You say NAND here, but you enable mmc2 for eMMC below. Please correct it. >>>>> >>>> >>>> ACK. >>>> >>>>>> and OTG, a WiFi/Bluetooth combo chip, a micro SD Card reader, 2 >>>>>> controllable buttons, an LVDS port with separated backlight and >>>>>> capacitive touch panel ports, an audio/microphone jack, a camera CSI >>>>>> port, 2 sets of 22 GPIOs and an accelerometer. >>>>> >>>>> I assume the board is this one: >>>>> >>>>> https://world.taobao.com/item/530374411673.htm >>>>> >>>> >>>> Definitely looks like it. >>>> >>>>>> Signed-off-by: Quentin Schulz <quentin.schulz@free-electrons.com> >>>>>> --- >>>>>> arch/arm/boot/dts/Makefile | 1 + >>>>>> arch/arm/boot/dts/sun8i-r16-parrot.dts | 333 +++++++++++++++++++++++++++++++++ >>>>>> 2 files changed, 334 insertions(+) >>>>>> create mode 100644 arch/arm/boot/dts/sun8i-r16-parrot.dts >>>>>> >>>>>> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile >>>>>> index 06b6c2d..1149512 100644 >>>>>> --- a/arch/arm/boot/dts/Makefile >>>>>> +++ b/arch/arm/boot/dts/Makefile >>>>>> @@ -760,6 +760,7 @@ dtb-$(CONFIG_MACH_SUN8I) += \ >>>>>> sun8i-a33-ippo-q8h-v1.2.dtb \ >>>>>> sun8i-a33-q8-tablet.dtb \ >>>>>> sun8i-a33-sinlinx-sina33.dtb \ >>>>>> + sun8i-r16-parrot.dtb \ >>>>>> sun8i-a83t-allwinner-h8homlet-v2.dtb \ >>>>>> sun8i-a83t-cubietruck-plus.dtb \ >>>>>> sun8i-h3-orangepi-2.dtb \ >>>>>> diff --git a/arch/arm/boot/dts/sun8i-r16-parrot.dts b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>>>>> new file mode 100644 >>>>>> index 0000000..75e2420 >>>>>> --- /dev/null >>>>>> +++ b/arch/arm/boot/dts/sun8i-r16-parrot.dts >>>>>> @@ -0,0 +1,333 @@ >>>>>> +/* >>>>>> + * Copyright 2015 Quentin Schulz >>>>>> + * >>>>>> + * Quentin Schulz <quentin.schulz@free-electrons.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-a33.dtsi" >>>>>> +#include "sunxi-common-regulators.dtsi" >>>>>> + >>>>>> +#include <dt-bindings/gpio/gpio.h> >>>>>> +#include <dt-bindings/input/input.h> >>>>>> + >>>>>> +/ { >>>>>> + model = "Allwinner Parrot EVB R16"; >>>>>> + compatible = "allwinner,parrot-evb-r16", "allwinner,sun8i-a33"; >>>>>> + >>>>>> + aliases { >>>>>> + serial0 = &uart0; >>>>>> + }; >>>>>> + >>>>>> + chosen { >>>>>> + stdout-path = "serial0:115200n8"; >>>>>> + }; >>>>>> + >>>>>> + leds { >>>>>> + compatible = "gpio-leds"; >>>>>> + pinctrl-names = "default"; >>>>>> + pinctrl-0 = <&led_pins_r16>; >>>>> >>>>> IMO r16 is too generic. You may want to add parrot_ or parrot_evb_ to it. >>>>> Same goes for all the other r16 identifier names. >>>>> >>>> >>>> ACK. >>>> >>>>>> + >>>>>> + led1 { >>>>>> + label = "r16:led1:usr"; >>>>>> + gpio = <&pio 4 17 GPIO_ACTIVE_HIGH>; /* PE17 */ >>>>>> + }; >>>>>> + >>>>>> + led2 { >>>>>> + label = "r16:led2:usr"; >>>>>> + gpio = <&pio 4 16 GPIO_ACTIVE_HIGH>; /* PE16 */ >>>>>> + }; >>>>>> + }; >>>>>> + >>>>>> + wifi_pwrseq: wifi_pwrseq { >>>>>> + compatible = "mmc-pwrseq-simple"; >>>>>> + reset-gpios = <&r_pio 0 6 GPIO_ACTIVE_LOW>; /* PL06 */ >>>>>> + }; >>>>>> + >>>>>> +}; >>>>>> + >>>>>> +&ehci0 { >>>>>> + status = "okay"; >>>>>> +}; >>>>>> + >>>>>> +&i2c1 { >>>>>> + pinctrl-names = "default"; >>>>>> + pinctrl-0 = <&i2c1_pins_a>; >>>>>> + status = "okay"; >>>>> >>>>> Nothing connected? A comment mentioning which connector this is on >>>>> if it's not directly connecting something on the board would be nice. >>>>> >>>> >>>> An accelerometer is connected to this i2c, but: >>>> 1) The given address of the i2c device given by i2cdetect is not the >>>> same as specified in both fex and schematics. >>>> 2) The accelerometer has a "product reference" on the schematics for a >>>> Broadcom BMA250 but the associated driver does not work with it. >>>> >>>> So there is an accelerometer connected to this i2c but I've not found >>>> yet what can drive it. I could add a comment specifying the >>>> accelerometer is attached to this i2c or remove the node? >>> >>> A comment will suffice, until you figure out what exactly is on there. >>> >>>> >>>>>> +}; >>>>>> + >>>>>> +&lradc { >>>>>> + vref-supply = <®_aldo3>; >>>>>> + status = "okay"; >>>>>> + >>>>>> + button@0 { >>>>>> + label = "V+"; >>>>>> + linux,code = <KEY_VOLUMEUP>; >>>>>> + channel = <0>; >>>>>> + voltage = <190000>; >>>>>> + }; >>>>>> + >>>>>> + button@1 { >>>>>> + label = "V-"; >>>>>> + linux,code = <KEY_VOLUMEDOWN>; >>>>>> + channel = <0>; >>>>>> + voltage = <390000>; >>>>>> + }; >>>>>> + >>>>>> +}; >>>>>> + >>>>>> +&mmc0 { >>>>>> + pinctrl-names = "default"; >>>>>> + pinctrl-0 = <&mmc0_pins_a>, <&mmc0_cd_pin_parrot>; >>>>>> + vmmc-supply = <®_dcdc1>; >>>>>> + cd-gpios = <&pio 3 14 GPIO_ACTIVE_LOW>; /* PD14 */ >>>>>> + bus-width = <4>; >>>>>> + status = "okay"; >>>>>> +}; >>>>>> + >>>>>> +&mmc1 { >>>>>> + pinctrl-names = "default"; >>>>>> + pinctrl-0 = <&mmc1_pins_a>, <&wifi_reset_pin_r16>; >>>>>> + vmmc-supply = <®_aldo1>; >>>>> >>>>> This looks fishy. See below. >>>>> >>>>>> + mmc-pwrseq = <&wifi_pwrseq>; >>>>>> + bus-width = <4>; >>>>>> + non-removable; >>>>>> + status = "okay"; >>>>>> +}; >>>>>> + >>>>>> +&mmc2 { >>>>>> + pinctrl-names = "default"; >>>>>> + pinctrl-0 = <&mmc2_8bit_pins>; >>>>>> + vmmc-supply = <®_dcdc1>; >>>>>> + bus-width = <8>; >>>>>> + non-removable; >>>>>> + cap-mmc-hw-reset; >>>>>> + status = "okay"; >>>>>> +}; >>>>>> + >>>>>> +&mmc2_8bit_pins { >>>>>> + allwinner,drive = <SUN4I_PINCTRL_40_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>>>> +}; >>>>>> + >>>>>> +&ohci0 { >>>>>> + status = "okay"; >>>>>> +}; >>>>>> + >>>>>> +&pio { >>>>>> + mmc0_cd_pin_parrot: mmc0_cd_pin@0 { >>>>> >>>>> _parrot suffix works as well. >>>>> >>>>>> + allwinner,pins = "PD14"; >>>>>> + allwinner,function = "gpio_in"; >>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>>>> + }; >>>>>> + >>>>>> + led_pins_r16: led_pins@0 { >>>>>> + allwinner,pins = "PE16", "PE17"; >>>>>> + allwinner,function = "gpio_out"; >>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>>>> + }; >>>>>> + >>>>>> + usb0_id_det: usb0_id_detect_pin@0 { >>>>>> + allwinner,pins = "PD10"; >>>>>> + allwinner,function = "gpio_in"; >>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_PULL_UP>; >>>>>> + }; >>>>>> + >>>>>> + usb1_vbus_pin_r16: usb1_vbus_pin@0 { >>>>>> + allwinner,pins = "PD12"; >>>>>> + allwinner,function = "gpio_out"; >>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>>>> + }; >>>>>> +}; >>>>>> + >>>>>> +&r_pio { >>>>>> + wifi_reset_pin_r16: wifi_reset_pin@3 { >>>>> >>>>> Why @3? >>>>> >>>> >>>> This is a typo, I'll correct it. >>>> >>>>>> + allwinner,pins = "PL6"; >>>>>> + allwinner,function = "gpio_out"; >>>>>> + allwinner,drive = <SUN4I_PINCTRL_10_MA>; >>>>>> + allwinner,pull = <SUN4I_PINCTRL_NO_PULL>; >>>>>> + }; >>>>>> +}; >>>>>> + >>>>>> +&r_rsb { >>>>>> + status = "okay"; >>>>>> + >>>>>> + axp22x: pmic@3a3 { >>>>>> + compatible = "x-powers,axp223"; >>>>>> + reg = <0x3a3>; >>>>>> + interrupt-parent = <&nmi_intc>; >>>>>> + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; >>>>>> + eldoin-supply = <®_dcdc1>; >>>>> >>>>> A drivevbus-supply referencing reg_vcc5v0 here would be better. >>>>> >>>> >>>> ACK. >>>> >>>>>> + x-powers,drive-vbus-en; >>>>>> + }; >>>>>> +}; >>>>>> + >>>>>> +#include "axp22x.dtsi" >>>>>> + >>>>>> +®_aldo1 { >>>>>> + regulator-always-on; >>>>>> + regulator-min-microvolt = <3000000>; >>>>>> + regulator-max-microvolt = <3000000>; >>>>>> + regulator-name = "aldo1"; >>>>> >>>>> What is this for exactly? Would turning it off render the system inoperable? >>>>> How was it referenced in the fex file? >>>>> >>>>> If this is for WiFi I/O VCC, then you should specify it in mmc1 with >>>>> vqmmc-supply. >>>>> >>>> >>>> In the fex, aldo1 is one of the three power inputs for the WiFi (the >>>> others being dldo1 and dldo2) and in the schematics it is linked to >>>> both VCC-USB and VCC-IO-WIFI. >>>> >>>> I tried to turn it off and, indeed, the system becomes inoperable. >>>> >>>> I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am >>>> wondering what to put in vmmc-supply for mmc1 since the WiFi module has >>>> three power inputs: dldo1, dldo2 and aldo1. In the fex, they are >>>> referenced as, respectively, module_power1, module_power2 and >>>> module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI >>>> while aldo1 is used for VCC-IO-WIFI (if it can help in any way). >>>> >>>> VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. >>>> VCC-IO-WIFI is connected to pin VDDIO of the chip. >>> >>> VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. >> >> Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one >> used to drop to 1.8V with the UHS cards? > > For eMMC, vmmc and vqmmc correspond to the pins on the eMMC chip. > vmmc provides power to the internals, while vqmmc provides power to the > I/O buffers on both sides. With SD this is a bit less clear. IIUC the SD > card itself supplies I/O voltage, dropping it to 1.8V if necessary, from > VDD (vmmc). vqmmc only affects the host side, and whatever external pull-ups > that might exist. > >> In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the >> wifi chip power rail, to be handled by a power sequence. > > For the WiFi chip, there is a separate IO-VCC rail, which is VCC-IO-WIFI. > This, from what I understand of the datasheet, is for I/O signaling, and > thus vqmmc. VCC-WIFI is what powers the chip, which is what vmmc is. > >>> About having 2 regulators, Maxime is working on a solution. In the >>> meantime, you could have dldo1/dldo2 always-on, and put a TODO >>> comment on them. >> >> Yep. >> >>> Not sure about it also linked to VCC-USB though. Is VCC-USB connected >>> to the SoC or just a trace on the board? If it's the latter, it might >>> also be designed to work with a USB based WiFi module. >> >> It seems to be one of the inputs of the SoC. > > A33 datasheet has a "VCC-USB" power pin, which it says is "Power supply > for the USB PHY". This should probably be tied to the USB PHY node. The > PHY framework has a common "phy-supply" property, though I don't know if > it works with sunxi's multiple PHY per node approach. I just checked and this will likely appear to work, but it is not a good idea, if we ever actually start turning phy-s off on an individual basis, then this will cause the phy-supply to get turned off on the power_off of the first phy. If this VCC-USB thing is shared we should probably just add support for it to phy-sun4i-usb.c and simple enable it at probe time and disable it at remove time. Regards, Hans ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-20 16:30 ` Chen-Yu Tsai 2016-06-20 16:55 ` Hans de Goede @ 2016-06-20 18:35 ` Maxime Ripard 2016-06-21 9:11 ` Chen-Yu Tsai 1 sibling, 1 reply; 12+ messages in thread From: Maxime Ripard @ 2016-06-20 18:35 UTC (permalink / raw) To: Chen-Yu Tsai Cc: Quentin Schulz, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni, Hans De Goede [-- Attachment #1: Type: text/plain, Size: 2686 bytes --] On Tue, Jun 21, 2016 at 12:30:25AM +0800, Chen-Yu Tsai wrote: > >> >>> +®_aldo1 { > >> >>> + regulator-always-on; > >> >>> + regulator-min-microvolt = <3000000>; > >> >>> + regulator-max-microvolt = <3000000>; > >> >>> + regulator-name = "aldo1"; > >> >> > >> >> What is this for exactly? Would turning it off render the system inoperable? > >> >> How was it referenced in the fex file? > >> >> > >> >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with > >> >> vqmmc-supply. > >> >> > >> > > >> > In the fex, aldo1 is one of the three power inputs for the WiFi (the > >> > others being dldo1 and dldo2) and in the schematics it is linked to > >> > both VCC-USB and VCC-IO-WIFI. > >> > > >> > I tried to turn it off and, indeed, the system becomes inoperable. > >> > > >> > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am > >> > wondering what to put in vmmc-supply for mmc1 since the WiFi module has > >> > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are > >> > referenced as, respectively, module_power1, module_power2 and > >> > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI > >> > while aldo1 is used for VCC-IO-WIFI (if it can help in any way). > >> > > >> > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. > >> > VCC-IO-WIFI is connected to pin VDDIO of the chip. > >> > >> VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. > > > > Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one > > used to drop to 1.8V with the UHS cards? > > For eMMC, vmmc and vqmmc correspond to the pins on the eMMC chip. > vmmc provides power to the internals, while vqmmc provides power to the > I/O buffers on both sides. With SD this is a bit less clear. IIUC the SD > card itself supplies I/O voltage, dropping it to 1.8V if necessary, from > VDD (vmmc). vqmmc only affects the host side, and whatever external pull-ups > that might exist. Ok. > > In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the > > wifi chip power rail, to be handled by a power sequence. > > For the WiFi chip, there is a separate IO-VCC rail, which is VCC-IO-WIFI. > This, from what I understand of the datasheet, is for I/O signaling, and > thus vqmmc. VCC-WIFI is what powers the chip, which is what vmmc is. Ok. but there's still the issue of the two regulators that needs to be kept in sync. Before, that, I'd rather stice to not tying them to the MMC bus, and putting a comment on top. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH] ARM: sun8i: Add Parrot Board DTS 2016-06-20 18:35 ` Maxime Ripard @ 2016-06-21 9:11 ` Chen-Yu Tsai 0 siblings, 0 replies; 12+ messages in thread From: Chen-Yu Tsai @ 2016-06-21 9:11 UTC (permalink / raw) To: Maxime Ripard Cc: Chen-Yu Tsai, Quentin Schulz, Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala, linux, devicetree, linux-arm-kernel, linux-kernel, Thomas Petazzoni, Hans De Goede On Tue, Jun 21, 2016 at 2:35 AM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Tue, Jun 21, 2016 at 12:30:25AM +0800, Chen-Yu Tsai wrote: >> >> >>> +®_aldo1 { >> >> >>> + regulator-always-on; >> >> >>> + regulator-min-microvolt = <3000000>; >> >> >>> + regulator-max-microvolt = <3000000>; >> >> >>> + regulator-name = "aldo1"; >> >> >> >> >> >> What is this for exactly? Would turning it off render the system inoperable? >> >> >> How was it referenced in the fex file? >> >> >> >> >> >> If this is for WiFi I/O VCC, then you should specify it in mmc1 with >> >> >> vqmmc-supply. >> >> >> >> >> > >> >> > In the fex, aldo1 is one of the three power inputs for the WiFi (the >> >> > others being dldo1 and dldo2) and in the schematics it is linked to >> >> > both VCC-USB and VCC-IO-WIFI. >> >> > >> >> > I tried to turn it off and, indeed, the system becomes inoperable. >> >> > >> >> > I'll add vqmmc-supply in mmc1 with aldo1 regulator. However, I am >> >> > wondering what to put in vmmc-supply for mmc1 since the WiFi module has >> >> > three power inputs: dldo1, dldo2 and aldo1. In the fex, they are >> >> > referenced as, respectively, module_power1, module_power2 and >> >> > module_power3 and in the schematics dldo1 and dldo2 are named VCC-WIFI >> >> > while aldo1 is used for VCC-IO-WIFI (if it can help in any way). >> >> > >> >> > VCC-WIFI is connected to pin VBAT of the Broadcom AP6212 WiFi chip. >> >> > VCC-IO-WIFI is connected to pin VDDIO of the chip. >> >> >> >> VCC-IO-WIFI is vqmmc, and VCC-WIFI is vmmc. >> > >> > Isn't vmmc supposed to be always powered at 3.3v, and vqmmc the one >> > used to drop to 1.8V with the UHS cards? >> >> For eMMC, vmmc and vqmmc correspond to the pins on the eMMC chip. >> vmmc provides power to the internals, while vqmmc provides power to the >> I/O buffers on both sides. With SD this is a bit less clear. IIUC the SD >> card itself supplies I/O voltage, dropping it to 1.8V if necessary, from >> VDD (vmmc). vqmmc only affects the host side, and whatever external pull-ups >> that might exist. > > Ok. > >> > In which case, VCC-IO-WIFI is vmmc, and VCC-WIFI is just one of the >> > wifi chip power rail, to be handled by a power sequence. >> >> For the WiFi chip, there is a separate IO-VCC rail, which is VCC-IO-WIFI. >> This, from what I understand of the datasheet, is for I/O signaling, and >> thus vqmmc. VCC-WIFI is what powers the chip, which is what vmmc is. > > Ok. but there's still the issue of the two regulators that needs to be > kept in sync. Before, that, I'd rather stice to not tying them to the > MMC bus, and putting a comment on top. OK. Let's do that for now. ChenYu ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2016-06-21 9:11 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-13 10:15 [PATCH] ARM: sun8i: Add Parrot Board DTS Quentin Schulz
2016-06-13 10:15 ` Quentin Schulz
[not found] ` <1465812931-22824-2-git-send-email-quentin.schulz-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
2016-06-13 12:43 ` kbuild test robot
2016-06-13 13:04 ` Chen-Yu Tsai
2016-06-14 12:59 ` Quentin Schulz
2016-06-14 13:19 ` Chen-Yu Tsai
2016-06-15 7:52 ` Hans de Goede
2016-06-20 15:44 ` Maxime Ripard
2016-06-20 16:30 ` Chen-Yu Tsai
2016-06-20 16:55 ` Hans de Goede
2016-06-20 18:35 ` Maxime Ripard
2016-06-21 9:11 ` Chen-Yu Tsai
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).