From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Date: Mon, 30 Mar 2020 13:23:47 -0600 Subject: [PATCH v2 6/6] dt-bindings: usb: document aspeed vhub device ID/string properties In-Reply-To: <20200315191632.12536-7-rentao.bupt@gmail.com> References: <20200315191632.12536-1-rentao.bupt@gmail.com> <20200315191632.12536-7-rentao.bupt@gmail.com> Message-ID: <20200330192347.GA6388@bogus> List-Id: To: linux-aspeed@lists.ozlabs.org MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit On Sun, Mar 15, 2020 at 12:16:32PM -0700, rentao.bupt at 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 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.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,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 72C62C43331 for ; Mon, 30 Mar 2020 19:23:55 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4CDB620784 for ; Mon, 30 Mar 2020 19:23:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1585596235; bh=6OlA2qF0ge60Y9qd4QRLz8Wg6xFIOo/oolkIEsWt7nQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=k01Vf8JusMUbOnCc5cSiiRVXw28LbFRHfVgvWSjxYeyxoGimzZED7m31A4CcGX4Ps yHq3hoPgr0ZMlM4iCG0IXr+ahR8DCCMrNwbaijfoeOT47ZSNeE+bR+W4Fvop95WuNy 2byae0f/7wg8yJd9vqwD6nOBypQPGJfli1rrWjTM= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728209AbgC3TXv (ORCPT ); Mon, 30 Mar 2020 15:23:51 -0400 Received: from mail-io1-f65.google.com ([209.85.166.65]:43691 "EHLO mail-io1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726781AbgC3TXv (ORCPT ); Mon, 30 Mar 2020 15:23:51 -0400 Received: by mail-io1-f65.google.com with SMTP id x9so12633343iom.10; 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=b2WK8r2tG8huW8HYFFvjSGY0TfaCN7piJfb0Q6GCQr2a3D7F1WfwdHRz/eIsnRs86h y09f5qGv5o3X3V6sdlUdZ85G+LVyGsiVn2O6zaTDfR/dxWWFN4wS+IuLsuOghqSAS16q JY7WtgUBEVR8UhHPFnmpe7aEd6AyuRYuLGOUvZ9KWwCR+B+w0+GcrzPTDVzJ0vZ7bfwA 0zSPfP/DNT2X+3VGFWso8z6aZa8wQ1yXJGxVB5OnNSQ08HeZK0RYyz7qEo/PQ+onL9kk plhszG505Bx2v9I6pFuUbqrnfCNukCl4Wy1rrPlN8SP0l3zdmzRFjFpFcET95+k7sKyF 8tmA== X-Gm-Message-State: ANhLgQ2L0jfR/zywO3xGCDo+Jo+oRoFsqB9+tnHPwhAgy5Z4YXYYXiGC /fynJzRe+bOxB9EsTf78WQ== 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 Cc: Felipe Balbi , Greg Kroah-Hartman , Joel Stanley , Andrew Jeffery , Benjamin Herrenschmidt , Chunfeng Yun , Colin Ian King , Stephen Boyd , Mark Rutland , linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-aspeed@lists.ozlabs.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, openbmc@lists.ozlabs.org, taoren@fb.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-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200315191632.12536-7-rentao.bupt@gmail.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.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 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