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 30 commits from battery_power into main
pull from: battery_power
merge into: sirlilpanda:main
Conversation 3 Commits 30 Files Changed 17 +44408 -10183
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
sirlilpanda added 15 commits 2026-06-15 15:26:40 -07:00
cleaned up sch 437cb1f7f4
made sheet A3, fixed suffix, removed en from input buck, fixed battery input, fixed TMR resistor, fixed decoupling caps voltage and value 3aff43f51c
renamed PWR to USB_input 053f2c8078
relinked files 2f26909ae8
bump c68a4acd42
cleaned up pos of decoupling caps 157730c2a2
auto commited 551e7e04c0
auto commited 19ab12ab3b
auto commited 4ee0467bee
auto commited 3ca7c37e71
auto commited 09188a2ec6
auto commited 3f45ca0df7
auto commited 8fa4decbed
auto commited e49af9f3f8
auto commited 914a60ed6d
sirlilpanda added 7 commits 2026-06-15 15:32:21 -07:00
fixed power switch d82330c539
auto commited 521e643e0a
auto commited 52c05d283c
auto commited 523e2dd9c6
auto commited 14932f43e9
auto commited 291f2c0b76
auto commited 254e2f2d56
sirlilpanda commented 2026-06-15 15:32:51 -07:00
Author
Owner
Copy Link

no switch on U12 since thats just a buck for the usbpd and the switch on u19 will turn off all down stream devices including battery power. However this does also mean when the switch is used to turn the device off the battery will not charge.

no switch on U12 since thats just a buck for the usbpd and the switch on u19 will turn off all down stream devices including battery power. However this does also mean when the switch is used to turn the device off the battery will not charge.
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?