From: Andrey Smirnov <andrew.smirnov@gmail.com>
To: Shawn Guo <shawnguo@kernel.org>
Cc: linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
Rob Herring <robh@kernel.org>, Liang Pan <Liang.Pan@zii.aero>,
linux-kernel <linux-kernel@vger.kernel.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@vger.kernel.org>, Chris Healy <cphealy@gmail.com>,
Fabio Estevam <festevam@gmail.com>,
Bob Langer <Bob.Langer@zii.aero>,
Lucas Stach <l.stach@pengutronix.de>
Subject: Re: [PATCH v2 1/2] ARM: dts: Add ZII support for ZII i.MX7 RMU2 board
Date: Mon, 24 Jun 2019 11:00:12 -0700 [thread overview]
Message-ID: <CAHQ1cqEPfdZZw9Tp0t78U7pRtQAsVyPU6JcSrrG4_zcLR+vsqg@mail.gmail.com> (raw)
In-Reply-To: <20190624002853.GC3800@dragon>
On Sun, Jun 23, 2019 at 5:29 PM Shawn Guo <shawnguo@kernel.org> wrote:
>
> On Mon, Jun 17, 2019 at 08:30:24AM -0700, Andrey Smirnov wrote:
> > Add support for ZII's i.MX7 based Remote Modem Unit 2 (RMU2) board.
> >
> > Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
> > Cc: Shawn Guo <shawnguo@kernel.org>
> > Cc: Rob Herring <robh@kernel.org>
> > Cc: Chris Healy <cphealy@gmail.com>
> > Cc: Lucas Stach <l.stach@pengutronix.de>
> > Cc: Fabio Estevam <festevam@gmail.com>
> > Cc: Bob Langer <Bob.Langer@zii.aero>
> > Cc: Liang Pan <Liang.Pan@zii.aero>
> > Cc: linux-arm-kernel@lists.infradead.org
> > Cc: linux-kernel@vger.kernel.org
> > Cc: devicetree@vger.kernel.org
> > ---
> >
> > Changes since [v1]:
> >
> > - Added missing #address-cells and #size-cells
> >
> > - Replaced reset-gpio -> reset-gpios
> >
> >
> > [v1] lore.kernel.org/r/20190614080317.16850-1-andrew.smirnov@gmail.com
> >
> > arch/arm/boot/dts/Makefile | 1 +
> > arch/arm/boot/dts/imx7d-zii-rmu2.dts | 361 +++++++++++++++++++++++++++
> > 2 files changed, 362 insertions(+)
> > create mode 100644 arch/arm/boot/dts/imx7d-zii-rmu2.dts
> >
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index 5559028b770e..516e2912236d 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -593,6 +593,7 @@ dtb-$(CONFIG_SOC_IMX7D) += \
> > imx7d-sdb.dtb \
> > imx7d-sdb-reva.dtb \
> > imx7d-sdb-sht11.dtb \
> > + imx7d-zii-rmu2.dtb \
> > imx7d-zii-rpu2.dtb \
> > imx7s-colibri-eval-v3.dtb \
> > imx7s-mba7.dtb \
> > diff --git a/arch/arm/boot/dts/imx7d-zii-rmu2.dts b/arch/arm/boot/dts/imx7d-zii-rmu2.dts
> > new file mode 100644
> > index 000000000000..e60b3232a090
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx7d-zii-rmu2.dts
> > @@ -0,0 +1,361 @@
> > +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> > +/*
> > + * Device tree file for ZII's RMU2 board
> > + *
> > + * RMU - Remote Modem Unit
> > + *
> > + * Copyright (C) 2019 Zodiac Inflight Innovations
> > + */
> > +
> > +/dts-v1/;
> > +#include <dt-bindings/thermal/thermal.h>
> > +#include "imx7d.dtsi"
> > +
> > +/ {
> > + model = "ZII RMU2 Board";
> > + compatible = "zii,imx7d-rmu2", "fsl,imx7d";
> > +
> > + chosen {
> > + stdout-path = &uart2;
> > + };
> > +
> > + gpio-leds {
> > + compatible = "gpio-leds";
> > + pinctrl-0 = <&pinctrl_leds_debug>;
> > + pinctrl-names = "default";
> > +
> > + debug {
> > + label = "zii:green:debug1";
> > + gpios = <&gpio2 8 GPIO_ACTIVE_HIGH>;
> > + linux,default-trigger = "heartbeat";
> > + };
> > + };
> > +};
> > +
> > +&cpu0 {
> > + arm-supply = <&sw1a_reg>;
> > +};
> > +
> > +&ecspi1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_ecspi1>;
> > + cs-gpios = <&gpio4 19 GPIO_ACTIVE_HIGH>;
> > + status = "okay";
> > +
> > + flash@0 {
> > + compatible = "jedec,spi-nor";
> > + spi-max-frequency = <20000000>;
> > + reg = <0>;
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + };
> > +};
> > +
> > +&fec1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_enet1>;
> > + assigned-clocks = <&clks IMX7D_ENET1_TIME_ROOT_SRC>,
> > + <&clks IMX7D_ENET1_TIME_ROOT_CLK>;
> > + assigned-clock-parents = <&clks IMX7D_PLL_ENET_MAIN_100M_CLK>;
> > + assigned-clock-rates = <0>, <100000000>;
> > + phy-mode = "rgmii";
> > + phy-handle = <&fec1_phy>;
> > + status = "okay";
> > +
> > + mdio {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + fec1_phy: phy@0 {
>
> ethernet-phy for node name.
>
OK, will change.
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_enet1_phy_reset>,
> > + <&pinctrl_enet1_phy_interrupt>;
> > + reg = <0>;
> > + interrupt-parent = <&gpio1>;
> > + interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> > + reset-gpios = <&gpio5 11 GPIO_ACTIVE_LOW>;
> > + };
> > + };
> > +};
> > +
> > +&i2c1 {
> > + clock-frequency = <100000>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_i2c1>;
> > + status = "okay";
> > +
> > + pmic: pmic@8 {
>
> Label can be more specific, so maybe:
>
> pfuze3000: pmic@8
>
Hmm, I don't think that label is used anywhere. I may as well just drop it.
> > + compatible = "fsl,pfuze3000";
> > + reg = <0x08>;
> > +
> > + regulators {
> > + sw1a_reg: sw1a {
> > + regulator-min-microvolt = <700000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + regulator-ramp-delay = <6250>;
> > + };
> > +
> > + sw1c_reg: sw1b {
> > + regulator-min-microvolt = <700000>;
> > + regulator-max-microvolt = <1475000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + regulator-ramp-delay = <6250>;
> > + };
> > +
> > + sw2_reg: sw2 {
> > + regulator-min-microvolt = <1500000>;
> > + regulator-max-microvolt = <1850000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + sw3a_reg: sw3 {
> > + regulator-min-microvolt = <900000>;
> > + regulator-max-microvolt = <1650000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + swbst_reg: swbst {
> > + regulator-min-microvolt = <5000000>;
> > + regulator-max-microvolt = <5150000>;
> > + };
> > +
> > + snvs_reg: vsnvs {
> > + regulator-min-microvolt = <1000000>;
> > + regulator-max-microvolt = <3000000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + vref_reg: vrefddr {
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + vgen1_reg: vldo1 {
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + vgen2_reg: vldo2 {
> > + regulator-min-microvolt = <800000>;
> > + regulator-max-microvolt = <1550000>;
> > + regulator-always-on;
> > + };
> > +
> > + vgen3_reg: vccsd {
> > + regulator-min-microvolt = <2850000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + vgen4_reg: v33 {
> > + regulator-min-microvolt = <2850000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + vgen5_reg: vldo3 {
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + vgen6_reg: vldo4 {
> > + regulator-min-microvolt = <1800000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > + };
> > + };
> > +
> > + eeprom@50 {
> > + compatible = "atmel,24c04";
> > + reg = <0x50>;
> > + };
> > +
> > + eeprom@52 {
> > + compatible = "atmel,24c04";
> > + reg = <0x52>;
> > + };
> > +};
> > +
> > +&uart2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_uart2>;
> > + assigned-clocks = <&clks IMX7D_UART2_ROOT_SRC>;
> > + assigned-clock-parents = <&clks IMX7D_OSC_24M_CLK>;
> > + status = "okay";
> > +};
> > +
> > +&uart4 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_uart4>;
> > + assigned-clocks = <&clks IMX7D_UART4_ROOT_SRC>;
> > + assigned-clock-parents = <&clks IMX7D_PLL_SYS_MAIN_240M_CLK>;
> > + status = "okay";
> > +
> > + rave-sp {
> > + compatible = "zii,rave-sp-rdu2";
> > + current-speed = <1000000>;
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > +
> > + watchdog {
> > + compatible = "zii,rave-sp-watchdog";
> > + };
> > +
> > + eeprom@a3 {
> > + compatible = "zii,rave-sp-eeprom";
> > + reg = <0xa3 0x4000>;
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + zii,eeprom-name = "main-eeprom";
> > + };
> > + };
> > +};
> > +
> > +&usbotg2 {
> > + dr_mode = "host";
> > + disable-over-current;
> > + status = "okay";
> > +};
> > +
> > +&usdhc1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usdhc1>;
> > + bus-width = <4>;
> > + no-1-8-v;
> > + no-sdio;
> > + keep-power-in-suspend;
> > + status = "okay";
> > +};
> > +
> > +&usdhc3 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usdhc3>;
> > + bus-width = <8>;
> > + no-1-8-v;
> > + non-removable;
> > + no-sdio;
> > + no-sd;
> > + keep-power-in-suspend;
> > + status = "okay";
> > +};
> > +
> > +&wdog1 {
> > + status = "disabled";
> > +};
> > +
> > +&snvs_rtc {
> > + status = "disabled";
> > +};
>
> Please sort it alphabetically in label name.
>
Will do.
> > +
> > +&snvs_pwrkey {
> > + status = "disabled";
> > +};
>
> We already queued up the patch below to disable snvs_pwrkey by default.
>
> https://lkml.org/lkml/2019/6/13/1170
>
OK, good to know, will drop.
> > +
> > +&iomuxc {
> > + pinctrl_ecspi1: ecspi1grp {
> > + fsl,pins = <
> > + MX7D_PAD_ECSPI1_SCLK__ECSPI1_SCLK 0x2
> > + MX7D_PAD_ECSPI1_MOSI__ECSPI1_MOSI 0x2
> > + MX7D_PAD_ECSPI1_MISO__ECSPI1_MISO 0x2
> > + MX7D_PAD_ECSPI1_SS0__GPIO4_IO19 0x59
> > + >;
> > + };
> > +
> > + pinctrl_enet1: enet1grp {
> > + fsl,pins = <
> > + MX7D_PAD_SD2_CD_B__ENET1_MDIO 0x3
> > + MX7D_PAD_SD2_WP__ENET1_MDC 0x3
> > + MX7D_PAD_ENET1_RGMII_TXC__ENET1_RGMII_TXC 0x1
> > + MX7D_PAD_ENET1_RGMII_TD0__ENET1_RGMII_TD0 0x1
> > + MX7D_PAD_ENET1_RGMII_TD1__ENET1_RGMII_TD1 0x1
> > + MX7D_PAD_ENET1_RGMII_TD2__ENET1_RGMII_TD2 0x1
> > + MX7D_PAD_ENET1_RGMII_TD3__ENET1_RGMII_TD3 0x1
> > + MX7D_PAD_ENET1_RGMII_TX_CTL__ENET1_RGMII_TX_CTL 0x1
> > + MX7D_PAD_ENET1_RGMII_RXC__ENET1_RGMII_RXC 0x1
> > + MX7D_PAD_ENET1_RGMII_RD0__ENET1_RGMII_RD0 0x1
> > + MX7D_PAD_ENET1_RGMII_RD1__ENET1_RGMII_RD1 0x1
> > + MX7D_PAD_ENET1_RGMII_RD2__ENET1_RGMII_RD2 0x1
> > + MX7D_PAD_ENET1_RGMII_RD3__ENET1_RGMII_RD3 0x1
> > + MX7D_PAD_ENET1_RGMII_RX_CTL__ENET1_RGMII_RX_CTL 0x1
> > + >;
> > + };
> > +
> > + pinctrl_enet1_phy_reset: enet1phyresetgrp {
> > + fsl,pins = <
> > + MX7D_PAD_SD2_RESET_B__GPIO5_IO11 0x14
> > +
> > + >;
> > + };
> > +
> > + pinctrl_i2c1: i2c1grp {
> > + fsl,pins = <
> > + MX7D_PAD_I2C1_SDA__I2C1_SDA 0x4000007f
> > + MX7D_PAD_I2C1_SCL__I2C1_SCL 0x4000007f
> > + >;
> > + };
> > +
> > + pinctrl_leds_debug: debuggrp {
>
> ledsgrp?
>
Sure, will change in next version.
Thanks,
Andrey Smirnov
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
prev parent reply other threads:[~2019-06-24 18:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-06-17 15:30 [PATCH v2 1/2] ARM: dts: Add ZII support for ZII i.MX7 RMU2 board Andrey Smirnov
2019-06-17 15:30 ` [PATCH v2 2/2] dt-bindings: arm: fsl: Add " Andrey Smirnov
2019-06-17 16:05 ` [PATCH v2 1/2] ARM: dts: Add ZII " Fabio Estevam
2019-06-24 0:28 ` Shawn Guo
2019-06-24 18:00 ` Andrey Smirnov [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAHQ1cqEPfdZZw9Tp0t78U7pRtQAsVyPU6JcSrrG4_zcLR+vsqg@mail.gmail.com \
--to=andrew.smirnov@gmail.com \
--cc=Bob.Langer@zii.aero \
--cc=Liang.Pan@zii.aero \
--cc=cphealy@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=l.stach@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).