rusEFI
The most advanced open source ECU
Loading...
Searching...
No Matches
instant_rpm_calculator.cpp
Go to the documentation of this file.
1
2#include "pch.h"
4
5/**
6 * sensorChartMode
7 */
8#include "engine_state.h"
9
10#if EFI_UNIT_TEST
11extern bool printTriggerDebug;
12#endif
13
14#if EFI_SHAFT_POSITION_INPUT
15
17 //https://en.cppreference.com/w/cpp/language/zero_initialization
18 timeOfLastEvent()
19 , instantRpmValue()
20 {
21}
22
24 // here we take timestamps of events which happened prior to synchronization and place them
25 // at appropriate locations
26 auto triggerSize = getTriggerCentral()->triggerShape.getLength();
27
28 size_t eventsToCopy = minI(spinningEventIndex, triggerSize);
29
30 size_t firstSrc;
31 size_t firstDst;
32
33 if (eventsToCopy >= triggerSize) {
34 // Only copy one trigger length worth of events, filling the whole buffer
35 firstSrc = spinningEventIndex - triggerSize;
36 firstDst = 0;
37 } else {
38 // There is less than one full cycle, copy to the end of the buffer
39 firstSrc = 0;
40 firstDst = triggerSize - spinningEventIndex;
41 }
42
43 memcpy(timeOfLastEvent + firstDst, spinningEvents + firstSrc, eventsToCopy * sizeof(timeOfLastEvent[0]));
44}
45
47 TriggerWaveform const & triggerShape, TriggerFormDetails *triggerFormDetails,
48 uint32_t current_index, efitick_t nowNt) {
49
50 // It's OK to truncate from 64b to 32b, ARM with single precision FPU uses an expensive
51 // software function to convert 64b int -> float, while 32b int -> float is very cheap hardware conversion
52 // The difference is guaranteed to be short (it's 90 degrees of engine rotation!), so it won't overflow.
53 uint32_t nowNt32 = nowNt;
54
55 assertIsInBoundsWithResult(current_index, timeOfLastEvent, "calc timeOfLastEvent", 0);
56
57 // Record the time of this event so we can calculate RPM from it later
58 timeOfLastEvent[current_index] = nowNt32;
59
60 // Determine where we currently are in the revolution
61 angle_t currentAngle = triggerFormDetails->eventAngles[current_index];
62 efiAssert(ObdCode::OBD_PCM_Processor_Fault, !std::isnan(currentAngle), "eventAngles", 0);
63
64 // Hunt for a tooth ~90 degrees ago to compare to the current time
65 angle_t previousAngle = currentAngle - 90;
66 wrapAngle(previousAngle, "prevAngle", ObdCode::CUSTOM_ERR_TRIGGER_ANGLE_RANGE);
67 int prevIndex = triggerShape.findAngleIndex(triggerFormDetails, previousAngle);
68
69 // now let's get precise angle for that event
70 angle_t prevIndexAngle = triggerFormDetails->eventAngles[prevIndex];
71 auto time90ago = timeOfLastEvent[prevIndex];
72
73 // No previous timestamp, instant RPM isn't ready yet
74 if (time90ago == 0) {
76 }
77
78 uint32_t time = nowNt32 - time90ago;
79 angle_t angleDiff = currentAngle - prevIndexAngle;
80
81 // Wrap the angle in to the correct range (ie, could be -630 when we want +90)
82 wrapAngle(angleDiff, "angleDiff", ObdCode::CUSTOM_ERR_6561);
83
84 // just for safety, avoid divide-by-0
85 if (time == 0) {
87 }
88
89 float instantRpm = (60000000.0 / 360 * US_TO_NT_MULTIPLIER) * angleDiff / time;
90 assertIsInBoundsWithResult(current_index, instantRpmValue, "instantRpmValue", 0);
91 instantRpmValue[current_index] = instantRpm;
92
93 // This fixes early RPM instability based on incomplete data
94 if (instantRpm < RPM_LOW_THRESHOLD) {
96 }
97
99
101
102 return instantRpm;
103}
104
106 // here we remember tooth timestamps which happen prior to synchronization
107 if (spinningEventIndex >= efi::size(spinningEvents)) {
108 // too many events while trying to find synchronization point
109 // todo: better implementation would be to shift here or use cyclic buffer so that we keep last
110 // 'PRE_SYNC_EVENTS' events
111 return;
112 }
113
114 uint32_t nowNt32 = nowNt;
116
117 // If we are using only rising edges, we never write in to the odd-index slots that
118 // would be used by falling edges
119 // TODO: don't reach across to trigger central to get this info
121}
122
124 uint32_t current_index,
125 TriggerWaveform const & triggerShape, TriggerFormDetails *triggerFormDetails,
126 uint32_t index, efitick_t nowNt) {
127 UNUSED(current_index);
128
129 m_instantRpm = calculateInstantRpm(triggerShape, triggerFormDetails, index,
130 nowNt);
131#if EFI_UNIT_TEST
132 if (printTriggerDebug) {
133 printf("instantRpm = %f\n", m_instantRpm);
134 }
135#endif
136
137}
138
139#endif // EFI_SHAFT_POSITION_INPUT
void setLastEventTimeForInstantRpm(efitick_t nowNt)
float instantRpmValue[PWM_PHASE_MAX_COUNT]
float calculateInstantRpm(TriggerWaveform const &triggerShape, TriggerFormDetails *triggerFormDetails, uint32_t index, efitick_t nowNt)
uint32_t timeOfLastEvent[PWM_PHASE_MAX_COUNT]
void updateInstantRpm(uint32_t current_index, TriggerWaveform const &triggerShape, TriggerFormDetails *triggerFormDetails, uint32_t index, efitick_t nowNt)
TriggerWaveform triggerShape
angle_t eventAngles[2 *PWM_PHASE_MAX_COUNT]
Trigger shape has all the fields needed to describe and decode trigger signal.
uint16_t findAngleIndex(TriggerFormDetails *details, angle_t angle) const
size_t getLength() const
TriggerCentral * getTriggerCentral()
Definition engine.cpp:590
One header which acts as gateway to current engine state.
bool printTriggerDebug
UNUSED(samplingTimeSeconds)
@ CUSTOM_ERR_TRIGGER_ANGLE_RANGE
@ CUSTOM_ERR_6561
@ OBD_PCM_Processor_Fault
instantRpm("sync: instant RPM", SensorCategory.SENSOR_INPUTS, FieldType.INT16, 326, 1.0, 0.0, 0.0, "rpm")
bool printTriggerDebug
void wrapAngle(angle_t &angle, const char *msg, ObdCode code)
printf("\n")