Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

common rail / direct injection scheduling control test PWM #557

Closed
rusefi opened this issue Jan 26, 2018 · 4 comments
Closed

common rail / direct injection scheduling control test PWM #557

rusefi opened this issue Jan 26, 2018 · 4 comments

Comments

@rusefi
Copy link
Collaborator

rusefi commented Jan 26, 2018

We need a piece of test code which would produce a test sequence signal:

100us on, 100us off, 200us on, 200us off, 500us on, 10ms off

@rusefi
Copy link
Collaborator Author

rusefi commented Jan 26, 2018

rusefi pushed a commit that referenced this issue Jan 27, 2018
rusefi pushed a commit that referenced this issue Jan 28, 2018
rusefi pushed a commit that referenced this issue Jan 28, 2018
@rusefi rusefi closed this as completed in 3fc39b8 Jan 28, 2018
@rusefi
Copy link
Collaborator Author

rusefi commented Jan 28, 2018

test #1 scenario
toggle pin after 100us, 100us, 200us, 200us, 500us, 500us, 500us, 5000us, LOOP
results captured with Saleae Logic Analyzer
Wow, results are better than I expected: https://docs.google.com/spreadsheets/d/1iCgO1wxHvhZnQScgKofgeSqCAZ8g0wbljH9tHGqvekA

image

Whole .csv file
test_557.zip

@rusefi
Copy link
Collaborator Author

rusefi commented Jan 28, 2018

test #2 scenario - similar to above but 20, 20, 50, 50, 100, 100, 200, 200, 500, 500, 500, 500, LOOP (app times in US)
test_557_2.zip

https://docs.google.com/spreadsheets/d/199qd9fB09Jn1go-XdfPJ1h0Jdo_SAJGDXYvOfOk6KaQ/edit?usp=sharing

image

image

@rusefi
Copy link
Collaborator Author

rusefi commented Jan 29, 2018

5us/5us/10us/10us is where things get worse. Looks like we have give or take 5us precision.

image

https://docs.google.com/spreadsheets/d/1byKMS8Ad8R2nBuTZhQ4ReqlcNcWmnJm2i09Pu9Bimh0/edit?usp=sharing

image

test_557_3.zip

@rusefi rusefi changed the title common rail / direct injection scheduling control test common rail / direct injection scheduling control test PWM Nov 21, 2018
@rusefi rusefi mentioned this issue May 17, 2020
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

No branches or pull requests

0 participants