From: Shawn Guo <shawnguo@kernel.org>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: Fabio Estevam <fabio.estevam@nxp.com>,
Gavin Schenk <g.schenk@eckelmann.de>,
NXP Linux Team <linux-imx@nxp.com>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/2] ARM: dts: Add devicetree for Eckelmann CI4x10
Date: Mon, 14 Jan 2019 09:39:11 +0800 [thread overview]
Message-ID: <20190114013909.GE15371@dragon> (raw)
In-Reply-To: <20190111164936.7152-1-u.kleine-koenig@pengutronix.de>
On Fri, Jan 11, 2019 at 05:49:35PM +0100, Uwe Kleine-König wrote:
> This is one of two boards that make use of the recently introduced SIOX
> bus. Apart from the devices described in the dts it features a display
> with touch that I didn't include here because it needs some non-mainline
> change to operate correctly.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> arch/arm/boot/dts/Makefile | 1 +
> arch/arm/boot/dts/imx6dl-eckelmann-ci4x10.dts | 390 ++++++++++++++++++
> 2 files changed, 391 insertions(+)
> create mode 100644 arch/arm/boot/dts/imx6dl-eckelmann-ci4x10.dts
>
> diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> index bd40148a15b2..b60d759e39c0 100644
> --- a/arch/arm/boot/dts/Makefile
> +++ b/arch/arm/boot/dts/Makefile
> @@ -397,6 +397,7 @@ dtb-$(CONFIG_SOC_IMX6Q) += \
> imx6dl-cubox-i-emmc-som-v15.dtb \
> imx6dl-cubox-i-som-v15.dtb \
> imx6dl-dfi-fs700-m60.dtb \
> + imx6dl-eckelmann-ci4x10.dtb \
> imx6dl-emcon-avari.dtb \
> imx6dl-gw51xx.dtb \
> imx6dl-gw52xx.dtb \
> diff --git a/arch/arm/boot/dts/imx6dl-eckelmann-ci4x10.dts b/arch/arm/boot/dts/imx6dl-eckelmann-ci4x10.dts
> new file mode 100644
> index 000000000000..ef7a7c9d55c3
> --- /dev/null
> +++ b/arch/arm/boot/dts/imx6dl-eckelmann-ci4x10.dts
> @@ -0,0 +1,390 @@
> +/*
> + * Copyright (C) 2016 Eckelmann AG.
> + * Copyright (C) 2013 Freescale Semiconductor, Inc.
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
Please use SPDX-License-Identifier.
> + */
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +#include "imx6dl.dtsi"
> +
> +/ {
> + model = "Freescale i.MX6 DualLite Eckelmann CI 4X10 Board";
It's not a board produced by Freescale, right?
> + compatible = "eckelmann,imx6dl-ci4x10", "fsl,imx6dl";
Document the compatible.
> +
> + chosen {
> + stdout-path = &uart3;
> + };
> +
> + memory {
device_type = "memory";
> + reg = <0x10000000 0x40000000>;
> + };
> +
> + clocks {
> + rmii_clk: rmii-clock {
clock-rmii for node name, and put it directly under root.
> + /* This clock is provided by the phy (KSZ8091RNB) */
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-frequency = <50000000>;
> + };
> + };
> +
> + reg_usb_h1_vbus: regulator@1 {
reg_xxx: regulator-xxx {
...
};
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_reg_usb_h1_vbus>;
> +
Drop the newline.
> + compatible = "regulator-fixed";
> + regulator-name = "usb_h1_vbus";
> + regulator-min-microvolt = <5000000>;
> + regulator-max-microvolt = <5000000>;
> + gpio = <&gpio3 31 0>;
Use the defines in include/dt-bindings/gpio/gpio.h.
> + enable-active-high;
> + };
> +
> + siox {
> + compatible = "eckelmann,siox-gpio";
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_siox>;
> +
Ditto
> + din-gpios = <&gpio6 11 0>;
> + dout-gpios = <&gpio6 8 0>;
> + dclk-gpios = <&gpio6 9 0>;
> + dld-gpios = <&gpio6 10 0>;
Ditto
> + };
> +};
> +
> +&can1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_flexcan1>;
> + status = "okay";
> +};
> +
> +&can2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_flexcan2>;
> + status = "okay";
> +};
> +
> +&ecspi2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_ecspi2>;
> + status = "okay";
Put the 'status' at end of property list.
> + cs-gpios = <&gpio5 12 GPIO_ACTIVE_HIGH>;
> +
> + mram: mr25h256@0 {
Use a generic node name.
> + compatible = "everspin,mr25h256";
Undocumented compatible.
> + reg = <0>;
> + spi-max-frequency = <15000000>;
> + };
> +};
> +
> +&ecspi1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_ecspi1>;
> + status = "okay";
Ditto.
> + cs-gpios = <&gpio5 25 GPIO_ACTIVE_HIGH>;
> +
> + tpm_spi_tis@0 {
Just 'tpm' for name maybe?
> + compatible = "infineon,slb9670", "tcg,tpm_tis-spi";
> + reg = <0>;
> + spi-max-frequency = <10000000>;
> + };
> +};
> +
> +&i2c1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_i2c1>;
> + status = "okay";
> +
> + temp@49 {
temperature-sensor
> + compatible = "ad,ad7414";
> + reg = <0x49>;
> + };
> +
> + rtc@51 {
> + compatible = "nxp,pcf2127";
> + reg = <0x51>;
> + };
> +};
> +
> +&iomuxc {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_hog>;
> +
> + pinctrl_hog: hog {
> + fsl,pins = <
> + MX6QDL_PAD_NANDF_D0__GPIO2_IO00 0x00000018 /* buzzer */
> + MX6QDL_PAD_KEY_COL1__GPIO4_IO08 0x00000018 /* OUT_1 */
> + MX6QDL_PAD_KEY_ROW1__GPIO4_IO09 0x00000018 /* OUT_2 */
> + MX6QDL_PAD_KEY_COL2__GPIO4_IO10 0x00000018 /* OUT_3 */
> + MX6QDL_PAD_NANDF_CS2__GPIO6_IO15 0x00000000 /* In1 */
> + MX6QDL_PAD_KEY_ROW0__GPIO4_IO07 0x00000000 /* In2 */
> + MX6QDL_PAD_GPIO_9__GPIO1_IO09 0x00000018 /* unused watchdog pin */
> + MX6QDL_PAD_SD1_DAT2__GPIO1_IO19 0x00000018 /* unused watchdog pin */
> +
> + >;
> + };
> +
> + pinctrl_ecspi1: ecspi1grp {
> + fsl,pins = <
> + MX6QDL_PAD_CSI0_DAT4__ECSPI1_SCLK 0x000100a0
> + MX6QDL_PAD_CSI0_DAT5__ECSPI1_MOSI 0x000100a0
> + MX6QDL_PAD_CSI0_DAT6__ECSPI1_MISO 0x000100a0
> + MX6QDL_PAD_CSI0_DAT7__GPIO5_IO25 0x000100a0
> + >;
> + };
> +
> + pinctrl_ecspi2: ecspi2grp {
> + fsl,pins = <
> + MX6QDL_PAD_DISP0_DAT19__ECSPI2_SCLK 0x000100b1
> + MX6QDL_PAD_EIM_CS1__ECSPI2_MOSI 0x000100b1
> + MX6QDL_PAD_EIM_OE__ECSPI2_MISO 0x000100b1
> + MX6QDL_PAD_DISP0_DAT18__GPIO5_IO12 0x000100b1
> + >;
> + };
> +
> + pinctrl_enet: enetgrp {
> + fsl,pins = <
> + MX6QDL_PAD_GPIO_16__ENET_REF_CLK 0x4001b0a8
> + MX6QDL_PAD_ENET_MDIO__ENET_MDIO 0x0001b098
> + MX6QDL_PAD_ENET_MDC__ENET_MDC 0x0001b098
> + MX6QDL_PAD_ENET_TXD0__ENET_TX_DATA0 0x0001b098
> + MX6QDL_PAD_ENET_TXD1__ENET_TX_DATA1 0x0001b098
> + MX6QDL_PAD_ENET_TX_EN__ENET_TX_EN 0x0001b098
> + MX6QDL_PAD_ENET_RX_ER__ENET_RX_ER 0x0001b0b0
> + MX6QDL_PAD_ENET_RXD0__ENET_RX_DATA0 0x0001b0b0
> + MX6QDL_PAD_ENET_RXD1__ENET_RX_DATA1 0x0001b0b0
> + MX6QDL_PAD_ENET_CRS_DV__ENET_RX_EN 0x0001b0b0
> + MX6QDL_PAD_SD1_CMD__GPIO1_IO18 0x00000018
> + >;
> + };
> +
> + pinctrl_flexcan1: flexcan1grp {
> + fsl,pins = <
> + MX6QDL_PAD_GPIO_7__FLEXCAN1_TX 0x0001b020
> + MX6QDL_PAD_GPIO_8__FLEXCAN1_RX 0x0001b0b0
> + >;
> + };
> +
> + pinctrl_flexcan2: flexcan2grp {
> + fsl,pins = <
> + MX6QDL_PAD_KEY_COL4__FLEXCAN2_TX 0x0001b020
> + MX6QDL_PAD_KEY_ROW4__FLEXCAN2_RX 0x0001b0b0
> + >;
> + };
> +
> + pinctrl_i2c1: i2c1grp {
> + fsl,pins = <
> + /* without SION i2c doesn't detect bus busy */
> + MX6QDL_PAD_CSI0_DAT9__I2C1_SCL 0x4001b820
> + MX6QDL_PAD_CSI0_DAT8__I2C1_SDA 0x4001b820
> + >;
> + };
> +
> + pinctrl_pcie: pciegrp {
> + fsl,pins = <
> + MX6QDL_PAD_SD1_CLK__GPIO1_IO20 0x00000018
> + >;
> + };
> +
> + pinctrl_reg_usb_h1_vbus: reg_usb_h1_vbusgrp {
> + fsl,pins = <
> + MX6QDL_PAD_EIM_D31__GPIO3_IO31 0x0001b0b0
> + >;
> + };
> +
> + pinctrl_siox: sioxgrp {
> + fsl,pins = <
> + MX6QDL_PAD_NANDF_CS0__GPIO6_IO11 0x0001b010 /* DIN */
> + MX6QDL_PAD_NANDF_ALE__GPIO6_IO08 0x0001b010 /* DOUT */
> + MX6QDL_PAD_NANDF_WP_B__GPIO6_IO09 0x0001b010 /* DCLK */
> + MX6QDL_PAD_NANDF_RB0__GPIO6_IO10 0x0001b010 /* DLD */
> + >;
> + };
> +
> + pinctrl_uart1_dte: uart1grp {
> + fsl,pins = <
> + MX6QDL_PAD_CSI0_DAT11__UART1_TX_DATA 0x0001b010
> + MX6QDL_PAD_CSI0_DAT10__UART1_RX_DATA 0x0001b010
> + MX6QDL_PAD_EIM_D19__UART1_RTS_B 0x0001b010
> + MX6QDL_PAD_EIM_D20__UART1_CTS_B 0x0001b010
> + MX6QDL_PAD_EIM_D23__GPIO3_IO23 0x0001b010 /* DCD */
> + MX6QDL_PAD_EIM_D24__GPIO3_IO24 0x0001b010 /* DTR */
> + MX6QDL_PAD_EIM_D25__GPIO3_IO25 0x0001b010 /* DSR */
> + >;
> + };
> +
> + pinctrl_uart2_dte: uart2grp {
> + fsl,pins = <
> + MX6QDL_PAD_EIM_D27__UART2_TX_DATA 0x0001b010
> + MX6QDL_PAD_EIM_D26__UART2_RX_DATA 0x0001b010
> + MX6QDL_PAD_EIM_D28__UART2_RTS_B 0x0001b010
> + MX6QDL_PAD_EIM_D29__UART2_CTS_B 0x0001b010
> + MX6QDL_PAD_NANDF_D1__GPIO2_IO01 0x0001b010 /* DCD */
> + MX6QDL_PAD_GPIO_18__GPIO7_IO13 0x0001b010 /* DTR */
> + MX6QDL_PAD_NANDF_CS3__GPIO6_IO16 0x0001b010 /* DSR */
> + >;
> + };
> +
> + pinctrl_uart3_dce: uart3grp {
> + fsl,pins = <
> + MX6QDL_PAD_SD4_CLK__UART3_RX_DATA 0x0001b010
> + MX6QDL_PAD_SD4_CMD__UART3_TX_DATA 0x0001b010
> + >;
> + };
> +
> + pinctrl_uart4_dce: uart4grp {
> + fsl,pins = <
> + MX6QDL_PAD_CSI0_DAT13__UART4_RX_DATA 0x0001b010
> + MX6QDL_PAD_CSI0_DAT12__UART4_TX_DATA 0x0001b010
> + MX6QDL_PAD_CSI0_DAT17__GPIO6_IO03 0x0001b010
> + >;
> + };
> +
> + pinctrl_uart5_dce: uart5grp {
> + fsl,pins = <
> + MX6QDL_PAD_CSI0_DAT15__UART5_RX_DATA 0x0001b010
> + MX6QDL_PAD_CSI0_DAT14__UART5_TX_DATA 0x0001b010
> + MX6QDL_PAD_CSI0_DAT19__GPIO6_IO05 0x0001b010 /* RTS */
> + >;
> + };
> +
> + pinctrl_usbh1: usbh1grp {
> + fsl,pins = <
> + MX6QDL_PAD_EIM_D30__USB_H1_OC 0x80000000
> + >;
> + };
> +
> + pinctrl_usdhc3: usdhc3grp {
> + fsl,pins = <
> + MX6QDL_PAD_SD3_CMD__SD3_CMD 0x00017059
> + MX6QDL_PAD_SD3_CLK__SD3_CLK 0x00010059
> + MX6QDL_PAD_SD3_DAT0__SD3_DATA0 0x00017059
> + MX6QDL_PAD_SD3_DAT1__SD3_DATA1 0x00017059
> + MX6QDL_PAD_SD3_DAT2__SD3_DATA2 0x00017059
> + MX6QDL_PAD_SD3_DAT3__SD3_DATA3 0x00017059
> + MX6QDL_PAD_SD3_DAT4__SD3_DATA4 0x00017059
> + MX6QDL_PAD_SD3_DAT5__SD3_DATA5 0x00017059
> + MX6QDL_PAD_SD3_DAT6__SD3_DATA6 0x00017059
> + MX6QDL_PAD_SD3_DAT7__SD3_DATA7 0x00017059
> + >;
> + };
> +};
> +
> +&gpio2 {
Out of alphabetic order.
> + gpio-line-names = "buzzer", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "";
> +};
> +
> +&gpio4 {
> + gpio-line-names = "", "", "", "", "", "", "", "in2",
> + "prio2", "prio1", "aux", "", "", "", "", "",
> + "", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "";
> +};
> +
> +&gpio6 {
> + gpio-line-names = "", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "in1",
> + "", "", "", "", "", "", "", "",
> + "", "", "", "", "", "", "", "";
> +};
> +
> +&fec {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_enet>;
> + phy-mode = "rmii";
> + phy-reset-gpios = <&gpio1 18 GPIO_ACTIVE_LOW>;
> + phy-handle = <&phy>;
> + clocks = <&clks IMX6QDL_CLK_ENET>, <&clks IMX6QDL_CLK_ENET>, <&rmii_clk>;
> + status = "okay";
> +
> + mdio {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + phy: ethernet-phy@1 {
> + compatible = "ethernet-phy-ieee802.3-c22";
> + reg = <1>;
> + };
> + };
> +};
> +
> +&pcie {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_pcie>;
> + reset-gpio = <&gpio1 20 0>;
GPIO_ACTIVE_XXX
> + status = "okay";
> +};
> +
> +&uart1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_uart1_dte>;
> + status = "okay";
> +
Drop the newline and move 'status' to end of list.
> + uart-has-rtscts;
> + fsl,dte-mode;
> + dcd-gpios = <&gpio3 23 GPIO_ACTIVE_LOW>;
> + dtr-gpios = <&gpio3 24 GPIO_ACTIVE_LOW>;
> + dsr-gpios = <&gpio3 25 GPIO_ACTIVE_LOW>;
> +};
> +
> +&uart2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_uart2_dte>;
> + status = "okay";
> +
Ditto
> + uart-has-rtscts;
> + fsl,dte-mode;
> + dcd-gpios = <&gpio2 1 GPIO_ACTIVE_LOW>;
> + dtr-gpios = <&gpio7 13 GPIO_ACTIVE_LOW>;
> + dsr-gpios = <&gpio6 16 GPIO_ACTIVE_LOW>;
> +};
> +
> +&uart3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_uart3_dce>;
> + status = "okay";
> +};
> +
> +&uart4 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_uart4_dce>;
> + status = "okay";
Move to end of list.
> + rts-gpios = <&gpio6 3 GPIO_ACTIVE_HIGH>;
> +};
> +
> +&uart5 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_uart5_dce>;
> + status = "okay";
Ditto
> + rts-gpios = <&gpio6 5 GPIO_ACTIVE_HIGH>;
> +};
> +
> +&usbh1 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_usbh1>;
> +
Drop the newline.
> + vbus-supply = <®_usb_h1_vbus>;
> + status = "okay";
> +};
> +
> +&usbotg {
> + status = "okay";
Move to end of list.
Shawn
> + dr_mode = "peripheral";
> +};
> +
> +&usdhc3 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_usdhc3>;
> + bus-width = <8>;
> + non-removable;
> + status = "okay";
> +};
> --
> 2.20.1
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-01-14 1:39 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-11 16:49 [PATCH 1/2] ARM: dts: Add devicetree for Eckelmann CI4x10 Uwe Kleine-König
2019-01-11 16:49 ` [PATCH 2/2] ARM: imx_v6_v7_defconfig: Enable SIOX bus Uwe Kleine-König
2019-01-14 1:40 ` Shawn Guo
2019-02-05 6:17 ` Uwe Kleine-König
2019-02-11 1:28 ` Shawn Guo
2019-01-14 1:39 ` Shawn Guo [this message]
2019-02-05 8:18 ` [PATCH 1/2] ARM: dts: Add devicetree for Eckelmann CI4x10 Uwe Kleine-König
2019-02-08 9:33 ` [PATCH v2] " Uwe Kleine-König
2019-02-08 10:48 ` Fabio Estevam
2019-02-11 10:50 ` Uwe Kleine-König
2019-02-11 1:38 ` [PATCH 1/2] " Shawn Guo
2019-02-11 10:59 ` Uwe Kleine-König
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=20190114013909.GE15371@dragon \
--to=shawnguo@kernel.org \
--cc=fabio.estevam@nxp.com \
--cc=g.schenk@eckelmann.de \
--cc=kernel@pengutronix.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=u.kleine-koenig@pengutronix.de \
/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).