Going back to 94170015
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
So does anyone have addresses the following for 94170015 ????
Injector Cranking Enrichment IPW Adder (Main)
ISCV Control tables (these are the newer one that help control throttle hang, not the idle stepper ones we've always had)
***Warmup Enrichment Load vs RPM (This is the table with the 255s in the warmup region)
Warmup Ignition Retard (This is the "unknown ignition table" with the hole of -5s)
Injector Cranking Enrichment IPW Adder (Main)
ISCV Control tables (these are the newer one that help control throttle hang, not the idle stepper ones we've always had)
***Warmup Enrichment Load vs RPM (This is the table with the 255s in the warmup region)
Warmup Ignition Retard (This is the "unknown ignition table" with the hole of -5s)
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
So Tephra posted the 94170715 rom on the v7 thread. The XML is a very weak though. We are working on getting it up to snuff.
I have all but the few tables above.
This rom works wonderfully.
Paul
I have all but the few tables above.
This rom works wonderfully.
Paul
Code:
IGNITION
<table name="***Warmup Enrichment Load vs RPM" category="Ignition Settings" address="6071" type="3D" level="1" scaling="uint8">
<table name="Engine Load" address="66f2" type="X Axis" elements="14" scaling="Load"/>
<table name="Engine Speed" address="6982" type="Y Axis" elements="8" scaling="RPM"/>
</table>
Interestingly, it is not in 94170015. However, there is a 2D table in the same section of code.
Table address = 36FE
Axis = 64D6
14 elements
The warmup stuff is actually pretty different between my 94170015 and 96530006 dis-assembly. My 94170015 isn't broken down that well though, so it may just be missing some sections.
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
Great find ! !
Looks like my y axis addy is wrong too though.
ER uh....Sorry I didn't understand what you said when I first read it.
Are you saying that the 3D table does not exist ? ?
Also, I edited the OP to reflect your observation about the table.
Looks like my y axis addy is wrong too though.
ER uh....Sorry I didn't understand what you said when I first read it.
Are you saying that the 3D table does not exist ? ?
Also, I edited the OP to reflect your observation about the table.
Last edited by Appauldd; Feb 3, 2010 at 07:29 AM. Reason: adding info
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
Maybe it should look like this....
<table name="Warmup Enrichment vs Load" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="Engine Load" address="64d6" type="X Axis" elements="14" scaling="Load"/>
</table>
It makes perfect sense too...The table appears to be enriched out to 100 load and then drops to 0
<table name="Warmup Enrichment vs Load" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="Engine Load" address="64d6" type="X Axis" elements="14" scaling="Load"/>
</table>
It makes perfect sense too...The table appears to be enriched out to 100 load and then drops to 0
I will say that the tables you have labeled "Warmup Ignition Retard" and "Ignition Unknown Control (warm-up?)" are part of the same subroutine in 96530006, and I haven't seen similar code or tables in any other ROM. To the best of my knowledge, they're unique to 9653. (A hint is that rather than using the typical load scaling that most other tables used, they defined a special one for this table only 8 elements long. This was an "afterthought" addition, I'd guess, to address problems with one specific model/market; if memory serves, 8859 doesn't have anything similar either.)
As for "***Warmup Enrichment Load vs RPM", the logic of the related routine in 9417 is a bit different; while 96530006 has a full 3D table that's used here, 9417 has a 14-element 2D table at 0x36FE, whose axis is RPM (at 0x64D6). Otherwise, they're used in the same place, in the same way. I haven't looked at how 8859 handles this, but I'd be surprised if it wasn't a 3D table there.
I was curious about that last table, so I did a little more digging: it's only used when RPM > the word at 0x1718 (RPM scaling), and that's true for both ROMs. On both ROMs, that appears to be set to an arbitrarily low value ("1" on 9653, "250" on 9417), so that check almost might as well not be there, but you could totally disable the use of those tables by setting 0x1718 to FFFF (or something higher than whatever max RPM you're going to hit is).
(Edit: 03whitegsr beat me to it.
)And, for "Injector Cranking Enrichment IPW Adder (Main)", it's definitely been posted before, although probably called "Cranking Primer Initial Pulse". I believe the address you're looking for is 0x56AE.
For the ISCV stuff, though, you're on your own. After going through all of mrfred's fuel findings in 9653 (which were pretty simple by comparison), I have no idea how Eric had the patience to go through that huge stack of ISCV locations.

There's a significant duplication of effort here, which is I really think is a shame. But again, I can't dictate how people spend their time.
Maybe it should look like this....
<table name="Warmup Enrichment vs Load" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="Engine Load" address="64d6" type="X Axis" elements="14" scaling="Load"/>
</table>
It makes perfect sense too...The table appears to be enriched out to 100 load and then drops to 0
<table name="Warmup Enrichment vs Load" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="Engine Load" address="64d6" type="X Axis" elements="14" scaling="Load"/>
</table>
It makes perfect sense too...The table appears to be enriched out to 100 load and then drops to 0
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
Ed,
Thank you for your insightful input. As I stated earlier, I do not think there will be any more development of this rom beyond what we have here. Honestly, I don't think it is really necessary either. 9653 certainly does work well, but requires a bit of ingenuity to get it to so so.
Even with 9653, I personally never messed with the ISCV tables anyhow. Therefore, in my limited testing of 94170715 I do not believe that they would be necessary. Throttle hang is not an issue with this rom.
Again, I sincerely thank you for your efforts.
Thank you for your insightful input. As I stated earlier, I do not think there will be any more development of this rom beyond what we have here. Honestly, I don't think it is really necessary either. 9653 certainly does work well, but requires a bit of ingenuity to get it to so so.
Even with 9653, I personally never messed with the ISCV tables anyhow. Therefore, in my limited testing of 94170715 I do not believe that they would be necessary. Throttle hang is not an issue with this rom.
Again, I sincerely thank you for your efforts.
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
Better ? ? ?
<table name="Warmup Enrichment vs RPM" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="RPM" address="64d6" type="X Axis" elements="14" scaling="RPM"/>
</table>
But in scaling this way the RPM displays all of the way down to 125. and the upper limit is 2250. This scaling would not impose that this is a warmup table at all.
<table name="Warmup Enrichment vs RPM" category="Warmup Settings" address="36fe" type="2D" level="1" scaling="uint8">
<table name="RPM" address="64d6" type="X Axis" elements="14" scaling="RPM"/>
</table>
But in scaling this way the RPM displays all of the way down to 125. and the upper limit is 2250. This scaling would not impose that this is a warmup table at all.
Thread Starter
Evolved Member
iTrader: (22)
Joined: Nov 2003
Posts: 2,408
Likes: 7
From: Northern KY near Cincy
Seeing that the differences in the warmup section do exist, I'd be interested in a quick trial run of a SD patched 94170015.
It would be nothing more then loading in my current SD and main fuel and ignition tables and going for a drive though.
Overall, I'm pretty happy with 96530006. Marginal throttle hang, good stable idle, no fast idle when cold.
I had all those issues initially, but then I reactivated the low throttle ignition advance routine and everything went away and the car ran great. I think most issues in 96530006 can be resolved by 1 or 2 periphery bits.
It would be nothing more then loading in my current SD and main fuel and ignition tables and going for a drive though.
Overall, I'm pretty happy with 96530006. Marginal throttle hang, good stable idle, no fast idle when cold.
I had all those issues initially, but then I reactivated the low throttle ignition advance routine and everything went away and the car ran great. I think most issues in 96530006 can be resolved by 1 or 2 periphery bits.


