Allow setting PWMAudio frequency before begin (#2726)

earlephilhower · web-flow · commit d84ff02f0322 · 2024-12-30T10:57:40.000-08:00
I2S allows setting freq before ::begin, so allow same for PWMAudio.
diff --git a/libraries/PWMAudio/src/PWMAudio.cpp b/libraries/PWMAudio/src/PWMAudio.cpp
@@ -101,7 +101,8 @@ bool PWMAudio::setFrequency(int frequency) {
         return true; // We're already at the right speed
     }
     if (_pacer < 0) {
-        return false;
+        _sampleRate = frequency;
+        return true;
     }
     uint16_t _pacer_D, _pacer_N;
     // Flip fraction(N for D, D for N) because we are using it as sys_clk * fraction(mechanic of dma_timer_set_fraction) for smaller than sys_clk values
@@ -152,6 +153,7 @@ bool PWMAudio::begin() {
     if (_pacer < 0) {
         return false;
     }
+
     uint16_t _pacer_D = 0;
     uint16_t _pacer_N = 0;
     // Flip fraction(N for D, D for N) because we are using it as sys_clk * fraction(mechanic of dma_timer_set_fraction) for smaller than sys_clk values

Original file line number	Diff line number	Diff line change
`@@ -101,7 +101,8 @@ bool PWMAudio::setFrequency(int frequency) {`
`101`	`101`	`return true; // We're already at the right speed`
`102`	`102`	`}`
`103`	`103`	`if (_pacer < 0) {`
`104`		`- return false;`
	`104`	`+ _sampleRate = frequency;`
	`105`	`+ return true;`
`105`	`106`	`}`
`106`	`107`	`uint16_t _pacer_D, _pacer_N;`
`107`	`108`	`// Flip fraction(N for D, D for N) because we are using it as sys_clk * fraction(mechanic of dma_timer_set_fraction) for smaller than sys_clk values`
`@@ -152,6 +153,7 @@ bool PWMAudio::begin() {`
`152`	`153`	`if (_pacer < 0) {`
`153`	`154`	`return false;`
`154`	`155`	`}`
	`156`	`+`
`155`	`157`	`uint16_t _pacer_D = 0;`
`156`	`158`	`uint16_t _pacer_N = 0;`
`157`	`159`	`// Flip fraction(N for D, D for N) because we are using it as sys_clk * fraction(mechanic of dma_timer_set_fraction) for smaller than sys_clk values`