On Fri, Dec 1, 2017 at 12:13 PM, Fabio Estevam [off-list ref] wrote:

On Thu, Nov 30, 2017 at 6:14 PM,  [off-list ref] wrote:

quoted

From: Tyler Baker <redacted>

Add device tree nodes for the USB hub, and USB OTG. i2c2 on this
platform supports low state retention power state so lets use it.

Signed-off-by: Tyler Baker <redacted>
---
 arch/arm/boot/dts/imx7d-cl-som-imx7.dts | 36 +++++++++++++++++++++++++--------
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/arch/arm/boot/dts/imx7d-cl-som-imx7.dts b/arch/arm/boot/dts/imx7d-cl-som-imx7.dts
index ae45af1..a9f690b 100644
--- a/arch/arm/boot/dts/imx7d-cl-som-imx7.dts
+++ b/arch/arm/boot/dts/imx7d-cl-som-imx7.dts

@@ -30,6 +30,16 @@
                gpio = <&gpio1 5 GPIO_ACTIVE_HIGH>;
                enable-active-high;
        };
+
+       reg_usbh_nreset: regulator at 4 {

You should not use @4. Please make sure that regulators are not used
inside simple-bus.

Good catch, will fix.

quoted

-
        pinctrl_uart1: uart1grp {
                fsl,pins = <
                        MX7D_PAD_UART1_TX_DATA__UART1_DCE_TX    0x79

@@ -284,4 +297,11 @@
                        MX7D_PAD_LPSR_GPIO1_IO05__GPIO1_IO5     0x14 /* OTG PWREN */
                >;
        };
-};

\ No newline at end of file

This text here looks strange :-)

Indeed, I'll scrub it.