2016-04-16 01:00:57

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH 1/2] clk: imx: do not sleep if IRQ's are still disabled

On 01/29, Stefan Agner wrote:
> If a clock gets enabled early during boot time, it can lead to a PLL
> startup. The wait_lock function makes sure that the PLL is really
> stareted up before it gets used. However, the function sleeps which
> leads to scheduling and an error:
> bad: scheduling from the idle thread!
> ...
>
> Use udelay in case IRQ's are still disabled.
>
> Signed-off-by: Stefan Agner <[email protected]>

This is really old. Shawn, are you picking these up? I'm removing
these from my queue for now.

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project


2016-04-18 01:58:38

by Shawn Guo

[permalink] [raw]
Subject: Re: [PATCH 1/2] clk: imx: do not sleep if IRQ's are still disabled

On Fri, Apr 15, 2016 at 06:00:53PM -0700, Stephen Boyd wrote:
> On 01/29, Stefan Agner wrote:
> > If a clock gets enabled early during boot time, it can lead to a PLL
> > startup. The wait_lock function makes sure that the PLL is really
> > stareted up before it gets used. However, the function sleeps which
> > leads to scheduling and an error:
> > bad: scheduling from the idle thread!
> > ...
> >
> > Use udelay in case IRQ's are still disabled.
> >
> > Signed-off-by: Stefan Agner <[email protected]>
>
> This is really old. Shawn, are you picking these up? I'm removing
> these from my queue for now.

Yes, I'm picking them up.

Shawn