Skip to content

Commit

Permalink
AP_EFI: Hirth: remove crankshaft sensor status
Browse files Browse the repository at this point in the history
There is no crankshaft sensor status reported by this EFI. This line is
misleading and should be removed. The sensor health bitmask is already
logged elsewhere.
  • Loading branch information
robertlong13 committed Sep 11, 2024
1 parent 07e9e76 commit de65762
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions libraries/AP_EFI/AP_EFI_Serial_Hirth.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@

#define ENGINE_RUNNING 4
#define THROTTLE_POSITION_FACTOR 10
#define CRANK_SHAFT_SENSOR_OK 0x0F
#define INJECTION_TIME_RESOLUTION 0.8
#define THROTTLE_POSITION_RESOLUTION 0.1
#define VOLTAGE_RESOLUTION 0.0049 /* 5/1024 */
Expand Down Expand Up @@ -301,7 +300,6 @@ void AP_EFI_Serial_Hirth::decode_data()

// EFI2 log
internal_state.engine_state = (Engine_State)record1->engine_status;
internal_state.crankshaft_sensor_status = (record1->sensor_ok & CRANK_SHAFT_SENSOR_OK) ? Crankshaft_Sensor_Status::OK : Crankshaft_Sensor_Status::ERROR;

// ECYL log
internal_state.cylinder_status.injection_time_ms = record1->injection_time * INJECTION_TIME_RESOLUTION;
Expand Down

0 comments on commit de65762

Please sign in to comment.