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

m_maximumretard broken, resulting in knock controller not working. Related to Closed Loop Fueling? #417

Closed
rk-iv opened this issue Apr 30, 2024 · 8 comments
Labels
🐞 bug 🐞 Something isn't working

Comments

@rk-iv
Copy link

rk-iv commented Apr 30, 2024

Proteus f7 BMM ecu for Mazda Miata NA6

When setting up the Max knock retard table with an array of values ranging from 2 through 10. The variable m_maximumretard does not correlate properly, I assumed if i set up a table of RPM, MAP, Knock: max retard i would see a table that represents my tune table. That was not the case. Furthermore, It made me realize it was not pulling timing when knock count incremented.

This is what I have found so far

  • ECU powered on W/ Closed Loop Fuel Control - ON
    - m_maximumretard is a constant 0 regardless of Max Knock Retard Table
  • ECU powered on W/ Closed Loop Fuel Control - ON, then Turned OFF
    - m_maximumretard changes from 0 to 2 regardless of Max Knock Retard Table
    - Upon power cycle, m_maximumretard value stayed 2
  • ECU powered on w/ Closed Loop Fuel Control ON, then OFF, Then ON again
    - m_maximumretard starts at 0, goes to 2.
    - Started the car, m_maximumretard goes to 8, stays at 8 constant regardless of Max Knock Retard Table
    - Drove around, took a log, both Closed loop fuel control and Knock Controller work as Expected. Pulling timing, and fuel trimming.
    - When I turned the car off, the m_maximumretard stayed constant at 8. When I power cycled ECU, value returned to 0.
@nmschulte
Copy link
Contributor

see also #246

@nmschulte nmschulte added the 🐞 bug 🐞 Something isn't working label Apr 30, 2024
@rk-iv
Copy link
Author

rk-iv commented May 1, 2024

M_maximumretard constant zero when ecu is powered on with Closed loop fuel control

Screenshot 2024-04-30 094508

m_maximumretard changes to to constant 2 when CLFC is turned Off.

Screenshot 2024-04-30 094539

Turned CLFC back on, and turned on car, Knock controller is Active

Screenshot 2024-04-30 094620

Screenshot 2024-04-30 181157
Screenshot 2024-04-30 181633

@rusefillc
Copy link
Contributor

is this rusefi/rusefi#6370?

@rk-iv
Copy link
Author

rk-iv commented May 1, 2024

@rusefillc Let me go give this a try, ill report back

@rk-iv
Copy link
Author

rk-iv commented May 1, 2024

With zero's in the first row, it did not pull timing, and m_maximumretard was Zero when the car was off, and remained zero when the car was on and didnt change regardless of RPM or MAP

It did not make it so that upon ecu power cycle that the m_maximumretard would be anything but constant zero.

changing the bottom cells back to Non zeroes, and doing the step of Closed loop fuel control off then on resulted in the knock controller responding.

My Max knock table isnt TPS vs rpm its MAP vs RPM

nmschulte added a commit to nmschulte/fome-fw that referenced this issue May 2, 2024
column and row definition were swapped w/re: value lookup

fixes FOME-Tech#246, FOME-Tech#417
mck1117 pushed a commit that referenced this issue May 4, 2024
* fix knock retard table init

column and row definition were swapped w/re: value lookup

fixes #246, #417

* initialize knock controller / max retard table on reset

* fix KnockController init

no need for previousConfig (or any config ref); use the current config
@mck1117
Copy link
Collaborator

mck1117 commented May 4, 2024

Not related to closed loop fueling, related to any config burn. If you do a burn, then the knock retard max table would actually work.

@nmschulte
Copy link
Contributor

@rk-iv is this resolved via #419?

@rk-iv
Copy link
Author

rk-iv commented Jun 1, 2024

@rk-iv is this resolved via #419?
yes, latest snapshot has resulted in the max knock table working as expected and not requiring a burn to configure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 bug 🐞 Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants