(EXT) Re: [PATCH 2/5] ARM: dts: imx6ul: add TQ-Systems MBa6ULx device trees
Alexander Stein
alexander.stein at ew.tq-group.com
Mon Feb 21 07:45:58 PST 2022
Hello Shawn,
thanks for your review.
Am Sonntag, 13. Februar 2022, 04:12:12 CET schrieb Shawn Guo:
> On Tue, Feb 08, 2022 at 02:42:20PM +0100, Alexander Stein wrote:
> > Add device trees for the MBa6ULx mainboard with TQMa6ULx SoMs.
> >
> > Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>
> > ---
> >
> > arch/arm/boot/dts/Makefile | 2 +
> > arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi | 215 ++++++
> > arch/arm/boot/dts/imx6ul-tqma6ul1-mba6ulx.dts | 56 ++
> > arch/arm/boot/dts/imx6ul-tqma6ul1.dtsi | 37 ++
> > arch/arm/boot/dts/imx6ul-tqma6ul2-mba6ulx.dts | 15 +
> > arch/arm/boot/dts/imx6ul-tqma6ul2.dtsi | 71 ++
> > arch/arm/boot/dts/imx6ul-tqma6ulx-common.dtsi | 43 ++
> > arch/arm/boot/dts/mba6ulx.dtsi | 615 ++++++++++++++++++
> > 8 files changed, 1054 insertions(+)
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ul1-mba6ulx.dts
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ul1.dtsi
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ul2-mba6ulx.dts
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ul2.dtsi
> > create mode 100644 arch/arm/boot/dts/imx6ul-tqma6ulx-common.dtsi
> > create mode 100644 arch/arm/boot/dts/mba6ulx.dtsi
> >
> > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
> > index 6a920b1a83bb..ce4673e270a2 100644
> > --- a/arch/arm/boot/dts/Makefile
> > +++ b/arch/arm/boot/dts/Makefile
> > @@ -686,6 +686,8 @@ dtb-$(CONFIG_SOC_IMX6UL) += \
> >
> > imx6ul-kontron-n6310-s.dtb \
> > imx6ul-kontron-n6310-s-43.dtb \
> > imx6ul-liteboard.dtb \
> >
> > + imx6ul-tqma6ul1-mba6ulx.dtb \
> > + imx6ul-tqma6ul2-mba6ulx.dtb \
> >
> > imx6ul-opos6uldev.dtb \
> > imx6ul-pico-dwarf.dtb \
> > imx6ul-pico-hobbit.dtb \
> >
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
> > b/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi new file mode 100644
> > index 000000000000..8cc20f04501c
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ul-common.dtsi
> > @@ -0,0 +1,215 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +/*
> > + * Common for
> > + * - TQMa6ULx
> > + * - TQMa6ULxL
> > + * - TQMa6ULLx
> > + * - TQMa6ULLxL
> > + */
> > +
> > +/ {
> > + memory at 80000000 {
> > + device_type = "memory";
> > + reg = <0x80000000 0x10000000>;
> > + };
> > +};
> > +
> > +&i2c4 {
> > + clock-frequency = <100000>;
> > + pinctrl-names = "default", "gpio";
> > + pinctrl-0 = <&pinctrl_i2c4>;
> > + pinctrl-1 = <&pinctrl_i2c4_recovery>;
> > + scl-gpios = <&gpio1 20 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> > + sda-gpios = <&gpio1 21 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>;
> > + status = "okay";
> > +
> > + pfuze3000: pmic at 8 {
> > + compatible = "fsl,pfuze3000";
> > + reg = <0x08>;
> > +
> > + regulators {
> > + reg_sw1a: sw1a {
> > + regulator-min-microvolt =
<700000>;
> > + regulator-max-microvolt =
<3300000>;
> > + regulator-boot-on;
> > + regulator-ramp-delay = <6250>;
> > + /* not used */
> > + };
> > +
> > + reg_sw1b_core: sw1b {
> > + regulator-min-microvolt =
<700000>;
> > + regulator-max-microvolt =
<1475000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + regulator-ramp-delay = <6250>;
> > + };
> > +
> > + reg_sw2: sw2 {
> > + regulator-min-microvolt =
<2500000>;
> > + regulator-max-microvolt =
<3300000>;
> > + };
> > +
> > + reg_sw3_ddr: sw3 {
> > + regulator-min-microvolt =
<900000>;
> > + regulator-max-microvolt =
<1650000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + reg_swbst: swbst {
> > + regulator-min-microvolt =
<5000000>;
> > + regulator-max-microvolt =
<5150000>;
> > + /* not used */
> > + };
> > +
> > + reg_snvs_3v0: vsnvs {
> > + regulator-min-microvolt =
<1000000>;
> > + regulator-max-microvolt =
<3000000>;
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + reg_vrefddr: vrefddr {
> > + regulator-boot-on;
> > + regulator-always-on;
> > + };
> > +
> > + reg_vccsd: vccsd {
> > + regulator-min-microvolt =
<2850000>;
> > + regulator-max-microvolt =
<3300000>;
> > + };
> > +
> > + reg_v33_3v3: v33 {
> > + regulator-min-microvolt =
<2850000>;
> > + regulator-max-microvolt =
<3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + reg_vldo1_3v3: vldo1 {
> > + regulator-min-microvolt =
<1800000>;
> > + regulator-max-microvolt =
<3300000>;
> > + /* not used */
> > + };
> > +
> > + reg_vldo2: vldo2 {
> > + regulator-min-microvolt =
<800000>;
> > + regulator-max-microvolt =
<1550000>;
> > + /* not used */
> > + };
> > +
> > + reg_vldo3: vldo3 {
> > + regulator-min-microvolt =
<1800000>;
> > + regulator-max-microvolt =
<3300000>;
> > + /* not used */
> > + };
> > +
> > + reg_vldo4: vldo4 {
> > + regulator-min-microvolt =
<1800000>;
> > + regulator-max-microvolt =
<1800000>;
> > + regulator-always-on;
> > + };
> > + };
> > + };
> > +
> > + jc42_1a: eeprom-temperature-sensor at 1a {
> > + compatible = "nxp,se97", "jedec,jc-42.4-temp";
> > + reg = <0x1a>;
> > + status = "okay";
>
> We generally use okay status only to flip disabled devices.
Got it, will be removed.
> > + };
> > +
> > + m24c64_50: eeprom at 50 {
> > + compatible = "atmel,24c64";
> > + reg = <0x50>;
> > + pagesize = <32>;
> > + status = "okay";
> > + };
> > +
> > + m24c02_52: eeprom at 52 {
> > + compatible = "nxp,se97b", "atmel,24c02";
> > + reg = <0x52>;
> > + pagesize = <16>;
> > + read-only;
> > + status = "okay";
> > + };
> > +
> > + rtc0: rtc at 68 {
> > + compatible = "dallas,ds1339";
> > + reg = <0x68>;
> > + status = "okay";
> > + };
> > +};
> > +
> > +&gpio4 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_pmic>;
> > +
> > + /*
> > + * PMIC & temperature sensor IRQ
> > + * Both do currently not use IRQ
> > + * potentially dangerous if used on baseboard
> > + */
> > + pmic-int-hog {
> > + gpio-hog;
> > + gpios = <24 0>;
> > + input;
> > + };
> > +};
> > +
> > +&qspi {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_qspi>;
> > + status = "okay";
> > +
> > + flash0: flash at 0 {
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + compatible = "jedec,spi-nor";
> > + spi-max-frequency = <33000000>;
> > + spi-rx-bus-width = <4>;
> > + spi-tx-bus-width = <1>;
> > + reg = <0>;
> > + };
> > +};
> > +
> > +/* eMMC */
> > +&usdhc2 {
> > + pinctrl-names = "default", "state_100mhz" , "state_200mhz";
> > + pinctrl-0 = <&pinctrl_usdhc2>;
> > + pinctrl-1 = <&pinctrl_usdhc2_100mhz>;
> > + pinctrl-2 = <&pinctrl_usdhc2_200mhz>;
> > +
> > + bus-width = <8>;
> > + disable-wp;
> > + non-removable;
> > + no-sdio;
> > + no-sd;
> > + status = "okay";
> > +};
> > +
> > +&iomuxc {
> > + pinctrl_i2c4: i2c4grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART2_TX_DATA__I2C4_SCL 0x4001b8b0
> > + MX6UL_PAD_UART2_RX_DATA__I2C4_SDA 0x4001b8b0
> > + >;
> > + };
> > +
> > + pinctrl_i2c4_recovery: i2c4recoverygrp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART2_TX_DATA__GPIO1_IO20
0x4001b8b0
> > + MX6UL_PAD_UART2_RX_DATA__GPIO1_IO21
0x4001b8b0
> > + >;
> > + };
> > +
> > + pinctrl_pmic: pmic {
> > + fsl,pins = <
> > + /* PMIC irq */
> > + MX6UL_PAD_CSI_DATA03__GPIO4_IO24 0x1b099
> > + >;
> > + };
> > +};
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul1-mba6ulx.dts
> > b/arch/arm/boot/dts/imx6ul-tqma6ul1-mba6ulx.dts new file mode 100644
> > index 000000000000..58f847ae2631
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ul1-mba6ulx.dts
> > @@ -0,0 +1,56 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "imx6ul-tqma6ul1.dtsi"
> > +#include "mba6ulx.dtsi"
> > +
> > +/ {
> > + model = "TQ-Systems TQMa6UL1 SoM on MBa6ULx board";
> > + compatible = "tq,imx6ul-tqma6ul1-mba6ulx", "tq,imx6ul-tqma6ul1",
> > "fsl,imx6ul"; +};
> > +
> > +/*
> > + * Note: can2 and fec2 are enabled on mba6ulx level (for i.MX6ULG2 usage)
> > + * and need to be disabled here again
> > + */
> > +
>
> Drop the newline.
It's gone.
> > +&can2 {
> > + status = "disabled";
> > +};
> > +
> > +&fec1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_enet1>, <&pinctrl_enet1_mdc>;
> > + status = "okay";
>
> Have a newline between properties and child node.
Yep, inserted.
> > + mdio {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + ethphy0: ethernet-phy at 0 {
> > + compatible = "ethernet-phy-ieee802.3-c22";
> > + max-speed = <100>;
> > + reg = <0>;
> > + };
> > + };
> > +};
> > +
> > +&fec2 {
> > + status = "disabled";
> > + /delete-property/ phy-handle;
> > + /delete-node/ mdio;
>
> It's disabled anyway, so why bothering?
I have to delete 'mdio' to avoid duplicate label 'ethphy0' and delete 'phy-
handle' to remove the reference to 'ethphy1'.
I can remove the status assignment though.
> > +};
> > +
> > +&iomuxc {
> > + pinctrl_enet1_mdc: enet1mdcgrp {
> > + fsl,pins = <
> > + /* mdio */
> > + MX6UL_PAD_GPIO1_IO07__ENET1_MDC
0x1b0b0
> > + MX6UL_PAD_GPIO1_IO06__ENET1_MDIO 0x1b0b0
> > + >;
> > + };
> > +};
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul1.dtsi
> > b/arch/arm/boot/dts/imx6ul-tqma6ul1.dtsi new file mode 100644
> > index 000000000000..24192d012ef7
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ul1.dtsi
> > @@ -0,0 +1,37 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +#include "imx6ul-tqma6ul2.dtsi"
> > +
> > +/ {
> > + model = "TQ-Systems TQMa6UL1 SoM";
> > + compatible = "tq,imx6ul-tqma6ul1", "fsl,imx6ul";
> > +};
> > +
> > +/*
> > + * There are no module specific differences compared to TQMa6UL2,
> > + * only external interfaces differ
> > + */
> > +
> > +/*
> > + * Devices not available on i.MX6ULG1 and should not be enabled on
> > + * mainboard level (again)
> > + */
> > +&can2 {
> > + status = "disabled";
> > +};
> > +
> > +&csi {
> > + status = "disabled";
> > +};
> > +
> > +&fec2 {
> > + status = "disabled";
> > +};
> > +
> > +&lcdif {
> > + status = "disabled";
> > +};
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul2-mba6ulx.dts
> > b/arch/arm/boot/dts/imx6ul-tqma6ul2-mba6ulx.dts new file mode 100644
> > index 000000000000..0757df2b8f48
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ul2-mba6ulx.dts
> > @@ -0,0 +1,15 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +/dts-v1/;
> > +
> > +#include "imx6ul-tqma6ul2.dtsi"
> > +#include "mba6ulx.dtsi"
> > +
> > +/ {
> > + model = "TQ-Systems TQMa6ULx SoM on MBa6ULx board";
> > + compatible = "tq,imx6ul-tqma6ul2-mba6ulx", "tq,imx6ul-tqma6ul2",
> > "fsl,imx6ul"; +};
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ul2.dtsi
> > b/arch/arm/boot/dts/imx6ul-tqma6ul2.dtsi new file mode 100644
> > index 000000000000..e2e95dd92263
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ul2.dtsi
> > @@ -0,0 +1,71 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +#include "imx6ul.dtsi"
> > +#include "imx6ul-tqma6ul-common.dtsi"
> > +#include "imx6ul-tqma6ulx-common.dtsi"
> > +
> > +/ {
> > + model = "TQ-Systems TQMa6UL2 SoM";
> > + compatible = "tq,imx6ul-tqma6ul2", "fsl,imx6ul";
> > +};
> > +
> > +&usdhc2 {
> > + fsl,tuning-step = <6>;
> > +};
> > +
> > +&iomuxc {
> > + pinctrl_usdhc2: usdhc2grp {
> > + fsl,pins = <
> > + MX6UL_PAD_NAND_RE_B__USDHC2_CLK
0x00017051
> > + MX6UL_PAD_NAND_WE_B__USDHC2_CMD
0x00017051
> > + MX6UL_PAD_NAND_DATA00__USDHC2_DATA0
0x00017051
> > + MX6UL_PAD_NAND_DATA01__USDHC2_DATA1
0x00017051
> > + MX6UL_PAD_NAND_DATA02__USDHC2_DATA2
0x00017051
> > + MX6UL_PAD_NAND_DATA03__USDHC2_DATA3
0x00017051
> > + MX6UL_PAD_NAND_DATA04__USDHC2_DATA4
0x00017051
> > + MX6UL_PAD_NAND_DATA05__USDHC2_DATA5
0x00017051
> > + MX6UL_PAD_NAND_DATA06__USDHC2_DATA6
0x00017051
> > + MX6UL_PAD_NAND_DATA07__USDHC2_DATA7
0x00017051
> > + /* rst */
> > + MX6UL_PAD_NAND_ALE__GPIO4_IO10
0x0001b051
> > + >;
> > + };
> > +
> > + pinctrl_usdhc2_100mhz: usdhc2-100mhzgrp {
> > + fsl,pins = <
> > + MX6UL_PAD_NAND_RE_B__USDHC2_CLK
0x000170e1
> > + MX6UL_PAD_NAND_WE_B__USDHC2_CMD
0x000170f1
> > + MX6UL_PAD_NAND_DATA00__USDHC2_DATA0
0x000170f1
> > + MX6UL_PAD_NAND_DATA01__USDHC2_DATA1
0x000170f1
> > + MX6UL_PAD_NAND_DATA02__USDHC2_DATA2
0x000170f1
> > + MX6UL_PAD_NAND_DATA03__USDHC2_DATA3
0x000170f1
> > + MX6UL_PAD_NAND_DATA04__USDHC2_DATA4
0x000170f1
> > + MX6UL_PAD_NAND_DATA05__USDHC2_DATA5
0x000170f1
> > + MX6UL_PAD_NAND_DATA06__USDHC2_DATA6
0x000170f1
> > + MX6UL_PAD_NAND_DATA07__USDHC2_DATA7
0x000170f1
> > + /* rst */
> > + MX6UL_PAD_NAND_ALE__GPIO4_IO10
0x0001b051
> > + >;
> > + };
> > +
> > + pinctrl_usdhc2_200mhz: usdhc2-200mhzgrp {
> > + fsl,pins = <
> > + MX6UL_PAD_NAND_RE_B__USDHC2_CLK
0x000170f1
> > + MX6UL_PAD_NAND_WE_B__USDHC2_CMD
0x000170e1
> > + MX6UL_PAD_NAND_DATA00__USDHC2_DATA0
0x000170e1
> > + MX6UL_PAD_NAND_DATA01__USDHC2_DATA1
0x000170e1
> > + MX6UL_PAD_NAND_DATA02__USDHC2_DATA2
0x000170e1
> > + MX6UL_PAD_NAND_DATA03__USDHC2_DATA3
0x000170e1
> > + MX6UL_PAD_NAND_DATA04__USDHC2_DATA4
0x000170e1
> > + MX6UL_PAD_NAND_DATA05__USDHC2_DATA5
0x000170e1
> > + MX6UL_PAD_NAND_DATA06__USDHC2_DATA6
0x000170e1
> > + MX6UL_PAD_NAND_DATA07__USDHC2_DATA7
0x000170e1
> > + /* rst */
> > + MX6UL_PAD_NAND_ALE__GPIO4_IO10
0x0001b051
> > + >;
> > + };
> > +};
> > diff --git a/arch/arm/boot/dts/imx6ul-tqma6ulx-common.dtsi
> > b/arch/arm/boot/dts/imx6ul-tqma6ulx-common.dtsi new file mode 100644
> > index 000000000000..5afb9046c202
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/imx6ul-tqma6ulx-common.dtsi
> > @@ -0,0 +1,43 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +/*
> > + * Common for
> > + * - TQMa6ULx
> > + * - TQMa6ULLx
> > + */
> > +
> > +&m24c64_50 {
> > + vcc-supply = <®_sw2>;
> > +};
> > +
> > +&m24c02_52 {
> > + vcc-supply = <®_sw2>;
> > +};
> > +
> > +®_sw2 {
> > + regulator-boot-on;
> > + regulator-always-on;
> > +};
> > +
> > +/* eMMC */
> > +&usdhc2 {
> > + vmmc-supply = <®_sw2>;
> > + vqmmc-supply = <®_vldo4>;
> > +};
> > +
> > +&iomuxc {
> > + pinctrl_qspi: qspigrp {
> > + fsl,pins = <
> > + MX6UL_PAD_NAND_WP_B__QSPI_A_SCLK 0x70b9
> > + MX6UL_PAD_NAND_READY_B__QSPI_A_DATA00 0x70b9
> > + MX6UL_PAD_NAND_CE0_B__QSPI_A_DATA01 0x70b9
> > + MX6UL_PAD_NAND_CE1_B__QSPI_A_DATA02 0x70b9
> > + MX6UL_PAD_NAND_CLE__QSPI_A_DATA03 0x70b9
> > + MX6UL_PAD_NAND_DQS__QSPI_A_SS0_B 0x70a1
> > + >;
> > + };
> > +};
> > diff --git a/arch/arm/boot/dts/mba6ulx.dtsi
> > b/arch/arm/boot/dts/mba6ulx.dtsi new file mode 100644
> > index 000000000000..3a35b1efdc1f
> > --- /dev/null
> > +++ b/arch/arm/boot/dts/mba6ulx.dtsi
> > @@ -0,0 +1,615 @@
> > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT)
> > +/*
> > + * Copyright 2018-2022 TQ-Systems GmbH
> > + * Author: Markus Niebel <Markus.Niebel at tq-group.com>
> > + */
> > +
> > +/ {
> > + model = "TQ-Systems MBA6ULx Baseboard";
> > +
> > + aliases {
> > + mmc0 = &usdhc2;
> > + mmc1 = &usdhc1;
> > + rtc0 = &rtc0;
> > + rtc1 = &snvs_rtc;
> > + };
> > +
> > + backlight: backlight {
> > + compatible = "pwm-backlight";
> > + power-supply = <®_mba6ul_3v3>;
> > + enable-gpios = <&expander_out0 4 GPIO_ACTIVE_HIGH>;
> > + status = "disabled";
> > + };
> > +
> > + beeper: beeper {
> > + compatible = "gpio-beeper";
> > + gpios = <&expander_out1 6 GPIO_ACTIVE_HIGH>;
> > + };
> > +
> > + chosen {
> > + stdout-path = &uart1;
> > + };
>
> Move 'chosen' node up.
Ok. The binding does not say anything regarding the order with respect to the
alias node. I've seen examples of both order. So I went with alias 1st, chosen
2nd.
> > +
> > + gpio_buttons: gpio-keys {
> > + compatible = "gpio-keys";
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_buttons>;
> > +
> > + button at 0 {
>
> No 'reg' property, no unit-address. So I guess you didn't run
> dtbs_check, did you?
Ehm, I though I did. I checked again and there are no warnings.
Anyway you are right and the will be removed.
> > + label = "s14";
> > + linux,code = <KEY_1>;
> > + gpios = <&expander_in0 0 GPIO_ACTIVE_LOW>;
> > + };
> > +
> > + button at 1 {
> > + label = "s6";
> > + linux,code = <KEY_2>;
> > + gpios = <&expander_in0 1 GPIO_ACTIVE_LOW>;
> > + };
> > +
> > + button at 2 {
> > + label = "s7";
> > + linux,code = <KEY_3>;
> > + gpios = <&expander_in0 2 GPIO_ACTIVE_LOW>;
> > + };
> > +
> > + button at 3 {
> > + label = "POWER";
> > + linux,code = <KEY_POWER>;
> > + gpios = <&gpio1 3 GPIO_ACTIVE_LOW>;
> > + gpio-key,wakeup;
> > + };
> > + };
> > +
> > + gpio-leds {
> > + compatible = "gpio-leds";
> > + status = "okay";
> > +
> > + led1 {
> > + label = "led1";
> > + gpios = <&expander_out1 4 GPIO_ACTIVE_HIGH>;
> > + linux,default-trigger = "default-on";
> > + };
> > +
> > + led2 {
> > + label = "led2";
> > + gpios = <&expander_out1 5 GPIO_ACTIVE_HIGH>;
> > + linux,default-trigger = "heartbeat";
> > + };
> > + };
> > +
> > + reg_lcd_pwr: regulator-lcd-pwr {
> > + compatible = "regulator-fixed";
> > + regulator-name = "lcd-pwr";
> > + gpio = <&expander_out0 1 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + status = "disabled";
> > + };
> > +
> > + reg_mba6ul_3v3: regulator-mba6ul-3v3 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "supply-mba6ul-3v3";
> > + regulator-min-microvolt = <3300000>;
> > + regulator-max-microvolt = <3300000>;
> > + regulator-always-on;
> > + };
> > +
> > + reg_mba6ul_5v0: regulator-mba6ul-5v0 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "supply-mba6ul-5v0";
> > + regulator-min-microvolt = <5000000>;
> > + regulator-max-microvolt = <5000000>;
> > + regulator-always-on;
> > + };
> > +
> > + reg_mpcie: regulator-mpcie-3v3 {
> > + compatible = "regulator-fixed";
> > + regulator-name = "mpcie-3v3";
> > + regulator-min-microvolt = <3300000>;
> > + regulator-max-microvolt = <3300000>;
> > + gpio = <&expander_out0 2 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + regulator-always-on;
> > + startup-delay-us = <500000>;
> > + vin-supply = <®_mba6ul_3v3>;
> > + };
> > +
> > + reg_otg2vbus_5v0: regulator-otg2-vbus-5v0 {
> > + compatible = "regulator-fixed";
> > + gpio = <&expander_out1 0 GPIO_ACTIVE_HIGH>;
> > + enable-active-high;
> > + regulator-name = "otg2-vbus-supply-5v0";
> > + regulator-min-microvolt = <5000000>;
> > + regulator-max-microvolt = <5000000>;
> > + vin-supply = <®_mpcie>;
> > + };
> > +
> > + reserved-memory {
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> > + ranges;
> > +
> > + linux,cma {
> > + compatible = "shared-dma-pool";
> > + reusable;
> > + size = <0x6000000>;
> > + linux,cma-default;
> > + };
> > + };
> > +
> > + sound {
> > + compatible = "fsl,imx-audio-tlv320aic32x4";
> > + model = "imx-audio-tlv320aic32x4";
> > + ssi-controller = <&sai1>;
> > + audio-codec = <&tlv320aic32x4>;
> > + audio-asrc = <&asrc>;
> > + };
> > +};
> > +
> > +&can1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_flexcan1>;
> > + xceiver-supply = <®_mba6ul_3v3>;
> > + status = "okay";
> > +};
> > +
> > +&can2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_flexcan2>;
> > + xceiver-supply = <®_mba6ul_3v3>;
> > + status = "okay";
> > +};
> > +
> > +&clks {
> > + assigned-clocks = <&clks IMX6UL_CLK_PLL4_AUDIO_DIV>;
> > + assigned-clock-rates = <768000000>;
> > +};
> > +
> > +&ecspi2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_ecspi2>;
> > + num-cs = <1>;
> > + status = "okay";
> > +
> > + spidev2_0: spidev2-0 at 0 {
> > + compatible = "tq,testdev";
>
> Not supported.
Sorry, that's from internal tree.
> > + spi-max-frequency = <5000000>;
> > + reg = <0>;
> > + status = "okay";
> > + };
> > +};
> > +
> > +&fec1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_enet1>;
> > + phy-mode = "rmii";
> > + phy-handle = <ðphy0>;
> > + phy-supply = <®_mba6ul_3v3>;
> > + phy-reset-gpios = <&expander_out1 1 GPIO_ACTIVE_LOW>;
> > + phy-reset-duration = <25>;
> > + phy-reset-post-delay = <1>;
>
> Handle them with ethernet-phy.
The reset doesn't work if the PHY is connected to the non-local MDIO bus.
Probably due to the same reason PHY probing does not work.
> > + status = "okay";
> > +};
> > +
> > +&fec2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_enet2>, <&pinctrl_enet2_mdc>;
> > + phy-mode = "rmii";
> > + phy-handle = <ðphy1>;
> > + phy-supply = <®_mba6ul_3v3>;
> > + phy-reset-gpios = <&expander_out1 2 GPIO_ACTIVE_LOW>;
> > + phy-reset-duration = <25>;
> > + phy-reset-post-delay = <1>;
> > + status = "okay";
> > +
> > + mdio {
> > + #address-cells = <1>;
> > + #size-cells = <0>;
> > +
> > + ethphy0: ethernet-phy at 0 {
> > + compatible = "ethernet-phy-ieee802.3-c22";
> > + clocks = <&clks IMX6UL_CLK_ENET_REF>;
> > + reg = <0>;
> > + max-speed = <100>;
> > + };
> > +
> > + ethphy1: ethernet-phy at 1 {
> > + compatible = "ethernet-phy-ieee802.3-c22";
> > + clocks = <&clks IMX6UL_CLK_ENET2_REF_125M>;
> > + reg = <1>;
> > + max-speed = <100>;
> > + };
> > + };
> > +};
> > +
> > +&flash0 {
> > + partition at 0 {
> > + label = "U-Boot";
> > + reg = <0x0 0xd0000>;
> > + };
> > +
> > + partition at d0000 {
> > + label = "ENV1";
> > + reg = <0xd0000 0x10000>;
> > + };
> > +
> > + partition at e0000 {
> > + label = "ENV2";
> > + reg = <0xe0000 0x10000>;
> > + };
> > +
> > + partition at f0000 {
> > + label = "DTB";
> > + reg = <0xf0000 0x10000>;
> > + };
> > +
> > + partition at 100000 {
> > + label = "Linux";
> > + reg = <0x100000 0x700000>;
> > + };
> > +
> > + partition at 800000 {
> > + label = "RootFS";
> > + reg = <0x800000 0x3800000>;
> > + };
> > +};
> > +
> > +&i2c4 {
> > + tlv320aic32x4: codec at 18 {
>
> audio-codec for the node name.
Thanks, got changed.
Thanks for your review points.
Regards,
Alexander
> > + compatible = "ti,tlv320aic32x4";
> > + reg = <0x18>;
> > + clocks = <&clks IMX6UL_CLK_SAI1>;
> > + clock-names = "mclk";
> > + ldoin-supply = <®_mba6ul_3v3>;
> > + iov-supply = <®_mba6ul_3v3>;
> > + };
> > +
> > + jc42: temperature-sensor at 19 {
> > + compatible = "nxp,se97", "jedec,jc-42.4-temp";
> > + reg = <0x19>;
> > + status = "okay";
> > + };
> > +
> > + expander_out0: gpio-expander at 20 {
> > + compatible = "nxp,pca9554";
> > + reg = <0x20>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
> > + status = "okay";
> > + };
> > +
> > + expander_in0: gpio-expander at 21 {
> > + compatible = "nxp,pca9554";
> > + reg = <0x21>;
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_expander_in0>;
> > + interrupt-parent = <&gpio4>;
> > + interrupts = <23 IRQ_TYPE_LEVEL_LOW>;
> > + interrupt-controller;
> > + #interrupt-cells = <2>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
> > + status = "okay";
> > +
> > + enet1_int-hog {
> > + gpio-hog;
> > + gpios = <6 0>;
> > + input;
> > + };
> > +
> > + enet2_int-hog {
> > + gpio-hog;
> > + gpios = <7 0>;
> > + input;
> > + };
> > + };
> > +
> > + expander_out1: gpio-expander at 22 {
> > + compatible = "nxp,pca9554";
> > + reg = <0x22>;
> > + gpio-controller;
> > + #gpio-cells = <2>;
> > + status = "okay";
> > + };
> > +
> > + analog_touch: touchscreen at 41 {
> > + compatible = "st,stmpe811";
> > + reg = <0x41>;
> > + interrupts = <21 IRQ_TYPE_EDGE_FALLING>;
> > + interrupt-parent = <&gpio4>;
> > + interrupt-controller;
> > + status = "disabled";
> > +
> > + stmpe_touchscreen {
> > + compatible = "st,stmpe-ts";
> > + st,adc-freq = <1>; /* 3.25 MHz ADC clock
speed */
> > + st,ave-ctrl = <3>; /* 8 sample average
control */
> > + st,fraction-z = <7>; /* 7 length
fractional part in z */
> > + /*
> > + * 50 mA typical 80 mA max touchscreen
drivers
> > + * current limit value
> > + */
> > + st,i-drive = <1>;
> > + st,mod-12b = <1>; /* 12-bit ADC */
> > + st,ref-sel = <0>; /* internal ADC
reference */
> > + st,sample-time = <4>; /* ADC converstion
time: 80 clocks */
> > + st,settling = <3>; /* 1 ms panel driver
settling time */
> > + st,touch-det-delay = <5>; /* 5 ms touch
detect interrupt delay */
> > + };
> > + };
> > +
> > + /* NXP SE97BTP with temperature sensor + eeprom */
> > + se97b: eeprom at 51 {
> > + compatible = "nxp,se97b", "atmel,24c02";
> > + reg = <0x51>;
> > + pagesize = <16>;
> > + status = "okay";
> > + };
> > +};
> > +
> > +&pwm2 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_pwm2>;
> > + status = "okay";
> > +};
> > +
> > +&sai1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_sai1>;
> > + assigned-clocks = <&clks IMX6UL_CLK_SAI1_SEL>,
> > + <&clks IMX6UL_CLK_SAI1>;
> > + assigned-clock-parents = <&clks IMX6UL_CLK_PLL4_AUDIO_DIV>;
> > + assigned-clock-rates = <0>, <24000000>;
> > + fsl,sai-mclk-direction-output;
> > + status = "okay";
> > +};
> > +
> > +&uart1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_uart1>;
> > + status = "okay";
> > +};
> > +
> > +&uart3 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_uart3>;
> > + status = "okay";
> > +};
> > +
> > +&uart6 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_uart6>;
> > + /* for DTE mode, add below change */
> > + /* fsl,dte-mode; */
> > + /* pinctrl-0 = <&pinctrl_uart6dte>; */
> > + uart-has-rtscts;
> > + linux,rs485-enabled-at-boot-time;
> > + rs485-rts-active-low;
> > + rs485-rx-during-tx;
> > + status = "okay";
> > +};
> > +
> > +/* otg-port */
> > +&usbotg1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usb_otg1>;
> > + power-active-high;
> > + over-current-active-low;
> > + /* we implement only dual role but not a fully featured OTG */
> > + hnp-disable;
> > + srp-disable;
> > + adp-disable;
> > + dr_mode = "otg";
> > + status = "okay";
> > +};
> > +
> > +/* 7-port usb hub */
> > +/* id, pwr, oc pins not connected */
> > +&usbotg2 {
> > + disable-over-current;
> > + vbus-supply = <®_otg2vbus_5v0>;
> > + dr_mode = "host";
> > + status = "okay";
> > +};
> > +
> > +&usdhc1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_usdhc1>;
> > + cd-gpios = <&gpio1 19 GPIO_ACTIVE_LOW>;
> > + wp-gpios = <&gpio1 18 GPIO_ACTIVE_HIGH>;
> > + bus-width = <4>;
> > + vmmc-supply = <®_mba6ul_3v3>;
> > + vqmmc-supply = <®_vccsd>;
> > + no-1-8-v;
> > + no-mmc;
> > + no-sdio;
> > + status = "okay";
> > +};
> > +
> > +&wdog1 {
> > + pinctrl-names = "default";
> > + pinctrl-0 = <&pinctrl_wdog1>;
> > + fsl,ext-reset-output;
> > + status = "okay";
> > +};
> > +
> > +&iomuxc {
> > + pinctrl_buttons: buttonsgrp {
> > + fsl,pins = <
> > + MX6UL_PAD_GPIO1_IO03__GPIO1_IO03 0x100b0
> > + >;
> > + };
> > +
> > + pinctrl_ecspi2: ecspi2grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART4_TX_DATA__ECSPI2_SCLK
0x1b020
> > + MX6UL_PAD_UART5_RX_DATA__ECSPI2_MISO
0x1b020
> > + MX6UL_PAD_UART5_TX_DATA__ECSPI2_MOSI
0x1b020
> > + MX6UL_PAD_UART4_RX_DATA__ECSPI2_SS0
0x1b020
> > + >;
> > + };
> > +
> > + pinctrl_enet1: enet1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_ENET1_RX_EN__ENET1_RX_EN
0x1b0b0
> > + MX6UL_PAD_ENET1_RX_ER__ENET1_RX_ER
0x1b0b0
> > + MX6UL_PAD_ENET1_RX_DATA0__ENET1_RDATA00
0x1b0b0
> > + MX6UL_PAD_ENET1_RX_DATA1__ENET1_RDATA01
0x1b0b0
> > + MX6UL_PAD_ENET1_TX_DATA0__ENET1_TDATA00
0x1b0b0
> > + MX6UL_PAD_ENET1_TX_DATA1__ENET1_TDATA01
0x1b0b0
> > + MX6UL_PAD_ENET1_TX_EN__ENET1_TX_EN
0x1b0b0
> > + MX6UL_PAD_ENET1_TX_CLK__ENET1_REF_CLK1
0x4001b0a8
> > + >;
> > + };
> > +
> > + pinctrl_enet2: enet2grp {
> > + fsl,pins = <
> > + MX6UL_PAD_ENET2_RX_EN__ENET2_RX_EN
0x1b0b0
> > + MX6UL_PAD_ENET2_RX_ER__ENET2_RX_ER
0x1b0b0
> > + MX6UL_PAD_ENET2_RX_DATA0__ENET2_RDATA00
0x1b0b0
> > + MX6UL_PAD_ENET2_RX_DATA1__ENET2_RDATA01
0x1b0b0
> > + MX6UL_PAD_ENET2_TX_DATA0__ENET2_TDATA00
0x1b0a0
> > + MX6UL_PAD_ENET2_TX_DATA1__ENET2_TDATA01
0x1b0a0
> > + MX6UL_PAD_ENET2_TX_EN__ENET2_TX_EN
0x1b0b0
> > + MX6UL_PAD_ENET2_TX_CLK__ENET2_REF_CLK2
0x4001b0a8
> > + >;
> > + };
> > +
> > + pinctrl_enet2_mdc: enet2mdcgrp {
> > + fsl,pins = <
> > + /* mdio */
> > + MX6UL_PAD_GPIO1_IO07__ENET2_MDC
0x1b0b0
> > + MX6UL_PAD_GPIO1_IO06__ENET2_MDIO 0x1b0b0
> > + >;
> > + };
> > +
> > + pinctrl_expander_in0: expanderin0grp {
> > + fsl,pins = <
> > + MX6UL_PAD_CSI_DATA02__GPIO4_IO23 0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_flexcan1: flexcan1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART3_RTS_B__FLEXCAN1_RX
0x1b020
> > + MX6UL_PAD_UART3_CTS_B__FLEXCAN1_TX
0x1b020
> > + >;
> > + };
> > +
> > + pinctrl_flexcan2: flexcan2grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART2_RTS_B__FLEXCAN2_RX
0x1b020
> > + MX6UL_PAD_UART2_CTS_B__FLEXCAN2_TX
0x1b020
> > + >;
> > + };
> > +
> > + pinctrl_pwm2: pwm2grp {
> > + fsl,pins = <
> > + /* 100 k PD, DSE 120 OHM, SPPEED LO */
> > + MX6UL_PAD_GPIO1_IO09__PWM2_OUT
0x00003050
> > + >;
> > + };
> > +
> > + pinctrl_sai1: sai1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_CSI_DATA05__SAI1_TX_BCLK
0x1b0b1
> > + MX6UL_PAD_CSI_DATA04__SAI1_TX_SYNC
0x1b0b1
> > + MX6UL_PAD_CSI_DATA07__SAI1_TX_DATA
0x1f0b8
> > + MX6UL_PAD_CSI_DATA06__SAI1_RX_DATA
0x110b0
> > + MX6UL_PAD_CSI_DATA01__SAI1_MCLK
0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_uart1: uart1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART1_TX_DATA__UART1_DCE_TX
0x1b0b1
> > + MX6UL_PAD_UART1_RX_DATA__UART1_DCE_RX
0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_uart3: uart3grp {
> > + fsl,pins = <
> > + MX6UL_PAD_UART3_TX_DATA__UART3_DCE_TX
0x1b0b1
> > + MX6UL_PAD_UART3_RX_DATA__UART3_DCE_RX
0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_uart6: uart6grp {
> > + fsl,pins = <
> > + MX6UL_PAD_CSI_MCLK__UART6_DCE_TX 0x1b0b1
> > + MX6UL_PAD_CSI_PIXCLK__UART6_DCE_RX
0x1b0b1
> > + MX6UL_PAD_CSI_VSYNC__UART6_DCE_RTS
0x1b0b1
> > + MX6UL_PAD_CSI_HSYNC__UART6_DCE_CTS
0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_uart6dte: uart6dte {
> > + fsl,pins = <
> > + MX6UL_PAD_CSI_PIXCLK__UART6_DTE_TX
0x1b0b1
> > + MX6UL_PAD_CSI_MCLK__UART6_DTE_RX 0x1b0b1
> > + MX6UL_PAD_CSI_HSYNC__UART6_DTE_RTS
0x1b0b1
> > + MX6UL_PAD_CSI_VSYNC__UART6_DTE_CTS
0x1b0b1
> > + >;
> > + };
> > +
> > + pinctrl_usb_otg1: usbotg1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_GPIO1_IO00__ANATOP_OTG1_ID
0x00017059
> > + MX6UL_PAD_GPIO1_IO01__USB_OTG1_OC 0x0001b0b0
> > + MX6UL_PAD_GPIO1_IO04__USB_OTG1_PWR
0x0001b099
> > + >;
> > + };
> > +
> > + pinctrl_usdhc1: usdhc1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_SD1_CLK__USDHC1_CLK
0x00017069
> > + MX6UL_PAD_SD1_CMD__USDHC1_CMD
0x00017059
> > + MX6UL_PAD_SD1_DATA0__USDHC1_DATA0 0x00017059
> > + MX6UL_PAD_SD1_DATA1__USDHC1_DATA1 0x00017059
> > + MX6UL_PAD_SD1_DATA2__USDHC1_DATA2 0x00017059
> > + MX6UL_PAD_SD1_DATA3__USDHC1_DATA3 0x00017059
> > + /* WP */
> > + MX6UL_PAD_UART1_CTS_B__GPIO1_IO18 0x0001b099
> > + /* CD */
> > + MX6UL_PAD_UART1_RTS_B__GPIO1_IO19 0x0001b099
> > + >;
> > + };
> > +
> > + pinctrl_usdhc1_100mhz: usdhc1grp100mhz {
> > + fsl,pins = <
> > + MX6UL_PAD_SD1_CLK__USDHC1_CLK
0x00017069
> > + MX6UL_PAD_SD1_CMD__USDHC1_CMD
0x000170b9
> > + MX6UL_PAD_SD1_DATA0__USDHC1_DATA0 0x000170b9
> > + MX6UL_PAD_SD1_DATA1__USDHC1_DATA1 0x000170b9
> > + MX6UL_PAD_SD1_DATA2__USDHC1_DATA2 0x000170b9
> > + MX6UL_PAD_SD1_DATA3__USDHC1_DATA3 0x000170b9
> > + /* WP */
> > + MX6UL_PAD_UART1_CTS_B__GPIO1_IO18 0x0001b099
> > + /* CD */
> > + MX6UL_PAD_UART1_RTS_B__GPIO1_IO19 0x0001b099
> > + >;
> > + };
> > +
> > + pinctrl_usdhc1_200mhz: usdhc1grp200mhz {
> > + fsl,pins = <
> > + MX6UL_PAD_SD1_CLK__USDHC1_CLK
0x00017069
> > + MX6UL_PAD_SD1_CMD__USDHC1_CMD
0x000170f9
> > + MX6UL_PAD_SD1_DATA0__USDHC1_DATA0 0x000170f9
> > + MX6UL_PAD_SD1_DATA1__USDHC1_DATA1 0x000170f9
> > + MX6UL_PAD_SD1_DATA2__USDHC1_DATA2 0x000170f9
> > + MX6UL_PAD_SD1_DATA3__USDHC1_DATA3 0x000170f9
> > + /* WP */
> > + MX6UL_PAD_UART1_CTS_B__GPIO1_IO18 0x0001b099
> > + /* CD */
> > + MX6UL_PAD_UART1_RTS_B__GPIO1_IO19 0x0001b099
> > + >;
> > + };
> > +
> > + pinctrl_wdog1: wdog1grp {
> > + fsl,pins = <
> > + MX6UL_PAD_GPIO1_IO08__WDOG1_WDOG_B
0x0001b099
> > + >;
> > + };
> > +};
More information about the linux-arm-kernel
mailing list