Re: [PATCH v2 1/1] pwm: pca9685: fix gpio-only operation.

[Sven Van Asbroeck]

On Wed, Apr 12, 2017 at 4:53 AM, Mika Westerberg
<mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> Looks like going to the right direction.

Thanks, I appreciate your comments !

>> +static void pca9685_pwm_gpio_free(struct gpio_chip *gpio, unsigned int offset)
> Why did you move the function here?

Because I added code that disables the output when the user releases
(unexports) the pwm or gpio.

If the output is not disabled on release (unexport):
- the output will remain in its previous state (possibly ON)
- when the last output is unexported, SLEEP is set, and all outputs
   are disabled at the same time
- this is counter-intuitive behaviour.

Expect a v3 patch soon which addresses your feedback.