Skip to content
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

[Examples/peripherals/usb_serial_jtag] Stack size increase required. (IDFGH-12190) #13245

Closed
3 tasks done
GTCLive opened this issue Feb 23, 2024 · 2 comments
Closed
3 tasks done
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally

Comments

@GTCLive
Copy link

GTCLive commented Feb 23, 2024

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

Tested on ESP32-S3
ESP-IDF: v5.3-dev-2087-g5f42493573

Crash loop occurs. The fix/workaround is to revise line 16 in usb_serial_echo_main.c
From
#define ECHO_TASK_STACK_SIZE (2048)
To
#define ECHO_TASK_STACK_SIZE (4096)

Ref:
https://github.com/espressif/esp-idf/blob/master/examples/peripherals/usb_serial_jtag/usb_serial_jtag_echo/main/usb_serial_echo_main.c

Amazing SDK, appreciate all the efforts guys. Keep it up.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Feb 23, 2024
@github-actions github-actions bot changed the title [Examples/peripherals/usb_serial_jtag] Stack size increase required. [Examples/peripherals/usb_serial_jtag] Stack size increase required. (IDFGH-12190) Feb 23, 2024
@GTCLive
Copy link
Author

GTCLive commented Feb 23, 2024

Or remove Line 44. Which doesn't print back anything on the JTAG port anyway (only uart ports perhaps?).
Line 44: ESP_LOG_BUFFER_HEXDUMP("Recv str: ", data, len, ESP_LOG_INFO);

The usb_serial_jtag_write_bytes() works perfectly fine otherwise!

@igrr
Copy link
Member

igrr commented Feb 23, 2024

print back anything on the JTAG port anyway (only uart ports perhaps?).

Yes, the idea is that it prints to the UART port.

Thanks for the issue report and the suggested fix!

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Opened Issue is new labels Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally
Projects
None yet
Development

No branches or pull requests

3 participants