From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8CCC9C3F2CD for ; Tue, 3 Mar 2020 06:31:19 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5BB3D21739 for ; Tue, 3 Mar 2020 06:31:19 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="RIE0MQwL"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="UbMUxmCC" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5BB3D21739 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=P08Gns0cwkz8xmjv6R0WrfzIkz5ECrgjjuk3MrG66Cg=; b=RIE0MQwL30oxED 88QzR43BS80TjpbSOaz9AwGh2fAmCX0Q8EP3KPDlhkUulxMbSxqzbifECVJvSMGPK31wj/FHs2XWF vU3T8layxcsy9Bb/eqPtI/xXI42CBmq0f62/HlCbCj/Zd6zujOV89uPLNzQ1wmQeTgGvv4KFkCQRX DSeCde6k5dwG0P8Qi4Rdk3T74ZH8A3xUL9hdqZKqsxz1Z/lP+wlhicmBe1OW1jO7/4Lprfs9wVAWl RXN9wk9Jo/Agj5iSszvoyHhe3bzSDa8ZC7sHEPyW9XKi0d0PlX5V0Rx/+LnL2GD7RrtWD0HwV1/O/ 9w6EmQc1sOohOC97DXBg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j915B-0006lx-HF; Tue, 03 Mar 2020 06:31:09 +0000 Received: from mail-pf1-x443.google.com ([2607:f8b0:4864:20::443]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j9157-0006lT-Ay for linux-arm-kernel@lists.infradead.org; Tue, 03 Mar 2020 06:31:07 +0000 Received: by mail-pf1-x443.google.com with SMTP id i13so950534pfe.3 for ; Mon, 02 Mar 2020 22:31:04 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=ndx+j6LnVMqCAXHMVDFZLXsjTKEuusKvWbflCc649JY=; b=UbMUxmCCJ1iqtAUi6lktmncZnowFCN1CA8b4uv3RRka2O4keDihSXfn8yE9a2iXPoL 9z7MgPOfyHIRosPIN1a1t2ACLwEftyMbcu3gmo9dew1aYCQFhNfkGldLuMWqX7FNANqi b3IkikKS9IgVFw5xhru6Tv/r/MSX3e890v5jp060Rpf0liav7ePur3okq8pvAjXZez3f RbpgV6gQE3ti6GZ9F7AQSEaz5PlWceHqzCoQ28ZFbN31V7noE07UcLBpPgX479J2mwQe pvsYq98tdanDbB4+zLyYdr0tGxAT/h0gaiN12AZbDOO0eCW7wu6gYGx8ehDn/5CpVpCJ a+pw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=ndx+j6LnVMqCAXHMVDFZLXsjTKEuusKvWbflCc649JY=; b=QcrHtWqEwwQn3Pci2Ja1u345MlikJPW8l021ZQKkEv1qTKXbi76K1Z22O8sj+DaVSX fuv1rL08U5+NW2Fu/pSDdWWApxLrBIfMITHQv03OofTN3Db4FJwjJR435ZiJFajNe5id ylWWW/AB0IFaz9zu8xLnIcmrucaLKfY5TXYzLoDbbGSYkRQSzMV6iJ76H5n+QpK91NPl ozQ6lR1hLlGwqETo1EuohZ326TMnQ0HOzQnF0Uo97xadMDOy9wljDdSjhy9X/v/d1IK7 A1N/x+rxqkvEsRmVnTusWy8UTGlVQhQPOT+1qmARTp1UdzkgmIx2awFpvNdtg81tKMVC y9Zw== X-Gm-Message-State: ANhLgQ0/aiEOESUGgVCx422NkPxaPmyuhFP+TZTVD8pEhCuieHrNBXgl rryIUacNh1keRxbzfg9tXKE= X-Google-Smtp-Source: ADFU+vuGZQszw51zqWe1fJX1aHCdONMS54hLYZPiwaStPENZVZ7FsvgKtDNPs8Yql9brDHHw68ftjQ== X-Received: by 2002:aa7:9726:: with SMTP id k6mr146994pfg.196.1583217064295; Mon, 02 Mar 2020 22:31:04 -0800 (PST) Received: from taoren-ubuntu-R90MNF91 (c-24-4-25-55.hsd1.ca.comcast.net. [24.4.25.55]) by smtp.gmail.com with ESMTPSA id 69sm4375913pfz.97.2020.03.02.22.31.03 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Mon, 02 Mar 2020 22:31:03 -0800 (PST) Date: Mon, 2 Mar 2020 22:30:57 -0800 From: Tao Ren To: Rob Herring Subject: Re: [PATCH v6 7/7] dt-bindings: usb: add documentation for aspeed usb-vhub Message-ID: <20200303063056.GA6751@taoren-ubuntu-R90MNF91> References: <20200228020757.10513-1-rentao.bupt@gmail.com> <20200228020757.10513-8-rentao.bupt@gmail.com> <20200302234744.GA21778@bogus> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200302234744.GA21778@bogus> User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200302_223105_405078_3A7F5F95 X-CRM114-Status: GOOD ( 23.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Felipe Balbi , linux-aspeed@lists.ozlabs.org, devicetree@vger.kernel.org, Andrew Jeffery , Benjamin Herrenschmidt , openbmc@lists.ozlabs.org, linux-usb@vger.kernel.org, taoren@fb.com, linux-kernel@vger.kernel.org, Stephen Boyd , Joel Stanley , Greg Kroah-Hartman , Chunfeng Yun , Colin Ian King , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Rob, Thanks for the careful review and I've addressed all your feedback in v7 (based on my understanding). Please kindly let me know if you have further suggestions. Cheers, Tao On Mon, Mar 02, 2020 at 05:47:44PM -0600, Rob Herring wrote: > On Thu, Feb 27, 2020 at 06:07:57PM -0800, rentao.bupt@gmail.com wrote: > > From: Tao Ren > > > > Add device tree binding documentation for the Aspeed USB 2.0 Virtual HUb > > Controller. > > > > Signed-off-by: Tao Ren > > Reviewed-by: Joel Stanley > > --- > > Changes in v6: > > - added 2 required properties into example and passed "make > > dt_binding_check". > > Changes in v5: > > - updated maintainer to Ben. > > - refined patch description per Joel's suggestion. > > No change in v2/v3/v4: > > - the patch is added to the patch series since v4. > > > > .../bindings/usb/aspeed,usb-vhub.yaml | 73 +++++++++++++++++++ > > 1 file changed, 73 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > > > > diff --git a/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > > new file mode 100644 > > index 000000000000..b9f33310e9a2 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > > @@ -0,0 +1,73 @@ > > +# SPDX-License-Identifier: GPL-2.0-or-later > > Dual license new bindings please. > > (GPL-2.0-only OR BSD-2-Clause) > > > +# Copyright (c) 2020 Facebook Inc. > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/usb/aspeed,usb-vhub.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: ASPEED USB 2.0 Virtual Hub Controller > > + > > +maintainers: > > + - Benjamin Herrenschmidt > > + > > +description: |+ > > + The ASPEED USB 2.0 Virtual Hub Controller implements 1 set of USB Hub > > + register and several sets of Device and Endpoint registers to support > > + the Virtual Hub's downstream USB devices. > > + > > + Supported number of devices and endpoints vary depending on hardware > > + revisions. AST2400 and AST2500 Virtual Hub supports 5 downstream devices > > + and 15 generic endpoints, while AST2600 Virtual Hub supports 7 downstream > > + devices and 21 generic endpoints. > > + > > +properties: > > + compatible: > > + enum: > > + - aspeed,ast2400-usb-vhub > > + - aspeed,ast2500-usb-vhub > > + - aspeed,ast2600-usb-vhub > > + > > + reg: > > + maxItems: 1 > > + description: Common configuration registers > > If 'Common' means something specific to this block, then keep. > Otherwise, you can drop ti. > > > + > > + clocks: > > + maxItems: 1 > > + description: The Virtual Hub Controller clock gate > > Drop this. > > > + > > + interrupts: > > + maxItems: 1 > > + > > + aspeed,vhub-downstream-ports: > > + description: Number of downstream ports supported by the Virtual Hub > > + allOf: > > + - $ref: /schemas/types.yaml#/definitions/uint32 > > Constraints? > > 'maximum: 5' ? > > default? > > > + > > + aspeed,vhub-generic-endpoints: > > + description: Number of generic endpoints supported by the Virtual Hub > > + allOf: > > + - $ref: /schemas/types.yaml#/definitions/uint32 > > Constraints? > > > + > > +required: > > + - compatible > > + - reg > > + - clocks > > + - interrupts > > + - aspeed,vhub-downstream-ports > > + - aspeed,vhub-generic-endpoints > > + > > +additionalProperties: false > > + > > +examples: > > + - | > > + #include > > + vhub: usb-vhub@1e6a0000 { > > + compatible = "aspeed,ast2500-usb-vhub"; > > + reg = <0x1e6a0000 0x300>; > > + interrupts = <5>; > > + clocks = <&syscon ASPEED_CLK_GATE_USBPORT1CLK>; > > + aspeed,vhub-downstream-ports = <5>; > > + aspeed,vhub-generic-endpoints = <15>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_usb2ad_default>; > > + }; > > -- > > 2.17.1 > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel