Re: [PATCH 2/2] ARM: omap5/dra7xx: Fix counter frequency drift for AM572x errata i856.

From: "Lennart Sorensen" <lsorense@csclub.uwaterloo.ca>
To: Nishanth Menon <nm@ti.com>
Cc: Tero Kristo <t-kristo@ti.com>, Lokesh Vutla <lokeshvutla@ti.com>,
	linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Sekhar Nori <nsekhar@ti.com>
Subject: Re: [PATCH 2/2] ARM: omap5/dra7xx: Fix counter frequency drift for AM572x errata i856.
Date: Wed, 17 Dec 2014 10:27:10 -0500	[thread overview]
Message-ID: <20141217152710.GT24110@csclub.uwaterloo.ca> (raw)
In-Reply-To: <20141217152225.GA737@kahuna>

On Wed, Dec 17, 2014 at 09:22:25AM -0600, Nishanth Menon wrote:
> A clock mux might do the job?
> 
> value 1, 2 , 3 will imply sysclk1 / 610
> value of 0 implies fixed 32768
> 
> soemthing like
> sys_clk32_crystal {
>                compatible = "fixed-clock";
>                clock-frequency = <32768>;
> }
> 
> sys_clk32_pseudo {
>                compatible = "fixed-clock";
>                compatible = "fixed-factor-clock";
>                clocks = <&sys_clkin1>;
>                clock-mult = <1>;
>                clock-div = <610>;
> }
> 
> sys_32k_ck: sys_32k_ck {
> 	compatible = "ti,mux-clock";
> 	clocks = <&sys_clk32_crystal>, <&sys_clk32_pseudo>, <&sys_clk32_pseudo>, <&sys_clk32_pseudo>;
> };
> 
> I think... The only issue is that the BOOTSTRAP register is not around
> the usual CM1,2 address region...

OK, I like where that idea is headed.  That looks promising.

-- 
Len Sorensen