feat(ledc): Replace log2 with __builtin_ctz

P-R-O-C-H-Y · P-R-O-C-H-Y · commit 1acdd7819b34 · 2024-02-19T14:37:13.000+01:00
diff --git a/cores/esp32/esp32-hal-ledc.c b/cores/esp32/esp32-hal-ledc.c
@@ -136,7 +136,7 @@ bool ledcAttach(uint8_t pin, uint32_t freq, uint8_t resolution)
         log_e("No more LEDC channels available! (maximum is %u channels)", LEDC_CHANNELS);
         return false;
     }
-    uint8_t channel = log2(free_channel & -free_channel);
+    uint8_t channel = __builtin_ctz(free_channel); // Convert the free_channel bit to channel number
 
     return ledcAttachChannel(pin, freq, resolution, channel);
 }

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ bool ledcAttach(uint8_t pin, uint32_t freq, uint8_t resolution)`
`136`	`136`	`log_e("No more LEDC channels available! (maximum is %u channels)", LEDC_CHANNELS);`
`137`	`137`	`return false;`
`138`	`138`	`}`
`139`		`- uint8_t channel = log2(free_channel & -free_channel);`
	`139`	`+ uint8_t channel = __builtin_ctz(free_channel); // Convert the free_channel bit to channel number`
`140`	`140`
`141`	`141`	`return ledcAttachChannel(pin, freq, resolution, channel);`
`142`	`142`	`}`