Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GDI PT2001 Sense resisters for injectors should be 15mOhm, or settings altered #4251

Closed
NOx-z opened this issue Jun 10, 2022 · 8 comments
Closed
Assignees

Comments

@NOx-z
Copy link
Contributor

NOx-z commented Jun 10, 2022

all of these sense resistors for Rev A are 10mOhm whereas all of our previous versions are 15mohm (except for 10mOhm used on the boost converter sense, sense number 4). This has implications of incorrectly setting the expected peak-hold currents with our current settings.

solutions:
A) change settings for 10mOhm
b) swap resistors to 15mOhm

in real life we have smaller R0.3 - DO WE HAVE BOM?

R0.3.2 was a major change - at least a much larger board, BOM

"0R010","R22,R12,R26,R21","Resistors_SMD:R_1206_HandSoldering","","C76294","Panasonic,ERJ-8CWFR015V","DIGI,P0.015BVCT-ND","JLCPCB,C201281"

https://rusefi.com/docs/ibom/Common_Rail_MC33816_0.3.2.html

@rusefillc
Copy link
Contributor

let's alter microcode

@Qwerty-OFF just FYI - no HW changes needed

@rusefillc rusefillc transferred this issue from rusefi/rusefi-hardware Jun 10, 2022
@rusefillc
Copy link
Contributor

I will fix my legacy board, who else has what legacy boards? @NOx-z @mck1117 do you have any legacy? safer to update those all

@rusefillc
Copy link
Contributor

Does this sound like rusefi/rusefi-hardware#92 reason or contributing factor?

@rusefillc
Copy link
Contributor

is this related to rusefi/rusefi-hardware#96 at all?

@rusefillc rusefillc changed the title Sense resisters for injectors should be 15mOhm, or settings altered GDI PT2000 Sense resisters for injectors should be 15mOhm, or settings altered Jun 11, 2022
@rusefillc rusefillc changed the title GDI PT2000 Sense resisters for injectors should be 15mOhm, or settings altered GDI PT2001 Sense resisters for injectors should be 15mOhm, or settings altered Jun 11, 2022
rusefillc pushed a commit that referenced this issue Jun 11, 2022
@mck1117
Copy link
Contributor

mck1117 commented Jun 11, 2022

@rusefillc can you confirm what resistors are on the board you started the car on previously? There are references in code to them being 10mOhm, but we thought they were maybe 15.

@rusefillc
Copy link
Contributor

Flight delayed by 40 minutes, ETA touch down 11pm EST

@rusefillc
Copy link
Contributor

So "R22,R12,R26,R21"?

https://rusefi.com/docs/ibom/Common_Rail_MC33816_0.3.2.html ?

i think https://rusefi.com/forum/viewtopic.php?p=37790#p37790 is the picture of my actual board

at least R010 matches

image

@mck1117
Copy link
Contributor

mck1117 commented Jun 17, 2022

OK, in that case we're all set here, no change to make.

@mck1117 mck1117 closed this as not planned Won't fix, can't repro, duplicate, stale Jun 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants