Skip to content

Rocket power board #93

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

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

Rocket power board #93

wants to merge 61 commits into from

Conversation

Pdada1
Copy link

@Pdada1 Pdada1 commented Nov 26, 2024

No description provided.

@StarlightDescender
Copy link
Contributor

Assuming this is good to review despite the PR still being marked as a draft?

  1. There's a lot of stuff on the root page, and the subsheets are basically empty. I think you could have your buck, battery charge circuitry, and current sensing on the same sheet? In general there's some messy stuff that would benefit from being tidied up, both for aesthetics but also readability.
  2. You should note the fuse current, and on that note I think 6.3A is too high? my napkin calculation says that 4A is sufficient - 2A max through the gecko on both the 5V and 12V lines, and the battery current draw due to the 5V is at most (2A * (5V/12V) * 1/e) = 0.92A (assuming an efficiency of 90%).
  3. On the 5V enable, I'm not sure why you're using the Drain-Source voltage to size R14/15? Should you not be using the Gate-Source Threshold voltage to calculate the divider such that V(HIGH) across the divider gives a voltage just above VGSth?
  4. Your P-MOSFET has an insufficient Continuous Body Diode Forward Current (1.5A) - from the buck and the gecko we can output up to 2A, is there a reason we're limiting it here?
  5. Is there a reason you don't have a low-pass RC filter on the INA2180? You should also make a note of the gain of the INA.
  6. Rather than add a solder jumper to the CAN termination resistor, you could mark it as DNP and only add it if you do actually need it.

@Pdada1 Pdada1 marked this pull request as ready for review January 15, 2025 00:15
@ManavToor
Copy link
Contributor

Please organize your schematic better, it is very difficult to read.

@Pdada1 Pdada1 requested a review from ManavToor January 29, 2025 23:51
Copy link
Contributor

@StarlightDescender StarlightDescender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Sorry I wasn't clear, I meant that you should put a differential low pass filter on the inputs to the INA.
    nvm im trolling, yes the low pass filter should be right at the ADC pin
  2. How did you size the fuse? What math/current draw/data has been referenced?
  3. Current sense resistors not sized still?
  4. Some of your footprint links are broken - There's a footprint from tutorial board, as well as several from other projects and some from folders in your downloads folder. These should all be in the project specific footprint library, and should be referenced with a relative filepath, rather than an absolute filepath.

@StarlightDescender
Copy link
Contributor

Still some weird schematic stuff - ex. lipo voltage from power subsheet goes to VCC on main sheet which only connects to 12V sense in on power subsheet?

  1. Why is R8 so small? if 2A max this gives you a V range from 0 to 2V - try to use most of the voltage range (for the PIC, 0-5V)
  2. I was thinking about this more - we also want current sense for the 12V rail to rocket, and battery charging current
  3. Not sure how I missed this one, but the buck converter wants 3 1uF input capacitors.
    image

Layout:
4. C13 and 14 are getting kind of far from the 12V input - probably best to do a 2x2 array of capacitors, if that makes sense?
5. You can move R1 and R2 closer to the buck
If you haven't already, read 9.4 of the buck datasheet, and also AN-1149

Copy link
Contributor

@StarlightDescender StarlightDescender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Fix DRC
    image
  2. Take a look at fig 2-3 (pg 15) of the datasheet for oscillator layout - you should copy this layout.
  3. center-to-center distance between the mounting holes should be some multiple of 0.5in, also rounded pcb corners please
  4. Connectors (screw terminal and gecko connector) should be right on the board edge
  5. in general you have some parts that are almost aligned but not quite? the board looks nicer if you can align components so they're all facing the same way (and so silkscreens are also all in one direction)
  6. How did you calculate the spacing for the via fence on the oscillator?

Copy link
Contributor

@StarlightDescender StarlightDescender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. totally missed this, please revert your changes to both RA and charging on the pad, and also tidy up the symbols/footprints (delete stuff you arent using - ex. don't need the L and M SOT-23-THIN footprints)
  2. Is there a reason you have CAN TX and RX on non-adjacent pins?
  3. Lots of places where you have components really smushed together - this wont be very fun to solder. You have lots of space in other places, maybe there's a way to take advantage of that?
    image
    image
    image
  4. Why run the CAN traces through two vias each?
    image
  5. There's some silkscreens that will get covered by components - L1, U4, and a few others.
  6. we cant have parts with thermal pads on both sides? currently we reflow one side, and then we try to reflow the other and then the first side melts and falls off which isnt great
  7. thermal pads should probably get thicccc traces so they can effectively dissipate heat. Something like 0.05in or so

@Joe-Joe-Joe-Joe
Copy link
Contributor

Spreadsheet is good. I wish every board designer did this
Schematic:
5V

  1. Does FLT need an external pullup? Can the PIC internal GPIO pullup source enough current? This is the only reference I could find and it says "no"
    image
    image
  2. schematic note for what this does?
    image
  3. This is kind of really broad but have you considered transient protection/reverse polarity protection? Most importantly is downstream, we don't want to reverse-voltage the 5V bus
    image
    main
  4. There is a builtin function for DNPs that makes it more obvious
    image
    12V
  5. Pullup for FLT - as above, schematic note for OVCSEL - as above
  6. Why do you use 0.01uF for Cin if datasheet recommends 0.1uF?
  7. Your schottky diode for the charge line is rated to 1A average fwd current. But your fuse is 5A to the battery (also consider that when charging you are also powering the entire bus. So the minimum current over the diode is (battery max charge current + 5V buck max input draw + 12V (2A since RILM is 475))
  8. Why is F1 a chip fuse instead of a replaceable clip fuse (I think there is a standard littlefuse holder we use elsewhere?)
  9. L1 is 42mR typ but Webench calls for 19mR
    image

PCB:

  1. Building on the charge current concerns above - are you exceeding the max current on a single gecko pin for the charge line?
    image
  2. standard hole spacing I believe is in increments of 0.5"
    image
  3. Dangling trace
    image
  4. I'm not going to check all your power trace sizes. But note that your cross-sectional area across this pour where it constricts is less than the width of the actual trace into the pin. You can almost certainly eke out a bit more space on the top-right of the gecko mounting hole
    image
  5. Bend osc traces in a "Y" shape rather than 90 deg
    image
  6. As Ash mentioned, try to spread components more evenly
  7. Your edge cut is not straight
    image
  8. Minor nit, rn your buck gnd plane goes through your pad. you can move C12 and R2 to give yourself a larger path
    image
  9. You can move this to the side to match and not overlap ss
    image
  10. Can move C13 upward to bridge pours directly, then add more GND vias below
    image
  11. Nit silkscreen labels should be oriented the same way and not overlap vias
    image

thats enough for now

@StarlightDescender
Copy link
Contributor

Joe raises a good comment re: externall PU resistor on FLT signal lines from both fuses - I see this still isn't addressed?

@StarlightDescender
Copy link
Contributor

Still haven't addressed my comment wrt both e-fuses needing to be on the same side?

@Pdada1
Copy link
Author

Pdada1 commented Mar 29, 2025

Still haven't addressed my comment wrt both e-fuses needing to be on the same side?

Was thinking we can just use hot air and get around this. Rio is pretty confident it should be ok

Copy link
Contributor

@StarlightDescender StarlightDescender left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
this is too thin if we're expecting anything close to 5A - 5A with 25C rise has a trace width of 1.6mm, so you should run a thicc trace to here rather than rely on zone fill. Same thing with the 12V output (and ideally we're not using a 25C temp increase)
2. ...what :crying-emoji-dies:
image
image

You should use one of the two central planes as a power plane.
3. In general traces are a bit messy (leaving pads at weird angles), silkscreens are overlapping with each other (or pads)

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

Successfully merging this pull request may close these issues.

Charging Board Re-respin todo
6 participants