Side effects of uninstalling the I2S driver in the AudioOutputI2S.cpp stop() method

[riraosan]

Hi. everyone.

about arduino-esp32 2.0.3...

The latest source code uninstalls the I2S driver in the stop() method, as shown in the image above.
This process actually seems to have a side effect.
Uninstalling I2S1=1 is causing the I2S0 driver to be uninstalled.
It looks like a bug in the i2s_driver_uninstall() driver.
How can I work around this? @earlephilhower

I was using I2S0 for CVBS and I2S1 for Audio, and I noticed that when I stop() Audio, the CVBS (NTSC) signal stops at the same time.

I hope this discussion will be useful for those who use I2S0 and I2S1 at the same time.

[FedericoBusero]

As a workaround, you can revert the change f95f15f
However, then you will have

• issue Repeated playback of sound (Mp3 or Wav), unable to fix Unable to install I2S drives error #561 : error "Unable to install I2S driver"
• issue ESP32 I2S DMA Buffer Memory Leak issues？ #488 : possible memory leak in destructor

I suggest another solution:

• I would propose to solve the memory leak in the destructor by introducing another variable next to i2sOn which is true in case the I2S driver has been installed (and doesn't go to false in stop() )
• just ignore the error "Unable to install I2S driver", as it pops up when the driver is already installed