Hi Pavel, On 2/26/20 3:35 PM, Pavel Machek wrote: >> On Fri, Jan 24, 2020 at 05:54:08PM +0100, Uwe Kleine-König wrote: >>> pwm_config(), pwm_enable() and pwm_disable() should get removed in the >>> long run. So update the driver to use the atomic API that is here to >>> stay. >>> >>> A few side effects: >>> >>> - led_pwm_set() now returns an error when setting the PWM fails. >>> - During .probe() the PWM isn't disabled implicitly by pwm_apply_args() >>> any more. >>> >>> Signed-off-by: Uwe Kleine-König <uwe@xxxxxxxxxxxxxxxxx> > >> And so: >> >> Tested-by: Jeff LaBundy <jeff@xxxxxxxxxxx> > > Thanks for patches, thanks for testing. > > 1,2 applied to -for-next. > > AFAICT 3/3 should really get the default-state: keep support before > being merged? Maybe I miss something, but I wonder why default-state: keep support isn't implemented in the core but in each led driver instead. (And as soon as someone implements it in the core, patch 3 is the right thing to do.) Best regards Uwe
Attachment:
signature.asc
Description: OpenPGP digital signature