Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layout Comments #5

Open
daveb377 opened this issue Jul 9, 2020 · 1 comment
Open

Layout Comments #5

daveb377 opened this issue Jul 9, 2020 · 1 comment
Labels
r0.1 Revision 0.1

Comments

@daveb377
Copy link

daveb377 commented Jul 9, 2020

  • The LED's are awfully close to the mounting holes. If the screwdriver slips, you might damage the LED's.
  • the top side needs a ground pour.
  • The R6/R7 voltage divider components need to be closer together. Shift R4 left, towards C9 and you can move R6/R7 closer.
  • If you rotate C9 180 degrees, the +28V line can be shortened up. tradeoff is the ground there gets a little worse, but adding a ground plane would mitigate this
  • The 28V1 test point is very close to the connector, it's going to be hard to probe.
  • LED on +28V line would be useful. 3 mA@28V isn't much power, and would be good to have.
@dkozel
Copy link
Owner

dkozel commented Jul 10, 2020

  • Added top ground pour
  • 28V testpoint moved away from the connector

I think reworking the entire path of the 28V line is worth doing and I'll take a shot at it for the r0.2. I think I improved it a little in this rev.

I'm going to remove the mounting holes in the second revision, so not worrying about that for now. Thanks for pointing it out though. I didn't want to add a 28V LED since the Vout Enable LED should be acting to signal that the output is on, hopefully it serves the purpose. Once the mounting holes are removed, there should be lots of space to move the LEDs to be next to the BNC connector.

@dkozel dkozel added the r0.1 Revision 0.1 label Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
r0.1 Revision 0.1
Projects
None yet
Development

No branches or pull requests

2 participants