Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935245AbdCXJ2L (ORCPT ); Fri, 24 Mar 2017 05:28:11 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:15957 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752206AbdCXJ15 (ORCPT ); Fri, 24 Mar 2017 05:27:57 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Fri, 24 Mar 2017 02:24:56 -0700 Subject: Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults unnecessarily To: Peter De Schrijver References: <1487846686-6388-1-git-send-email-pdeschrijver@nvidia.com> <1487846686-6388-6-git-send-email-pdeschrijver@nvidia.com> <20170324092546.GA22128@tbergstrom-lnx.Nvidia.com> CC: Prashant Gaikwad , Michael Turquette , Stephen Boyd , "Stephen Warren" , Thierry Reding , Alexandre Courbot , Rob Herring , "Mark Rutland" , Rhyland Klein , , , , From: Jon Hunter Message-ID: <6a7a5f58-08e1-c8e8-c32f-f57b63690863@nvidia.com> Date: Fri, 24 Mar 2017 09:27:50 +0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <20170324092546.GA22128@tbergstrom-lnx.Nvidia.com> X-Originating-IP: [10.21.132.151] X-ClientProxiedBy: UKMAIL102.nvidia.com (10.26.138.15) To UKMAIL101.nvidia.com (10.26.138.13) Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 879 Lines: 28 On 24/03/17 09:25, Peter De Schrijver wrote: > 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. Thanks. I will take a closer look at what is going on either today or Monday and let you know what I find. Cheers Jon -- nvpublic