Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1590901 > unrolled thread

[PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf

Started byMeng Dongyang <daniel.meng@rock-chips.com>
First post2017-03-02 09:00 +0100
Last post2017-03-03 08:30 +0100
Articles 4 — 4 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf Meng Dongyang <daniel.meng@rock-chips.com> - 2017-03-02 09:00 +0100
    Re: [PATCH v2 2/3] Documentation: bindings: add DT documentation for  u2phy and u2phy grf Rob Herring <robh@kernel.org> - 2017-03-03 08:00 +0100
      Re: [PATCH v2 2/3] Documentation: bindings: add DT documentation for  u2phy and u2phy grf "rock-chips(daniel.meng)" <daniel.meng@rock-chips.com> - 2017-03-03 08:00 +0100
      Re: [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf Heiko Stuebner <heiko@sntech.de> - 2017-03-03 08:30 +0100

#1590901 — [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf

FromMeng Dongyang <daniel.meng@rock-chips.com>
Date2017-03-02 09:00 +0100
Subject[PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf
Message-ID<tgtOx-7vk-9@gated-at.bofh.it>
Due to the u2phy registers are separated from general grf, we need to
add u2phy grf node and place u2phy node in it. So this patch add u2phy
grf node.

Changes in v2:
 - add u2phy grf specification

Signed-off-by: Meng Dongyang <daniel.meng@rock-chips.com>
---
 Documentation/devicetree/bindings/soc/rockchip/grf.txt | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/Documentation/devicetree/bindings/soc/rockchip/grf.txt b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
index 013e71a..6ac23f9 100644
--- a/Documentation/devicetree/bindings/soc/rockchip/grf.txt
+++ b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
@@ -7,6 +7,9 @@ From RK3368 SoCs, the GRF is divided into two sections,
 - GRF, used for general non-secure system,
 - PMUGRF, used for always on system
 
+On RK3328 SoCs, the GRF is divided into three sections,
+except GRF and PMUGRF, add a section for USB2PHYGRF,
+
 Required Properties:
 
 - compatible: GRF should be one of the followings
@@ -19,6 +22,8 @@ Required Properties:
 - compatible: PMUGRF should be one of the followings
    - "rockchip,rk3368-pmugrf", "syscon": for rk3368
    - "rockchip,rk3399-pmugrf", "syscon": for rk3399
+- compatible: USB2GRF should be one of the followings
+   - "rockchip,rk3328-usb2phy-grf", "syscon": for rk3328
 - reg: physical base address of the controller and length of memory mapped
   region.
 
@@ -33,3 +38,13 @@ Example: GRF and PMUGRF of RK3399 SoCs
 		compatible = "rockchip,rk3399-grf", "syscon";
 		reg = <0x0 0xff770000 0x0 0x10000>;
 	};
+
+Example: USB2GRF of RK3328 SoCs
+
+	usb2phy_grf: syscon@ff450000 {
+		compatible = "rockchip,rk3328-usb2phy-grf", "syscon",
+			     "simple-mfd";
+		reg = <0x0 0xff450000 0x0 0x10000>;
+		#address-cells = <1>;
+		#size-cells = <1>;
+	};
-- 
1.9.1

[toc] | [next] | [standalone]

#1591696 — Re: [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf

FromRob Herring <robh@kernel.org>
Date2017-03-03 08:00 +0100
SubjectRe: [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf
Message-ID<tgPm3-5Hb-23@gated-at.bofh.it>
In reply to#1590901
On Thu, Mar 02, 2017 at 03:49:04PM +0800, Meng Dongyang wrote:
> Due to the u2phy registers are separated from general grf, we need to
> add u2phy grf node and place u2phy node in it. So this patch add u2phy
> grf node.

Similar comment on the subject.

> 
> Changes in v2:
>  - add u2phy grf specification
> 
> Signed-off-by: Meng Dongyang <daniel.meng@rock-chips.com>
> ---
>  Documentation/devicetree/bindings/soc/rockchip/grf.txt | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/soc/rockchip/grf.txt b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> index 013e71a..6ac23f9 100644
> --- a/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> +++ b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> @@ -7,6 +7,9 @@ From RK3368 SoCs, the GRF is divided into two sections,
>  - GRF, used for general non-secure system,
>  - PMUGRF, used for always on system
>  
> +On RK3328 SoCs, the GRF is divided into three sections,
> +except GRF and PMUGRF, add a section for USB2PHYGRF,
> +
>  Required Properties:
>  
>  - compatible: GRF should be one of the followings
> @@ -19,6 +22,8 @@ Required Properties:
>  - compatible: PMUGRF should be one of the followings
>     - "rockchip,rk3368-pmugrf", "syscon": for rk3368
>     - "rockchip,rk3399-pmugrf", "syscon": for rk3399
> +- compatible: USB2GRF should be one of the followings
> +   - "rockchip,rk3328-usb2phy-grf", "syscon": for rk3328
>  - reg: physical base address of the controller and length of memory mapped
>    region.
>  
> @@ -33,3 +38,13 @@ Example: GRF and PMUGRF of RK3399 SoCs
>  		compatible = "rockchip,rk3399-grf", "syscon";
>  		reg = <0x0 0xff770000 0x0 0x10000>;
>  	};
> +
> +Example: USB2GRF of RK3328 SoCs
> +
> +	usb2phy_grf: syscon@ff450000 {
> +		compatible = "rockchip,rk3328-usb2phy-grf", "syscon",
> +			     "simple-mfd";

simple-mfd is not documented. Do you have child nodes?

What makes this a syscon? It has non-PHY registers?


> +		reg = <0x0 0xff450000 0x0 0x10000>;
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +	};
> -- 
> 1.9.1
> 
>

[toc] | [prev] | [next] | [standalone]

#1591697 — Re: [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf

From"rock-chips(daniel.meng)" <daniel.meng@rock-chips.com>
Date2017-03-03 08:00 +0100
SubjectRe: [PATCH v2 2/3] Documentation: bindings: add DT documentation for u2phy and u2phy grf
Message-ID<tgPm3-5Hb-25@gated-at.bofh.it>
In reply to#1591696

On 2017/3/3 14:21, Rob Herring wrote:
> On Thu, Mar 02, 2017 at 03:49:04PM +0800, Meng Dongyang wrote:
>> Due to the u2phy registers are separated from general grf, we need to
>> add u2phy grf node and place u2phy node in it. So this patch add u2phy
>> grf node.
> Similar comment on the subject.
>
>> Changes in v2:
>>   - add u2phy grf specification
>>
>> Signed-off-by: Meng Dongyang <daniel.meng@rock-chips.com>
>> ---
>>   Documentation/devicetree/bindings/soc/rockchip/grf.txt | 15 +++++++++++++++
>>   1 file changed, 15 insertions(+)
>>
>> diff --git a/Documentation/devicetree/bindings/soc/rockchip/grf.txt b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
>> index 013e71a..6ac23f9 100644
>> --- a/Documentation/devicetree/bindings/soc/rockchip/grf.txt
>> +++ b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
>> @@ -7,6 +7,9 @@ From RK3368 SoCs, the GRF is divided into two sections,
>>   - GRF, used for general non-secure system,
>>   - PMUGRF, used for always on system
>>   
>> +On RK3328 SoCs, the GRF is divided into three sections,
>> +except GRF and PMUGRF, add a section for USB2PHYGRF,
>> +
>>   Required Properties:
>>   
>>   - compatible: GRF should be one of the followings
>> @@ -19,6 +22,8 @@ Required Properties:
>>   - compatible: PMUGRF should be one of the followings
>>      - "rockchip,rk3368-pmugrf", "syscon": for rk3368
>>      - "rockchip,rk3399-pmugrf", "syscon": for rk3399
>> +- compatible: USB2GRF should be one of the followings
>> +   - "rockchip,rk3328-usb2phy-grf", "syscon": for rk3328
>>   - reg: physical base address of the controller and length of memory mapped
>>     region.
>>   
>> @@ -33,3 +38,13 @@ Example: GRF and PMUGRF of RK3399 SoCs
>>   		compatible = "rockchip,rk3399-grf", "syscon";
>>   		reg = <0x0 0xff770000 0x0 0x10000>;
>>   	};
>> +
>> +Example: USB2GRF of RK3328 SoCs
>> +
>> +	usb2phy_grf: syscon@ff450000 {
>> +		compatible = "rockchip,rk3328-usb2phy-grf", "syscon",
>> +			     "simple-mfd";
> simple-mfd is not documented. Do you have child nodes?
do you mean we need to add a child node of phy here?
>
> What makes this a syscon? It has non-PHY registers?
rk3328 devides grf into three parts and we put all u2phy registers into 
the part of u2phy grf,
so here we add a simple usb2phy_grf.
>
>
>> +		reg = <0x0 0xff450000 0x0 0x10000>;
>> +		#address-cells = <1>;
>> +		#size-cells = <1>;
>> +	};
>> -- 
>> 1.9.1
>>
>>
>
>

[toc] | [prev] | [next] | [standalone]

#1591718

FromHeiko Stuebner <heiko@sntech.de>
Date2017-03-03 08:30 +0100
Message-ID<tgPP4-66b-5@gated-at.bofh.it>
In reply to#1591696
Am Freitag, 3. März 2017, 00:21:56 CET schrieb Rob Herring:
> On Thu, Mar 02, 2017 at 03:49:04PM +0800, Meng Dongyang wrote:
> > Due to the u2phy registers are separated from general grf, we need to
> > add u2phy grf node and place u2phy node in it. So this patch add u2phy
> > grf node.
> 
> Similar comment on the subject.
> 
> > Changes in v2:
> >  - add u2phy grf specification
> > 
> > Signed-off-by: Meng Dongyang <daniel.meng@rock-chips.com>
> > ---
> > 
> >  Documentation/devicetree/bindings/soc/rockchip/grf.txt | 15
> >  +++++++++++++++
> >  1 file changed, 15 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> > b/Documentation/devicetree/bindings/soc/rockchip/grf.txt index
> > 013e71a..6ac23f9 100644
> > --- a/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> > +++ b/Documentation/devicetree/bindings/soc/rockchip/grf.txt
> > @@ -7,6 +7,9 @@ From RK3368 SoCs, the GRF is divided into two sections,
> > 
> >  - GRF, used for general non-secure system,
> >  - PMUGRF, used for always on system
> > 
> > +On RK3328 SoCs, the GRF is divided into three sections,
> > +except GRF and PMUGRF, add a section for USB2PHYGRF,
> > +
> > 
> >  Required Properties:
> >  
> >  - compatible: GRF should be one of the followings
> > 
> > @@ -19,6 +22,8 @@ Required Properties:
> >  - compatible: PMUGRF should be one of the followings
> >  
> >     - "rockchip,rk3368-pmugrf", "syscon": for rk3368
> >     - "rockchip,rk3399-pmugrf", "syscon": for rk3399
> > 
> > +- compatible: USB2GRF should be one of the followings
> > +   - "rockchip,rk3328-usb2phy-grf", "syscon": for rk3328
> > 
> >  - reg: physical base address of the controller and length of memory
> >  mapped
> >  
> >    region.
> > 
> > @@ -33,3 +38,13 @@ Example: GRF and PMUGRF of RK3399 SoCs
> > 
> >  		compatible = "rockchip,rk3399-grf", "syscon";
> >  		reg = <0x0 0xff770000 0x0 0x10000>;
> >  	
> >  	};
> > 
> > +
> > +Example: USB2GRF of RK3328 SoCs
> > +
> > +	usb2phy_grf: syscon@ff450000 {
> > +		compatible = "rockchip,rk3328-usb2phy-grf", "syscon",
> > +			     "simple-mfd";
> 
> simple-mfd is not documented. Do you have child nodes?

Yes, this will have child nodes and I guess we don't really need the 
additional example, that only differs in the compatible.


> What makes this a syscon? It has non-PHY registers?

It is still a "General Register Files" block in all of the socs manual
and yes, that special area also contains registers not used by the phy driver.


Heiko

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web