On Sat, Sep 16, 2017 at 02:31:09AM +0300, Serge Semin wrote: > USB2517i hubs are very like USB251xb devices series. They have almost > the same configuration registers space except number of ports, led > configurations and lack of battery settings. All these peculiarities > are reflected in this patch. > > Signed-off-by: Serge Semin <fancer.lan...@gmail.com> > --- > Documentation/devicetree/bindings/usb/usb251xb.txt | 4 +-
Though Greg wants the code split, I want the binding as one change. H/w doesn't gain features one by one. It's preferred to split bindings to a separate patch. > drivers/usb/misc/usb251xb.c | 84 > +++++++++++++++++++--- > 2 files changed, 78 insertions(+), 10 deletions(-) > > diff --git a/Documentation/devicetree/bindings/usb/usb251xb.txt > b/Documentation/devicetree/bindings/usb/usb251xb.txt > index 3957d4eda..3d84626d3 100644 > --- a/Documentation/devicetree/bindings/usb/usb251xb.txt > +++ b/Documentation/devicetree/bindings/usb/usb251xb.txt > @@ -6,7 +6,8 @@ Hi-Speed Controller. > Required properties : > - compatible : Should be "microchip,usb251xb" or one of the specific types: > "microchip,usb2512b", "microchip,usb2512bi", "microchip,usb2513b", > - "microchip,usb2513bi", "microchip,usb2514b", "microchip,usb2514bi" > + "microchip,usb2513bi", "microchip,usb2514b", "microchip,usb2514bi", > + "microchip,usb2517", "microchip,usb2517i" > - reset-gpios : Should specify the gpio for hub reset > - reg : I2C address on the selected bus (default is <0x2C>) > > @@ -36,6 +37,7 @@ Optional properties : > an invalid value is given, the default is used instead. > - compound-device : indicate the hub is part of a compound device > - port-mapping-mode : enable port mapping mode > + - speed-led-mode : led speed indiation mode selection (usb2517 only) This is a boolean or has values? What are valid values? This needs a vendor prefix. Somehow the other properties got in without. > - string-support : enable string descriptor support (required for > manufacturer, > product and serial string configuration) > - non-removable-ports : Should specify the ports which have a non-removable
