Re: [PATCH] input: misc: max8997: Switch to pwm_apply()
From: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Date: 2021-03-21 22:24:34
Also in:
linux-pwm
Hi Uwe, On Tue, Mar 16, 2021 at 09:38:13PM +0100, Uwe Kleine-König wrote:
max8997_haptic_enable() is the only caller of max8997_haptic_set_duty_cycle(). For the non-external case the PWM is already enabled in max8997_haptic_set_duty_cycle(), so this can be done
Are you sure about that? I think the intent was to enable it in max8997_haptic_configure(), and only after "inmotor" regulator is enabled. If the device is enabled earlier then I'd say we need to make sure we disable it until it is needed. Thanks. -- Dmitry