Notices
ECU Flash

SD - first test success

Thread Tools
 
Search this Thread
 
Old Feb 4, 2009, 02:52 PM
  #1  
Evolved Member
Thread Starter
 
jcsbanks's Avatar
 
Join Date: May 2006
Location: UK
Posts: 2,399
Likes: 0
Received 5 Likes on 4 Posts
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  
Old Feb 4, 2009, 02:59 PM
  #2  
Evolved Member
iTrader: (31)
 
justboosted02's Avatar
 
Join Date: Jun 2006
Location: northeast
Posts: 1,898
Received 10 Likes on 9 Posts
wow, cant wait for this to be refined and ported to other roms
Old Feb 4, 2009, 03:31 PM
  #3  
EvoM Guru
iTrader: (6)
 
tephra's Avatar
 
Join Date: Feb 2007
Location: Melbourne, Australia
Posts: 9,486
Received 66 Likes on 42 Posts
awesome.

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

why isn't the VE map 3d to take into account rpm and load? as opposed to just rpm?
Old Feb 4, 2009, 03:49 PM
  #6  
Evolved Member
iTrader: (2)
 
l2r99gst's Avatar
 
Join Date: Mar 2004
Location: CA
Posts: 3,499
Likes: 0
Received 4 Likes on 4 Posts
Very nice. Can't wait to see some more testing.
Old Feb 4, 2009, 03:53 PM
  #7  
Evolving Member
iTrader: (10)
 
Cyloc's Avatar
 
Join Date: May 2006
Location: Tampa Area
Posts: 465
Likes: 0
Received 0 Likes on 0 Posts
All I can say is........WOW......this will open the stock ecu to the next level.
Old Feb 4, 2009, 03:56 PM
  #8  
Evolved Member
 
cossie1's Avatar
 
Join Date: Aug 2007
Location: UK
Posts: 569
Likes: 0
Received 1 Like on 1 Post
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.
Old Feb 4, 2009, 04:07 PM
  #9  
Evolved Member
 
cossie1's Avatar
 
Join Date: Aug 2007
Location: UK
Posts: 569
Likes: 0
Received 1 Like on 1 Post
Quick question.

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

What location is it programmed to?
Old Feb 4, 2009, 04:50 PM
  #11  
Account Disabled
iTrader: (3)
 
dan l's Avatar
 
Join Date: Apr 2006
Location: USA
Posts: 1,029
Likes: 0
Received 0 Likes on 0 Posts
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.
Old Feb 4, 2009, 05:16 PM
  #12  
Evolved Member
iTrader: (2)
 
wreckleford's Avatar
 
Join Date: Jun 2003
Location: Jamaica
Posts: 1,171
Received 11 Likes on 9 Posts
Wow. JCSBanks is the man! I need to read this and the other thread over a few times to fully understand.
Old Feb 4, 2009, 06:40 PM
  #13  
EvoM Staff Alumni
iTrader: (16)
 
MR Turco's Avatar
 
Join Date: May 2007
Location: Massachusetts
Posts: 3,233
Received 3 Likes on 3 Posts
Great work!
Old Feb 4, 2009, 07:17 PM
  #14  
Evolved Member
iTrader: (2)
 
l2r99gst's Avatar
 
Join Date: Mar 2004
Location: CA
Posts: 3,499
Likes: 0
Received 4 Likes on 4 Posts
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.
Old Feb 4, 2009, 07:39 PM
  #15  
EvoM Guru
iTrader: (50)
 
mrfred's Avatar
 
Join Date: Mar 2006
Location: Tri-Cities, WA // Portland, OR
Posts: 9,675
Received 128 Likes on 96 Posts
Originally Posted by RoadSpike
Do we need an air temp sensor for this mod?

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


Quick Reply: SD - first test success



All times are GMT -7. The time now is 12:19 AM.