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

original mcu review #39

Open
wassfila opened this issue Oct 20, 2020 · 3 comments
Open

original mcu review #39

wassfila opened this issue Oct 20, 2020 · 3 comments
Assignees

Comments

@wassfila
Copy link
Collaborator

Reviewer : Raspi Team : Seth K
here are a couple easy steps that can make it easier to read.
First, I'd recommend using SOME net labels on page

  1. This would reduce the wiring clutter, and make the more important signals stand out.
  2. I would recommend adding expanding each circuit block so the circuits have more room to "breathe", and adding a text label under each block to indicate what they are/do.
@Vincent-Stragier
Copy link
Collaborator

82af7b3, adds comments to describe the function of most of the blocks.

@wassfila
Copy link
Collaborator Author

Feedback from team meeting review notes from @Informaticore
Meeting Notes:

Schematic review to da max

- freeze schematics after last change for regulators
- create ticket and reviews/findings from everybody there
- https://github.com/Informaticore/SuperPower/issues/39
- check requirements against the current schematics if they are met

Components annotation for BOM

  • use same unit for components (100nF vs. 0.1uF)
  • deciding for footprints
    • Resistor: 0603
    • avoid BGA
    • capacitor X7R rating
      • <10uF 0603
      • '>10uF 0805
    • wisley choose inductors
  • set the foot prints
  • swap components by multible different values to reduce component number
  • one person should do that cleanup/setting

ToDo:

  • put LED indicator for Power on VIN on the charger sheet

@wassfila
Copy link
Collaborator Author

wassfila commented Nov 16, 2020

  • check again if the +5V reg should not get a Mosfet instead of driving the Enable pin
    uA current won tbd.

Edit: The MCU regulator should be switched as well.

@MantaRayDeeJay MantaRayDeeJay added the Schematic Board Schematic label Feb 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants