You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/07/26 07:32:55 UTC

[GitHub] [incubator-nuttx] michallenc opened a new pull request #4227: pwm: add option to break the loops when using multiple PWM channels

michallenc opened a new pull request #4227:
URL: https://github.com/apache/incubator-nuttx/pull/4227


   ## 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.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] michallenc commented on pull request #4227: pwm: add option to break the loops when using multiple PWM channels

Posted by GitBox <gi...@apache.org>.
michallenc commented on pull request #4227:
URL: https://github.com/apache/incubator-nuttx/pull/4227#issuecomment-886460660


   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).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] acassis merged pull request #4227: pwm: add option to break the loops when using multiple PWM channels

Posted by GitBox <gi...@apache.org>.
acassis merged pull request #4227:
URL: https://github.com/apache/incubator-nuttx/pull/4227


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4227: pwm: add option to break the loops when using multiple PWM channels

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #4227:
URL: https://github.com/apache/incubator-nuttx/pull/4227#issuecomment-886536826


   > 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.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org