On Sat, Oct 10, 2020 at 04:43:12PM +0800, Chunfeng Yun wrote: > Add some optional properties which are needed for hard wired devices > > Signed-off-by: Chunfeng Yun <chunfeng.yun@xxxxxxxxxxxx> > --- > v2 changes suggested by Rob: > 1. modify pattern to support any USB class > 2. refer to usb-device.yaml instead of usb-device.txt > --- > .../devicetree/bindings/usb/usb-hcd.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) You can fold this into the first patch. While not explicit before, it was implied. Rob > > diff --git a/Documentation/devicetree/bindings/usb/usb-hcd.yaml b/Documentation/devicetree/bindings/usb/usb-hcd.yaml > index 7263b7f2b510..42b295afdf32 100644 > --- a/Documentation/devicetree/bindings/usb/usb-hcd.yaml > +++ b/Documentation/devicetree/bindings/usb/usb-hcd.yaml > @@ -22,9 +22,28 @@ properties: > description: > Name specifier for the USB PHY > > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 0 > + > +patternProperties: > + "^[a-f]+@[0-9a-f]+$": Just define the unit-address here: "@[0-9a-f]+$" > + type: object > + $ref: /usb/usb-device.yaml > + description: The hard wired USB devices Need to also define 'reg' and 'compatible' here. > + > examples: > - | > usb { > phys = <&usb2_phy1>, <&usb3_phy1>; > phy-names = "usb"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + hub@1 { > + compatible = "usb5e3,610"; > + reg = <1>; > + }; > }; > -- > 2.18.0