search

ethanho154 / VariableLightF18

0 stars 0 forks source link

PCB Review #2

Open ethanho154 opened 6 years ago

ethanho154 commented 6 years ago

Did not assign Matt Brown yet since I forgot to add him as a collaborator before.

brinnaebent commented 6 years ago

Schematic-connections are fine. Clean things up-label N$1 something informative (just good schematic practice)

PCB- change route size from 6 to 12 mil Polygon pour will make your job routeing much easier Missing routes (any yellow line is a missing route) Are you using the Arduino? If so, check spacing for headers. Right now your Arduino wouldn't fit. Disregard if using a different microcontroller Your board is very large. Consider consolidating parts and making the board smaller.

ethanho154 commented 6 years ago

Thank you for the feedback, I just updated my schematic and PCB based on the feedback and was wondering if I could get it checked again.

sphaerobolus commented 6 years ago

I've assigned it to me to remind me to check it in a day or so.

sphaerobolus commented 6 years ago

I see 6 mil traces. As Brinnae said, those should be at least 12 mil. Better yet, 20 mil.

Your headers don't connect to any MCU/board known to me. As Brinnae said, make sure that you use the correct pin spacing.

As Brinnae said, board is quite large for the components you have. You could easily place all components within the perimeter formed by the headers to connect to the MCU.

If this is to connect to an Uno, the voltage regulator is unnecessary.

I don't think we have the exact connector you've placed for 9V. You could just use a 2-pin header. The connector you've used is surface mount. It's possible, but difficult and unreliable, to solder wires to that.

ethanho154 commented 6 years ago

Hi Matt, thanks for the previous feedback! I just updated the schematic and PCB again and I think I finally made the changes you guys were talking about, could I get some feedback on the newest one? Thanks in advance!