Activity since 2025-06-28

esp-rs/espflash repo issues and pull requests
- Indicate in CLI when a partition was skipped or being verified, fix incorrect chunk size being shown. #908 [nullstalgia]
  This is meant to extend #904 to utilize the "skipped" flag in CLI, show an example usage of the verifying signal I...
  - jessebraham: Very nice, thanks for this!
- Investigate espflash library usage #794 [MabezDev] status:needs-investigation
  We don't have much from crates.io rev deps, but what we can, and should do is do a code search on Github/Gitlab to...
- Document all public types #909 [jessebraham] skip-changelog
  Probably still room for improvement with regards to docs, but this at least ensures all public types are documented...
  - SergioGasquez: LGTM! Thanks
- Fix clippy warnings #911 [JurajSadel] skip-changelog
  Those annoys me in my another branch 😆
  - JurajSadel: Sorry for the noise, I forgot to rebase on the latest commit.
- Target module fixups #910 [MabezDev] skip-changelog
  - SergioGasquez: Mmmm you went too far with search and replace
    - MabezDev: Oops, I hit match whole word instead of match word and case in vscode 😅
  - SergioGasquez: LGTM!
  - jessebraham: Thanks!
- Rust API guidelines: `connection` module #913 [SergioGasquez] skip-changelog
  Slightly touched flasher module too for C-CONV to have more sense
  - jessebraham: Thanks, LGTM!
- Apply relevant Rust API Guidelines to _library_ sections of espflash #864 [MabezDev]
  https://rust-lang.github.io/api-guidelines/checklist.html To see the relevant API only build the docs locally via...
- Apply relevant Rust API guidelines #915 [jessebraham]
  Probably still missing some minor things, but I think this is good enough for the release. Most of these things can...
  - SergioGasquez: LGTM! Just one nit question
    - SergioGasquez: Is there a reason not to import cli::parse:partition_table and use it here?
      - jessebraham: Yes, that function is gated behind the cli feature so it's not available when this feature is...
  - MabezDev: Thanks! LGTM
- Cannot connect to esp32s3 without `--no-stub` #914 [skibon02]
  Information Board: YD-ESP32-23 (https://github.com/rtek1000/YD-ESP32-23?tab=readme-ov-file) Connection: both type-c...
  - bugadani: I can understand that the UART port doesn't work given it's a USB-UART bridge we don't test for,...
  - skibon02: Well, when I monitor D+ and D- ports I see constant communication even when espflash is not...
  - bugadani: I meant what does espflash tell you?
  - skibon02: It is the same: $ espflash board-info [2025-06-30T16:24:54Z INFO ] Serial port:...
  - SergioGasquez: What espflash version are you using?
  - skibon02: $ espflash --version espflash 3.3.0
  - SergioGasquez: $ espflash --version espflash 3.3.0 Try using espflash@main : cargo install espflash --git...
  - skibon02: Now it works fine on v4.0.0-dev: $ espflash board-info [2025-07-01T08:43:09Z INFO ] Serial port:...
  - SergioGasquez: We just created #917 to fix this issue! Thanks for reporting it!
- Fix SPI detection on s3 by using the right base address #917 [MabezDev] skip-changelog
  Skipping change log because we haven't released since I broke it :D
  - JurajSadel
  - bugadani: C2 and C3 also used to use 0x6000_2000
  - SergioGasquez: Esp32c2 and Esp32c3 also have 0x6000_2000 base address
  - SergioGasquez
  - bugadani
- Version 4.0.0 #916 [jessebraham]
  - MabezDev
  - SergioGasquez: LGTM! Thanks!
- Incompatible with rust edition "2024" #833 [justbuchanan]
  I generated a simple project from template with cargo generate esp-rs/esp-idf-template cargo and was able to use...
  - SergioGasquez: espflash@4.0 is now released, so updating it would solve this issue!
- "efuse blk rev" error with new bootloaders on qemu #919 [SFrijters]
  I've read the similar issues that seem to have been resolved, but I'm still a bit out of my depth. I maintain a...
  - MabezDev: You can build an custom bootloader with more relaxed efuse min versions, and pass it via the...
  - jessebraham: You can build an custom bootloader with more relaxed efuse min versions, and pass it via the...
  - SFrijters: Is this referring to 18a4e38 ? I did see that commit and tried to apply the suggestion but it...
  - SFrijters: Ok, the problem was on my end, probably mixed up some combination of probably "fixes": after...
  - SergioGasquez: I think we should still check if the app descriptor is there in save-image, can you give a try...
  - SFrijters: You should now get: Error: × The app descriptor is not present in the `esp-hal` based...
- Add app descriptor check to save image #920 [SergioGasquez]
  Add app descriptor check to save image Rename check_idf_bootloader to check_app_descriptor Only check_app_descriptor...
  - jessebraham: This is a breaking change
    - SergioGasquez: Upps forgot that we cant do breaking changes anymore
  - jessebraham: Thanks!
- Decide where efuses code should live #851 [SergioGasquez]
  > I'm also aware that we also have some efuse field generation in esp-hal's xtask, is the plan to retire that and...
  - jessebraham: I guess now that we are exporting the eFuse field definitions, they will need to stay where...
  - MabezDev: We can always reexport from the new crate, if we choose to go down that route. That might...
- Xtask tests #918 [SergioGasquez] skip-changelog
  Do not merge
- Update check takes a *very* long time to fail #879 [darconeous] status:awaiting-response
  I noticed espflash recently started taking waaaaay longer to run on my system. A quick check of spindump indicates...
  - jessebraham: Not entirely sure what the issue is here, the timeout should be 5 seconds by default. I've...
- Improve missing app-desc errors #921 [SergioGasquez] skip-changelog
  Improves #919 (comment), related to esp-rs/esp-hal#3750
  - bugadani: Removing this is a breaking change
  - MabezDev: I think instead of making the breaking change to error, we can just improve these error...
  - MabezDev: Perfect, thank you!
- Installation failure due to yanked dependency #922 [robertbastian]
  % cargo install espflash Updating crates.io index Downloaded espflash v4.0.0 Downloaded 1 crate (233.9 KB)...
  - MabezDev: Based on https://github.com/esp-rs/espflash/blob/main/espflash/README.md#installation, use...
  - robertbastian: Ok thanks. Indicatif has released 0.18 because they made a breaking change, they probably won't...
- Add xtask command for running HIL tests #912 [JurajSadel] skip-changelog
  Opening as a draft so I can test on the HIL runners.
  - bugadani: Why do you need libc for this? Child has a kill method
    - JurajSadel: Removed, thanks!
  - bugadani: These don't seem to be used anywhere?
  - SergioGasquez: This should be git ignored (?)
  - jessebraham: The efuse_generator.rs file is out of sync with the current implementation in main, please update this.
  - jessebraham: This dependency is unused
  - jessebraham: Couple small issues noted, will do a proper review tomorrow
  - MabezDev
    - MabezDev: Suggested change /// Run...
    - MabezDev: This tests the system installed espflash, not the changes in a given PR/local working tree
  - MabezDev: This can be simplified to cargo run -p espflash --release --, I think?
  - SergioGasquez: We can just use a bool and have it false by default, this way we dont need a value parser and we...
    - JurajSadel: The reason I used Option is to make sure when no flag is provided, the espflash will be built...
    - SergioGasquez: Then we should just use default true
- esp32c6 flash reboot loop "Failed to fetch app description header" / "Factory app partition is not bootable" #868 [thecubic]
  After flashing a known good Rust program (tested successful with probe-rs on two Xiao C6s), a run --release with...
  - skibon02: I have the same issue. However, espflash v3.3.0 works fine When I add esp-bootloader-esp-idf as...
  - skibon02: Update My reboots are a little different: I (22) boot: ESP-IDF v5.4.1-426-g3ad36321ea 2nd stage...
  - skibon02: Can be reproduced with default esp-idf-template generated project when built without...
  - skibon02: Also, there is a difference in bootloader output on the same chip with the same...