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

Fix hardfault in ledstrip driver due to division by zero #2790

Merged
merged 1 commit into from
Feb 18, 2018

Conversation

digitalentity
Copy link
Member

@digitalentity digitalentity commented Feb 18, 2018

Fixes #2782

@digitalentity digitalentity added this to the 1.9 milestone Feb 18, 2018
@digitalentity
Copy link
Member Author

@stronnag can you test this as well? Seems to work properly on my OMNIBUSF4

@stronnag
Copy link
Collaborator

yes, just building it

@stronnag
Copy link
Collaborator

Patched onto current dev ...

# version
# INAV/MATEKF405 1.9.0 Feb 18 2018 / 11:58:53 (a2a58544)

# status
System Uptime: 15 seconds
Current Time: 0000-01-01T00:00:00.000+00:00
Voltage: 0.0V (3S battery - OK)
CPU Clock=168MHz, GYRO=MPU6500, ACC=MPU6500
STM32 system clocks:
  SYSCLK = 168 MHz
  HCLK   = 168 MHz
  PCLK1  = 42 MHz
  PCLK2  = 84 MHz
Sensor status: GYRO=OK, ACC=OK, MAG=NONE, BARO=NONE, RANGEFINDER=NONE, OPFLOW=NONE, GPS=NONE
SD card: Startup failed
Stack size: 6144, Stack address: 0x10010000
I2C Errors: 0, config size: 3494, max available config: 131072
ADC channel usage:
   BATTERY : configured = ADC 1, used = none
      RSSI : configured = ADC 3, used = none
   CURRENT : configured = ADC 2, used = none
  AIRSPEED : configured = none, used = none
System load: 1, cycle time: 1003, PID rate: 997, RX rate: 49, System rate: 9
Arming disabled flags: RX CLI

# feature
Enabled: TELEMETRY LED_STRIP BLACKBOX AIRMODE PWM_OUTPUT_ENABLE 

Confirmed working.

@digitalentity digitalentity merged commit cfafebc into development Feb 18, 2018
@digitalentity digitalentity deleted the de_fix_ledstrip_hardfault branch February 18, 2018 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants