Skip to content

Dev board revisions#4

Open
mgordon2646 wants to merge 4 commits intomainfrom
Dev-board-revisions
Open

Dev board revisions#4
mgordon2646 wants to merge 4 commits intomainfrom
Dev-board-revisions

Conversation

@mgordon2646
Copy link
Collaborator

Still the same PCB (or is it?)

It's still the same PCB
Still the same PCB
@Joe-Joe-Joe-Joe
Copy link
Contributor

Joe-Joe-Joe-Joe commented Oct 3, 2025

  1. Please do not export the gerbers to the top level folder. Specifically, what you should do is install this JLC PCB toolkit plugin. This adds a button to your PCB editor which you just click, and it puts all your gerbers in the specific format JLC wants them, that you can zip and send directly to JLC
image
  1. GND symbols pointing up is a sin
image
  1. I don't know what solenoids you are spec'd for, but your NMOS has a max current of 200mA, and your diodes max out at 500mA. That seems concerningly low.
  2. Part number for ferrite bead. Look at processor for what was used there?
  3. I'll be very impressed if TC2/TC3 are your own inventions. That being said, seems like TC1 will win in the end. I do not have to pick apart TC2/3 but they seem cool
  4. I think Ash is too anal about daisy chaining but this is a good example of where it does actually make a difference between the board working and not working
image
  1. I am 98% sure the blue diodes we stock are also 1206
image
  1. You have infinite layout space please just label every screw terminal. I promise it will save so many headaches down the line
image
  1. In general it is not good to jump planes, esp ground planes, like this. For reference, a 0.2mm trace is ~62mR/inch, and your PT sense resistor is +/- 150mR. So in this case, probably wont make a difference, but its not nothing.
image
  1. Please super clearly mark how to orient the programmer on this
image
  1. A lot of silkscreen labels are at various rotations. We try to keep all silkscreen labels all pointing the same direction - makes it easier to read

- Copied Ferrite Bead P/N from processor
- Added external oscillator
- Made status LED pins pull-down
- Copied processor programming header
Updated Diode and NMOS to support 1A max. Fixed valve state LED to draw from BATT instead of MCU.
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.

2 participants