From: Rob Herring <robh@kernel.org> To: Gregory CLEMENT <gregory.clement@bootlin.com> Cc: "Stephen Boyd" <sboyd@kernel.org>, "Mike Turquette" <mturquette@baylibre.com>, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, "Jason Cooper" <jason@lakedaemon.net>, "Andrew Lunn" <andrew@lunn.ch>, "Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com>, "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>, linux-arm-kernel@lists.infradead.org, "Antoine Tenart" <antoine.tenart@bootlin.com>, "Miquèl Raynal" <miquel.raynal@bootlin.com>, "Maxime Chevallier" <maxime.chevallier@bootlin.com> Subject: Re: [PATCH v5 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file Date: Fri, 26 Apr 2019 09:30:27 -0500 [thread overview] Message-ID: <20190426143027.GA17777@bogus> (raw) In-Reply-To: <20190423095107.21091-2-gregory.clement@bootlin.com> On Tue, Apr 23, 2019 at 11:51:02AM +0200, Gregory CLEMENT wrote: > Document the device tree binding for the cluster clock controllers found > in the Armada 7K/8K SoCs. > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > --- > .../arm/marvell/ap806-system-controller.txt | 26 +++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > index 7b8b8eb0191f..a65d3e9ff915 100644 > --- a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > +++ b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > @@ -143,3 +143,29 @@ ap_syscon1: system-controller@6f8000 { > #thermal-sensor-cells = <1>; > }; > }; > + > +Cluster clocks: > +--------------- > + > +Device Tree Clock bindings for cluster clock of AP806 Marvell. Each > +cluster contain up to 2 CPUs running at the same frequency. > + > +Required properties: > +- compatible: must be "marvell,ap806-cpu-clock"; > +- #clock-cells : should be set to 1. > +- clocks : shall be the input parents clock phandle for the clock. > +- reg: register range associated the cluster clocks, offset must be 0 > + and the size have to be the whole size of the system-controller > + > + > +ap_syscon1: system-controller@6f8000 { > + compatible = "syscon", "simple-mfd"; This should have a specific compatible. > + reg = <0x6f8000 0x1000>; > + > + cpu_clk: clock-cpu@0 { > + compatible = "marvell,ap806-cpu-clock"; > + clocks = <&ap_clk 0>, <&ap_clk 1>; > + #clock-cells = <1>; > + reg = <0x0 0x1000>; This takes the entire address range of the parent, then why the child node? You can't add any other child nodes without creating an overlap. Your example also won't compile. Rob
WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh@kernel.org> To: Gregory CLEMENT <gregory.clement@bootlin.com> Cc: devicetree@vger.kernel.org, "Jason Cooper" <jason@lakedaemon.net>, "Andrew Lunn" <andrew@lunn.ch>, "Stephen Boyd" <sboyd@kernel.org>, "Mike Turquette" <mturquette@baylibre.com>, linux-kernel@vger.kernel.org, "Maxime Chevallier" <maxime.chevallier@bootlin.com>, "Antoine Tenart" <antoine.tenart@bootlin.com>, "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>, "Miquèl Raynal" <miquel.raynal@bootlin.com>, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, "Sebastian Hesselbarth" <sebastian.hesselbarth@gmail.com> Subject: Re: [PATCH v5 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file Date: Fri, 26 Apr 2019 09:30:27 -0500 [thread overview] Message-ID: <20190426143027.GA17777@bogus> (raw) In-Reply-To: <20190423095107.21091-2-gregory.clement@bootlin.com> On Tue, Apr 23, 2019 at 11:51:02AM +0200, Gregory CLEMENT wrote: > Document the device tree binding for the cluster clock controllers found > in the Armada 7K/8K SoCs. > > Signed-off-by: Gregory CLEMENT <gregory.clement@bootlin.com> > --- > .../arm/marvell/ap806-system-controller.txt | 26 +++++++++++++++++++ > 1 file changed, 26 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > index 7b8b8eb0191f..a65d3e9ff915 100644 > --- a/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > +++ b/Documentation/devicetree/bindings/arm/marvell/ap806-system-controller.txt > @@ -143,3 +143,29 @@ ap_syscon1: system-controller@6f8000 { > #thermal-sensor-cells = <1>; > }; > }; > + > +Cluster clocks: > +--------------- > + > +Device Tree Clock bindings for cluster clock of AP806 Marvell. Each > +cluster contain up to 2 CPUs running at the same frequency. > + > +Required properties: > +- compatible: must be "marvell,ap806-cpu-clock"; > +- #clock-cells : should be set to 1. > +- clocks : shall be the input parents clock phandle for the clock. > +- reg: register range associated the cluster clocks, offset must be 0 > + and the size have to be the whole size of the system-controller > + > + > +ap_syscon1: system-controller@6f8000 { > + compatible = "syscon", "simple-mfd"; This should have a specific compatible. > + reg = <0x6f8000 0x1000>; > + > + cpu_clk: clock-cpu@0 { > + compatible = "marvell,ap806-cpu-clock"; > + clocks = <&ap_clk 0>, <&ap_clk 1>; > + #clock-cells = <1>; > + reg = <0x0 0x1000>; This takes the entire address range of the parent, then why the child node? You can't add any other child nodes without creating an overlap. Your example also won't compile. Rob _______________________________________________ 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:[~2019-04-26 14:30 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-04-23 9:51 [PATCH resend v5 0/6] Add CPU clock support for Armada 7K/8K Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-23 9:51 ` [PATCH v5 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-26 14:30 ` Rob Herring [this message] 2019-04-26 14:30 ` Rob Herring 2019-04-23 9:51 ` [PATCH v5 2/6] clk: mvebu: add helper file for Armada AP and CP clocks Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-23 9:51 ` [PATCH v5 3/6] clk: mvebu: add CPU clock driver for Armada 7K/8K Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-23 9:51 ` [PATCH v5 4/6] clk: mvebu: ap806: Fix clock name for the cluster Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-23 9:51 ` [PATCH v5 5/6] arm64: marvell: enable the Armada 7K/8K CPU clk driver Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-23 9:51 ` [PATCH v5 6/6] arm64: dts: marvell: Add cpu clock node on Armada 7K/8K Gregory CLEMENT 2019-04-23 9:51 ` Gregory CLEMENT 2019-04-26 0:46 ` [PATCH resend v5 0/6] Add CPU clock support for " Stephen Boyd 2019-04-26 0:46 ` Stephen Boyd -- strict thread matches above, loose matches on Subject: below -- 2019-04-23 8:33 [PATCH " Gregory CLEMENT 2019-04-23 8:33 ` [PATCH v5 1/6] dt-bindings: ap806: add the cluster clock node in the syscon file Gregory CLEMENT 2019-04-23 8:33 ` Gregory CLEMENT
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=20190426143027.GA17777@bogus \ --to=robh@kernel.org \ --cc=andrew@lunn.ch \ --cc=antoine.tenart@bootlin.com \ --cc=devicetree@vger.kernel.org \ --cc=gregory.clement@bootlin.com \ --cc=jason@lakedaemon.net \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=maxime.chevallier@bootlin.com \ --cc=miquel.raynal@bootlin.com \ --cc=mturquette@baylibre.com \ --cc=sboyd@kernel.org \ --cc=sebastian.hesselbarth@gmail.com \ --cc=thomas.petazzoni@bootlin.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.