Skip to content

fix esp32c3 chip description#205

Merged
brianignacio5 merged 1 commit intomainfrom
bugfix/esp32c3-chip-desc
Aug 4, 2025
Merged

fix esp32c3 chip description#205
brianignacio5 merged 1 commit intomainfrom
bugfix/esp32c3-chip-desc

Conversation

@brianignacio5
Copy link
Copy Markdown
Collaborator

This pull request refactors the ESP32C3ROM class in src/targets/esp32c3.ts to enhance chip versioning and description functionality. The changes include splitting the chip version retrieval into separate methods for major and minor versions and improving the chip description logic to include detailed revision information.

Enhancements to chip versioning:

  • Added getMinorChipVersion(loader: ESPLoader): Promise<number> method to calculate the minor chip version by reading specific EFUSE register bits.
  • Added getMajorChipVersion(loader: ESPLoader): Promise<number> method to calculate the major chip version by reading specific EFUSE register bits.

Improvements to chip description:

  • Refactored getChipDescription(loader: ESPLoader) to include a mapping of chip package types and detailed revision information using the newly added major and minor version methods.

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.

@brianignacio5 brianignacio5 self-assigned this Aug 4, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 4, 2025

Download the artifacts for this pull request:

@brianignacio5 brianignacio5 merged commit 612f57e into main Aug 4, 2025
1 check passed
@brianignacio5 brianignacio5 deleted the bugfix/esp32c3-chip-desc branch August 4, 2025 07:38
tyeth pushed a commit to tyeth/read-partitions-esptool-js that referenced this pull request Oct 15, 2025
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.

1 participant