-
Notifications
You must be signed in to change notification settings - Fork 85
Add XIAO ESP32-C5 board definition #393
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,51 @@ | ||||||
| { | ||||||
| "build": { | ||||||
| "arduino": { | ||||||
| "ldscript": "esp32c5_out.ld", | ||||||
| "partitions": "default_8MB.csv", | ||||||
| "memory_type": "qio_qspi" | ||||||
| }, | ||||||
| "core": "esp32", | ||||||
| "extra_flags": [ | ||||||
| "-DARDUINO_XIAO_ESP32C5", | ||||||
| "-DBOARD_HAS_PSRAM", | ||||||
| "-DARDUINO_USB_MODE=1", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why do you set USB_MODE to the fixed value 1 in the board manifest? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @sivar2311 Is correct the board has only the CDC/JTAG port. |
||||||
| "-DARDUINO_USB_CDC_ON_BOOT=1" | ||||||
| ], | ||||||
| "f_cpu": "240000000L", | ||||||
| "f_flash": "80000000L", | ||||||
| "flash_mode": "qio", | ||||||
| "psram_type": "qspi", | ||||||
| "hwids": [ | ||||||
| [ | ||||||
| "0x303a", | ||||||
| "0x1001" | ||||||
| ] | ||||||
| ], | ||||||
| "mcu": "esp32c5", | ||||||
| "variant": "XIAO_ESP32C5" | ||||||
| }, | ||||||
| "connectivity": [ | ||||||
| "wifi", | ||||||
| "bluetooth", | ||||||
| "zigbee", | ||||||
| "thread" | ||||||
| ], | ||||||
| "debug": { | ||||||
| "openocd_target": "esp32c5.cfg" | ||||||
| }, | ||||||
| "frameworks": [ | ||||||
| "arduino", | ||||||
| "espidf" | ||||||
| ], | ||||||
| "name": "Seeed Studio XIAO ESP32C5", | ||||||
| "upload": { | ||||||
| "flash_size": "8MB", | ||||||
| "maximum_ram_size": 327680, | ||||||
Jason2866 marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
| "maximum_size": 8388608, | ||||||
| "require_upload_port": true, | ||||||
| "speed": 460800 | ||||||
| }, | ||||||
| "url": "https://wiki.seeedstudio.com/SeeedStudio_XIAO_Series_Introduction/", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. URL points to the generic XIAO series page instead of the XIAO ESP32-C5–specific wiki. The board's URL should link directly to the C5 getting-started guide for PlatformIO users who follow it from the IDE. 💡 Suggested correction- "url": "https://wiki.seeedstudio.com/SeeedStudio_XIAO_Series_Introduction/",
+ "url": "https://wiki.seeedstudio.com/xiao_esp32c5_getting_started/",📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| "vendor": "Seeed Studio" | ||||||
| } | ||||||
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.
🧩 Analysis chain
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 360
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 1184
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 2636
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 721
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 4064
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 161
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 932
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 704
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 1038
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 440
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 3994
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 1676
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 2046
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 722
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 1346
🏁 Script executed:
Repository: pioarduino/platform-espressif32
Length of output: 5125
Move partition table and memory_type definitions to build. (flat) structure; remove unused ldscript.*
The
ldscriptkey underbuild.arduinois unused by the Arduino framework builder and will cause ESP-IDF builds to fail—this is the only board in the repository that defines it, and no other board needs it. Movepartitionsandmemory_typeto the flatbuild.*level (not nested underbuild.arduino) to match both the expected framework resolution paths and the configuration pattern used by seeed_xiao_esp32s3.json:Suggested structure
Remove the
build.arduino.ldscriptkey entirely.🤖 Prompt for AI Agents