sirlilpanda/esp32-racer
1
0
Fork 0
generated from sirlilpanda/kicad-project-template-actionless
Code Issues 3 Pull Requests Actions Packages Projects Releases Wiki Activity

General comments #5

New Issue
Open
opened 2026-03-09 22:26:57 -07:00 by jordan · 2 comments
jordan commented 2026-03-09 22:26:57 -07:00
Copy Link

The following comments are too tedious to break out into their own issues, aren't that much of a fuss, or are just cool things that I noticed.

  1. Symbol D34 is backwards.
  2. Consider writing design voltages near connectors or components that are sensitive to voltage, e.g. INA232 common mode voltage, battery voltage range etc. You have done this for the USB, but I recommend not using a netlabel for this.
  3. I appreciate the addresses and modes written by each IC.
  4. Hierarchical labels should indicate the flow of control or power.
  5. I haven't checked component choices or calculations. Let me know if you want anything particular checked.
The following comments are too tedious to break out into their own issues, aren't that much of a fuss, or are just cool things that I noticed. 1. Symbol D34 is backwards. 2. Consider writing design voltages near connectors or components that are sensitive to voltage, e.g. INA232 common mode voltage, battery voltage range etc. You have done this for the USB, but I recommend not using a netlabel for this. 3. I appreciate the addresses and modes written by each IC. 4. Hierarchical labels should indicate the flow of control or power. 5. I haven't checked component choices or calculations. Let me know if you want anything particular checked.
sirlilpanda commented 2026-03-09 23:12:29 -07:00
Owner
Copy Link

  1. check link to falstad sim below the fet, this is just a protection diode that should stop the fet from blowing up when turning on and off juice from the battery

  2. :)

  3. yeah yeah yeah, i fix these for the mcu pins the rest should be alright

  4. have a goose at the bucks make sure they are spitting out the right thing

1. check link to falstad sim below the fet, this is just a protection diode that should stop the fet from blowing up when turning on and off juice from the battery 3. :) 4. yeah yeah yeah, i fix these for the mcu pins the rest should be alright 5. have a goose at the bucks make sure they are spitting out the right thing
jordan commented 2026-03-09 23:16:10 -07:00
Author
Copy Link

The Falstad simulation does not include gate capacitance so overstates how quickly the FET will switch. Add a couple hundred pico-Farads across the gate and see what happens. (Not relevant to this issue.)

D34 is not the protection diode. It is on VBUS

image.png

~~The Falstad simulation does not include gate capacitance so overstates how quickly the FET will switch. Add a couple hundred pico-Farads across the gate and see what happens.~~ (Not relevant to this issue.) D34 is not the protection diode. It is on VBUS ![image.png](/attachments/d5d8acfa-853c-477a-aae8-e3156f81ebea)
image.png
6.3 KiB
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Zekkos sirlilpanda
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sirlilpanda/esp32-racer#5
No description provided.
Delete Branch "%!s()"

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?