Re: [PATCH 1/5] usb: usb251xb: Add USB2517/i hub support

From: Rob Herring
Date: Wed Sep 20 2017 - 17:01:26 EST


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.lancer@xxxxxxxxx>
> ---
> 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