Page 32 of 37
Re: PCM Hammer Release 021
Posted: Tue Aug 15, 2023 8:28 am
by antus
Yep the 07 is the p12b 2mb. I was able to reproduce this last night and I think the timeout is borderline too short for the larger sectors on the amd flash chip with the xpro timeout calcs. I'll try and get this sorted soon and send you a test build.
Re: PCM Hammer Release 021
Posted: Tue Aug 15, 2023 10:07 am
by Bingus
Yep! I was able to reproduce exactly that by building locally and debugging. Stepping through line by line I got a success response from the kernel and was able to write! Now I have 2 happy P12 pcms
I'm still willing to test that fix to confirm - hopefully it'll help others!
Re: PCM Hammer Release 021
Posted: Tue Aug 15, 2023 2:10 pm
by Tazzi
Bingus wrote:Yep! I was able to reproduce exactly that by building locally and debugging. Stepping through line by line I got a success response from the kernel and was able to write! Now I have 2 happy P12 pcms
I'm still willing to test that fix to confirm - hopefully it'll help others!
If using the OBDX Pro VT on Bench for some of these older style ECUS like P12/P04 ect, we found that you may need to add a pull down resistor (connect resistor between VPW and ground) to help with communication stability.
This is due to those style ecus were typically in vehicles which had multiple modules that would all pull the bus down slightly (loading the bus) which acts as the method for pulling the VPW line low. Without these pulldowns, some modules will 'float' there VPW voltage back to ground which can cause stability issues at high speed as this takes too long.
Quick fix for this is adding a 2k-5k resistor from VPW pin to ground.
Some J2534 tools may apply a slightly stronger pull down inside the tool which also accommodates to this, so may not be needed on something like a GM MDI.
Re: PCM Hammer Release 021
Posted: Wed Aug 16, 2023 2:13 am
by DavidBraley
Tazzi,
Does the (sometimes) need for a pull-down resistor between the VPW line and ground need to be close to the ecu? Or can it be an addition to the bench harness?
A switched resistance path between the VPW line and ground would be super easy to add to a bench harness.
Re: PCM Hammer Release 021
Posted: Wed Aug 16, 2023 4:17 am
by Tazzi
I only have the pull down for the VT, and mines near the scantool
Itself, but I believe it could be done anywhere in the harness.
This all only applies when using a bench harness with the VT.
Re: PCM Hammer Release 021
Posted: Wed Aug 16, 2023 9:25 am
by antus
I will just add that I havn't needed this with my VT, so its not an always thing either. Also I am having trouble with the first write packet on P12, but I think it is something else as I have added that resistor and it made no difference for me. I am testing the new asm kernels that are not released though. I still think its a timing issue, but I don't know for sure yet. I'd like to get this new asm kernel issue solved as I think all other issues are solved with it and it'll be good to look towards the next pcmhammer release supporting P04 and P08 as well.
Re: PCM Hammer Release 021
Posted: Wed Aug 16, 2023 10:29 am
by Tazzi
antus wrote:I will just add that I havn't needed this with my VT, so its not an always thing either. Also I am having trouble with the first write packet on P12, but I think it is something else as I have added that resistor and it made no difference for me. I am testing the new asm kernels that are not released though. I still think its a timing issue, but I don't know for sure yet. I'd like to get this new asm kernel issue solved as I think all other issues are solved with it and it'll be good to look towards the next pcmhammer release supporting P04 and P08 as well.
If it is timing, it can be adjusted by commands to the VT also if thats the case
I would expect seeing issues trying to upload a kernel/loader if timing was a problem though for large chunks. Maybe looping through uploading a chunk of data to the ECU over and over could test the theory?
Re: PCM Hammer Release 021
Posted: Wed Aug 16, 2023 10:35 am
by antus
Yep its just time. It all fundamentally works - it probably just needs a small tweak... in the right place.
Re: PCM Hammer Release 021
Posted: Thu Aug 17, 2023 10:51 am
by ScarabEpic22
antus wrote:Yep the 07 is the p12b 2mb. I was able to reproduce this last night and I think the timeout is borderline too short for the larger sectors on the amd flash chip with the xpro timeout calcs. I'll try and get this sorted soon and send you a test build.
Can you confirm the 07 actually has 2MB? GM talked about releasing a 2MB P12, haven't seen one yet (but want to be proven wrong here)!
Re: PCM Hammer Release 021
Posted: Thu Aug 17, 2023 2:36 pm
by Bingus
Here is the bin I pulled from the 07 P12b I have. It is indeed 2MB.
It also took a lot longer to read due to failures - wondering if the resistor trick would help. I'm surprised at how unstable the P12b read was compared to the P12 I have.