sparkfun/OpenScale

BRD and SCH Issues v01

lewispg228 opened this issue · 3 comments

lewispg228 commented

  1. Designed by engineer name is not showing up in board file. Maybe it's worth just adding in a text object until Hord gets the new frame technique working 100%

  2. Route directly away from TMP102 (and not underneath). Please see BOB for a good example: https://github.com/sparkfun/Digital_Temperature_Sensor_Breakout_-_TMP102/

  3. Route straight out from legs on FT31X chip. The two "left-most" pads have traces routed out forming unnecessary "T"s.

  4. Version text should be "v01", not just "01"

  5. Prefixes incorrect on J1, J3, X1, LED6, LED7.

  6. Mirror Vreg circuit so it goes left to right.

  7. U6 (the ftdi chip) should be "FT231X", not "FTS31X"

  8. I don't think you want to connect 5V to the VIO pin of the FT231X. The data sheet says the absolute max for that pin is 4V. Maybe tie it to the 3V3OUT from the chip itself (this is how Jim did it in the Redboard). http://dlnmh9ip6v2uc.cloudfront.net/datasheets/Components/General%20IC/DS_FT231X.pdf (page 16)

  9. Add test point for VCC

  10. I think it would be worth breaking out the I2C lines with our standard 4-pin header layout somewhere on the board (perhaps in the upper right). This way the user can easily add other I2C sensors. But if you think the likelihood of this ever happening is too low, then leave it out. Also note, if this option is added, this would probably benefit from the PU jumper.

  11. RX LED should be yellow. (says red in sch).

  12. Add Measures to layer 47 for both outer dimensions and standoffs.

  13. Add top silk for B- B+ headers

  14. Please update 328, FTDI and TMP102 packages to bump outline lines to 8 mil, and put the polarity marking outside the package (so it is visible by production after the part is populated).

nseidle commented

  1. Fixed. Was on tplace layer. Now on tdocu.

  2. Good call. Fixed.

  3. Fixed.

  4. Fixed.

  5. Fixed.

  6. Yep. Fixed.

  7. Weird. Comes from FT231x breakout so keep an eye out. That error may be a lot of places.

  8. NICE CATCH. Fixed.

  9. Added exposed via under HX711 for VCC testing.

  10. It's low. Leave out.

  11. Fixed.

  12. Yep. Added.

  13. Added.

  14. Yep. Indicators moved and silk thickened.

lewispg228 commented

Look great. Thanks, Nate.

One small thing about the test point you added for VCC. We like to use the test point device (added into the schematic). You used a via a named it "VCC", which works, but it's nice to have the symbol in the schematic as well.

Also, it's nice to only have the stop mask happen on the bottom side of the board. The way you have it now, the via would be exposed on both top and bottom. Using the device, the package only has bstop.

(The part lives in the passive lbr called test point)

nseidle commented

Thanks Pete! I knew we had one but couldn't find the lib. Fixed in latest rev.