Hello Sean, On Mon, Oct 02, 2023 at 09:20:20AM +0100, Sean Young wrote: > Having said that, the extra call to pwm_apply_state() may have benefits, > see this comment in the pwm-sifive driver: > > * - When changing both duty cycle and period, we cannot prevent in > * software that the output might produce a period with mixed > * settings (new period length and old duty cycle). You don't gain anything here I think. Disabling a PWM might also result in a constant active level. If you want to prevent this, your best bet is to never disable the PWM. Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König | Industrial Linux Solutions | https://www.pengutronix.de/ |
Attachment:
signature.asc
Description: PGP signature