On Fri, Feb 25, 2022 at 05:16:01PM +0800, Song Chen wrote: > Introduce apply in pwm_ops to replace legacy operations, > like enable, disable, config and set_polarity. Why? You are saying what you are doing here, but nothing about why this is needed, or what it will help with, or what it will fix. You need to explain why a change is needed. thanks, greg k-h _______________________________________________ greybus-dev mailing list -- greybus-dev@xxxxxxxxxxxxxxxx To unsubscribe send an email to greybus-dev-leave@xxxxxxxxxxxxxxxx