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.7 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,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 E9F61C43331 for ; Mon, 30 Mar 2020 19:23:58 +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 C2BFF2072E for ; Mon, 30 Mar 2020 19:23:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="dHInFaFt" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C2BFF2072E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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=fKr8puw6qMldz35PPqt327WZ6kotMjnh76LGS9qFkUk=; b=dHInFaFtqj80of 6gs/f0IODe/KEUzKWMQwGAUeF0YdbgbJ8UgYF/lQtvQFZA/SoDTK6B6Q+RbpnIYIIrSqElWdIuiQO z1gf54vkxXjJJ9aTy9FF6/XlA3HY4eXx4kL3Z4VA2NyZFeybK4GcJkrK4Q2sJMYciQM37/ZXTFlez WB1Drgi0xuKzmXPxmAIQcRkxsHHnkqKqaqHGe6s3azyd6tX4ggQi8x25jGI+d60Xuz4FiZrBN4r4y ojZem0ZSAP0EHovzIFyHLQxQIFVWL2VEo+O+mM3U73Wf2UlfRRuV9aYtDyj6xt8kdkqD+/Tv5Oi2e MFIZa0d6TTVdnu7Oy2tg==; 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 1jJ00r-00082X-Jp; Mon, 30 Mar 2020 19:23:57 +0000 Received: from mail-io1-f66.google.com ([209.85.166.66]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jJ00o-00081W-D3 for linux-arm-kernel@lists.infradead.org; Mon, 30 Mar 2020 19:23:55 +0000 Received: by mail-io1-f66.google.com with SMTP id n10so5019599iom.3 for ; Mon, 30 Mar 2020 12:23:50 -0700 (PDT) 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=MXO0cFSUgdRH+a4P/o/GAi4s4MndCf1pU3SuIpqOxH8=; b=I8Y+J2CIbpKQ1Qqmzx4wzA1NgHEs5P+2FRYzWE80jTCAvBxYCIcxrRC7h7TUPKFsMk cta4EdFyGV2XMUApbRrc4+cEj9ubjf+AdpuoVguqpeQK9FiQbQ4oKOrMhiTDZ8qaRxOV oPPXWV/8pSsz8hsv9i2dAO4GjS4zkK34uqmrYtkBsZna2Fl8aHRFJgDxcZnl3DOnMmF2 pRDurqdGr0tsAmYqjB8fJcL+PEmINj8Dx+Ep9tuzSNKGMHM45+o55BZAhNbwF6aWYRhP OImeSWidd5cykn5MfRBNOGi9ca+1S0lMLdLvvU0JlfZygedQkv7sY+aeKmUGebhfw6uc rq3Q== X-Gm-Message-State: ANhLgQ3QWilV9v9vH5XJFqxnyvZjR/k43oOL//Q/+n8pS2CcSo9ogTIZ RQKXrKiz7QAx7lD7gT7CIQ== X-Google-Smtp-Source: ADFU+vul8Oe41tCjMTU3VB/Pazto4kKTIj+AVWhsXgGw9jz3HDJI0g0/B3oHxwfVXEbonu12CN0a1g== X-Received: by 2002:a6b:b989:: with SMTP id j131mr12170677iof.6.1585596229713; Mon, 30 Mar 2020 12:23:49 -0700 (PDT) Received: from rob-hp-laptop ([64.188.179.250]) by smtp.gmail.com with ESMTPSA id p189sm4307507iof.17.2020.03.30.12.23.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 Mar 2020 12:23:49 -0700 (PDT) Received: (nullmailer pid 22585 invoked by uid 1000); Mon, 30 Mar 2020 19:23:47 -0000 Date: Mon, 30 Mar 2020 13:23:47 -0600 From: Rob Herring To: rentao.bupt@gmail.com Subject: Re: [PATCH v2 6/6] dt-bindings: usb: document aspeed vhub device ID/string properties Message-ID: <20200330192347.GA6388@bogus> References: <20200315191632.12536-1-rentao.bupt@gmail.com> <20200315191632.12536-7-rentao.bupt@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200315191632.12536-7-rentao.bupt@gmail.com> User-Agent: Mutt/1.10.1 (2018-07-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200330_122354_443270_88557BE5 X-CRM114-Status: GOOD ( 15.89 ) 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 On Sun, Mar 15, 2020 at 12:16:32PM -0700, rentao.bupt@gmail.com wrote: > From: Tao Ren > > Update device tree binding document for aspeed vhub's device IDs and > string properties. > > Signed-off-by: Tao Ren > --- > No change in v2: > - the patch is added into the series since v2. > > .../bindings/usb/aspeed,usb-vhub.yaml | 68 +++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > index 06399ba0d9e4..5b2e8d867219 100644 > --- a/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > +++ b/Documentation/devicetree/bindings/usb/aspeed,usb-vhub.yaml > @@ -52,6 +52,59 @@ properties: > minimum: 1 > maximum: 21 > > + vhub-vendor-id: > + description: vhub Vendor ID > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - maximum: 65535 > + > + vhub-product-id: > + description: vhub Product ID > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - maximum: 65535 There's already standard 'vendor-id' and 'device-id' properties. Use those. > + > + vhub-device-revision: Specific to USB, not vhub. > + description: vhub Device Revision in binary-coded decimal > + allOf: > + - $ref: /schemas/types.yaml#/definitions/uint32 > + - maximum: 65535 > + > + vhub-strings: > + type: object > + > + properties: > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > + patternProperties: > + '^string@[0-9a-f]+$': > + type: object > + description: string descriptors of the specific language > + > + properties: > + reg: > + maxItems: 1 > + description: 16-bit Language Identifier defined by USB-IF > + > + manufacturer: > + description: vhub manufacturer > + allOf: > + - $ref: /schemas/types.yaml#/definitions/string > + > + product: > + description: vhub product name > + allOf: > + - $ref: /schemas/types.yaml#/definitions/string > + > + serial-number: > + description: vhub device serial number > + allOf: > + - $ref: /schemas/types.yaml#/definitions/string For all of this, it's USB specific, not vhub specific. I'm not sure this is the right approach. It might be better to just define properties which are just raw USB descriptors rather than inventing some DT format that then has to be converted into USB descriptors. Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel