On Tue, Dec 08, 2015 at 12:30:59AM -0200, Fabio Estevam wrote: > On Mon, Dec 7, 2015 at 11:37 PM, Peter Chen <peter.c...@freescale.com> wrote: > > Add dt-binding documentation for generic onboard USB HUB. > > > > Signed-off-by: Peter Chen <peter.c...@freescale.com> > > --- > > .../bindings/usb/generic-onboard-hub.txt | 28 > > ++++++++++++++++++++++ > > 1 file changed, 28 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/usb/generic-onboard-hub.txt > > > > diff --git a/Documentation/devicetree/bindings/usb/generic-onboard-hub.txt > > b/Documentation/devicetree/bindings/usb/generic-onboard-hub.txt > > new file mode 100644 > > index 0000000..ea92205 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/generic-onboard-hub.txt > > @@ -0,0 +1,28 @@ > > +Generic Onboard USB HUB > > + > > +Required properties: > > +- compatible: should be "generic-onboard-hub" > > + > > +Optional properties: > > +- clocks: the input clock for HUB. > > + > > +- clock-names: Should be "external_clk" > > + > > +- hub-reset-gpios: Should specify the GPIO for reset. > > + > > +- hub-reset-active-high: the active reset signal is high, if this property > > is > > + not set, the active reset signal is low. > > + > > +- hub-reset-duration-us: the duration for assert reset signal, the time > > unit > > + is microsecond. > > + > > +Example: > > + > > + usb_hub1 { > > + compatible = "generic-onboard-hub"; > > + clocks = <&clks IMX6QDL_CLK_CKO>; > > + clock-names = "external_clk"; > > + hub-reset-gpios = <&gpio7 12 0>; > > + hub-reset-active-high; > > I think you could drop the 'hub-reset-active-high' property and do > like this instead: > > hub-reset-gpios = <&gpio7 12 GPIO_ACTIVE_HIGH>; > > or > > hub-reset-gpios = <&gpio7 12 GPIO_ACTIVE_LOW>;
Yes, you are right, would you have a test to see if can work for udoo board? -- Best Regards, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html