Messages in this thread | | | Date | Wed, 18 Oct 2023 23:00:22 +0800 | From | Jisheng Zhang <> | Subject | Re: [PATCH] dt-bindings: usb: vialab,vl817: remove reset-gpios from required list |
| |
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@kernel.org> > > --- > > .../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.
| |