[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <56DFE35F.3010400@rock-chips.com>
Date: Wed, 09 Mar 2016 16:48:31 +0800
From: Elaine Zhang <zhangqing@...k-chips.com>
To: Caesar Wang <caesar.upstream@...il.com>, wxt@...k-chips.com
CC: heiko@...ech.de, khilman@...libre.com, huangtao@...k-chips.com,
xxx@...k-chips.com, linux-kernel@...r.kernel.org,
linux-rockchip@...ts.infradead.org, zyw@...k-chips.com,
jay.xu@...k-chips.com, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v5 5/6] dt/bindings: rockchip: modify document of Rockchip
power domains
On 03/08/2016 07:09 PM, Caesar Wang wrote:
>
>
> 在 2016年03月03日 16:09, Elaine Zhang 写道:
>> Add binding documentation for the power domains
>> found on Rockchip RK3399 SoCs.
>> RK3399 pd on/off not need to enable clk which in this pd.
>> So remove the clocks in the rk3399 pd example.
>>
>> Signed-off-by: Elaine Zhang <zhangqing@...k-chips.com>
>> ---
>> .../bindings/soc/rockchip/power_domain.txt | 37
>> ++++++++++++++++++++++
>> 1 file changed, 37 insertions(+)
>>
>> diff --git
>> a/Documentation/devicetree/bindings/soc/rockchip/power_domain.txt
>> b/Documentation/devicetree/bindings/soc/rockchip/power_domain.txt
>> index 13dc6a3..98085c8 100644
>> --- a/Documentation/devicetree/bindings/soc/rockchip/power_domain.txt
>> +++ b/Documentation/devicetree/bindings/soc/rockchip/power_domain.txt
>> @@ -7,6 +7,7 @@ Required properties for power domain controller:
>> - compatible: Should be one of the following.
> [....]
>> Example of the node using power domain:
>> @@ -65,3 +96,9 @@ Example of the node using power domain:
>> power-domains = <&power RK3368_PD_GPU_1>;
>> /* ... */
>> };
>> +
>> + node {
>> + /* ... */
>> + power-domains = <&power RK3399_PD_VOPB>;
>> + /* ... */
>> + };
>
> Trivial: typo
> power: power-controller {
> @@ -92,10 +92,10 @@ Example of the node using power domain:
> };
>
> node {
> - /* ... */
> - power-domains = <&power RK3368_PD_GPU_1>;
> - /* ... */
> - };
> + /* ... */
> + power-domains = <&power RK3368_PD_GPU_1>;
> + /* ... */
> + };
This is a example.I think there is no problem.
>
>
>
>
>
Powered by blists - more mailing lists