Message ID | 1675188609-20913-6-git-send-email-ssengar@linux.microsoft.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:eb09:0:0:0:0:0 with SMTP id s9csp54023wrn; Tue, 31 Jan 2023 10:13:57 -0800 (PST) X-Google-Smtp-Source: AK7set9gDhF6P+UoWvuq6jlsr8o5SJtQP43CPxt495A4J6hXpHfO9voEFXIrKGBl6w8ZFmSchgd3 X-Received: by 2002:a17:902:e809:b0:196:7a94:41ac with SMTP id u9-20020a170902e80900b001967a9441acmr12230888plg.66.1675188837165; Tue, 31 Jan 2023 10:13:57 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1675188837; cv=none; d=google.com; s=arc-20160816; b=f4wcN7fe31ud1zL6BGUEQDhOOIRL4NMgRT1mVvS14IwM4RdWicIoNduiJYH0U/y5iI AiR2vjtYN5KRpjklDClMkf0QYQS/AqItfSad9McMn4kzxyj1cd2WBvZqbvaIkoj2Vwpt jeHcIC5+MiUBF0IF2lGLNzZchOu9QC3qW10n33hbtjQKQ+Tl4wzkrODddK5Qt1QIrBui yAa0YQhbJhYe+iscVH5n4S3xMJM1GloSieo2F5IP3Jw9/JjcRNAKcmta1CeAlpVR+8q4 jhJgihMKswSiUs6Gbk3rQl7oOpofp42mUGlrG/I93TqvLjP6sfbAIif88d2T4r/AYRn+ Z8KA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:references:in-reply-to:message-id:date:subject :to:from:dkim-signature:dkim-filter; bh=aolDgf0ZKmA1dIvXP/OLbXrz+i5/r4RkpCnFJWskn6w=; b=GOZBNQg09+tk6exD59FW0N43kE1lCuHE8dEZ+APXVjTtWhhuxwGX8TtK02x9X3GSOe Y4Tu6S6FX9CEU3tWK79ulRKzBZ3kH7QNSU4d2cTthE9OmHok3EqMTOCPkEtQo/ONfh2W WvXx18c2y/ykkaLi5jvzEJrSgR2/Kex2jRev9X32WvhkTwqZ5Xog/ncCqPx1OG8lQm8S WXiiiJ7sVYipi7D+6NwmclbaSqIwULjSh7JUfrfi0wqjGSm4uetvB9B3Rldc6CMgPI++ /g0O/DscpAnaJiCvg31FUr6TbvMEglHO15lYAmuZp86M5lV5EoW5pxMI0JChomovhOhQ bN6Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux.microsoft.com header.s=default header.b=XhFYOZ5z; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linux.microsoft.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id f6-20020a170902ce8600b00189239e6a29si18451380plg.95.2023.01.31.10.13.44; Tue, 31 Jan 2023 10:13:57 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linux.microsoft.com header.s=default header.b=XhFYOZ5z; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linux.microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230321AbjAaSKc (ORCPT <rfc822;maxin.john@gmail.com> + 99 others); Tue, 31 Jan 2023 13:10:32 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37896 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231644AbjAaSKT (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Tue, 31 Jan 2023 13:10:19 -0500 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 47BBF2884D; Tue, 31 Jan 2023 10:10:18 -0800 (PST) Received: from linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net (linux.microsoft.com [13.77.154.182]) by linux.microsoft.com (Postfix) with ESMTPSA id 9A96520DFDBF; Tue, 31 Jan 2023 10:10:17 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 9A96520DFDBF DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1675188617; bh=aolDgf0ZKmA1dIvXP/OLbXrz+i5/r4RkpCnFJWskn6w=; h=From:To:Subject:Date:In-Reply-To:References:From; b=XhFYOZ5zsW06gmNDjCX8xlCrymk2F9BhPsEXvR9/xI2wHtfEIBQr1s3Bs3F3GJg6e vohocTceW2F4c2LX+iYT0AOi9q7dEutF/VcR4tV3wLpkAFX4noleRLV+9F0+goj98u QPBIkVVVrqWCpKTp7PovS5nSvRe43RO4V2jJl7o4= From: Saurabh Sengar <ssengar@linux.microsoft.com> To: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, kys@microsoft.com, haiyangz@microsoft.com, wei.liu@kernel.org, decui@microsoft.com, daniel.lezcano@linaro.org, tglx@linutronix.de, virtualization@lists.linux-foundation.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hyperv@vger.kernel.org, mikelley@microsoft.com, ssengar@microsoft.com Subject: [PATCH v2 5/6] dt-bindings: hypervisor: Add dt-bindings for VMBus Date: Tue, 31 Jan 2023 10:10:08 -0800 Message-Id: <1675188609-20913-6-git-send-email-ssengar@linux.microsoft.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1675188609-20913-1-git-send-email-ssengar@linux.microsoft.com> References: <1675188609-20913-1-git-send-email-ssengar@linux.microsoft.com> X-Spam-Status: No, score=-19.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,ENV_AND_HDR_SPF_MATCH,RCVD_IN_DNSWL_MED, SPF_HELO_PASS,SPF_PASS,USER_IN_DEF_DKIM_WL,USER_IN_DEF_SPF_WL autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1756562810232493035?= X-GMAIL-MSGID: =?utf-8?q?1756562810232493035?= |
Series |
Device tree support for Hyper-V VMBus driver
|
|
Commit Message
Saurabh Singh Sengar
Jan. 31, 2023, 6:10 p.m. UTC
Add dt-bindings for Hyper-V VMBus
Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com>
---
.../bindings/hypervisor/msft,vmbus.yaml | 50 +++++++++++++++++++
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
MAINTAINERS | 1 +
3 files changed, 53 insertions(+)
create mode 100644 Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml
Comments
On 31/01/2023 19:10, Saurabh Sengar wrote: > Add dt-bindings for Hyper-V VMBus > > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> > --- > .../bindings/hypervisor/msft,vmbus.yaml | 50 +++++++++++++++++++ > .../devicetree/bindings/vendor-prefixes.yaml | 2 + > MAINTAINERS | 1 + > 3 files changed, 53 insertions(+) > create mode 100644 Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > > diff --git a/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > new file mode 100644 > index 000000000000..8f50d6097c48 > --- /dev/null > +++ b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > @@ -0,0 +1,50 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/hypervisor/msft,vmbus.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Microsoft Hyper-V VMBus device tree bindings This is a friendly reminder during the review process. It seems my previous comments were not fully addressed. Maybe my feedback got lost between the quotes, maybe you just forgot to apply it. Please go back to the previous discussion and either implement all requested changes or keep discussing them. Thank you. (other places as well...) Best regards, Krzysztof
On 31/01/2023 19:10, Saurabh Sengar wrote: > > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> > + - | > + / { > + compatible = "foo"; > + model = "foo"; > + #address-cells = <0x02>; > + #size-cells = <0x02>; Except previous comments (all of them were ignored), also: Drop entire part. Not related, not correct, not helping and you cannot have top level nodes in example. > + > + vmbus@ff0000000 { > + #address-cells = <0x02>; > + #size-cells = <0x01>; > + compatible = "msft,vmbus"; > + ranges = <0x00 0x00 0x0f 0xf0000000 0x10000000>; > + }; > + }; > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > index 1f7a519a936f..ab74ea97535f 100644 > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > @@ -876,6 +876,8 @@ patternProperties: > deprecated: true > "^mscc,.*": > description: Microsemi Corporation > + "^msft,.*": > + description: Microsoft Corporation Don't duplicate vendor prefixes. Drop and use correct prefix. Best regards, Krzysztof
On Tue, Jan 31, 2023 at 07:51:20PM +0100, Krzysztof Kozlowski wrote: > On 31/01/2023 19:10, Saurabh Sengar wrote: > > Add dt-bindings for Hyper-V VMBus > > > > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> > > --- > > .../bindings/hypervisor/msft,vmbus.yaml | 50 +++++++++++++++++++ > > .../devicetree/bindings/vendor-prefixes.yaml | 2 + > > MAINTAINERS | 1 + > > 3 files changed, 53 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > > > > diff --git a/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > > new file mode 100644 > > index 000000000000..8f50d6097c48 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml > > @@ -0,0 +1,50 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/hypervisor/msft,vmbus.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Microsoft Hyper-V VMBus device tree bindings > > This is a friendly reminder during the review process. > > It seems my previous comments were not fully addressed. Maybe my > feedback got lost between the quotes, maybe you just forgot to apply it. > Please go back to the previous discussion and either implement all > requested changes or keep discussing them. > > Thank you. > > (other places as well...) Hi Krzysztof, Thank you for the review. Sorry I missed this, will fix in V3. The patches have gone significant modification, and I have thought I have fixed all the comments you have provided hence didnt send the follow up discussion. Apparently, I may have missed few will look again and fix in next version. Regards, Saurabh > > Best regards, > Krzysztof
On Tue, Jan 31, 2023 at 07:54:56PM +0100, Krzysztof Kozlowski wrote: > On 31/01/2023 19:10, Saurabh Sengar wrote: > > > > Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> > > > + - | > > + / { > > + compatible = "foo"; > > + model = "foo"; > > + #address-cells = <0x02>; > > + #size-cells = <0x02>; > > Except previous comments (all of them were ignored), Thank you for your comments, I have tried to address them all in this version but I may have missed few. I will go and look again all the emails, but if there is any thing which you can point again and we can start a new dicussion from here will be very helpful. I think one concern was related to use of 'reg' or 'ranges', and I thought this patch will give clarity that I intend to use 'ranges' without any child node. Is this acceptable ? > also: > Drop entire part. Not related, not correct, not helping and you cannot > have top level nodes in example. If I dont use the top level node, the parent address cells are assumed to be 1, and I get below warning. If there is better way to address this warning, please suggest I will work on it. Warning (ranges_format): /example-0/vmbus@ff0000000:ranges: "ranges" property has invalid length (20 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1) > > > + > > + vmbus@ff0000000 { > > + #address-cells = <0x02>; > > + #size-cells = <0x01>; > > + compatible = "msft,vmbus"; > > + ranges = <0x00 0x00 0x0f 0xf0000000 0x10000000>; > > + }; > > + }; > > diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > index 1f7a519a936f..ab74ea97535f 100644 > > --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml > > +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml > > @@ -876,6 +876,8 @@ patternProperties: > > deprecated: true > > "^mscc,.*": > > description: Microsemi Corporation > > + "^msft,.*": > > + description: Microsoft Corporation > > Don't duplicate vendor prefixes. Drop and use correct prefix. Sure, will fix this. Regards, Saurabh > > Best regards, > Krzysztof
On 01/02/2023 02:57, Saurabh Singh Sengar wrote: > On Tue, Jan 31, 2023 at 07:54:56PM +0100, Krzysztof Kozlowski wrote: >> On 31/01/2023 19:10, Saurabh Sengar wrote: >>> >>> Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> >> >>> + - | >>> + / { >>> + compatible = "foo"; >>> + model = "foo"; >>> + #address-cells = <0x02>; >>> + #size-cells = <0x02>; >> >> Except previous comments (all of them were ignored), > > Thank you for your comments, I have tried to address them all in this version > but I may have missed few. I will go and look again all the emails, but if > there is any thing which you can point again and we can start a new dicussion > from here will be very helpful. > > I think one concern was related to use of 'reg' or 'ranges', and I > thought this patch will give clarity that I intend to use 'ranges' > without any child node. Is this acceptable ? There were several comments. > > >> also: >> Drop entire part. Not related, not correct, not helping and you cannot >> have top level nodes in example. > > If I dont use the top level node, the parent address cells are assumed to be 1, > and I get below warning. If there is better way to address this warning, please > suggest I will work on it. > > Warning (ranges_format): /example-0/vmbus@ff0000000:ranges: "ranges" property has invalid length (20 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1) Then use soc just like every other example. Best regards, Krzysztof
On Wed, Feb 01, 2023 at 08:23:48AM +0100, Krzysztof Kozlowski wrote: > On 01/02/2023 02:57, Saurabh Singh Sengar wrote: > > On Tue, Jan 31, 2023 at 07:54:56PM +0100, Krzysztof Kozlowski wrote: > >> On 31/01/2023 19:10, Saurabh Sengar wrote: > >>> > >>> Signed-off-by: Saurabh Sengar <ssengar@linux.microsoft.com> > >> > >>> + - | > >>> + / { > >>> + compatible = "foo"; > >>> + model = "foo"; > >>> + #address-cells = <0x02>; > >>> + #size-cells = <0x02>; > >> > >> Except previous comments (all of them were ignored), > > > > Thank you for your comments, I have tried to address them all in this version > > but I may have missed few. I will go and look again all the emails, but if > > there is any thing which you can point again and we can start a new dicussion > > from here will be very helpful. > > > > I think one concern was related to use of 'reg' or 'ranges', and I > > thought this patch will give clarity that I intend to use 'ranges' > > without any child node. Is this acceptable ? > > There were several comments. Ok, let me reply to all of your comments from previous thread to avoid any confusion. Hope this is fine. > > > > > > >> also: > >> Drop entire part. Not related, not correct, not helping and you cannot > >> have top level nodes in example. > > > > If I dont use the top level node, the parent address cells are assumed to be 1, > > and I get below warning. If there is better way to address this warning, please > > suggest I will work on it. > > > > Warning (ranges_format): /example-0/vmbus@ff0000000:ranges: "ranges" property has invalid length (20 bytes) (parent #address-cells == 1, child #address-cells == 2, #size-cells == 1) > > Then use soc just like every other example. Thanks for suggestion, I will fix this in V3 Regards, Saurabh > > > Best regards, > Krzysztof
diff --git a/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml new file mode 100644 index 000000000000..8f50d6097c48 --- /dev/null +++ b/Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml @@ -0,0 +1,50 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/hypervisor/msft,vmbus.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Microsoft Hyper-V VMBus device tree bindings + +maintainers: + - Saurabh Sengar <ssengar@linux.microsoft.com> + +description: + VMBus is a software bus that implement the protocols for communication + between the root or host OS and guest OSs (virtual machines). + +properties: + compatible: + const: msft,vmbus + + ranges: true + + '#address-cells': + const: 2 + + '#size-cells': + const: 1 + +required: + - compatible + - ranges + - '#address-cells' + - '#size-cells' + +additionalProperties: false + +examples: + - | + / { + compatible = "foo"; + model = "foo"; + #address-cells = <0x02>; + #size-cells = <0x02>; + + vmbus@ff0000000 { + #address-cells = <0x02>; + #size-cells = <0x01>; + compatible = "msft,vmbus"; + ranges = <0x00 0x00 0x0f 0xf0000000 0x10000000>; + }; + }; diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml index 1f7a519a936f..ab74ea97535f 100644 --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml @@ -876,6 +876,8 @@ patternProperties: deprecated: true "^mscc,.*": description: Microsemi Corporation + "^msft,.*": + description: Microsoft Corporation "^msi,.*": description: Micro-Star International Co. Ltd. "^mstar,.*": diff --git a/MAINTAINERS b/MAINTAINERS index 9bc5b88b723a..c749782b0cff 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -9496,6 +9496,7 @@ S: Supported T: git git://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git F: Documentation/ABI/stable/sysfs-bus-vmbus F: Documentation/ABI/testing/debugfs-hyperv +F: Documentation/devicetree/bindings/hypervisor/msft,vmbus.yaml F: Documentation/virt/hyperv F: Documentation/networking/device_drivers/ethernet/microsoft/netvsc.rst F: arch/arm64/hyperv