From: Jianqun Xu <jay.xu@rock-chips.com>
To: Heiko Stuebner <heiko@sntech.de>
Cc: Rob Herring <robh@kernel.org>,
pawel.moll@arm.com, mark.rutland@arm.com,
ijc+devicetree@hellion.org.uk, galak@codeaurora.org,
jwerner@chromium.org, broonie@kernel.org,
catalin.marinas@arm.com, will.deacon@arm.com,
sboyd@codeaurora.org, linus.walleij@linaro.org,
sjoerd.simons@collabora.co.uk, huangtao@rock-chips.com,
linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
Xing Zheng <zhengxing@rock-chips.com>
Subject: Re: [PATCH 5/6] dt-bindings: add documentation of rk3399 clock controller
Date: Fri, 19 Feb 2016 09:05:13 +0800 [thread overview]
Message-ID: <56C66A49.2080501@rock-chips.com> (raw)
In-Reply-To: <1758156.nKISImzkM9@phil>
Hi Heiko
在 19/02/2016 08:53, Heiko Stuebner 写道:
> Am Freitag, 19. Februar 2016, 08:48:15 schrieb Jianqun Xu:
>> Hi Rob
>>
>> 在 18/02/2016 22:36, Rob Herring 写道:
>>> On Wed, Feb 17, 2016 at 09:54:49AM +0800, jianqun.xu wrote:
>>>> From: Xing Zheng <zhengxing@rock-chips.com>
>>>>
>>>> Add the devicetree binding for the cru on the rk3399 which quite
>>>> similar structured as previous clock controllers.
>>>>
>>>> Signed-off-by: Xing Zheng <zhengxing@rock-chips.com>
>>>> ---
>>>>
>>>> .../bindings/clock/rockchip,rk3399-cru.txt | 82
>>>> ++++++++++++++++++++++ 1 file changed, 82 insertions(+)
>>>> create mode 100644
>>>> Documentation/devicetree/bindings/clock/rockchip,rk3399-cru.txt>>
>>>> diff --git
>>>> a/Documentation/devicetree/bindings/clock/rockchip,rk3399-cru.txt
>>>> b/Documentation/devicetree/bindings/clock/rockchip,rk3399-cru.txt new
>>>> file mode 100644
>>>> index 0000000..07bcc6e
>>>> --- /dev/null
>>>> +++ b/Documentation/devicetree/bindings/clock/rockchip,rk3399-cru.txt
>>>> @@ -0,0 +1,82 @@
>>>> +* Rockchip RK3399 Clock and Reset Unit
>>>
>>> [...]
>>>
>>>> +Example: General Register Files
>>>> +
>>>> + pmugrf: syscon@ff320000 {
>>>> + compatible = "rockchip,rk3399-pmugrf", "syscon";
>>>
>>> Is this documented?
>>
>> Since there is no documentation for rockchip grf from early SoCs, I
>> think if we need to add a new devicetree documentation, such as
>> Documentation/devicetree/bindings/clock/rockchip,rk3399-grf.txt ?
>
> The grf is not clock-specific, so the directory is probably not correct.
> I'd think bindings/arm/rockchip/grf.txt might be better or
> bindings/soc/rockchip/grf.txt or somewhere else. And define the properties
> for all per-soc GRFs in there.
>
> In any case we really should add the missing grf binding documentation (in a
> separate patch of course), as the grf is a pretty commonly used shared
> component.
>
Got it, I will add bindings/soc/rockchip/grf.txt in next patch
Thanks !
>
>>
>> If you agree, I will add it in next patch
>>
>>>> + reg = <0x0 0xff320000 0x0 0x1000>;
>>>> + };
>>>> +
>>>> + grf: syscon@ff770000 {
>>>> + compatible = "rockchip,rk3399-grf", "syscon";
>>>
>>> ditto.
>>>
>>>> + reg = <0x0 0xff770000 0x0 0x10000>;
>>>> + };
>>>> +
>>>> +Example: Clock controller node:
>>>> +
>>>> + pmucru: pmu-clock-controller@ff750000 {
>>>> + compatible = "rockchip,rk3399-pmucru";
>>>> + reg = <0x0 0xff750000 0x0 0x1000>;
>>>> + #clock-cells = <1>;
>>>> + #reset-cells = <1>;
>>>> + };
>>>> +
>>>> + cru: clock-controller@ff760000 {
>>>> + compatible = "rockchip,rk3399-cru";
>>>> + reg = <0x0 0xff760000 0x0 0x1000>;
>>>> + rockchip,grf = <&grf>;
>>>> + #clock-cells = <1>;
>>>> + #reset-cells = <1>;
>>>> + };
>>>> +
>>>> +Example: UART controller node that consumes the clock generated by the
>>>> clock + controller:
>>>> +
>>>> + uart0: serial@ff1a0000 {
>>>> + compatible = "rockchip,rk3399-uart", "snps,dw-apb-uart";
>>>> + reg = <0x0 0xff180000 0x0 0x100>;
>>>> + clocks = <&cru SCLK_UART0>, <&cru PCLK_UART0>;
>>>> + clock-names = "baudclk", "apb_pclk";
>>>> + interrupts = <GIC_SPI 99 IRQ_TYPE_LEVEL_HIGH>;
>>>> + reg-shift = <2>;
>>>> + reg-io-width = <4>;
>>>> + };
>>>> --
>>>> 1.9.1
>
>
>
>
next prev parent reply other threads:[~2016-02-19 1:05 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-17 1:53 [PATCH 0/6] Add core dtsi for rk3399 from Rockchip jianqun.xu
2016-02-17 1:53 ` [PATCH 1/6] clk: rockchip: add dt-binding header for rk3399 jianqun.xu
2016-02-17 1:53 ` [PATCH 2/6] spi: rockchip: add bindings for rk3399 spi jianqun.xu
2016-02-18 14:37 ` Rob Herring
2016-02-17 1:53 ` [PATCH 3/6] ASoC: rockchip: add bindings for rk3399 i2s jianqun.xu
2016-02-17 11:03 ` Mark Brown
2016-02-18 14:37 ` Rob Herring
2016-02-18 14:59 ` Mark Brown
2016-02-18 14:36 ` Rob Herring
2016-02-17 1:53 ` [PATCH 4/6] pinctrl: rockchip: add bindings for rk3399 pinctrl jianqun.xu
2016-02-17 6:47 ` Heiko Stuebner
2016-02-17 7:23 ` Jianqun Xu
2016-02-17 1:54 ` [PATCH 5/6] dt-bindings: add documentation of rk3399 clock controller jianqun.xu
2016-02-18 14:36 ` Rob Herring
2016-02-19 0:48 ` Jianqun Xu
2016-02-19 0:53 ` Heiko Stuebner
2016-02-19 1:05 ` Jianqun Xu [this message]
2016-02-17 2:01 ` [PATCH 6/6] ARM64: dts: rockchip: add core dtsi file for rk3399 jianqun.xu
2016-02-17 7:00 ` Heiko Stuebner
2016-02-18 1:43 ` Jianqun Xu
2016-02-17 11:27 ` Marc Zyngier
2016-02-17 11:46 ` Mark Rutland
2016-02-18 1:07 ` Jianqun Xu
2016-02-17 2:04 ` [PATCH 1/6] clk: rockchip: add dt-binding header " jianqun.xu
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=56C66A49.2080501@rock-chips.com \
--to=jay.xu@rock-chips.com \
--cc=broonie@kernel.org \
--cc=catalin.marinas@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=heiko@sntech.de \
--cc=huangtao@rock-chips.com \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jwerner@chromium.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=pawel.moll@arm.com \
--cc=robh@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=sjoerd.simons@collabora.co.uk \
--cc=will.deacon@arm.com \
--cc=zhengxing@rock-chips.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).