Skip to content

Commit

Permalink
Make r4.5 and older boards work again
Browse files Browse the repository at this point in the history
For the DRV8323 used on r4.5 and older, the enable line shouldn't be
read until after the enable sequence is completed.  We just do that
everywhere to make things safer.
  • Loading branch information
jpieper committed Dec 15, 2023
1 parent 2d9b911 commit 146e9ae
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion fw/drv8323.cc
Original file line number Diff line number Diff line change
Expand Up @@ -428,7 +428,7 @@ void Drv8323::Power(bool value) { impl_->Power(value); }

bool Drv8323::fault() {
return impl_->status_.fault_config ||
((impl_->enable_.read() != 0) &&
((impl_->enable_state_ == kEnabled) &&
((g_measured_hw_family == 0 && g_measured_hw_rev == 3) ?
// This revision seems to be unable to read the fault line
// properly. Thus we get a laggier version over SPI.
Expand Down

0 comments on commit 146e9ae

Please sign in to comment.