Hi Peter, On 03.03.2016 11:01, Peter Chen wrote: > From: Peter Chen <peter.chen@xxxxxxxxxxxxx> > > The current dts describes USB HUB's property at USB controller's > entry, it is improper. The USB HUB should be the child node > under USB controller. > > Signed-off-by: Peter Chen <peter.chen@xxxxxxxxxxxxx> Thanks for patches! VID:PID for hub on UDOO board is 0424:2514. I've tested these patches on this board and was able to make USB work again with following DT changes: diff --git a/arch/arm/boot/dts/imx6qdl-udoo.dtsi b/arch/arm/boot/dts/imx6qdl-udoo.dtsi index d3e54e4..4956de7 100644 --- a/arch/arm/boot/dts/imx6qdl-udoo.dtsi +++ b/arch/arm/boot/dts/imx6qdl-udoo.dtsi @@ -9,6 +9,8 @@ * */ +#include <dt-bindings/gpio/gpio.h> + / { chosen { stdout-path = &uart2; @@ -18,21 +27,11 @@ reg = <0x10000000 0x40000000>; }; - regulators { - compatible = "simple-bus"; - #address-cells = <1>; - #size-cells = <0>; - - reg_usb_h1_vbus: regulator@0 { - compatible = "regulator-fixed"; - reg = <0>; - regulator-name = "usb_h1_vbus"; - regulator-min-microvolt = <5000000>; - regulator-max-microvolt = <5000000>; - enable-active-high; - startup-delay-us = <2>; /* USB2415 requires a POR of 1 us minimum */ - gpio = <&gpio7 12 0>; - }; + usb2415_pwrseq: usb2415_pwrseq { + compatible = "usb-pwrseq"; + clocks = <&clks IMX6QDL_CLK_CKO>; + reset-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>; + reset-duration-us = <3000>; }; sound { @@ -163,9 +218,16 @@ &usbh1 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_usbh>; - vbus-supply = <®_usb_h1_vbus>; - clocks = <&clks 201>; status = "okay"; + + #address-cells = <1>; + #size-cells = <0>; + + hub: usb2415@01 { + compatible = "usb424,2514"; + reg = <1>; + usb-pwrseq = <&usb2415_pwrseq>; + }; }; &usdhc3 { Maciej -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html