On Tue, Nov 24, 2015 at 02:43:12PM +0800, Axel Lin wrote: > The .can_sleep flag is used for some PWM controllers that can't be used > in atomic context. Not such case for this driver, so don't set the > can_sleep flag. > > Signed-off-by: Axel Lin <[email protected]> > Acked-by: Florian Fainelli <[email protected]> > Reviewed-by: Brian Norris <[email protected]> > --- > This patch was sent on http://www.spinics.net/lists/linux-pwm/msg03506.html > > drivers/pwm/pwm-brcmstb.c | 1 - > 1 file changed, 1 deletion(-)
There's some discussion currently about marking all drivers can_sleep and modify all users to cope with it correctly. The reason is that we introduced a mutex in pwm_enable() and pwm_set_polarity() which means that PWM devices effectively can't be used from interrupt or atomic contexts. It also means that the driver is actually currently correct, so I'd like to leave it as-is until the users have been sorted out, at which point we can simply get rid of can_sleep. Thierry
signature.asc
Description: PGP signature
