Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pwm: add option to break the loops when using multiple PWM channels #4227

Merged
merged 1 commit into from Jul 26, 2021

Conversation

michallenc
Copy link
Contributor

Summary

PWM drivers currently use channel number 0 for the channels that are not
used by the application. This commit adds number -1 which indicates that
all following channels are not configured and that the loop can be broken.

Impact

PWM drivers supporting multiple channels option.

Testing

Tested with iMXRT and STM32 boards.

@michallenc
Copy link
Contributor Author

Hi @acassis @btashton, @xiaoxiang781216, any ideas what might have caused the docker to fail? I am not sure how this is code related (if it is).

@xiaoxiang781216
Copy link
Contributor

Hi @acassis @btashton, @xiaoxiang781216, any ideas what might have caused the docker to fail? I am not sure how this is code related (if it is).

look like github stuck, please rebase your change to the latest mainline and push again if there are new change commit.

PWM drivers currently use channel number 0 for the channels that are not
used by the application. This commit adds number -1 which indicates that
all following channels are not configured and that the loop can be broken.

Signed-off-by: Michal Lenc <michallenc@seznam.cz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants