On Tue, Oct 17, 2023 at 06:53:56PM +0200, Krzysztof Kozlowski wrote: > On 17/10/2023 18:01, Jisheng Zhang wrote: > > The "reset-gpios" is optional in real case, for example reset pin is > > is hard wired to "high". And this fact is also reflected by the > > devm_gpio_get_optional() calling in driver code. > > > > Signed-off-by: Jisheng Zhang <jszhang@xxxxxxxxxx> > > --- > > .../bindings/usb/thead,th1520-usb.yaml | 42 ++++++++++--------- > > .../devicetree/bindings/usb/vialab,vl817.yaml | 1 - > > 2 files changed, 23 insertions(+), 20 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/usb/thead,th1520-usb.yaml b/Documentation/devicetree/bindings/usb/thead,th1520-usb.yaml > > index afb618eb5013..ce1cab75f0ff 100644 > > --- a/Documentation/devicetree/bindings/usb/thead,th1520-usb.yaml > > +++ b/Documentation/devicetree/bindings/usb/thead,th1520-usb.yaml > > @@ -28,6 +28,10 @@ properties: > > > > ranges: true > > > > + thead,misc-syscon: > > + $ref: /schemas/types.yaml#/definitions/phandle-array > > + description: a phandle to the misc system register controller syscon node. > > Either you combined two unrelated changes or this should be resets from > reset-controller. oops, this change isn't expected... I was working on TH1520 usb but add local changes step by step, but then found vl817 need to be patched. I forgot to reset TH1520 usb part. Sorry for noise. V2 will be sent out soon.