adafruit / Adafruit_TinyUSB_Arduino

Arduino library for TinyUSB
MIT License
487 stars 129 forks source link

Conflicting Types in 3.1.3 #415

Open coofercat opened 6 months ago

coofercat commented 6 months ago

Operating System

MacOS

Arduino IDE version

Arduino 2.3.2

Board

DollaTek ESP32-S3-DevKitC-1

ArduinoCore version

~1.8.6~ 2.0.13

TinyUSB Library version

3.1.3

Sketch as ATTACHED TXT

examples/hid_boot_mouse.ino (https://github.com/adafruit/Adafruit_TinyUSB_Arduino/blob/master/examples/HID/hid_boot_mouse/hid_boot_mouse.ino)

Compiled Log as ATTACHED TXT

/Users/x/Documents/Arduino/libraries/Adafruit_TinyUSB_Library/src/host/usbh.c:756:6: error: conflicting types for 'tuh_edpt_abort_xfer'
 bool tuh_edpt_abort_xfer(uint8_t daddr, uint8_t ep_addr) {
      ^~~~~~~~~~~~~~~~~~~
/Users/x/Documents/Arduino/libraries/Adafruit_TinyUSB_Library/src/host/usbh.c:720:15: note: previous implicit declaration of 'tuh_edpt_abort_xfer' was here
               tuh_edpt_abort_xfer(daddr, tu_edpt_addr(epnum, dir));
               ^~~~~~~~~~~~~~~~~~~
exit status 1
Error compiling for board ESP32S3 Dev Module.

What happened ?

In version 3.1.3, I can't compile anything using the library and get the error above. If I revert to the next previous version available (3.1.1), it compiles correctly.

How to reproduce ?

  1. With an (almost) completely fresh Arduino IDE, install Adafruit TinyUSB and its dependencies
  2. Compile the examples/hid_boot_mouse.ino sketch
  3. Observe error
  4. Downgrade the Adafruit TinyUSB library to verion 3.1.1
  5. Compile - observe it compiles and uploads

Debug Log

No response

Screenshots

No response

jenschr commented 6 months ago

I can confirm. Tested with Arduino 2.3.2 and Adafruit Feather S3 NoPSRAM, I get the same error.

hathach commented 6 months ago

BSP 1.8.6 is too old, please update to at least 2.0.14/15

coofercat commented 6 months ago

Sorry, I'm not sure where 1.8.6 came from - if I look in Boards Manager, the Arduino ESP32 Boards version is 2.0.13. I don't have an option for 14/15, but I'll look to add one of those and re-try.

Wycers commented 6 months ago

Same issue here

hathach commented 6 months ago

I will look at this soon, though anyone with “same issue” please post your setup detail (check 1st post)