-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ESP32-S2-Mini constantly reboots #4392
base: main
Are you sure you want to change the base?
Conversation
…stanly reboot itself, thus never allowing the Access Point to initialize
DEBUG_PRINT(F("Opening access point ")); | ||
DEBUG_PRINTLN(apSSID); | ||
WiFi.softAPConfig(IPAddress(4, 3, 2, 1), IPAddress(4, 3, 2, 1), IPAddress(255, 255, 255, 0)); | ||
WiFi.softAP(apSSID, apPass, apChannel, apHide); | ||
#ifdef ARDUINO_ARCH_ESP32 | ||
WiFi.setTxPower(wifi_power_t(txPower)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤔 the "solution" basicially disables WiFi.setTxPower(wifi_power_t(txPower))
, because trying to set TX power before AP is started will be ignored.
I don't think this can be a general solution, because some boards require setting a reduced tx power otherwise wifi will not work.
Its interesting that this solves any problem ... maybe the default TX power is too high.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As explained (see above) I don't agree with the "solution" because it's just making the TX power feature useless in AP mode.
I think we need a different solution, maybe by only setting TX power when it's different from the default value. Or an extra option in wifi settings, like "default" or "no change".
I understand that it makes TX power useless, but nonetheless, this solution
worked for me. I have nine of the ESP32-S2-mini's and seven of them would
constantly reboot when executing "WiFi.softAPConfig(IPAddress(4, 3, 2, 1),
IPAddress(4, 3, 2, 1), IPAddress(255, 255, 255, 0));". I didn't debug
further because I was working issues outside of the WLED library, but I
surmise that maybe there is a bug within the WiFi library for the mini's?
I do understand if you don't want to merge this request. I'll just keep a
one-off version for my own use. Great product though!
Thanks,
David Markson
*Cell*: (817) 584-4755
*E-mail*: ***@***.***
…On Sat, Dec 14, 2024 at 7:19 AM Frank ***@***.***> wrote:
***@***.**** requested changes on this pull request.
As explained (see above) I don't agree with the "solution" because it's
just making the TX power feature useless in AP mode.
I think we need a different solution, maybe by only setting TX power when
it's different from the default value. Or an extra option in wifi settings,
like "default" or "no change".
—
Reply to this email directly, view it on GitHub
<#4392 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BMJ3GZRGCOUUXCG3JYZKVE32FQV65AVCNFSM6AAAAABTSZEB4GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDKMBTHA3TANBRGI>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@da-markson can you provide crash dump using exception decoder? |
Unfortunately I don't have an external JTAG debugger, so I can't produce a
crash dump on the ESP32-S2-mini.
Thanks,
David Markson
*Cell*: (817) 584-4755
*E-mail*: ***@***.***
…On Sat, Dec 14, 2024 at 10:07 AM Blaž Kristan ***@***.***> wrote:
@da-markson <https://github.com/da-markson> can you provide crash dump
using exception decoder?
Without it this change makes little sense.
—
Reply to this email directly, view it on GitHub
<#4392 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BMJ3GZSKD47S5S2YSM4CXEL2FRJSNAVCNFSM6AAAAABTSZEB4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGE3DQOBSHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
the IDF has an option to "reduce TX power on brown out", but I am not sure this is enabled in WLED and also I did not check on how that is implemented and setting TX power manually will probably overrule that anyway. |
@da-markson could you check if your S2 board still works when you just comment out the line with WiFi.setTxPower(wifi_power_t(txPower)) ? |
@da-markson you do not need JTAG to get crash dumps. connect USB serial, add -D WLED_DEBUG, add |
BTW I have several Lolin S2 Mini and all of them work as expected. Perhaps there is a HW issue with your boards. |
I will complete all the requests after the holidays as all my ESP32's are
currently running in my light display.
Thanks,
David Markson
*Cell*: (817) 584-4755
*E-mail*: ***@***.***
…On Sat, Dec 14, 2024 at 2:33 PM Blaž Kristan ***@***.***> wrote:
@da-markson <https://github.com/da-markson> you do not need JTAG to get
crash dumps. connect USB serial, add -D WLED_DEBUG, add monitor_filters =
esp32_exception_decoder and monior serial output using PIO
—
Reply to this email directly, view it on GitHub
<#4392 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/BMJ3GZTHDHJBI5JIWENOGKL2FSIYVAVCNFSM6AAAAABTSZEB4GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBTGMZTMNJYGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Soultion addresses problem where SOME of the ESP-S2-Mini's would constanly reboot itself, thus never allowing the Access Point to initialize