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

From: Nicolin Chen
Date: Thu Oct 19 2017 - 05:29:59 EST


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 preemption is disabled in the start_kernel before calling
> time_init, tegra210_init_pllu is actually in an atomic context while
> it includes a readl_relaxed_poll_timeout that might sleep.
>
> So this patch just changes this readl_relaxed_poll_timeout() to its
> atomic version.
>
> Signed-off-by: Nicolin Chen <nicoleotsuka@xxxxxxxxx>
> Acked-By: Peter De Schrijver <pdeschrijver@xxxxxxxxxx>

Thierry, can you also take a look at this one? I sent a month ago. Thanks.