From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [Patch v2 02/10] ARM: dts: dra7: add cam clkctrl node Date: Fri, 8 Nov 2019 08:58:12 -0800 Message-ID: <20191108165812.GG5610@atomide.com> References: <20191104204853.4355-1-bparrot@ti.com> <20191104204853.4355-3-bparrot@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20191104204853.4355-3-bparrot@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Benoit Parrot Cc: Tero Kristo , linux-omap@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org, Rob Herring , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org List-Id: linux-omap@vger.kernel.org * Benoit Parrot [191104 20:49]: > Add clkctrl nodes for CAM domain. This too should mention that we need to use custom node naming. Other than that this looks good to me. Again, I can queue the dts changes once I have an immutable clock changes branch to use as base from Tero. And up to Tero to consider if he wants to pick this clock node patch too. Regards, Tony > Signed-off-by: Benoit Parrot > --- > arch/arm/boot/dts/dra7xx-clocks.dtsi | 14 ++++++++++++++ > 1 file changed, 14 insertions(+) > > diff --git a/arch/arm/boot/dts/dra7xx-clocks.dtsi b/arch/arm/boot/dts/dra7xx-clocks.dtsi > index d1c2406ec71c..199f8d483f75 100644 > --- a/arch/arm/boot/dts/dra7xx-clocks.dtsi > +++ b/arch/arm/boot/dts/dra7xx-clocks.dtsi > @@ -1734,6 +1734,20 @@ > }; > }; > > + cam_cm: cam-cm@1000 { > + compatible = "ti,omap4-cm"; > + reg = <0x1000 0x100>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0 0x1000 0x100>; > + > + cam_clkctrl: cam-clkctrl@20 { > + compatible = "ti,clkctrl"; > + reg = <0x20 0x2c>; > + #clock-cells = <2>; > + }; > + }; > + > dss_cm: dss-cm@1100 { > compatible = "ti,omap4-cm"; > reg = <0x1100 0x100>; > -- > 2.17.1 >