lkml.org 
[lkml]   [2017]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/5] usb: usb251xb: Add USB2517/i hub support
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@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

\
 
 \ /
  Last update: 2017-09-21 19:24    [W:0.353 / U:0.452 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site