Peugeot 505 V6 60-2-2-2 trigger wheel

[Manu]

Hello developers,

I'm currently mount a MS2 on a Peugeot 505 V6 car. This car have a trigger wheel with 60-2-2-2 pattern. This pattern is very similar to Renix 66-2-2-2. The motor is the same. We won't do COP because the motor will keep 'stock'.

Trigger wheel pattern : 60-2-2-2
each tooth is 6°
the pattern 20-2 is repeated each 120°. There is 18 tooth and 2 missing ones, for a total of 20 each 120°.

Based on old Renix code, I make several changes to match this pattern. The code compile well. Can you please tell me if I do a mistake ? I want to be sure before flashing the code to the MS. Here is my new code :

Code: Select all

       /* ----------------------  Peugeot 505V6 60-2-2-2 ------------------------*/

    } else if (spkmode == 16) {
        no_teeth = 18;  // for single coil
        last_tooth = no_teeth;
        no_triggers = 1;
        deg_per_tooth[0] =  60; // 6° degrees per tooth
        deg_per_tooth[1] =  60;
        deg_per_tooth[2] =  60;
        deg_per_tooth[3] =  60;
        deg_per_tooth[4] =  60;
        deg_per_tooth[5] =  60;
        deg_per_tooth[6] =  60;
        deg_per_tooth[7] =  60;
        deg_per_tooth[8] =  60;
        deg_per_tooth[9] =  60;
        deg_per_tooth[10] =  60;
        deg_per_tooth[11] =  60;
        deg_per_tooth[12] =  60;
        deg_per_tooth[13] =  60;
        deg_per_tooth[14] =  60;
        deg_per_tooth[15] =  60;
        deg_per_tooth[16] =  60;
        deg_per_tooth[17] =  180; // double missing tooth = time * 3

        smallest_tooth_crk = 60;
        smallest_tooth_cam = 0;

        trigger_teeth[0] = 14;
        trig_angs[0] = -120 + tmp_offset; // 12 ATDC

        /* ----------------------  Suzuki swift ------------------------*/
    }

I will adjust the offset (if needed) using a strobe.

At least, I'm currently at the customer house and I'm a little urged

Best regards,
Manu

[Manu]

Thank James for your fast reply.

I'd test this code and it give me a sync error each 4 patterns. I think there is something wrong in it.

I will try this one today :

Code: Select all

/* ----------------------  Peugeot 505V6 60-2-2-2 ------------------------*/

    } 		else if (spkmode == 16) {
        int t;

        smallest_tooth_cam = 0;
        no_teeth = 18;  // for single coil
        no_triggers = 1;

		if (num_cyl == 6) {
            for (t = 0 ; t < 17; t++) {
                deg_per_tooth[t] = 60 + (t & 1); // 6 degrees per tooth
            }
            deg_per_tooth[17] =  180; // double missing tooth = time * 3
            smallest_tooth_crk = 60;
            trigger_teeth[0] = 14;
            trig_angs[0] = -120 + tmp_offset; // 16.4 ATDC
        } 
		else {
            conf_err = 1;
        }

        last_tooth = no_teeth;

        /* ----------------------  Suzuki swift ------------------------*/
    }

Bests,
Manu

[Manu]

What I don't understand is what those variables do :

Code: Select all

no_triggers =
trigger_teeth[0] =
trig_angs[0] =

If someone can explain it to me ;-)

What I think is that :

• trigger_teeth[0] is the number of the teeth that will trigger the ign event
• trig_angs[0] is the angle between the teeth that will trigger the ign event AND the #1TDC
• no_triggers = is the number of triggers that we should have for the pattern described in the code

Correct ?
Thx in advance

[Manu]

Attached is a csv and msq from the car I work on

[Manu]

Hi,

Just an update : when I use the second style of code, the problem remain the same.

Any idea ?

thx,
Manu

[Manu]

I will try to descibe the pull pattern in the code :

Code: Select all

else if (spkmode == 16) {
		no_teeth = 54;  // for single coil
        last_tooth = no_teeth;
        no_triggers = 3;
        deg_per_tooth[0] =  60; // 6° degrees per tooth
        deg_per_tooth[1] =  60;
        deg_per_tooth[2] =  60;
        deg_per_tooth[3] =  60;
        deg_per_tooth[4] =  60;
        deg_per_tooth[5] =  60;
        deg_per_tooth[6] =  60;
        deg_per_tooth[7] =  60;
        deg_per_tooth[8] =  60;
        deg_per_tooth[9] =  60;
        deg_per_tooth[10] =  60;
        deg_per_tooth[11] =  60;
        deg_per_tooth[12] =  60;
        deg_per_tooth[13] =  60;
        deg_per_tooth[14] =  60;
        deg_per_tooth[15] =  60;
        deg_per_tooth[16] =  60;
        deg_per_tooth[17] =  180; // double missing tooth = time * 3
        deg_per_tooth[18] =  60;
        deg_per_tooth[19] =  60;
        deg_per_tooth[20] =  60;
        deg_per_tooth[21] =  60;
        deg_per_tooth[22] =  60;
        deg_per_tooth[23] =  60;
        deg_per_tooth[24] =  60;
        deg_per_tooth[25] =  60;
        deg_per_tooth[26] =  60;
        deg_per_tooth[27] =  60;
        deg_per_tooth[28] =  60;
        deg_per_tooth[29] =  60;
        deg_per_tooth[30] =  60;
        deg_per_tooth[31] =  60;
        deg_per_tooth[32] =  60;
        deg_per_tooth[33] =  60;
        deg_per_tooth[34] =  60;
        deg_per_tooth[35] =  180; // double missing tooth = time * 3
        deg_per_tooth[36] =  60; 
        deg_per_tooth[37] =  60;
        deg_per_tooth[38] =  60;
        deg_per_tooth[39] =  60;
        deg_per_tooth[40] =  60;
        deg_per_tooth[41] =  60;
        deg_per_tooth[42] =  60;
        deg_per_tooth[43] =  60;
        deg_per_tooth[44] =  60;
        deg_per_tooth[45] =  60;
        deg_per_tooth[46] =  60;
        deg_per_tooth[47] =  60;
        deg_per_tooth[48] =  60;
        deg_per_tooth[49] =  60;
        deg_per_tooth[50] =  60;
        deg_per_tooth[51] =  60;
        deg_per_tooth[52] =  60;
        deg_per_tooth[53] =  180; // double missing tooth = time * 3
		
        smallest_tooth_crk = 60;
        smallest_tooth_cam = 0;

        trigger_teeth[0] = 12;
		trigger_teeth[1] = 30;
		trigger_teeth[2] = 48;
        trig_angs[0] = -120 + tmp_offset; // 12 ATDC
		trig_angs[1] = -120 + tmp_offset; // 12 ATDC
		trig_angs[2] = -120 + tmp_offset; // 12 ATDC

[Manu]

Hello,

I would like to thank James who point me to the "re-sync" code for trigger wheels. It look like it work. Now I have to check the timing with a strobe.

I still would like to know what those settings do :

no_triggers =
trigger_teeth[x] =
trig_angs[x] =

Regarding the code, it look like no_triggers = tell how much ign events should be triggered for the pattern it's related to. But I didn't understand, if my supposition is correct, what the 2 others settings do.

If someone can explain this to me. Thx.
Manu

[jsmcortina]

trigger_teeth[x] tells the code what tooth to use as the trigger.
trig_angs[x] tells it what crank angle (BTDC) that trigger tooth is at.

James

[Manu]

thx James.

But what trigger this setting report ? The main (crank) trigger or the second (cam) ? I think it's crank one but I unsure.

What I sure is that current Renix Code is not correct. You describe 2 patterns (4 with COP) in this wheel settings. 44-2-2 and 66-2-2-2. For the first pattern we have a 22-2 pattern each 180°. In the second one we have a 22-2 pattern each 120°.

So I think that trig_angs, at the same trigger_teeth can't be the same between 44-2-2 and 66-2-2-2 ? Actually the code say 16,4° for both.

Assuming your choice for trigger_teeth is teeth 14 (a random choice or is there a logic here ?), trig_angs should be (max_tooth - trigger_teeth) x teeth_angle ?

• So for 44-2-2 it should be 20-14 x ang = 6 x 8.2 + 1x 24.6 = 73,8° because when teeth 14 pass behind the crank sensor, the engine should rev approximately 73,8° more before #1 TDC. In the code we have 16.4° so I think I misunderstood something here.
• Same for 66-2-2-2 it should be 20-14 x ang = 6 x 5.4 + 1x 16.5 = 48,9° because when teeth 14 pass behind the crank sensor, the engine should rev approximately 48,9° more before #1 TDC. In the code we have 16.4°.

In all cases, is there something I can do to help you to fix this ? I got both cars to test.

Bests,
Manu

[jsmcortina]

trig_angs should typically be -100 or less.
Pick a value for trig_tooth that achieves this.

James