Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1607544 > unrolled thread

Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults unnecessarily

Started byJon Hunter <jonathanh@nvidia.com>
First post2017-03-23 15:40 +0100
Last post2017-03-24 10:30 +0100
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults  unnecessarily Jon Hunter <jonathanh@nvidia.com> - 2017-03-23 15:40 +0100
    Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults  unnecessarily Jon Hunter <jonathanh@nvidia.com> - 2017-03-24 10:30 +0100
    Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults  unnecessarily Peter De Schrijver <pdeschrijver@nvidia.com> - 2017-03-24 10:30 +0100

#1607544 — Re: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults unnecessarily

FromJon Hunter <jonathanh@nvidia.com>
Date2017-03-23 15:40 +0100
SubjectRe: [PATCH v2 5/7] clk: tegra: don't warn for PLL defaults unnecessarily
Message-ID<toc49-3pL-1@gated-at.bofh.it>
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 <pdeschrijver@nvidia.com>

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.

Cheers
Jon

[0]
https://nvtb.github.io//linux-next/test_next-20170323/20170323021534/boot/tegra210-smaug/tegra210-smaug/defconfig_log.txt

-- 
nvpublic

[toc] | [next] | [standalone]


#1608260

FromJon Hunter <jonathanh@nvidia.com>
Date2017-03-24 10:30 +0100
Message-ID<totHH-7Sh-1@gated-at.bofh.it>
In reply to#1607544
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 <pdeschrijver@nvidia.com>
>>
>> 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

[toc] | [prev] | [next] | [standalone]


#1608262

FromPeter De Schrijver <pdeschrijver@nvidia.com>
Date2017-03-24 10:30 +0100
Message-ID<totHH-7Sh-3@gated-at.bofh.it>
In reply to#1607544
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 <pdeschrijver@nvidia.com>
> 
> 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.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web