Skip to content

Support for RP2350 A2/A3/A4 along with code improvements#267

Merged
kilograham merged 15 commits intodevelopfrom
revisions_cleanup
Aug 7, 2025
Merged

Support for RP2350 A2/A3/A4 along with code improvements#267
kilograham merged 15 commits intodevelopfrom
revisions_cleanup

Conversation

@kilograham
Copy link
Copy Markdown
Contributor

  • replaced the ocean of one-off IFs with a more OO abstraction

…he ocean of one-off IFs with a more OO abstraction
@kilograham kilograham requested a review from will-v-pi August 6, 2025 13:43
@lurch
Copy link
Copy Markdown
Contributor

lurch commented Aug 6, 2025

CI seems very unhappy
EDIT: fixed

Comment thread main.cpp Outdated
Comment thread main.cpp Outdated
Comment thread picoboot_connection/picoboot_connection_cxx.h Outdated
@lurch lurch changed the title Support for RP2350 A2/A4/A4 along with code improvements Support for RP2350 A2/A3/A4 along with code improvements Aug 6, 2025
Comment thread README.md
Comment thread main.cpp Outdated
Comment thread main.cpp Outdated
Comment thread main.cpp Outdated
Comment thread main.cpp Outdated
Comment thread main.cpp Outdated
Copy link
Copy Markdown
Contributor

@will-v-pi will-v-pi left a comment

Choose a reason for hiding this comment

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

Tested and all LGTM

@kilograham kilograham merged commit c276650 into develop Aug 7, 2025
45 checks passed
@kilograham kilograham deleted the revisions_cleanup branch August 7, 2025 19:57
kilograham added a commit that referenced this pull request Aug 7, 2025
* Support for RP2350 A2/A4/A4 along with code improvements, replacing the ocean of one-off IFs with a more OO abstraction

Co-authored-by: William Vinnicombe <william.vinnicombe@raspberrypi.com>
Co-authored-by: will-v-pi <108662275+will-v-pi@users.noreply.github.com>
Co-authored-by: Andrew Scheller <andrew.scheller@raspberrypi.com>
Co-authored-by: graham sanderson <graham@spiff.local>
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