From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: richard.yu@hpe.com, verdun@hpe.com, nick.hawkins@hpe.com, gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux@armlinux.org.uk, balbi@kernel.org, linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v1 4/7] dt-bindings: usb: hpe,gxp-vuhc: add binding for gxp vEHCI Date: Thu, 3 Nov 2022 12:32:26 -0400 [thread overview] Message-ID: <4acf1dd3-afb3-b69a-5c03-05c74c4ad69c@linaro.org> (raw) In-Reply-To: <20221103160625.15574-5-richard.yu@hpe.com> On 03/11/2022 12:06, richard.yu@hpe.com wrote: > From: Richard Yu <richard.yu@hpe.com> > > Create documentation for the hpe,gxp-vuhc binding to support access to > the gxp virtual EHCI controller. > Previous comments apply. > Signed-off-by: Richard Yu <richard.yu@hpe.com> > --- > .../devicetree/bindings/usb/hpe,gxp-vuhc.yaml | 34 +++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > > diff --git a/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml b/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > new file mode 100644 > index 000000000000..1411a2550092 > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > @@ -0,0 +1,34 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/usb/hpe,gxp-vuhc.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: HPE GXP Virtual Universal Host Controller (vUHC) This is getting confusing. If it is EHCI, you miss usb-hcd.yaml reference. But in fact you describe at syscon, maybe to avoid proper hardware description. That does not look right. > + > +maintainers: > + - Richard Yu <richard.yu@hpe.com> > + - Jean-Marie Verdun <verdun@hpe.com> > + - Nick Hawkins <nick.hawkins@hpe.com> > + > +properties: > + compatible: > + items: > + - const: hpe,gxp-vuhc > + - const: syscon > + > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + vuhc@80400000 { Node names should be generic. https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation > + compatible = "hpe,gxp-vuhc", "syscon"; > + reg = <0x80400000 0x80>; > + }; Best regards, Krzysztof
WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> To: richard.yu@hpe.com, verdun@hpe.com, nick.hawkins@hpe.com, gregkh@linuxfoundation.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, linux@armlinux.org.uk, balbi@kernel.org, linux-usb@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v1 4/7] dt-bindings: usb: hpe,gxp-vuhc: add binding for gxp vEHCI Date: Thu, 3 Nov 2022 12:32:26 -0400 [thread overview] Message-ID: <4acf1dd3-afb3-b69a-5c03-05c74c4ad69c@linaro.org> (raw) In-Reply-To: <20221103160625.15574-5-richard.yu@hpe.com> On 03/11/2022 12:06, richard.yu@hpe.com wrote: > From: Richard Yu <richard.yu@hpe.com> > > Create documentation for the hpe,gxp-vuhc binding to support access to > the gxp virtual EHCI controller. > Previous comments apply. > Signed-off-by: Richard Yu <richard.yu@hpe.com> > --- > .../devicetree/bindings/usb/hpe,gxp-vuhc.yaml | 34 +++++++++++++++++++ > 1 file changed, 34 insertions(+) > create mode 100644 Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > > diff --git a/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml b/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > new file mode 100644 > index 000000000000..1411a2550092 > --- /dev/null > +++ b/Documentation/devicetree/bindings/usb/hpe,gxp-vuhc.yaml > @@ -0,0 +1,34 @@ > +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/usb/hpe,gxp-vuhc.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: HPE GXP Virtual Universal Host Controller (vUHC) This is getting confusing. If it is EHCI, you miss usb-hcd.yaml reference. But in fact you describe at syscon, maybe to avoid proper hardware description. That does not look right. > + > +maintainers: > + - Richard Yu <richard.yu@hpe.com> > + - Jean-Marie Verdun <verdun@hpe.com> > + - Nick Hawkins <nick.hawkins@hpe.com> > + > +properties: > + compatible: > + items: > + - const: hpe,gxp-vuhc > + - const: syscon > + > + reg: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + > +additionalProperties: false > + > +examples: > + - | > + vuhc@80400000 { Node names should be generic. https://devicetree-specification.readthedocs.io/en/latest/chapter2-devicetree-basics.html#generic-names-recommendation > + compatible = "hpe,gxp-vuhc", "syscon"; > + reg = <0x80400000 0x80>; > + }; Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-11-03 16:34 UTC|newest] Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-11-03 16:06 [PATCH v1 0/7] Add USB Driver for HPE GXP Architecture richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:06 ` [PATCH v1 1/7] usb: gadget: udc: gxp_udc: add gxp USB support richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:06 ` [PATCH v1 2/7] dt-bindings: usb: hpe,gxp-udc: Add binding for gxp gadget richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:29 ` Krzysztof Kozlowski 2022-11-03 16:29 ` Krzysztof Kozlowski 2022-11-04 20:03 ` Yu, Richard 2022-11-04 20:03 ` Yu, Richard 2022-11-04 20:09 ` Krzysztof Kozlowski 2022-11-04 20:09 ` Krzysztof Kozlowski 2022-11-07 20:16 ` Yu, Richard 2022-11-07 20:16 ` Yu, Richard 2022-11-08 11:30 ` Krzysztof Kozlowski 2022-11-08 11:30 ` Krzysztof Kozlowski 2022-11-09 3:37 ` Yu, Richard 2022-11-09 3:37 ` Yu, Richard 2022-11-11 8:20 ` Krzysztof Kozlowski 2022-11-11 8:20 ` Krzysztof Kozlowski 2022-11-03 16:06 ` [PATCH v1 3/7] dt-bindings: usb: hpe,gxp-udcg: Add binding for gxp gadget group richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:30 ` Krzysztof Kozlowski 2022-11-03 16:30 ` Krzysztof Kozlowski 2022-11-03 16:06 ` [PATCH v1 4/7] dt-bindings: usb: hpe,gxp-vuhc: add binding for gxp vEHCI richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:32 ` Krzysztof Kozlowski [this message] 2022-11-03 16:32 ` Krzysztof Kozlowski 2022-11-03 16:06 ` [PATCH v1 5/7] ARM: dts: hpe: Add UDC nodes richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-08 11:28 ` Krzysztof Kozlowski 2022-11-08 11:28 ` Krzysztof Kozlowski 2022-11-03 16:06 ` [PATCH v1 6/7] ARM: configs: multi_v7_defconfig: Enable HPE GXP USB Driver richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:32 ` Krzysztof Kozlowski 2022-11-03 16:32 ` Krzysztof Kozlowski 2022-11-03 16:06 ` [PATCH v1 7/7] MAINTAINERS: add USB support to GXP richard.yu 2022-11-03 16:06 ` richard.yu 2022-11-03 16:21 ` Krzysztof Kozlowski 2022-11-03 16:21 ` Krzysztof Kozlowski 2022-11-04 14:05 ` kernel test robot 2022-11-04 14:05 ` kernel test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4acf1dd3-afb3-b69a-5c03-05c74c4ad69c@linaro.org \ --to=krzysztof.kozlowski@linaro.org \ --cc=balbi@kernel.org \ --cc=devicetree@vger.kernel.org \ --cc=gregkh@linuxfoundation.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-usb@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=nick.hawkins@hpe.com \ --cc=richard.yu@hpe.com \ --cc=robh+dt@kernel.org \ --cc=verdun@hpe.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.