On 27/09/2013 19:33, Milo Kim wrote: > Sorry, I meant atmel_pwm_free(), not atmel_pwm_disable(). > - copy & paste disaster! :( > >> 2) Move atmel PWM register code to atmel_pwm_request() and _free() >> >> static int atmel_pwm_request(struct pwm_chip *chip, struct pwm_device >> *pwm) >> { >> struct atmel_pwm_chip *atmel_pwm = to_atmel_pwm_chip(chip); >> >> atmel_pwm_writel(atmel_pwm, PWM_ENA, 1 << pwm->hwpwm); >> return 0; >> } >> >> static void atmel_pwm_disable(struct pwm_chip *chip, struct pwm_device >> *pwm) > > static void atmel_pwm_free(struct pwm_chip *chip, struct pwm_device *pwm) > > Ok, I tested that and it seems to be working fine. Thanks ! -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html