[PATCH v5] ARM: davinci: da8xx: Fix sleeping function called from invalid context

Sekhar Nori nsekhar at ti.com
Tue Dec 6 01:33:54 PST 2016


On Monday 05 December 2016 07:43 PM, Alexandre Bailon wrote:
> Everytime the usb20 phy is enabled, there is a
> "sleeping function called from invalid context" BUG.
> 
> clk_enable() from arch/arm/mach-davinci/clock.c uses spin_lock_irqsave()
> before to invoke the callback usb20_phy_clk_enable().
> usb20_phy_clk_enable() uses clk_get() and clk_enable_prepapre()
> which may sleep.
> Move clk_get() to da8xx_register_usb20_phy_clk() and
> replace clk_prepare_enable() by clk_enable().
> 
> Signed-off-by: Alexandre Bailon <abailon at baylibre.com>

This will still cause the recursive locking problem reported by David.
Not sure what the point of sending this version was.

Thanks,
Sekhar




More information about the linux-arm-kernel mailing list