board_spec: correct StopWatch / Dial / StickCPlus storage and fix the host test - #323
Merged
Merged
Conversation
StopWatch carries 8 MB of octal PSRAM and 16 MB of flash (the quad PSRAM driver fails to start on it); it was listed without PSRAM. Dial (ESP32-S3FN8) and StickCPlus (ESP32-PICO-D4) had no storage entry; record that they have no PSRAM and 8 MB / 4 MB of flash.
power2101 now starts by disabling the DCDC protections (REG 0x23), so the legacy sequence the host test compares against needs the same first step; the C++ host suite no longer compiled (operation count 5 vs 6).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small fixes to the board specification catalog.
Storage entries
Values were read from the hardware: the ESP32 Arduino core's chip report (flash chip size, PSRAM size). On StopWatch the quad PSRAM driver fails to start ("PSRAM chip is not connected, or wrong PSRAM line mode") and the octal one brings up 8 MB. The PSRAM pin roles follow the other ESP32-S3 octal PSRAM boards (e.g. AtomS3R).
generated/resolvedand the editor bundle are regenerated.Host test
power2101now starts by disabling the AXP2101 DCDC protections (REG 0x23), but the legacy sequence intest/ops_host.cppstill had 5 steps, so the C++ host suite no longer compiled (static_assert5 vs 6). The legacy sequence gets the same first step.Checks
npm test: 160 tests, 160 pass (before: the C++ host suite fails)node cli/spec.js check: OK