Review 37d6a50 #1

Closed
opened 2026-03-06 15:15:06 +00:00 by Ferris · 0 comments

Review for 37d6a501a9

LedBoard

  • C* Silkscreen overlaps
  • [Verify] Screw pressure cleared (no pad clearance)

MainBoard

  • J5 Angled connector but center of board?
  • Expand unused pins to unpopulated headers
    • Currently many pin on the ESP are unused; while it's okay to not connect them, if there is space it's a good idea to put them on a generic 2xN or 1xN header so the boards can be easiyl adapted/repurposed for testing
  • LED/3V3/Prog routing can be improved
    • The LED can be placed on any pin; choose a pin closed to the power (if that is the desired topology)
    • 3.3V & Prog header are currently very far removed; re-arranging the header to be on the left of the ESP will simplify the routing
  • LED 5V is not quite safe
    • Technically it can work, but it's bad practice to do it this way
    • Why it works: 5V would break the ESP32S3 normally, but with a LED in series it drops ~1.2V, making the voltage the ESP32S3 sees 3.7V; still too high but tolerable.
    • What to do instead, connect it as an push-pull to 3.3V, or as a open-drain to 3.3V
  • [Production] Add QR code
    • We typically mark PCBs with a QR which leads to our website (as broken link initially), this allows us to add documentation later for the end-user if needed.
    • Typically the format is https://qea.info/projects/{project-name{board-name}}?src=PCB{VERSION}
  • [Production] Add Fiducial
    • This is required for the stencil & pick-and-place machine; if there is no spot for it left we can also add it on the outer frame when paneling
Review for https://git.devd.pw/tom/Inventarisatiesysteem/commit/37d6a501a94835053e1e0f4ba73b051804e5d2dd # LedBoard * [x] C* Silkscreen overlaps * [x] [Verify] Screw pressure cleared (no pad clearance) # MainBoard * [x] J5 Angled connector but center of board? * [x] Expand unused pins to unpopulated headers * Currently many pin on the ESP are unused; while it's okay to not connect them, if there is space it's a good idea to put them on a generic 2xN or 1xN header so the boards can be easiyl adapted/repurposed for testing * [x] LED/3V3/Prog routing can be improved * [x] The LED can be placed on any pin; choose a pin closed to the power (if that is the desired topology) * [ ] 3.3V & Prog header are currently very far removed; re-arranging the header to be on the left of the ESP will simplify the routing * [x] LED 5V is not quite safe * Technically it can *work*, but it's bad practice to do it this way * Why it works: 5V would break the ESP32S3 normally, but with a LED in series it drops ~1.2V, making the voltage the ESP32S3 sees 3.7V; still too high but tolerable. * What to do instead, connect it as an push-pull to 3.3V, or as a open-drain to 3.3V * [x] [Production] Add QR code * We typically mark PCBs with a QR which leads to our website (as broken link initially), this allows us to add documentation later for the end-user if needed. * Typically the format is `https://qea.info/projects/{project-name{board-name}}?src=PCB{VERSION}` * [x] [Production] Add Fiducial * This is required for the stencil & pick-and-place machine; if there is no spot for it left we can also add it on the outer frame when paneling
Ferris changed title from Review https://git.devd.pw/tom/Inventarisatiesysteem/commit/37d6a501a94835053e1e0f4ba73b051804e5d2dd to Review 2026-03-06 15:15:13 +00:00
Ferris changed title from Review to Review 37d6a50 2026-03-06 15:15:30 +00:00
tom closed this issue 2026-03-09 13:09:35 +00:00
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
tom/Inventarisatiesysteem#1
No description provided.