|
RomRaider
Documentation
Community
Developers
|
| Author |
Message |
|
joshuaj
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Mon Mar 11, 2019 6:15 pm |
|
 |
| Newbie |
Joined: Sun Jan 20, 2019 7:52 pm Posts: 11
|
Hey guys, I've been traveling for work and just got back around to this. When I flash the car with a modified map (just the rev limiter to keep it simple and validate function) I get a p0605 dtc for ROM error. I clear the code and the rev limiter works as I set it but the ECU faults on start up every time. Looks like this is a checksum issue. When I compare the 2 bin files in wols the sum and xor are the same even though they should be different.. Code: <checksum type="std" start="0" end="0x7FFFF" sumloc="0x67f0" xorloc="0x67e8" /> Romraider should calculate and update the checksum automatically on saving correct? Or am I just not doing it right? Thanks!
|
|
| Top |
|
 |
|
joshuaj
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Mon Mar 11, 2019 6:16 pm |
|
 |
| Newbie |
Joined: Sun Jan 20, 2019 7:52 pm Posts: 11
|
a33b wrote: I didn't think this would take so long. Ran into a bit of trouble and not sure what's going on with these parameters. I've double-checked the addresses and they are correct, but the values displayed start 0x1 position lower. I'm going to try deleting all the comments to see if they're doing something. I was using the wrong ROM. Attached and I've updated the git repo. Download the repo linked above to use the definition attached. Thanks for doing that!
|
|
| Top |
|
 |
|
fenugrec
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Mon Mar 11, 2019 6:24 pm |
|
 |
| Experienced |
 |
Joined: Thu Jan 09, 2014 3:07 am Posts: 652
|
joshuaj wrote: Romraider should calculate and update the checksum automatically on saving correct? Are you using that build of RR that includes checksums ? the main download (0.59 RC something) doesn't have it
_________________ If you like nisprog + npkern, you can support me via https://liberapay.com/fenugrec/ For sending me encrypted/secure messages, use PGP key 0xBAC61AEB3A3E6531 available from pool.sks-keyservers.net
|
|
| Top |
|
 |
|
dschultz
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Mon Mar 11, 2019 8:09 pm |
|
 |
| RomRaider Developer |
Joined: Thu May 21, 2009 1:49 am Posts: 7323 Location: Canada eh!
|
|
| Top |
|
 |
|
joshuaj
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Mon Mar 11, 2019 9:28 pm |
|
 |
| Newbie |
Joined: Sun Jan 20, 2019 7:52 pm Posts: 11
|
fenugrec wrote: joshuaj wrote: Romraider should calculate and update the checksum automatically on saving correct? Are you using that build of RR that includes checksums ? the main download (0.59 RC something) doesn't have it That would explain it. 
|
|
| Top |
|
 |
|
rodfeel
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Thu Apr 11, 2019 4:33 pm |
|
 |
| Newbie |
Joined: Wed Apr 03, 2019 7:33 pm Posts: 18
|
|
Hello, I downloaded the zip and when trying to launch the batch file, I get an error saying "the system cannot find the file specified". Please help !!!
You do not have the required permissions to view the files attached to this post.
|
|
| Top |
|
 |
|
a33b
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Fri Apr 12, 2019 1:02 am |
|
 |
| Experienced |
Joined: Sat Jun 24, 2017 6:23 pm Posts: 315
|
rodfeel wrote: Hello, I downloaded the zip and when trying to launch the batch file, I get an error saying "the system cannot find the file specified". Please help !!! Did you extract the contents of the zip prior to running the .bat?
|
|
| Top |
|
 |
|
rodfeel
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Fri Apr 12, 2019 9:14 pm |
|
 |
| Newbie |
Joined: Wed Apr 03, 2019 7:33 pm Posts: 18
|
|
hello, yes I did! I moved the whole folder to system32 and it worked somehow... Thanks anyway !!
|
|
| Top |
|
 |
|
Alex-Angarsk
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Wed Jun 12, 2019 1:32 pm |
|
 |
| Experienced |
 |
Joined: Sat Mar 30, 2019 7:04 am Posts: 372
|
|
nissandefs.xml from "murphys_law" and "a33b" , why different latency units? Milliseconds... Microseconds
You do not have the required permissions to view the files attached to this post.
_________________ SKYLINE 06`CPV35/MT6/VQ35DET/Cosworth/Eagle/ACL/ARP/Supertech/Cometic/MOCAL/ KAKIMOTO RACING/FUJITSUBO/Greddy/HKS/OBX RACING/AEROMOTIVE/UpRev
|
|
| Top |
|
 |
|
fenugrec
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Wed Jun 12, 2019 2:40 pm |
|
 |
| Experienced |
 |
Joined: Thu Jan 09, 2014 3:07 am Posts: 652
|
Alex-Angarsk wrote: Milliseconds... Microseconds murph' has not maintained his repo for a long time, and a lot has happened since that, in particular the release of that ZB060 A2L file. In this case a33b's def has that parameter described as "mTS14VZ ..." which is straight from the A2L. The A2L, gives the following scaling for that param: Code: /begin COMPU_METHOD COMP_0231 "10μS" RAT_FUNC "%8.2" "μS" COEFFS 0 1 0 0 0 10 /end COMPU_METHOD So, I would definitely trust a33b's repo over murph's in cases like these.
_________________ If you like nisprog + npkern, you can support me via https://liberapay.com/fenugrec/ For sending me encrypted/secure messages, use PGP key 0xBAC61AEB3A3E6531 available from pool.sks-keyservers.net
|
|
| Top |
|
 |
|
Alex-Angarsk
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Thu Jun 13, 2019 3:31 pm |
|
 |
| Experienced |
 |
Joined: Sat Mar 30, 2019 7:04 am Posts: 372
|
fenugrec wrote: Alex-Angarsk wrote: Milliseconds... Microseconds murph' has not maintained his repo for a long time, and a lot has happened since that, in particular the release of that ZB060 A2L file. In this case a33b's def has that parameter described as "mTS14VZ ..." which is straight from the A2L. The A2L, gives the following scaling for that param: Code: /begin COMPU_METHOD COMP_0231 "10μS" RAT_FUNC "%8.2" "μS" COEFFS 0 1 0 0 0 10 /end COMPU_METHOD So, I would definitely trust a33b's repo over murph's in cases like these. Strongly disagree, the murph' (with the right) value is correct, the a33b' (from left) is understating by 2 times the well-known tuner developers
You do not have the required permissions to view the files attached to this post.
_________________ SKYLINE 06`CPV35/MT6/VQ35DET/Cosworth/Eagle/ACL/ARP/Supertech/Cometic/MOCAL/ KAKIMOTO RACING/FUJITSUBO/Greddy/HKS/OBX RACING/AEROMOTIVE/UpRev
|
|
| Top |
|
 |
|
Sasha_A80
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Thu Jun 13, 2019 5:10 pm |
|
 |
| Senior Member |
Joined: Mon Jan 19, 2009 6:31 pm Posts: 1615 Location: Moscow, Russia
|
|
Picture number 2 looks more realistic.
|
|
| Top |
|
 |
|
Alex-Angarsk
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Thu Jun 13, 2019 9:35 pm |
|
 |
| Experienced |
 |
Joined: Sat Mar 30, 2019 7:04 am Posts: 372
|
|
.
You do not have the required permissions to view the files attached to this post.
_________________ SKYLINE 06`CPV35/MT6/VQ35DET/Cosworth/Eagle/ACL/ARP/Supertech/Cometic/MOCAL/ KAKIMOTO RACING/FUJITSUBO/Greddy/HKS/OBX RACING/AEROMOTIVE/UpRev
|
|
| Top |
|
 |
|
Alex-Angarsk
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Thu Jun 13, 2019 9:54 pm |
|
 |
| Experienced |
 |
Joined: Sat Mar 30, 2019 7:04 am Posts: 372
|
Sasha_A80 wrote: Picture number 2 looks more realistic. Alexander, hello. Tuners have long been identified with "Injector Minimum Pulse Width". Now I observe that the choice was made in the direction of halved value.
_________________ SKYLINE 06`CPV35/MT6/VQ35DET/Cosworth/Eagle/ACL/ARP/Supertech/Cometic/MOCAL/ KAKIMOTO RACING/FUJITSUBO/Greddy/HKS/OBX RACING/AEROMOTIVE/UpRev
|
|
| Top |
|
 |
|
a33b
|
Post subject: Re: Nissan ECU Definitions moved over to github Posted: Fri Jun 14, 2019 1:15 am |
|
 |
| Experienced |
Joined: Sat Jun 24, 2017 6:23 pm Posts: 315
|
|
My repo is going to be defined by the best source of info available, which is currently the A2L. It wouldn't be the first time the well known tuner developers made a mistake. See the last 15 years of ignition timing as an example.
You do not have the required permissions to view the files attached to this post.
|
|
| Top |
|
 |
Who is online |
Users browsing this forum: No registered users and 2 guests |
|
You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot post attachments in this forum
|
|