Arduino
Arduino copied to clipboard
Waveform: int/uint8_t inconsistency and implied stop PWM
@earlephilhower This is basically a cleanup, and I'm asking for you to review this, because it's your code.
I'm head scratching about the weak-ref magic you performed, it looks convoluted, but appears to be the only working variant - albeit I am worried that enablePhaseLockedWaveform(); does not remove the PWM bits from the final BSP like it was with the previous config menu?
@s-hadinger I am curious if you would be willing to spend some time on giving an opinion with regard to this PR and/or #8011 performing with LED strips. Real-life use cases are the most interesting, after all :-)
Sorry I'm not familiar with this code. I'm afraid I can't help.
@s-hadinger I am sorry, too, I was just going by things like https://github.com/esp8266/Arduino/pull/7231#issuecomment-616758135 and #7057 and all your much appreciated feedback to #7022