search

Edzelf / ESP32-Radio

Internet radio based on ESP32, VS1053 and a TFT screen.
GNU General Public License v3.0
965 stars 227 forks source link

Update Esp32_radio_init.ino #411

Closed mitchbear closed 3 years ago

mitchbear commented 3 years ago

The rest of the project and the documentation uses pin 32 for DCS. The initialization tool sets it to pin 16. This makes things consistent.

Edzelf commented 3 years ago

The initialization tool is meant to get things started. Once the wifi is working, you can define the pins in the config page of the webinterface.

mitchbear commented 3 years ago

Agreed, but a user will be able to get things working faster and have more immediate satisfaction if the initialization tool made the radio work according to the wiring instructions in the documentation. It took me a few hours to find out why the initialization tool did not get the radio working.

On Tue, Nov 24, 2020 at 1:39 AM Ed Smallenburg notifications@github.com wrote:

The initialization tool is meant to get things started. Once the wifi is working, you can define the pins in the config page of the webinterface.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Edzelf/ESP32-Radio/pull/411#issuecomment-732777360, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACHTZC7WHWUS6LOFCSRKKLTSRN5MLANCNFSM4UAJSKLA .

mitchbear commented 3 years ago

No problem. Thanks.