Skip to content

Conversation

@jaylikesbunda
Copy link
Contributor

@jaylikesbunda jaylikesbunda commented Nov 27, 2025

Description

Could not flash my C5 with esptool-js and wanted to fix it for myself and other customers.

Related

espressif/esptool@81bdb76
espressif/esptool#1140

Testing

flashed all my c5 variants to make sure it didn't regress compatibility


Checklist

Before submitting a Pull Request, please ensure the following:

  • 🚨 This PR does not introduce breaking changes.
  • All CI checks (GH Actions) pass.
  • Documentation is updated as needed.
  • Tests are updated or added as necessary.
  • Code is well-commented, especially in complex areas.
  • Git history is clean — commits are squashed to the minimum necessary.

jaylikesbunda and others added 2 commits November 27, 2025 17:23
Co-Authored-By: Radim Karniš <57687985+radimkarnis@users.noreply.github.com>
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@brianignacio5 brianignacio5 merged commit e390a0d into espressif:main Dec 9, 2025
1 check passed
@TD-er
Copy link

TD-er commented Dec 9, 2025

Just a question, to help me understand...
Was the original value also a valid value, but then for pre-production silicon? (or a first revision)
Or was it just a copy/paste error or a typo?

@brianignacio5
Copy link
Collaborator

I think it was a valid value for older boards.

@TD-er
Copy link

TD-er commented Dec 9, 2025

I think it was a valid value for older boards.

Then both should be checked, right?

@brianignacio5
Copy link
Collaborator

Seems that esptool master is just using this values for all C5 targets unlike C61 which uses multiples values depending on board revision.

I might be wrong and it was not working for previous board but was not tested.

@jaylikesbunda
Copy link
Contributor Author

jaylikesbunda commented Dec 9, 2025

I think it was a valid value for older boards.

Then both should be checked, right?

can confirm it still works with v1.1 and v1.2 devkits fine

@TD-er
Copy link

TD-er commented Dec 9, 2025

Since it has been merged, I assume this code is already online here right?

When using the USB to UART port:

esptool.js
Serial port WebSerial VendorID 0x10c4 ProductID 0xea60
Connecting...
Detecting chip type... ESP32-C5
Chip is ESP32-C5 (revision v1.0)
Features: Wi-Fi 6 (dual-band),BT 5 (LE)
Crystal is 48MHz
MAC: 30:ed:a0:e4:23:24
Uploading stub...
Running stub...
Stub running...
Flash ID: 0
WARNING: Failed to communicate with the flash chip,
                                                   read/write operations will fail.
                                                                                   Try checking the chip connections or removing
        any other hardware connected to IOs.
Configuring flash size...
Could not auto-detect Flash size. defaulting detect
Detected flash size set to 4MB

This is one with a 4017 flash (vendor 0x20)

@TD-er
Copy link

TD-er commented Dec 9, 2025

Ah nevermind, with the 'Show debug log' checked, I can see I have the same Chip Magic as in the merged commit:

Debug: Chip Magic 5fd1406f
Detecting chip type... ESP32-C5
Chip is ESP32-C5 (revision v1.0)
Features: Wi-Fi 6 (dual-band),BT 5 (LE)
Crystal is 48MHz
MAC: 30:ed:a0:e4:23:24
Uploading stub...

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.

3 participants