On Wed, Jan 11, 2017 at 08:02:29PM -0200, Sergio Prado wrote: > Adds the device tree bindings description for Samsung S3C2410 and > compatible USB device controller. > > Signed-off-by: Sergio Prado <sergio.prado@xxxxxxxxxxxxxx> > --- > .../devicetree/bindings/usb/s3c2410-usb.txt | 28 ++++++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/s3c2410-usb.txt b/Documentation/devicetree/bindings/usb/s3c2410-usb.txt > index e45b38ce2986..28353eea31fd 100644 > --- a/Documentation/devicetree/bindings/usb/s3c2410-usb.txt > +++ b/Documentation/devicetree/bindings/usb/s3c2410-usb.txt > @@ -20,3 +20,31 @@ usb0: ohci@49000000 { > clocks = <&clocks UCLK>, <&clocks HCLK_USBH>; > clock-names = "usb-bus-host", "usb-host"; > }; > + > +Samsung S3C2410 and compatible USB device controller > + > +Required properties: > + - compatible: Should be one of the following > + "samsung,s3c2410-udc" > + "samsung,s3c2440-udc" > + - reg: address and length of the controller memory mapped region > + - interrupts: interrupt number for the USB device controller > + - clocks: Should reference the bus and host clocks > + - clock-names: Should contain two strings > + "uclk" for the USB bus clock > + "usb-device" for the USB device clock Perhaps just "bus" and "device". > + > +Optional properties: > + - samsung,vbus-gpio: specifies a gpio that allows to detect whether > + vbus is present - USB is connected (active high, input). > + - samsung,pullup-gpio: If present, specifies a gpio to control the > + USB D+ pullup (active high, output). "-gpios", not "-gpio" is preferred. These should be common property names if we're going to have them. The problem with just "vbus-gpios" is does that mean an enable control or status as you have. I guess in the former case, that should always be modeled as a regulator. Also, these should all be part of a connector node as these controls are more related to the USB connector than the controller. And I don't mean extcon here because those bindings are a mess. Rob -- 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