Skip to content

Update to include Rover MEMS trigger patterns#1

Open
mike501 wants to merge 1 commit intomasterfrom
mike501-patch-1
Open

Update to include Rover MEMS trigger patterns#1
mike501 wants to merge 1 commit intomasterfrom
mike501-patch-1

Conversation

@mike501
Copy link
Copy Markdown
Owner

@mike501 mike501 commented Nov 3, 2020

Rover MEMS patterns added without cam details as they're not currently available. Patterns as below with trigger at 55 degress BTDC,
MEMS pre 1.9 (1.0, 1.3 etc) - 36-1-1 gaps at 0 degrees and 180 degrees
MEMS 1.9 36-1-1-1-1 gaps at 0, 100, 180, 310
MEMS 2.0 36-1-1-1-1 gaps at 30,60, 210, 250
MEMS 3.0 36-1-1-1-1 gaps at 80, 110, 260, 300

Rover MEMS patterns added without cam details as they're not currently available. Patterns as below with trigger at 55 degress BTDC, 
MEMS pre 1.9 (1.0, 1.3 etc) - 36-1-1 gaps at 0 degrees and 180 degrees
MEMS 1.9 36-1-1-1-1 gaps at 0, 100, 180, 310
MEMS 2.0 36-1-1-1-1 gaps at 30,60, 210, 250
MEMS 3.0 36-1-1-1-1 gaps at 80, 110, 260, 300
@mike501
Copy link
Copy Markdown
Owner Author

mike501 commented Nov 8, 2020

Patterns will need updating - found a bug with original update. Have updated code but still learning git to push update out.

mike501 pushed a commit that referenced this pull request Mar 21, 2023
@trevorgetty
Copy link
Copy Markdown

Brilliant work mike - I want to put this Ardu-stim work /branch onto mine to test on the SPI Mini I have. So I have now verified the 36-1-1-1 ( 4 gap pattern ) on the t16 2.0 t series engine, on MEMS1.6, and the later MEMS3 VVC k series.

So the mini uses the earlier 36-1-1 ( 2 gap ) pattern, and I have a few new ECUs, some where mems1.3 and some mems1.6 and they are a little difficult to get mapped without desoldering and putting on a new eeprom. So if I could get the SPI mini config onto speeduino and tested using this would be great. I will let you know how I get on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants