incorrect figure/description in PWM documentation for nrf52840

[tague]

it seems that Figure 2 on this page https://infocenter.nordicsemi.com/index.jsp?topic=%2Fps_nrf52840%2Fpwm.html is incorrect, or at a minimum the figure does not do what the text on the page describes.  the caption describes a falling-edge PWM but the figure seems to show a rising-edge PWM.  my team spent a lot of time debugging PWM driver issues due to confusion around this issue.

[Karl Ylvisaker]

Hello,

it seems that Figure 2 on this page https://infocenter.nordicsemi.com/index.jsp?topic=%2Fps_nrf52840%2Fpwm.html is incorrect, or at a minimum the figure does not do what the text on the page describes.  the caption describes a falling-edge PWM but the figure seems to show a rising-edge PWM.  my team spent a lot of time debugging PWM driver issues due to confusion around this issue.

You are absolutely correct - the PWM waveforms depicted in Figure 2 both have Rising Edge polarity. 

I have created an internal ticket about this to our technical writers, and they should have the documentation updated as soon as possible.
I have also asked them to clarify the polarity text parts, in order to make explain the polarity parameter better.
Thank you for bringing this to our attention, we appreciate the feedback!

I am sorry for any confusion this may have caused.

Best regards,
Karl