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