From: Biju Das <biju.das.jz@bp.renesas.com> To: Rob Herring <robh@kernel.org> Cc: Kishon Vijay Abraham I <kishon@ti.com>, Vinod Koul <vkoul@kernel.org>, Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>, "linux-phy@lists.infradead.org" <linux-phy@lists.infradead.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Geert Uytterhoeven <geert+renesas@glider.be>, Chris Paterson <Chris.Paterson2@renesas.com>, Biju Das <biju.das@bp.renesas.com>, Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>, "linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org> Subject: RE: [PATCH v2 01/11] dt-bindings: phy: renesas: Document RZ/G2L USB PHY Control bindings Date: Fri, 25 Jun 2021 10:11:58 +0000 [thread overview] Message-ID: <OS0PR01MB59227E0C129668A5FFF861E186069@OS0PR01MB5922.jpnprd01.prod.outlook.com> (raw) In-Reply-To: <OS0PR01MB5922742488532EC506334AC786089@OS0PR01MB5922.jpnprd01.prod.outlook.com> Hi Rob, > Subject: RE: [PATCH v2 01/11] dt-bindings: phy: renesas: Document RZ/G2L > USB PHY Control bindings > > Hi Rob, > > Thanks for the feedback. > > > Subject: Re: [PATCH v2 01/11] dt-bindings: phy: renesas: Document > > RZ/G2L USB PHY Control bindings > > > > On Wed, Jun 23, 2021 at 7:38 AM Biju Das <biju.das.jz@bp.renesas.com> > > wrote: > > > > > > Hi Rob, > > > > > > Thanks for the feedback. > > > > > > > Subject: Re: [PATCH v2 01/11] dt-bindings: phy: renesas: Document > > > > RZ/G2L USB PHY Control bindings > > > > > > > > On Mon, Jun 21, 2021 at 10:39:33AM +0100, Biju Das wrote: > > > > > Add device tree binding document for RZ/G2L USB PHY control > driver. > > > > > > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > > > > Reviewed-by: Lad Prabhakar > > > > > <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > > --- > > > > > V1->V2: > > > > > * Add clock properties > > > > > --- > > > > > .../phy/renesas,rzg2l-usbphyctrl.yaml | 65 > > +++++++++++++++++++ > > > > > 1 file changed, 65 insertions(+) create mode 100644 > > > > > Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl.y > > > > > am > > > > > l > > > > > > > > > > diff --git > > > > > a/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl > > > > > .y > > > > > aml > > > > > b/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl > > > > > .y > > > > > aml > > > > > new file mode 100644 > > > > > index 000000000000..8e8ba43f595d > > > > > --- /dev/null > > > > > +++ b/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphy > > > > > +++ ct > > > > > +++ rl.y > > > > > +++ aml > > > > > @@ -0,0 +1,65 @@ > > > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML > > > > > +1.2 > > > > > +--- > > > > > +$id: > > > > > +https://jpn01.safelinks.protection.outlook.com/?url=http%3A%2F% > > > > > +2F > > > > > +devi > > > > > +cetree.org%2Fschemas%2Fphy%2Frenesas%2Crzg2l-usbphyctrl.yaml%23 > > > > > +&a > > > > > +mp;d > > > > > +ata=04%7C01%7Cbiju.das.jz%40bp.renesas.com%7Cc6bbf5f6ce334eaa72 > > > > > +2a > > > > > +08d9 > > > > > +359f07ad%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C637599779 > > > > > +42 > > > > > +1910 > > > > > +039%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzI > > > > > +iL > > > > > +CJBT > > > > > +iI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Jcf6Om4DehifCe1KO1rm > > > > > +t5 > > > > > +LxTB > > > > > +6jtGoQLD1MoqWGM%2F0%3D&reserved=0 > > > > > +$schema: > > > > > +https://jpn01.safelinks.protection.outlook.com/?url=http%3A%2F% > > > > > +2F > > > > > +devi > > > > > +cetree.org%2Fmeta- > > schemas%2Fcore.yaml%23&data=04%7C01%7Cbiju.das. > > > > > +jz%40bp.renesas.com%7Cc6bbf5f6ce334eaa722a08d9359f07ad%7C53d825 > > > > > +71 > > > > > +da19 > > > > > +47e49cb4625a166a4a2a%7C0%7C0%7C637599779421910039%7CUnknown%7CT > > > > > +WF > > > > > +pbGZ > > > > > +sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC > > > > > +I6 > > > > > +Mn0% > > > > > +3D%7C1000&sdata=LlqPRLf9%2BGrEdSapxCFhwxVKcXTVh9ECr%2FXPN0S > > > > > +Iz > > > > > +i4%3 > > > > > +D&reserved=0 > > > > > + > > > > > +title: Renesas RZ/G2L USB2.0 PHY Control > > > > > + > > > > > +maintainers: > > > > > + - Biju Das <biju.das.jz@bp.renesas.com> > > > > > + > > > > > +description: > > > > > + The RZ/G2L USB2.0 PHY Control mainly controls reset and power > > > > > +down of the > > > > > + USB/PHY. > > > > > + > > > > > +properties: > > > > > + compatible: > > > > > + items: > > > > > + - enum: > > > > > + - renesas,r9a07g044-usbphyctrl # RZ/G2{L,LC} > > > > > + - const: renesas,rzg2l-usbphyctrl > > > > > + > > > > > + reg: > > > > > + maxItems: 1 > > > > > + > > > > > + clocks: > > > > > + maxItems: 1 > > > > > + > > > > > + resets: > > > > > + maxItems: 1 > > > > > + > > > > > + power-domains: > > > > > + maxItems: 1 > > > > > + > > > > > + '#phy-cells': > > > > > + # see phy-bindings.txt in the same directory > > > > > + const: 1 > > > > > + description: | > > > > > + The phandle's argument in the PHY specifier is the phy > > > > > + reset > > > > control bit > > > > > + of usb phy control. > > > > > + 0 = Port 1 Phy reset > > > > > + 1 = Port 2 Phy reset > > > > > + enum: [ 0, 1 ] > > > > > > > > You already have the const, so this doesn't do anything. > > > > > > OK, will take out const. > > > > No, 'const' is correct. This is the value of '#phy-cells', not the > > contents (we don't have a way to express schema for that). > > OK. > > > > > > > > +required: > > > > > + - compatible > > > > > + - reg > > > > > + - clocks > > > > > + - '#phy-cells' > > > > > + > > > > > +additionalProperties: false > > > > > + > > > > > +examples: > > > > > + - | > > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h> > > > > > + > > > > > + usbphyctrl@11c40000 { > > > > > > > > usb-phy@... > > > > > > The IP is called USBPHY control. It mainly controls reset and power > > > down > > of the USB2.0/PHY. > > > > Sounds like it should be using the reset binding... OK, Will model this as a reset binding. Since the IP mainly controls reset and power down Of the USB2.0/PHY. So it is better to have reset binding. Regards, Biju > This IP has reset, clock control , connection control , clock status and > power down setting registers. > Currenty we are using reset registers for turning ON USB/PHY block. > > Since it has extra registers I thought of modelling it as a phy device. > But we could model as reset device as well. > But it has extra functionalities apart from reset. > > So what do you propose here? Model as a reset device or phy device since > it is related to phy? > Please share your opinion on this. > > Regards, > Biju > > > > > > > So not sure usb-phy is right one here ? I prefer usb-phy-ctrl instead. > > Is it ok? Please let me know. > > > > A node with #phy-cells should use the standard phy node names unless > > it has other controls. > > Apart from reset, it has other controls like clock control , connection > control , clock status and powerdown setting registers. > > Cheers, > Biju > > As I said, this doesn't seem to be a phy, so using > > #phy-cells here is what seems wrong. > > > > > > > + compatible = "renesas,r9a07g044-usbphyctrl", > > > > > + "renesas,rzg2l-usbphyctrl"; > > > > > + reg = <0x11c40000 0x10000>; > > > > > + clocks = <&cpg CPG_MOD R9A07G044_USB_PCLK>; > > > > > + resets = <&cpg R9A07G044_USB_PCLK>; > > > > > + power-domains = <&cpg>; > > > > Also, are these all resources of the usbphyctrl block and not just > > resources you happen to want in the driver? For example, the > > power-domain should be the power island that this block resides in. > > > > > > > + #phy-cells = <1>; > > > > > + }; > > > > > -- > > > > > 2.17.1 > > > > > > > > > >
WARNING: multiple messages have this Message-ID (diff)
From: Biju Das <biju.das.jz@bp.renesas.com> To: Rob Herring <robh@kernel.org> Cc: Kishon Vijay Abraham I <kishon@ti.com>, Vinod Koul <vkoul@kernel.org>, Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>, "linux-phy@lists.infradead.org" <linux-phy@lists.infradead.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Geert Uytterhoeven <geert+renesas@glider.be>, Chris Paterson <Chris.Paterson2@renesas.com>, Biju Das <biju.das@bp.renesas.com>, Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>, "linux-renesas-soc@vger.kernel.org" <linux-renesas-soc@vger.kernel.org> Subject: RE: [PATCH v2 01/11] dt-bindings: phy: renesas: Document RZ/G2L USB PHY Control bindings Date: Fri, 25 Jun 2021 10:11:58 +0000 [thread overview] Message-ID: <OS0PR01MB59227E0C129668A5FFF861E186069@OS0PR01MB5922.jpnprd01.prod.outlook.com> (raw) In-Reply-To: <OS0PR01MB5922742488532EC506334AC786089@OS0PR01MB5922.jpnprd01.prod.outlook.com> Hi Rob, > Subject: RE: [PATCH v2 01/11] dt-bindings: phy: renesas: Document RZ/G2L > USB PHY Control bindings > > Hi Rob, > > Thanks for the feedback. > > > Subject: Re: [PATCH v2 01/11] dt-bindings: phy: renesas: Document > > RZ/G2L USB PHY Control bindings > > > > On Wed, Jun 23, 2021 at 7:38 AM Biju Das <biju.das.jz@bp.renesas.com> > > wrote: > > > > > > Hi Rob, > > > > > > Thanks for the feedback. > > > > > > > Subject: Re: [PATCH v2 01/11] dt-bindings: phy: renesas: Document > > > > RZ/G2L USB PHY Control bindings > > > > > > > > On Mon, Jun 21, 2021 at 10:39:33AM +0100, Biju Das wrote: > > > > > Add device tree binding document for RZ/G2L USB PHY control > driver. > > > > > > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > > > > Reviewed-by: Lad Prabhakar > > > > > <prabhakar.mahadev-lad.rj@bp.renesas.com> > > > > > --- > > > > > V1->V2: > > > > > * Add clock properties > > > > > --- > > > > > .../phy/renesas,rzg2l-usbphyctrl.yaml | 65 > > +++++++++++++++++++ > > > > > 1 file changed, 65 insertions(+) create mode 100644 > > > > > Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl.y > > > > > am > > > > > l > > > > > > > > > > diff --git > > > > > a/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl > > > > > .y > > > > > aml > > > > > b/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphyctrl > > > > > .y > > > > > aml > > > > > new file mode 100644 > > > > > index 000000000000..8e8ba43f595d > > > > > --- /dev/null > > > > > +++ b/Documentation/devicetree/bindings/phy/renesas,rzg2l-usbphy > > > > > +++ ct > > > > > +++ rl.y > > > > > +++ aml > > > > > @@ -0,0 +1,65 @@ > > > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML > > > > > +1.2 > > > > > +--- > > > > > +$id: > > > > > +https://jpn01.safelinks.protection.outlook.com/?url=http%3A%2F% > > > > > +2F > > > > > +devi > > > > > +cetree.org%2Fschemas%2Fphy%2Frenesas%2Crzg2l-usbphyctrl.yaml%23 > > > > > +&a > > > > > +mp;d > > > > > +ata=04%7C01%7Cbiju.das.jz%40bp.renesas.com%7Cc6bbf5f6ce334eaa72 > > > > > +2a > > > > > +08d9 > > > > > +359f07ad%7C53d82571da1947e49cb4625a166a4a2a%7C0%7C0%7C637599779 > > > > > +42 > > > > > +1910 > > > > > +039%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzI > > > > > +iL > > > > > +CJBT > > > > > +iI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=Jcf6Om4DehifCe1KO1rm > > > > > +t5 > > > > > +LxTB > > > > > +6jtGoQLD1MoqWGM%2F0%3D&reserved=0 > > > > > +$schema: > > > > > +https://jpn01.safelinks.protection.outlook.com/?url=http%3A%2F% > > > > > +2F > > > > > +devi > > > > > +cetree.org%2Fmeta- > > schemas%2Fcore.yaml%23&data=04%7C01%7Cbiju.das. > > > > > +jz%40bp.renesas.com%7Cc6bbf5f6ce334eaa722a08d9359f07ad%7C53d825 > > > > > +71 > > > > > +da19 > > > > > +47e49cb4625a166a4a2a%7C0%7C0%7C637599779421910039%7CUnknown%7CT > > > > > +WF > > > > > +pbGZ > > > > > +sb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVC > > > > > +I6 > > > > > +Mn0% > > > > > +3D%7C1000&sdata=LlqPRLf9%2BGrEdSapxCFhwxVKcXTVh9ECr%2FXPN0S > > > > > +Iz > > > > > +i4%3 > > > > > +D&reserved=0 > > > > > + > > > > > +title: Renesas RZ/G2L USB2.0 PHY Control > > > > > + > > > > > +maintainers: > > > > > + - Biju Das <biju.das.jz@bp.renesas.com> > > > > > + > > > > > +description: > > > > > + The RZ/G2L USB2.0 PHY Control mainly controls reset and power > > > > > +down of the > > > > > + USB/PHY. > > > > > + > > > > > +properties: > > > > > + compatible: > > > > > + items: > > > > > + - enum: > > > > > + - renesas,r9a07g044-usbphyctrl # RZ/G2{L,LC} > > > > > + - const: renesas,rzg2l-usbphyctrl > > > > > + > > > > > + reg: > > > > > + maxItems: 1 > > > > > + > > > > > + clocks: > > > > > + maxItems: 1 > > > > > + > > > > > + resets: > > > > > + maxItems: 1 > > > > > + > > > > > + power-domains: > > > > > + maxItems: 1 > > > > > + > > > > > + '#phy-cells': > > > > > + # see phy-bindings.txt in the same directory > > > > > + const: 1 > > > > > + description: | > > > > > + The phandle's argument in the PHY specifier is the phy > > > > > + reset > > > > control bit > > > > > + of usb phy control. > > > > > + 0 = Port 1 Phy reset > > > > > + 1 = Port 2 Phy reset > > > > > + enum: [ 0, 1 ] > > > > > > > > You already have the const, so this doesn't do anything. > > > > > > OK, will take out const. > > > > No, 'const' is correct. This is the value of '#phy-cells', not the > > contents (we don't have a way to express schema for that). > > OK. > > > > > > > > +required: > > > > > + - compatible > > > > > + - reg > > > > > + - clocks > > > > > + - '#phy-cells' > > > > > + > > > > > +additionalProperties: false > > > > > + > > > > > +examples: > > > > > + - | > > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h> > > > > > + > > > > > + usbphyctrl@11c40000 { > > > > > > > > usb-phy@... > > > > > > The IP is called USBPHY control. It mainly controls reset and power > > > down > > of the USB2.0/PHY. > > > > Sounds like it should be using the reset binding... OK, Will model this as a reset binding. Since the IP mainly controls reset and power down Of the USB2.0/PHY. So it is better to have reset binding. Regards, Biju > This IP has reset, clock control , connection control , clock status and > power down setting registers. > Currenty we are using reset registers for turning ON USB/PHY block. > > Since it has extra registers I thought of modelling it as a phy device. > But we could model as reset device as well. > But it has extra functionalities apart from reset. > > So what do you propose here? Model as a reset device or phy device since > it is related to phy? > Please share your opinion on this. > > Regards, > Biju > > > > > > > So not sure usb-phy is right one here ? I prefer usb-phy-ctrl instead. > > Is it ok? Please let me know. > > > > A node with #phy-cells should use the standard phy node names unless > > it has other controls. > > Apart from reset, it has other controls like clock control , connection > control , clock status and powerdown setting registers. > > Cheers, > Biju > > As I said, this doesn't seem to be a phy, so using > > #phy-cells here is what seems wrong. > > > > > > > + compatible = "renesas,r9a07g044-usbphyctrl", > > > > > + "renesas,rzg2l-usbphyctrl"; > > > > > + reg = <0x11c40000 0x10000>; > > > > > + clocks = <&cpg CPG_MOD R9A07G044_USB_PCLK>; > > > > > + resets = <&cpg R9A07G044_USB_PCLK>; > > > > > + power-domains = <&cpg>; > > > > Also, are these all resources of the usbphyctrl block and not just > > resources you happen to want in the driver? For example, the > > power-domain should be the power island that this block resides in. > > > > > > > + #phy-cells = <1>; > > > > > + }; > > > > > -- > > > > > 2.17.1 > > > > > > > > > > -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy
next prev parent reply other threads:[~2021-06-25 10:12 UTC|newest] Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-06-21 9:39 [PATCH v2 00/11] Add USB2.0 support Biju Das 2021-06-21 9:39 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 01/11] dt-bindings: phy: renesas: Document RZ/G2L USB PHY Control bindings Biju Das 2021-06-21 9:39 ` Biju Das 2021-06-21 17:40 ` Rob Herring 2021-06-21 17:40 ` Rob Herring 2021-06-22 16:58 ` Rob Herring 2021-06-22 16:58 ` Rob Herring 2021-06-23 13:38 ` Biju Das 2021-06-23 13:38 ` Biju Das 2021-06-23 14:13 ` Rob Herring 2021-06-23 14:13 ` Rob Herring 2021-06-23 14:29 ` Geert Uytterhoeven 2021-06-23 14:29 ` Geert Uytterhoeven 2021-06-23 15:45 ` Biju Das 2021-06-23 15:45 ` Biju Das 2021-06-25 10:11 ` Biju Das [this message] 2021-06-25 10:11 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 02/11] drivers: clk: renesas: r9a07g044-cpg: Add USB clocks Biju Das 2021-06-21 9:39 ` [PATCH v2 03/11] phy: renesas: Add RZ/G2L usb phy control driver Biju Das 2021-06-21 9:39 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 04/11] arm64: configs: defconfig: Enable RZ/G2L USB PHY " Biju Das 2021-06-21 9:39 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 05/11] dt-bindings: phy: renesas,usb2-phy: Document RZ/G2L phy bindings Biju Das 2021-06-21 9:39 ` [PATCH v2 05/11] dt-bindings: phy: renesas, usb2-phy: " Biju Das 2021-06-22 15:33 ` [PATCH v2 05/11] dt-bindings: phy: renesas,usb2-phy: " Geert Uytterhoeven 2021-06-22 15:33 ` Geert Uytterhoeven 2021-06-21 9:39 ` [PATCH v2 06/11] dt-bindings: usb: generic-ohci: Document RZ/G2L SoC bindings Biju Das 2021-06-21 17:40 ` Rob Herring 2021-06-22 10:03 ` Biju Das 2021-06-22 19:30 ` Rob Herring 2021-06-23 14:20 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 07/11] dt-bindings: usb: generic-ehci: " Biju Das 2021-06-21 17:40 ` Rob Herring 2021-06-22 10:04 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 08/11] arm64: dts: renesas: r9a07g044: Add USB2.0 phy and host support Biju Das 2021-06-21 9:39 ` [PATCH v2 09/11] dt-bindings: usb: renesas,usbhs: Document RZ/G2L bindings Biju Das 2021-06-21 9:39 ` [PATCH v2 10/11] phy: renesas: phy-rcar-gen3-usb2: Add OTG support for RZ/G2L Biju Das 2021-06-21 9:39 ` Biju Das 2021-06-22 15:29 ` Geert Uytterhoeven 2021-06-22 15:29 ` Geert Uytterhoeven 2021-06-23 7:04 ` Yoshihiro Shimoda 2021-06-23 7:04 ` Yoshihiro Shimoda 2021-06-23 11:52 ` Biju Das 2021-06-23 11:52 ` Biju Das 2021-06-23 8:02 ` Yoshihiro Shimoda 2021-06-23 8:02 ` Yoshihiro Shimoda 2021-06-23 11:54 ` Biju Das 2021-06-23 11:54 ` Biju Das 2021-06-21 9:39 ` [PATCH v2 11/11] arm64: dts: renesas: r9a07g044: Add USB2.0 device support Biju Das
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=OS0PR01MB59227E0C129668A5FFF861E186069@OS0PR01MB5922.jpnprd01.prod.outlook.com \ --to=biju.das.jz@bp.renesas.com \ --cc=Chris.Paterson2@renesas.com \ --cc=biju.das@bp.renesas.com \ --cc=devicetree@vger.kernel.org \ --cc=geert+renesas@glider.be \ --cc=kishon@ti.com \ --cc=linux-phy@lists.infradead.org \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=prabhakar.mahadev-lad.rj@bp.renesas.com \ --cc=robh@kernel.org \ --cc=vkoul@kernel.org \ --cc=yoshihiro.shimoda.uh@renesas.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.