Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-20 Thread Nicolin Chen
On Fri, Oct 20, 2017 at 11:20:24AM +0100, Jon Hunter wrote: > > On 19/10/17 19:42, Nicolin Chen wrote: > > On Thu, Oct 19, 2017 at 11:44:22AM +0200, Thierry Reding wrote: > Below is the call trace of tegra210_init_pllu() function: > start_kernel() > -> time_init() > --> o

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-20 Thread Thierry Reding
On Fri, Sep 15, 2017 at 12:10:13PM -0700, Nicolin Chen wrote: > Below is the call trace of tegra210_init_pllu() function: > start_kernel() > -> time_init() > --> of_clk_init() > ---> tegra210_clock_init() > > tegra210_pll_init() > -> tegra210_init_pllu() > > Because the preempt

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-20 Thread Jon Hunter
On 19/10/17 19:42, Nicolin Chen wrote: > On Thu, Oct 19, 2017 at 11:44:22AM +0200, Thierry Reding wrote: Below is the call trace of tegra210_init_pllu() function: start_kernel() -> time_init() --> of_clk_init() ---> tegra210_clock_init() > tegra210_pll_i

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-19 Thread Nicolin Chen
On Thu, Oct 19, 2017 at 11:42:24AM -0700, Nicolin Chen wrote: > On Thu, Oct 19, 2017 at 11:44:22AM +0200, Thierry Reding wrote: > > > > Below is the call trace of tegra210_init_pllu() function: > > > > start_kernel() > > > > -> time_init() > > > > --> of_clk_init() > > > > ---> tegra210_clo

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-19 Thread Nicolin Chen
On Thu, Oct 19, 2017 at 11:44:22AM +0200, Thierry Reding wrote: > > > Below is the call trace of tegra210_init_pllu() function: > > > start_kernel() > > > -> time_init() > > > --> of_clk_init() > > > ---> tegra210_clock_init() > > > > tegra210_pll_init() > > > -> tegra210_init_p

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-19 Thread Thierry Reding
On Thu, Oct 19, 2017 at 02:29:20AM -0700, Nicolin Chen wrote: > On Fri, Sep 15, 2017 at 12:10:13PM -0700, Nicolin Chen wrote: > > Below is the call trace of tegra210_init_pllu() function: > > start_kernel() > > -> time_init() > > --> of_clk_init() > > ---> tegra210_clock_init() > > >

Re: [PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-10-19 Thread Nicolin Chen
On Fri, Sep 15, 2017 at 12:10:13PM -0700, Nicolin Chen wrote: > Below is the call trace of tegra210_init_pllu() function: > start_kernel() > -> time_init() > --> of_clk_init() > ---> tegra210_clock_init() > > tegra210_pll_init() > -> tegra210_init_pllu() > > Because the preempt

[PATCH v2] clk: tegra: Use readl_relaxed_poll_timeout_atomic in tegra210_clock_init

2017-09-15 Thread Nicolin Chen
Below is the call trace of tegra210_init_pllu() function: start_kernel() -> time_init() --> of_clk_init() ---> tegra210_clock_init() > tegra210_pll_init() -> tegra210_init_pllu() Because the preemption is disabled in the start_kernel before calling time_init, tegra210_init_pllu