From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thierry Reding Date: Fri, 24 Jan 2014 15:13:54 +0100 Subject: [U-Boot] [PATCH V2] ARM: tegra: don't exceed AVP limits when configuring PLLP In-Reply-To: <1390503493-32180-1-git-send-email-swarren@wwwdotorg.org> References: <1390503493-32180-1-git-send-email-swarren@wwwdotorg.org> Message-ID: <20140124141353.GF25720@ulmo.nvidia.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thu, Jan 23, 2014 at 11:58:13AM -0700, Stephen Warren wrote: [...] > diff --git a/arch/arm/cpu/tegra-common/clock.c b/arch/arm/cpu/tegra-common/clock.c [...] > @@ -575,3 +575,91 @@ void clock_init(void) > /* Do any special system timer/TSC setup */ > arch_timer_init(); > } > + > +static void set_avp_clock_to(u32 src) Perhaps a better name would be "set_avp_clock_source()"? _to had me confused at first because I assumed it would be setting a frequency. > +void tegra30_or_later_adjust_pllp(void) I would've called this simply tegra30_adjust_pllp(), where it's implied that when you call tegra30_*() from code targetted at later generations these would be compatible with Tegra30. But if you'd rather make it implicit that's fine too. Either way: Reviewed-by: Thierry Reding Tested-by: Thierry Reding Acked-by: Thierry Reding Thierry -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: not available URL: