Skip to content

task: Trim down the obsolete functionality#11

Merged
tuxuser merged 8 commits intomainfrom
task/trim_down_again
May 30, 2025
Merged

task: Trim down the obsolete functionality#11
tuxuser merged 8 commits intomainfrom
task/trim_down_again

Conversation

@tuxuser
Copy link
Copy Markdown
Contributor

@tuxuser tuxuser commented May 30, 2025

This pull request removes functionality of the firmware that is obsolete now with the release of XboxPostcodeMonitor.

While this seems like a downgrade, this is done for a better overall usability.

  • It is not feasible to update the Pico firmware for every newly discovered code or fixup
    • It does degrade the flash storage's lifetime over time
  • As this tool is most likely used by people working on varying revisions of the console hardware, there needs to be a quick way to switch between console revisions. The REPL menu or hotkeys in POST monitor mode do not seem very intuitive for that task.
  • I2C scan was performing flaky since the beginning. People of course made use of that function and misdiagnosed their consoles due to that.
    • The POST codes ideally serve the same purpose! They tell you when an I2C address is unreachable and that specific component needs investigation.

As mentioned initially, the solution is to use the new XboxPostcodeMonitor.

  • It is available for all kinds of architectures: Windows (x64, ARM64), MacOS (x64, ARM64) and Linux (x64, ARM64) !

Thank you for your understanding.

Changes:

  • ce95e18 chore: Remove obsolete scripts/
  • d1de094 chore: Only read the received byte in one place, in i2cdata receive loop
  • cb6be7d chore: Add link to errors.xboxresearch.com in help menu
  • cdc644b chore: In help, add link to XboxPostcodeMonitor
  • 918f54e chore: Remove isErrorCode flag
  • c0bfcee chore: Remove POST code names
  • f0da03e chore: Remove flaky I2C scan functionality - POST codes have the better informational value

@tuxuser tuxuser merged commit 69da0c0 into main May 30, 2025
2 checks passed
@tuxuser tuxuser deleted the task/trim_down_again branch May 30, 2025 04:24
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