raiden00pl opened a new pull request, #18916:
URL: https://github.com/apache/nuttx/pull/18916

   ## Summary
   
   BREAKING CHANGE: separate pulse count feature from PWM driver.
       
   Coupling PWM driver with pulse count feature was bad decision from the 
beginning, these are two different things:
       
   - PWM is a modulation scheme: it continuously represents a value by varying 
duty cycle, usually at a fixed frequency.
   - Pulse train generation is a finite waveform transaction: generate N 
edges/pulses with selected timing, then complete.
       
   This change introduce a new pulse count driver with new API. Now user can 
generate pulse train by providing:
       
   - high pulse length in ns
   - low pulse length in ns
   - pulse count
       
    All architectures supporting pulse count have been modified.
   
   ## Impact
   Breaking change. Users must migrate their code to use the new driver with 
new API.
   
   ## Testing
   
   TODO: I can test on stm32 archs. All stm32 and at32 are based on the same 
stm32 driver. I'll add it later.
   
   Help with tests would be useful for tivia and tlsr82 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to