On Wed, Apr 09, 2014 at 09:19:54PM +0800, Axel Lin wrote:
> Current code only works when pdev->id is 1. Fix it by passing correct 
> bitvalues
> argument to abx500_mask_and_set_register_interruptible.
> 
> Having DISABLE_PWM/ENABLE_PWM does not make the code better in readability
> because the bitvalues depends on pdev->id.
> Thus drop defines for DISABLE_PWM/ENABLE_PWM.
> 
> This patch also removes a unnecessary return in ab8500_pwm_disable.
> 
> Signed-off-by: Axel Lin <[email protected]>
> Acked-by: Linus Walleij <[email protected]>
> Acked-by: Alexandre BOURDIOL <[email protected]>
> Acked-by: Philippe Begnic <[email protected]>
> ---
>  drivers/pwm/pwm-ab8500.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)

Applied, thanks.

Thierry

Attachment: pgpgFxMOHLWtA.pgp
Description: PGP signature

Reply via email to