This commit is contained in:
rusefi 2018-02-28 07:57:04 -05:00
parent f1cdc789d2
commit 923a5e97a2
3 changed files with 8 additions and 2 deletions

View File

@ -417,11 +417,15 @@ typedef enum {
Force_4b_pin_input_mode = ENUM_32_BITS,
} pin_input_mode_e;
#define CRANK_MODE_MULTIPLIER 2.0f
// todo: better enum name
typedef enum {
OM_NONE = 0,
/**
* 720 degree engine cycle but trigger is defined using a 360 cycle which is when repeated
* 720 degree engine cycle but trigger is defined using a 360 cycle which is when repeated.
* For historical reasons we have a pretty weird approach where one crank trigger revolution is
* defined as if it's stretched to 720 degress. See CRANK_MODE_MULTIPLIER
*/
FOUR_STROKE_CRANK_SENSOR = 1,
/**

View File

@ -447,6 +447,7 @@ static void configureNeon1995TriggerShapeCommon(bool withCam, TriggerShape *s DE
}
void configureNeon1995TriggerShapeOnlyCrank(TriggerShape *s DECLARE_ENGINE_PARAMETER_SUFFIX) {
s->initialize(FOUR_STROKE_CRANK_SENSOR, false);
configureNeon1995TriggerShapeCommon(false, s PASS_ENGINE_PARAMETER_SUFFIX);
}

View File

@ -13,7 +13,7 @@ void configureGmTriggerShape(TriggerShape *s DECLARE_ENGINE_PARAMETER_SUFFIX) {
// all angles are x2 here - so, 5 degree width is 10
float w = 10;
float m = 2.0;
float m = CRANK_MODE_MULTIPLIER;
s->addEvent2(m * 60 - w, T_PRIMARY, TV_RISE PASS_ENGINE_PARAMETER_SUFFIX);
s->addEvent2(m * 60, T_PRIMARY, TV_FALL PASS_ENGINE_PARAMETER_SUFFIX);
@ -51,6 +51,7 @@ void initGmLS24(TriggerShape *s DECLARE_ENGINE_PARAMETER_SUFFIX) {
/**
* based on http://rusefi.com/forum/viewtopic.php?f=3&t=936&start=30 logs
* todo: do we have one cycle or two cycles here? where is CRANK_MODE_MULTIPLIER?
*/
s->addEvent2(22.2733333333334, ch, TV_RISE PASS_ENGINE_PARAMETER_SUFFIX);
s->addEvent2(27.59333333333338, ch, TV_FALL PASS_ENGINE_PARAMETER_SUFFIX);