Informaticore/SuperPower

original mcu review

Opened this issue · 3 comments

wassfila commented

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 commented

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

wassfila commented

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 commented
  • 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.

  • 👍1