rusefi / alphax-4chan

3 stars 4 forks source link

use 9924 on one of VR channels #160

Closed rusefillc closed 8 months ago

ElDominio commented 10 months ago

bumping

9924 is a godsend on CKP, not really on CMP. Discrete is very good at decoding lower tooth cam signals, and 9924 is not so

Also, some cars (toyotas) are 3x VR. CKP should go to 9924, and 2x cam to discrete

C2/C3 to discrete1 E3/E4 to discrete2 C4/C7 to 9924

Qwerty-OFF commented 8 months ago

image Should I move the VR input from the previously used E5/E6? Is it necessary to maintain HALL support for the third VR input? The MAX9924 allows to natively use one of the lines as a HALL input, and this functionality can be activated either by manually soldering a resistor or programmatically with a number of additional components. To use both inputs as HALL, the scheme currently used for C2/C3 can be replicated.

ElDominio commented 8 months ago

I miswrote/was dumb, I meant E5/E6 as discrete2. My idea would be keep C2/C3 and E5/E6 as in revH (with same amount of pins useable as hall), while being able to have 9924 on pins C4/C7. Being able to use them both as HALL inputs allows use to not lose digital pins when C4/C7 are unused (since C4 is 2Step)

So C2/C3 = discrete1 (both pins can be used as hall) E5/E6 = discrete2 (only one pin can be used as hall, as is in the current revision) C4/C7 = 9924 (both pins can be used as hall programatically)

Qwerty-OFF commented 8 months ago

The ability to enable pulldown on the C4/C7 lines will be removed. The pullup for use as a Hall sensor will be similar to the C2, C3, E6 lines.