FOME-Tech / fome-fw

Free Open Motorsports ECU
https://www.fome.tech
Other
34 stars 17 forks source link

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

Closed rk-iv closed 1 month ago

rk-iv commented 2 months ago

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

nmschulte commented 2 months ago

see also https://github.com/FOME-Tech/fome-fw/issues/246

rk-iv commented 2 months ago

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 commented 2 months ago

is this https://github.com/rusefi/rusefi/issues/6370?

rk-iv commented 2 months ago

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

rk-iv commented 2 months ago

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

mck1117 commented 2 months ago

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 commented 1 month ago

@rk-iv is this resolved via #419?

rk-iv commented 1 month ago

@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