From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter De Schrijver Subject: Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults unnecessarily Date: Fri, 24 Mar 2017 11:25:46 +0200 Message-ID: <20170324092546.GA22128@tbergstrom-lnx.Nvidia.com> References: <1487846686-6388-1-git-send-email-pdeschrijver@nvidia.com> <1487846686-6388-6-git-send-email-pdeschrijver@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Jon Hunter Cc: Prashant Gaikwad , Michael Turquette , Stephen Boyd , Stephen Warren , Thierry Reding , Alexandre Courbot , Rob Herring , Mark Rutland , Rhyland Klein , linux-clk-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org On Thu, Mar 23, 2017 at 02:37:23PM +0000, Jon Hunter wrote: > Hi Peter, > > On 23/02/17 10:44, Peter De Schrijver wrote: > > If the PLL is on, only warn if the defaults are not yet set. Otherwise be > > silent. > > > > Signed-off-by: Peter De Schrijver > > This patch is breaking boot for Tegra210 Smaug on -next [0]. Reverting > this on top of -next allows the board to boot again. I have not had > chance to dig into this any further yet, but can unless you have some > thoughts. My guess would be pllm. Maybe the bootloader on Smaug switches to pllm for a higher emc frequency? On jetson cboot doesn't do that, so at least that could be a difference. Peter.