Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

target: Add RP2350 support #1718

Merged
merged 1 commit into from
Feb 22, 2025
Merged

target: Add RP2350 support #1718

merged 1 commit into from
Feb 22, 2025

Conversation

konkers
Copy link

@konkers konkers commented Aug 8, 2024

No description provided.

@rgrr
Copy link
Contributor

rgrr commented Dec 2, 2024

any chance to have this merged?

@ithinuel ithinuel changed the base branch from main to develop February 20, 2025 08:25
Copy link
Member

@ithinuel ithinuel left a comment

Choose a reason for hiding this comment

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

The Address map for RP2350 is a bit different on some parts (eg SRAM6-9 aren’t mapped) but that’s good enough for a start.

@ithinuel
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ithinuel
Copy link
Member

Ran the tests on a Pico and a Pico2.

------ TEST RESULTS ------
Target         Test                 Result         Time           

rp2040         Basic Test           Pass           3.723          
rp2040         Json Lists Test      Pass           2.747          
rp2040         Connect Test         Pass           2.506          
rp2040         Speed Test           Pass           34.324         
rp2040         Cortex Test          Fail           4.056          
rp2040         Concurrency Test     Pass           4.022          
rp2040         Flash Test           Fail           412.727        
rp2040         Flash Loader Test    Fail           4.710          
rp2040         Debug Context Test   Pass           1.138          
rp2040         Gdb Test             Fail [ignored] 311.783        
rp2040         Commands Test        Pass           4.478          
rp2040         Commander Test       Pass           0.329          
rp2040         Probeserver Test     Pass           2.213          
rp2040         User Script Test     Pass           0.348          
rp2350         Basic Test           Pass           3.646          
rp2350         Json Lists Test      Pass           2.513          
rp2350         Connect Test         Pass           2.340          
rp2350         Speed Test           Pass           32.232         
rp2350         Cortex Test          Fail           4.151          
rp2350         Concurrency Test     Pass           3.882          
rp2350         Flash Test           Fail           124.095        
rp2350         Flash Loader Test    Pass           2.831          
rp2350         Debug Context Test   Pass           0.773          
rp2350         Gdb Test             Fail [ignored] 312.155        
rp2350         Commands Test        Pass           3.019          
rp2350         Commander Test       Pass           0.458          
rp2350         Probeserver Test     Pass           1.532          
rp2350         User Script Test     Pass           0.408          

Test Time: 1283.266
One or more tests has failed!


------ TEST RESULTS ------

Not perfect but still an improvement over the rp2040’s current support that has no tests. (and fail on similar tests when added).

Full report: result.txt

@ithinuel ithinuel merged commit a9ded22 into pyocd:develop Feb 22, 2025
24 checks passed
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