From patchwork Tue Oct 17 16:01:58 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jisheng Zhang X-Patchwork-Id: 734983 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A11A847340; Tue, 17 Oct 2023 16:14:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="qx23junq" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 89D25C433CA; Tue, 17 Oct 2023 16:14:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697559254; bh=Ye90HBHDZzB2aDp/S5UGI++JNCzZ7q1PX2LEgBDO3FA=; h=From:To:Cc:Subject:Date:From; b=qx23junqDkaU3YjKyYdbbhUyl6RLIT071nSnCTOJMkBDZHvV4szsntrGDj3OEDqEj S897OKmOGZfSvjmRBVYtCU5LPOFJtMX6OJ82xXXEgA9Go4HgUsrSH/WNRLxFm58O8f ZUGjc4jgUhf1hAAv7P4kejGqwOn95zb7UDdwHe5jGFc/VkkSJxel6Mw98eYqq+Rcc4 tBK+OWo+yQhKO7JIk3NlSwDXApMJ1XBfqcDzzy/hOuQFWXCMLlVaVtnCNqYoT9Pa/g YzIH4zPCVPoff2t6r9ayWW3UQfdZm/HSJfCmPpvxm89cLOBoVT8Iku16qe2NZVa9L9 7FyQzoGAsPMNA== From: Jisheng Zhang To: Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Anand Moon Cc: linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] dt-bindings: usb: vialab,vl817: remove reset-gpios from required list Date: Wed, 18 Oct 2023 00:01:58 +0800 Message-Id: <20231017160158.1065-1-jszhang@kernel.org> X-Mailer: git-send-email 2.40.0 Precedence: bulk X-Mailing-List: linux-usb@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 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 --- .../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. + '#address-cells': enum: [ 1, 2 ] @@ -51,23 +55,23 @@ additionalProperties: false examples: - | - - usb { - compatible = "thead,th1520-usb"; - reg = <0xec03f000 0x1000>; - clocks = <&clk 1>, - <&clk 2>, - <&clk 3>, - <&clk 4>; - clock-names = "ref", "bus_early", "phy", "suspend"; - ranges; - #address-cells = <1>; - #size-cells = <1>; - - usb@e7040000 { - compatible = "snps,dwc3"; - reg = <0xe7040000 0x10000>; - interrupts = <68>; - dr_mode = "host"; - }; + usb@ec03f000 { + compatible = "thead,th1520-usb"; + reg = <0xec03f000 0x1000>; + thead,misc-syscon = <&misc_syscon>; + clocks = <&clk 1>, + <&clk 2>, + <&clk 3>, + <&clk 4>; + clock-names = "ref", "bus_early", "phy", "suspend"; + ranges; + #address-cells = <1>; + #size-cells = <1>; + + usb@e7040000 { + compatible = "snps,dwc3"; + reg = <0xe7040000 0x10000>; + interrupts = <68>; + dr_mode = "host"; + }; }; diff --git a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml index 76db9071b352..c815010ba9c2 100644 --- a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml +++ b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml @@ -37,7 +37,6 @@ properties: required: - compatible - reg - - reset-gpios - vdd-supply - peer-hub