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

Bring up issues on the Silicon v1.0 design #53

Open
andrewgreenberg opened this issue Jan 9, 2025 · 2 comments
Open

Bring up issues on the Silicon v1.0 design #53

andrewgreenberg opened this issue Jan 9, 2025 · 2 comments
Assignees
Labels

Comments

@andrewgreenberg
Copy link
Member

andrewgreenberg commented Jan 9, 2025

Critical Fixes:

  • U4 (INA226)
    • SDA and SCL are connected to the wrong pins (SCL to SDA, SDA to SCL).
    • vbus should be connected to Vin-, not Vin+.
    • Reduce R36 and R37 to either zero ohms or 10 ohms. Add a capacitor across them (see datasheet).
    • Double check the footprint. Are the pads unnecessarily long?
  • U3 (LT1618)
    • Datasheet does not have any resistance between shunt and ISP/ISN. Remove resistors R34 and R35.
    • Double check the footprint. Are the pads unnecessarily long?
    • Kill TP5. I think there's no reason to have it.
  • U2 and U11 (TMP101) are just... weird. Still! It's like a curse on this idea. Make the symbol match the pin out in the datasheet. Then fix the footprint to match, and make sure that the RIGHT PIN ONE is flagged. It's current wrong on the footprint. Also, while you're fixing the footprint, consider making that slot narrower to better grab the little SOT-23-6 feet.
  • Q1 is in backward. Furthermore, there's no reason it needs to be a DMP3050. It's not like we use that anywhere else? So switch to a SOT-23 P channel transistor like our standard NTR4101.
  • See the rest of the issues; I think they have very valid points, especially adding a separate shunt resistor for the INA226 and the LT1618. Sharing them is cute, but might be super noisy for the INA226. But there might not be room.
  • Add a white serial number area at LEAST on the component surface, but a small one on the side on the cell surface would be great.
@andrewgreenberg
Copy link
Member Author

Other notes:

  • You can officially kill TP27, TP26, and TP28, TP29

@JackkP
Copy link
Contributor

JackkP commented Jan 10, 2025

Additional Comment, need to change the PN for the smaller cells in the middle from KXOB2502X8F to KXOB2502X8TF the difference being cell thickness. New PN is the thin revision from anysolar.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants