From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933551AbdKBAGU (ORCPT ); Wed, 1 Nov 2017 20:06:20 -0400 Received: from mail-oi0-f66.google.com ([209.85.218.66]:45127 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753002AbdKBAGS (ORCPT ); Wed, 1 Nov 2017 20:06:18 -0400 X-Google-Smtp-Source: ABhQp+QQKnW+a1lVXpkQe2fJ2QeSadX721Nn9mn1V7eyJaPlv52vlSoPvjE7vLdqmB6t6/TOte4byA== Date: Wed, 1 Nov 2017 19:06:16 -0500 From: Rob Herring To: Manivannan Sadhasivam Cc: mturquette@baylibre.com, sboyd@codeaurora.org, afaerber@suse.de, mark.rutland@arm.com, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, davem@davemloft.net, mchehab@kernel.org, rdunlap@infradead.org, daniel.thompson@linaro.org, amit.kucheria@linaro.org, liuwei@actions-semi.com Subject: Re: [PATCH 1/3] arm64: dts: actions: add s900 clock controller nodes Message-ID: <20171102000616.ug7s7s6fmnubq7y5@rob-hp-laptop> References: <1509479663-8985-1-git-send-email-manivannan.sadhasivam@linaro.org> <1509479663-8985-2-git-send-email-manivannan.sadhasivam@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1509479663-8985-2-git-send-email-manivannan.sadhasivam@linaro.org> User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Nov 01, 2017 at 01:24:21AM +0530, Manivannan Sadhasivam wrote: > This patch adds clock controller nodes for Actions Semi s900 SoC > > Signed-off-by: Manivannan Sadhasivam > --- > arch/arm64/boot/dts/actions/s900.dtsi | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/arch/arm64/boot/dts/actions/s900.dtsi b/arch/arm64/boot/dts/actions/s900.dtsi > index 11406f6..c099ef1 100644 > --- a/arch/arm64/boot/dts/actions/s900.dtsi > +++ b/arch/arm64/boot/dts/actions/s900.dtsi > @@ -5,6 +5,7 @@ > */ > > #include > +#include > > / { > compatible = "actions,s900"; > @@ -88,6 +89,24 @@ > #clock-cells = <0>; > }; > > + losc: losc { > + compatible = "fixed-clock"; > + clock-frequency = <32768>; > + #clock-cells = <0>; > + }; > + > + diff_24M: diff_24M { > + compatible = "fixed-clock"; > + clock-frequency = <24000000>; > + #clock-cells = <0>; > + }; These 2 are inputs to the clock ctrlr? The clock controller should have a clocks property then. > + > + clock: clock-controller@e0160000 { > + compatible = "actions,s900-clock"; > + reg = <0 0xe0160000 0 0x1000>; > + #clock-cells = <1>; > + }; > + > soc { > compatible = "simple-bus"; > #address-cells = <2>; > -- > 2.7.4 >