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/12/28 10:58:19 UTC

[GitHub] [incubator-nuttx] normanr opened a new pull request #5093: Use userspace chosen channel numbers when starting bl602 pwm

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


   ## Summary
   commit 2889315c207fcf7bb0ccd46f9955ba1fe323ebed added support for pwm but didn't read the channel numbers provided by user-space. They should be, otherwise it's not possible to start a sub-set of channels that are not the first "n" channels.
   
   ## Impact
   bl602 pwm channels can now be chosen by userspace, where-as previously they were "hardcoded" to [0,1,2,3,4].
   
   This includes a fix for a typo in the description of `pwm_expired`. I'm happy to split it out into a separate PR if that's preferred.
   
   ## Testing
   Tested with apps/examples/pwm on a magichome LED controller (with a bl602 chipset)


-- 
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 merged pull request #5093: Use userspace chosen channel numbers when starting bl602 pwm

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


   


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