Skip to content

Commit e428e80

Browse files
willmmilesArcitec
authored andcommitted
bus_wrapper: Update comments to reflect RMT usage
1 parent cfa8b73 commit e428e80

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

wled00/bus_wrapper.h

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1436,7 +1436,8 @@ class PolyBus {
14361436
// ESP32-S2 only has 4 RMT channels
14371437
if (_useParallelI2S) {
14381438
if (num > 11) return I_NONE;
1439-
if (num < 8) offset = 1; // use x8 parallel I2S0 channels (use last to allow Audioreactive)
1439+
if (num < 8) offset = 1; // use x8 parallel I2S0 channels followed by RMT
1440+
// Note: conflicts with AudioReactive if enabled
14401441
} else {
14411442
if (num > 4) return I_NONE;
14421443
if (num > 3) offset = 1; // only one I2S0 (use last to allow Audioreactive)
@@ -1449,15 +1450,15 @@ class PolyBus {
14491450
// On ESP32-S3 only the first 4 RMT channels are usable for transmitting
14501451
if (_useParallelI2S) {
14511452
if (num > 11) return I_NONE;
1452-
if (num < 8) offset = 1; // use x8 parallel I2S LCD channels
1453+
if (num < 8) offset = 1; // use x8 parallel I2S LCD channels, followed by RMT
14531454
} else {
14541455
if (num > 3) return I_NONE; // do not use single I2S (as it is not supported)
14551456
}
14561457
#else
14571458
// standard ESP32 has 8 RMT and x1/x8 I2S1 channels
14581459
if (_useParallelI2S) {
14591460
if (num > 15) return I_NONE;
1460-
if (num < 8) offset = 1; // 8 RMT followed by 8 I2S
1461+
if (num < 8) offset = 1; // 8 I2S followed by 8 RMT
14611462
} else {
14621463
if (num > 9) return I_NONE;
14631464
if (num == 0) offset = 1; // prefer I2S1 for 1st bus (less flickering but more RAM needed)

0 commit comments

Comments
 (0)