From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tero Kristo Subject: Re: [PATCH] ARM: dts: am43xx-clocks: use ti,fixed-factor-clock for dpll_per_clkdcoldo Date: Mon, 19 May 2014 15:25:16 +0300 Message-ID: <5379F82C.3020702@ti.com> References: <1396040071-17900-1-git-send-email-d-gerlach@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from comal.ext.ti.com ([198.47.26.152]:37032 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752984AbaESMY7 (ORCPT ); Mon, 19 May 2014 08:24:59 -0400 In-Reply-To: <1396040071-17900-1-git-send-email-d-gerlach@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Dave Gerlach , linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org, devicetree@vger.kernel.org Cc: Paul Walmsley , Tony Lindgren , Benoit Cousson On 03/28/2014 10:54 PM, Dave Gerlach wrote: > Use the ti,fixed-factor-clock version so that autoidle for > dpll_per_clkdcoldo is properly controlled after power management code > is introduced. Without this the clock may be held active even when > it is gated. > > Signed-off-by: Dave Gerlach Thanks, queued for 3.16/clk-dt. > --- > arch/arm/boot/dts/am43xx-clocks.dtsi | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi > index 142009c..1d9b6bb 100644 > --- a/arch/arm/boot/dts/am43xx-clocks.dtsi > +++ b/arch/arm/boot/dts/am43xx-clocks.dtsi > @@ -609,10 +609,13 @@ > > dpll_per_clkdcoldo: dpll_per_clkdcoldo { > #clock-cells = <0>; > - compatible = "fixed-factor-clock"; > + compatible = "ti,fixed-factor-clock"; > clocks = <&dpll_per_ck>; > - clock-mult = <1>; > - clock-div = <1>; > + ti,clock-mult = <1>; > + ti,clock-div = <1>; > + ti,autoidle-shift = <8>; > + reg = <0x2e14>; > + ti,invert-autoidle-bit; > }; > > dll_aging_clk_div: dll_aging_clk_div { > From mboxrd@z Thu Jan 1 00:00:00 1970 From: t-kristo@ti.com (Tero Kristo) Date: Mon, 19 May 2014 15:25:16 +0300 Subject: [PATCH] ARM: dts: am43xx-clocks: use ti,fixed-factor-clock for dpll_per_clkdcoldo In-Reply-To: <1396040071-17900-1-git-send-email-d-gerlach@ti.com> References: <1396040071-17900-1-git-send-email-d-gerlach@ti.com> Message-ID: <5379F82C.3020702@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 03/28/2014 10:54 PM, Dave Gerlach wrote: > Use the ti,fixed-factor-clock version so that autoidle for > dpll_per_clkdcoldo is properly controlled after power management code > is introduced. Without this the clock may be held active even when > it is gated. > > Signed-off-by: Dave Gerlach Thanks, queued for 3.16/clk-dt. > --- > arch/arm/boot/dts/am43xx-clocks.dtsi | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/am43xx-clocks.dtsi b/arch/arm/boot/dts/am43xx-clocks.dtsi > index 142009c..1d9b6bb 100644 > --- a/arch/arm/boot/dts/am43xx-clocks.dtsi > +++ b/arch/arm/boot/dts/am43xx-clocks.dtsi > @@ -609,10 +609,13 @@ > > dpll_per_clkdcoldo: dpll_per_clkdcoldo { > #clock-cells = <0>; > - compatible = "fixed-factor-clock"; > + compatible = "ti,fixed-factor-clock"; > clocks = <&dpll_per_ck>; > - clock-mult = <1>; > - clock-div = <1>; > + ti,clock-mult = <1>; > + ti,clock-div = <1>; > + ti,autoidle-shift = <8>; > + reg = <0x2e14>; > + ti,invert-autoidle-bit; > }; > > dll_aging_clk_div: dll_aging_clk_div { >