Notices
ECU Flash

SD - first test success

Thread Tools
 
Search this Thread
 
Old Feb 4, 2009 | 02:52 PM
  #1  
jcsbanks's Avatar
Thread Starter
Evolved Member
 
Joined: May 2006
Posts: 2,399
Likes: 6
From: UK
SD - first test success

First test is a success without even having to change my first values, it drives and cold/warm starts like stock, closed loop trims are good, throttle response fine without changing any acceleration enrichment settings. It is freezing with snow on the ground so I have not done a test with boost to fine tune the tables, but this I expect will be the easy bit as it is closed loop and transients that are hard to get right with SD. It is easily good enough to go around my farmyard 60 ft "roundabout" with the awesome S-AYC holding the tail out

See the attached screen grab of my new Ecuflash tables. I did not make any changes to a stock ECU other than what you can see, so do not be overwhelmed. My work over the last few months on understanding all the ECU routines has been so that I can hit just a few key targets and do everything I want without huge changes. This way it should be bug free and easily transferred (once tested a bit more!) to lots of other ECUs, perhaps as a Tephra patch? The only two tables that are user configurable are the SD RPM VE (which is an old table that I just changed the values in) and a new SD MAP sensor calibration table which is actually looking after most of my VE calibration. I have, in the end, replicated the OEM airflow and load values based on Evoscan's excellent map tracing logging which I used to produce a MAP vs RPM vs VE display, from which I concluded the above settings, which so far, just work.

For the coders:

The four values of 0009 are NOPs to get the stock ECU to:
Replace MAFSOURCEMAIN and MAFSOURCEMAINxMAFMULTIP with the calculated values
Remove load limits
Remove MAF sensor unplugged error (still to do a MAT into IAT change in software, and disable the baro unplugged error)

The DA56 changes a table lookup to an axis lookup. The E273 loads r2 with 0x73 (calc explained below).

29E4 is a space I found nearby to put the map axis for the SD MAP sensor calibration table. FFFF6ABC is the raw MAP sensor value, which is used to interpolate the top row of the SD MAP sensor calibration table. FFFF699C (and the 699C reference with the FFFF not shown) is the result of this interpolation which is used to lookup the bottom row of the SD MAP sensor calibration table.

The calculation with two worked examples:

FIRST WORKED EXAMPLE:

MAP = 121kPa, RPM below 5500:

MAP lookup for 121 kPa is 121/1.334 = 91 (1.334 is the JDM MAP scaling from raw to kPa absolute)
RPM lookup is 100% = 128
r2 multiplier of 0x73 = 115

MAFSOURCEMAIN = 91*128*115/0x4000*8=654

From this the ECU calculates load:

Load = MAFSOURCEMAIN*0.596 = 1157*0.596 = 390

Load is scaled in the ECU as 10/32 so load is 690*10/32 = 121

Note how my choice of 0x73 as a multiplier result in load=MAP(kPa) when the RPM VE is 100% and when the MAP lookup is 1:1. On a near standard car above 120kPa and below 5500 RPM this is pretty near, but I still have to do the actual boost testing when the snow melts.

SECOND WORKED EXAMPLE (abbreviated):

MAP = 31kPa, lookup 24, RPM VE again 100%
Load = 24/1.334*128*0x73/0x4000*8*0.596*10/32=24

Other thoughts:

If the baro sensor remains in the inlet tract it will pull down the load at high RPM. So the RPM VE map will have to be adjusted less at high RPM if the baro sensor is still present.
Attached Thumbnails SD - first test success-sdtest1.png  
Reply
Old Feb 4, 2009 | 02:59 PM
  #2  
justboosted02's Avatar
Evolved Member
15 Year Member
Liked
Loved
Community Favorite
iTrader: (31)
 
Joined: Jun 2006
Posts: 1,926
Likes: 14
From: northeast
wow, cant wait for this to be refined and ported to other roms
Reply
Old Feb 4, 2009 | 03:31 PM
  #3  
tephra's Avatar
EvoM Guru
15 Year Member
iTrader: (6)
 
Joined: Feb 2007
Posts: 9,486
Likes: 67
From: Melbourne, Australia
awesome.

glad someone has the proper time to get into this
Reply
Old Feb 4, 2009 | 03:33 PM
  #4  
KevOVIII's Avatar
Evolving Member
iTrader: (10)
 
Joined: Aug 2005
Posts: 246
Likes: 0
From: San Francisco, CA
Good stuff!
Reply
Old Feb 4, 2009 | 03:42 PM
  #5  
tephra's Avatar
EvoM Guru
15 Year Member
iTrader: (6)
 
Joined: Feb 2007
Posts: 9,486
Likes: 67
From: Melbourne, Australia
quick reread

why isn't the VE map 3d to take into account rpm and load? as opposed to just rpm?
Reply
Old Feb 4, 2009 | 03:49 PM
  #6  
l2r99gst's Avatar
Evolved Member
iTrader: (2)
 
Joined: Mar 2004
Posts: 3,499
Likes: 4
From: CA
Very nice. Can't wait to see some more testing.
Reply
Old Feb 4, 2009 | 03:53 PM
  #7  
Cyloc's Avatar
Evolving Member
iTrader: (10)
 
Joined: May 2006
Posts: 465
Likes: 0
From: Tampa Area
All I can say is........WOW......this will open the stock ecu to the next level.
Reply
Old Feb 4, 2009 | 03:56 PM
  #8  
cossie1's Avatar
Evolved Member
 
Joined: Aug 2007
Posts: 569
Likes: 1
From: UK
WOW that didn't take long, when I asked you the other day, and you said it was closer, I didn't think it was that close.

Fair play John.
Reply
Old Feb 4, 2009 | 04:07 PM
  #9  
cossie1's Avatar
Evolved Member
 
Joined: Aug 2007
Posts: 569
Likes: 1
From: UK
Quick question.

How will / does this effect the boost table / limits ?
Reply
Old Feb 4, 2009 | 04:09 PM
  #10  
RoadSpike's Avatar
Evolved Member
iTrader: (5)
 
Joined: Oct 2006
Posts: 3,805
Likes: 2
From: Sacramento, CA
Do we need an air temp sensor for this mod?

What location is it programmed to?
Reply
Old Feb 4, 2009 | 04:50 PM
  #11  
dan l's Avatar
Account Disabled
iTrader: (3)
 
Joined: Apr 2006
Posts: 1,029
Likes: 0
From: USA
Originally Posted by tephra
quick reread

why isn't the VE map 3d to take into account rpm and load? as opposed to just rpm?
I have the same question. Looks good though.
Reply
Old Feb 4, 2009 | 05:16 PM
  #12  
wreckleford's Avatar
Evolved Member
iTrader: (2)
 
Joined: Jun 2003
Posts: 1,171
Likes: 11
From: Jamaica
Wow. JCSBanks is the man! I need to read this and the other thread over a few times to fully understand.
Reply
Old Feb 4, 2009 | 06:40 PM
  #13  
MR Turco's Avatar
EvoM Staff Alumni
iTrader: (16)
 
Joined: May 2007
Posts: 3,233
Likes: 3
From: Massachusetts
Great work!
Reply
Old Feb 4, 2009 | 07:17 PM
  #14  
l2r99gst's Avatar
Evolved Member
iTrader: (2)
 
Joined: Mar 2004
Posts: 3,499
Likes: 4
From: CA
John,

Just so I understand and what we were discussing in the other thread....you left your fuel and timing maps alone, correct? So, the axes stayed as load vs RPM, but the load calculation is just coming from map rather than maf?

If so, then that's great, but I thought you said that you were going to change the axes to be map vs RPM instead of load vs RPM. I hope I misunderstood this and you are keeping the axes as load vs RPM.
Reply
Old Feb 4, 2009 | 07:39 PM
  #15  
mrfred's Avatar
EvoM Guru
iTrader: (50)
 
Joined: Mar 2006
Posts: 9,675
Likes: 132
From: Tri-Cities, WA // Portland, OR
Originally Posted by RoadSpike
Do we need an air temp sensor for this mod?

...
Yes. Proper location would be in the IM.
Reply



All times are GMT -7. The time now is 05:28 AM.