From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Zhang Subject: Re: [PATCH 2/2] clk: tegra: Make gr2d and gr3d clocks children of pll_c Date: Mon, 01 Apr 2013 14:03:40 +0800 Message-ID: <5159233C.9060809@gmail.com> References: <1364502688-5135-1-git-send-email-thierry.reding@avionic-design.de> <1364502688-5135-2-git-send-email-thierry.reding@avionic-design.de> <51560BC4.8020400@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <51560BC4.8020400-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Stephen Warren Cc: Thierry Reding , Mike Turquette , linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Peter De Schrijver , Prashant Gaikwad , =?ISO-8859-1?Q?Terje_Bergstr=F6?= =?ISO-8859-1?Q?m?= List-Id: linux-tegra@vger.kernel.org On 03/30/2013 05:46 AM, Stephen Warren wrote: > On 03/28/2013 02:31 PM, Thierry Reding wrote: >> By default these clocks are children of pll_m, but in downstream kernels >> they are reparented to pll_c. While at it, decrease their frequencies to >> 300 MHz because the defaults aren't in the specified range. >> >> gr2d can reportedly run at much higher frequencies, but 300 MHz works >> and is a more conservative default. > > Questions on this patch: > > Do we need to do the same thing for Tegra30 and/or Tegra114? > I think yes. For Tegra114, it's pll_c2. > Is 300MHz the right value? > Right. I use 300MHz on Tegra114 and it seems OK. I recall sometimes 276MHz can be used as well. Mark > I'm hoping that Peter, Prashant, and/or Terje can provide guidance here. > -- > To unsubscribe from this list: send the line "unsubscribe linux-tegra" in > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >