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

Remove ext_io_conf, otg_io_conf from usb_phy_config_t phy_config in usb_serial_init #1

Closed
wants to merge 1 commit into from

Conversation

smartin015
Copy link

Hello!

In attempting to use this lib with PlatformIO on a Lolin s2 mini, I came across this build error:

.pio/libdeps/lolin_s2_mini/esp32-usb-serial/src/esp32_usb_serial.cpp: In function 'esp_err_t usb_serial_init()':
.pio/libdeps/lolin_s2_mini/esp32-usb-serial/src/esp32_usb_serial.cpp:121:3: error: 'usb_phy_config_t' has no non-static data member named 'ext_io_conf'

In the test file

those fields aren't specified, so I figured it was probably safe to remove them.

My barebones PlatformIO example is available at https://github.com/smartin015/esp32-usb-serial-pio/ - currently builds, but I haven't attempted to run it on a device yet.

@luc-github
Copy link
Owner

luc-github commented Jul 11, 2024

what idf version are you using with your arduino esp32 core? and what arduino core version are you using ?

@luc-github
Copy link
Owner

no feedback means not valid issue
I close

@luc-github luc-github closed this Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants