Skip to content
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

enable slew-rate control to prevent overcurrent #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arthurbenemann
Copy link
Contributor

Slew-rate limited

slew_rate1

No slew-rate (v1.0.0 software)

slew_rate3
slew_rate2

@arthurbenemann
Copy link
Contributor Author

Some extra screenshots showing the current on the system. The displayed currents are double because of limitations on the test setup.

Factory software

scope_2

Slew rate control

scope_3

scope_4

@AJHong
Copy link

AJHong commented Sep 30, 2015

Note: Here in the lab, we have seen 3 Solos where the I2C has failed with the slew-rate ESC firmware. Symptoms have been all green LEDs, and no Battery telemetry.
When we try and flash these ESCs back to "E2" factory firmware, we have seen a cases where the ESC will short out when powered on by Solo.
My best guess would be some hardware component is being damaged while slew ESC firmware is loaded, and when you reflash to E2 firmware it shorts out that component.

@sim-
Copy link

sim- commented Sep 30, 2015

Do you have pull-downs on the drive pins to prevent them from floating or staying on when the MCU is tristated? This is a critical piece that is sometimes overlooked. If you are using the bs_nfet pin arrangement, the high side is inverted, which means that all high-side FETs will be on if the MCU tristates, but there definitely needs to still be a path to drain the gates of the low-side FETs to prevent shoot-through.

@arthurbenemann
Copy link
Contributor Author

This is the bridge arrangement (PC3/PB0 are connected straight to the MCU):
image

There doesn't seem to be any pull-down resistors. A shoot -though would explain the high-current spikes

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.

3 participants