sirlilpanda/the-shrimpt-keyboard
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

battery_power #4

Open
sirlilpanda wants to merge 8 commits from battery_power into main
pull from: battery_power
merge into: sirlilpanda:main
Conversation 2 Commits 8 Files Changed 8 +37395 -7158
sirlilpanda commented 2026-06-05 16:20:54 -07:00
Owner
Copy Link

added power circuitry for powering the keyboard from a battery.

sub circuits created

  • usbpd buck converted
  • battery charger and input mux
  • battery protection circuits
  • power monitor circuit
  • boost converter for USB hub devices
  • boost converter for system
  • LDO for 3v3
  • ADC for power rail monitoring
added power circuitry for powering the keyboard from a battery. sub circuits created - usbpd buck converted - battery charger and input mux - battery protection circuits - power monitor circuit - boost converter for USB hub devices - boost converter for system - LDO for 3v3 - ADC for power rail monitoring
sirlilpanda added 8 commits 2026-06-05 16:20:54 -07:00
added rtc and connected pins to new power circuit 9781403d7c
added battery charger, buck for usbPD input, 2 boots for usb hub and power, 3v3 reg, power monitor and adc for voltage rail reading 363f24d3e0
fixed usb pd input 1c50a9a380
bump 6399e11017
bump 1b8043023f
attached modules togeather 1c4ae03c3d
cleaned up decoupling caps e68279169b
bump 60fbfede6f
sirlilpanda requested review from jordan 2026-06-05 16:21:03 -07:00
jordan changed title from battery_power to WIP: battery_power 2026-06-05 17:13:37 -07:00
jordan changed title from WIP: battery_power to battery_power 2026-06-05 17:15:10 -07:00
jordan commented 2026-06-05 17:31:33 -07:00
Collaborator
Copy Link

PWR.kicad_sch:

  1. Rename to usb_input or usb_frontend or similar
  2. D2 is backwards
  3. U6's diode between pins 5 and 2 will conduct once VBUS is raised above approximately 6 V
  4. 4-way junction above R32 should be replaced by two 3 way junctions, net label would be quite acceptable to avoid the DISCH crossing as well
`PWR.kicad_sch`: 1. Rename to `usb_input` or `usb_frontend` or similar 2. D2 is backwards 3. U6's diode between pins 5 and 2 will conduct once VBUS is raised above approximately 6 V 4. 4-way junction above R32 should be replaced by two 3 way junctions, net label would be quite acceptable to avoid the DISCH crossing as well
jordan commented 2026-06-05 18:12:01 -07:00
Collaborator
Copy Link

POWER.kicad_sch:

  1. Very tight layout, use A3 to improve readability.
  2. Suggest R suffix on R52 e.g. 30 -> 30R
  3. Suggest unit separator on R101, C81 e.g. 36.5k -> 36k5
  4. Why not put SW4 on U12? No need to run U12 if no load downstream
  5. What are the UVLO parameters used to set the circuit controlling EN on U12? The values selected imply that the system won't start until the input voltage exceeds 6.6 V. Is that intended?
  6. Suggest fuse before RVP circuitry on battery. Protect against RVP failure
  7. D12 should be on the batter side of Q4, use schottky to keep voltage diff low in RVP case
  8. R57 probably not needed. 3.7 V likely to be well within limits
  9. Consider renaming the +3.7V net to PWR_BATT or similar. Should be clear that it isn't for use elsewhere
  10. I think the instruction for the TMR resistor on U19 should read "use a resistor between 18k and 72k". This should be sized based on a desired length of the safety timer. Perhaps leave as unpopulated resistor if unsure?
  11. 22u bypass on U25 seems quite large
`POWER.kicad_sch`: 1. Very tight layout, use A3 to improve readability. 2. Suggest R suffix on R52 e.g. `30` -> `30R` 3. Suggest unit separator on R101, C81 e.g. `36.5k` -> `36k5` 4. Why not put SW4 on U12? No need to run U12 if no load downstream 5. What are the UVLO parameters used to set the circuit controlling EN on U12? The values selected imply that the system won't start until the input voltage exceeds 6.6 V. Is that intended? 6. Suggest fuse before RVP circuitry on battery. Protect against RVP failure 7. D12 should be on the batter side of Q4, use schottky to keep voltage diff low in RVP case 8. R57 probably not needed. 3.7 V likely to be well within limits 9. Consider renaming the +3.7V net to PWR_BATT or similar. Should be clear that it isn't for use elsewhere 10. I think the instruction for the TMR resistor on U19 should read "use a resistor between 18k and 72k". This should be sized based on a desired length of the safety timer. Perhaps leave as unpopulated resistor if unsure? 11. 22u bypass on U25 seems quite large
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin battery_power:battery_power
git checkout battery_power
Sign in to join this conversation.
Reviewers
No Reviewers
jordan
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
jordan sirlilpanda
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sirlilpanda/the-shrimpt-keyboard#4
No description provided.
Delete Branch "battery_power"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?