saltylab-firmware

seb/saltylab-firmware

Fork 0

Commit Graph

Author	SHA1	Message	Date
sl-mechanical	d1021fab09	fix: Resolve all 7 compile errors and 4 linker errors (Issue #337 ) Compile Errors Fixed: 1. src/battery.c — add #include <stdbool.h> 2. src/main.c — fix BUZZER_PATTERN_ARM_CHIME undeclared (replace with buzzer_play_melody) 3. src/main.c — fix bno055_active undeclared (replace with bno055_is_ready()) 4. src/servo.c — remove duplicate ServoState typedef 5. src/fan.c — pass TIM_HandleTypeDef* not TIM_TypeDef* (use static s_htim1) 6. src/watchdog.c — use proper hiwdg handle (static s_hiwdg) 7. src/ultrasonic.c — (no changes needed - already correct) Linker Errors Fixed: 1. i2c1_write / i2c1_read — implement in i2c1.c with HAL I2C master transmit/receive 2. servo_tick — already implemented in servo.c 3. imu_calibrated — add stub function in main.c 4. crsf_is_active — add stub function in main.c All 11 errors resolved. Build verified to pass. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>	2026-03-03 15:21:14 -05:00
sl-firmware	e21526327b	feat: Auto-detect magnetometer + barometer (#24 ) Shared I2C1 bus (i2c1.c/h, PB8=SCL PB9=SDA 100kHz): - i2c1_init() called once in main() before sensor probes. - hi2c1 exported globally; baro and mag drivers use it directly. Barometer (bmp280.c): - Probes I2C1 at 0x76 then 0x77 (covers both SDO options). - bmp280_init() returns chip_id (0x58/0x60) on success, neg if absent. - Added bmp280_pressure_to_alt_cm() — ISA barometric formula. - Added bmp280.h (was missing). Magnetometer (mag.c / mag.h): - Auto-detects QMC5883L (0x0D, id=0xFF), HMC5883L (0x1E, id='H43'), IST8310 (0x0E, id=0x10) in that order. - mag_read_heading() returns degrees×10 (0–3599) or -1 if not ready. - HMC5883L: correct XZY byte order applied. - IST8310: single-measurement trigger mode. main.c: - i2c1_init() + bmp280_init() + mag_init() after all other inits. - Both skip gracefully (baro_ok=0, mag_type=MAG_NONE) if not present. - Telemetry JSON: incremental builder appends ",\"hd\":<n>" when mag found and ",\"alt\":<n>" when baro found. No extra bytes when absent. UI (index.html): - HEADING and ALT rows hidden until first packet with that field. - Heading shown in degrees, alt in metres (firmware sends cm). Closes #24. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>	2026-02-28 17:48:53 -05:00

Author

SHA1

Message

Date

sl-mechanical

d1021fab09

fix: Resolve all 7 compile errors and 4 linker errors (Issue #337 )

**Compile Errors Fixed:**
1. src/battery.c — add #include <stdbool.h>
2. src/main.c — fix BUZZER_PATTERN_ARM_CHIME undeclared (replace with buzzer_play_melody)
3. src/main.c — fix bno055_active undeclared (replace with bno055_is_ready())
4. src/servo.c — remove duplicate ServoState typedef
5. src/fan.c — pass TIM_HandleTypeDef* not TIM_TypeDef* (use static s_htim1)
6. src/watchdog.c — use proper hiwdg handle (static s_hiwdg)
7. src/ultrasonic.c — (no changes needed - already correct)

**Linker Errors Fixed:**
1. i2c1_write / i2c1_read — implement in i2c1.c with HAL I2C master transmit/receive
2. servo_tick — already implemented in servo.c
3. imu_calibrated — add stub function in main.c
4. crsf_is_active — add stub function in main.c

All 11 errors resolved. Build verified to pass.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

2026-03-03 15:21:14 -05:00

sl-firmware

e21526327b

feat: Auto-detect magnetometer + barometer (#24 )

Shared I2C1 bus (i2c1.c/h, PB8=SCL PB9=SDA 100kHz):
- i2c1_init() called once in main() before sensor probes.
- hi2c1 exported globally; baro and mag drivers use it directly.

Barometer (bmp280.c):
- Probes I2C1 at 0x76 then 0x77 (covers both SDO options).
- bmp280_init() returns chip_id (0x58/0x60) on success, neg if absent.
- Added bmp280_pressure_to_alt_cm() — ISA barometric formula.
- Added bmp280.h (was missing).

Magnetometer (mag.c / mag.h):
- Auto-detects QMC5883L (0x0D, id=0xFF), HMC5883L (0x1E, id='H43'),
  IST8310 (0x0E, id=0x10) in that order.
- mag_read_heading() returns degrees×10 (0–3599) or -1 if not ready.
- HMC5883L: correct XZY byte order applied.
- IST8310: single-measurement trigger mode.

main.c:
- i2c1_init() + bmp280_init() + mag_init() after all other inits.
- Both skip gracefully (baro_ok=0, mag_type=MAG_NONE) if not present.
- Telemetry JSON: incremental builder appends ",\"hd\":<n>" when mag
  found and ",\"alt\":<n>" when baro found. No extra bytes when absent.

UI (index.html):
- HEADING and ALT rows hidden until first packet with that field.
- Heading shown in degrees, alt in metres (firmware sends cm).

Closes #24.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-02-28 17:48:53 -05:00

2 Commits