Fix schematic and PCB errors #1

Open
AllSpiceAlice wants to merge 8 commits from develop into main
2 changed files with 0 additions and 0 deletions
Showing only changes of commit d815fc8d16 - Show all commits

LOADING
Review

Adjusted based on RPI design

!thumbnail[](RPI-PICO-R3-PUBLIC.DSN){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" doc-id="67612bb56aa7d3a48b11" diff-visibility="full" variant="default" view-coords="47.3,28.2,55.1,43.5" aspect-ratio="1.405" } Adjusted based on RPI design
Review

Added LED connected to GPIO25

!thumbnail[](RPI-PICO-R3-PUBLIC.DSN){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" doc-id="67612bb56aa7d3a48b11" diff-visibility="full" variant="default" view-coords="76.0,32.5,94.4,46.3" aspect-ratio="1.405" } Added LED connected to GPIO25
Review

@RevaReviewa , can you take a look at this new BOOTSEL switch?

!thumbnail[](RPI-PICO-R3-PUBLIC.DSN){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" doc-id="67612bb56aa7d3a48b11" diff-visibility="full" variant="default" view-coords="6.6,26.5,32.0,52.7" aspect-ratio="1.405" } @RevaReviewa , can you take a look at this new BOOTSEL switch?
Review

It looks great. Thank you for using the reference design from the library!

It looks great. Thank you for using the reference design from the library!

LOADING
brendan marked this conversation as resolved Outdated

@AllSpiceAlice, mechanical review complete. No change to board outline, or mounting features. All new components (and old components) are below the height limit. It think it's ready!

@AllSpiceAlice, mechanical review complete. No change to board outline, or mounting features. All new components (and old components) are below the height limit. It think it's ready! !thumbnail[](RPI-PICO-R3a-PUBLIC.brd){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" layers="81,82,1,32,116,115,111,67,66,42,41,65,64,40,37,38,35,36,110,109,108,107,33,34,58,57,106,105,104,103" diff-visibility="full" variant="default" view-coords="2.3,1.3,96.9,99.0" aspect-ratio="0.510" }
brendan marked this conversation as resolved Outdated

Thank you for removing the alternate crystal footprint.

!thumbnail[](RPI-PICO-R3a-PUBLIC.brd){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" layers="81,82,1,32,116,115,111,67,66,42,41,65,64,40,37,38,35,36,110,109,108,107,33,34,58,57,106,105,104,103" diff-visibility="full" variant="default" view-coords="28.3,61.1,63.9,74.4" aspect-ratio="0.510" } Thank you for removing the alternate crystal footprint.

@AllSpiceAlice , the spec for the mounting holes is 2.1mm (± 0.05mm). We're getting reports that the prototypes aren't hitting this spec. Can you check the actual file?

!thumbnail[](RPI-PICO-R3a-PUBLIC.brd){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...d815fc8d16966b6c4b65eb4b0f26d05d4212cde7" pr="1" layers="81,82,1,32,67,66,42,41,65,64,40,37,38,35,36,33,34,57" diff-visibility="full" variant="default" view-coords="23.5,7.0,80.3,17.8" aspect-ratio="0.510" } @AllSpiceAlice , the spec for the mounting holes is 2.1mm (± 0.05mm). We're getting reports that the prototypes aren't hitting this spec. Can you check the actual file?
Review

@daniel Hey do we really need 10mil trace here?

!thumbnail[](RPI-PICO-R3a-PUBLIC.brd){ diff="AllSpice-Demos/OrCAD-Demo:ad10cfced55b1abdba8d8a1c924de5d19d891bd7...b0d764f6a36b18d467021a4ffdd30a93ac150476" pr="1" layers="81,82,1,32,67,66,42,41,65,64,40,37,38,35,36,33,34,57" diff-visibility="full" variant="default" view-coords="13.0,12.5,58.6,27.7" aspect-ratio="0.510" } @daniel Hey do we really need 10mil trace here?