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

GDI4 runs out of juice Isense4_high #221

Closed
rusefillc opened this issue Jul 19, 2023 · 22 comments
Closed

GDI4 runs out of juice Isense4_high #221

rusefillc opened this issue Jul 19, 2023 · 22 comments
Assignees

Comments

@rusefillc
Copy link
Contributor

rusefillc commented Jul 19, 2023

user report:
boost voltage setting 40v
user has one proteus with two GDI4 boxes (8 injectors sequential) for a GDI flow bench

at 3000RPM 2.5ms injector pulses 39v volts are observed at peak voltage as EXPECTED
at 4000rpm voltage drops to 30v, that's an ISSUE

L2
10uH

@Qwerty-OFF
Copy link
Collaborator

Это измерение напряжения на форсунке или на тестовой точке высокого напряжения на плате? Хочу узнать что в этой точке:
image

@rusefillc
Copy link
Contributor Author

@Qwerty-OFF GDI4 case is closed that's on injector wires

@rusefillc
Copy link
Contributor Author

rev D

no activity
vboost=64.7v

low RPM bench
vboost=64.7v

image

1200 RPM
vboost=64.7v

image

@rusefillc
Copy link
Contributor Author

there is a very sharp drop of vboost around 5200 rpm

image

4900 everything pretty fine
WhatsApp Image 2023-08-18 at 00 21 29

5100
WhatsApp Image 2023-08-18 at 00 22 03

5300 rpm
WhatsApp Image 2023-08-18 at 00 22 17

@rusefillc
Copy link
Contributor Author

just attempted bumping max boost time from 400 to 600 us, did not help. tried bumping max current targer from 13A to 15A I think it made matters worse

reducing boostCurrent to 10A has helped - there is no more voltage drop

@mck1117
Copy link
Contributor

mck1117 commented Aug 18, 2023

reducing boostCurrent to 10A has helped - there is no more voltage drop

this suggests that the inductor was saturating at 13A

@rusefillc
Copy link
Contributor Author

rusefillc commented Aug 18, 2023

this suggests that the inductor was saturating at 13A

what is the right way to improve things? do I change inductor or just change default boostCurrent to 10A and live happily ever after?

@mck1117
Copy link
Contributor

mck1117 commented Aug 18, 2023

Wait - I read that wrong, I thought you were adjusting the SMPS parameters, not the injector parameters. Changing the boost current will change the injector's opening time and potentially other things.

max boost time from 400 to 600 us

This just allows the injector to take that amount of time for the boost threshold to be reached, it doesn't change behavior at all so long as the injector hits the current limit faster than that time.

@rusefillc
Copy link
Contributor Author

@mck1117 I was changing the below I assume I was not changing injector parameters?

image

@mck1117
Copy link
Contributor

mck1117 commented Aug 18, 2023

Those are absolutely injector parameters. They control the behavior during the boost phase of the injection (high current "slam it open" part).

@rusefillc
Copy link
Contributor Author

Are "boost current target" and "peak phase current target" both injector parameters? I was assuming that only the latter is directly related to individual injector control while "boost current target" is something to do with DCDC core same as boost voltage is a DCDC setting?

@rusefillc
Copy link
Contributor Author

current plan is to try https://www.mouser.com/ProductDetail/EPCOS-TDK/B82559A5103A033?qs=amGC7iS6iy9mEhDlIgUEKA%3D%3D Power Inductors - SMD 10uH 12% 1.2mOhm ERU33 AEC-Q200

@kb1gtt @NOx-z do you guys have opinions on what could be going on here?

@NOx-z
Copy link
Collaborator

NOx-z commented Aug 18, 2023

I would like to see the vboost (pin) on an oscilloscope as multimeters are slow and don't give the full story.. (and I only have two test injectors, which may not be enough to replicate)

we may be able to setup vboost_mon_en which can set an IRQ (a certain pin, maybe) for when undervoltage of VBoost occurs, so we can detect/flag/log on this issue.

We have two additional settings, at least in the microcode, (I do not know if they are accessed elsewhere ) for the current sensing of the dc-dc converter:
#define Isense4_high 2;
#define Isense4_low 3;

We may be able to tweak those to allow more current, but we should do more research into this before just setting new values.

I am reviewing some of the microcode now.

@rusefillc
Copy link
Contributor Author

happy 3000 rpm

WhatsApp Image 2023-08-19 at 12 15 11
WhatsApp Image 2023-08-19 at 12 15 25

@rusefillc
Copy link
Contributor Author

unhappy 5300

WhatsApp Image 2023-08-19 at 12 19 15

@rusefillc
Copy link
Contributor Author

sad 5300 video - note how thing are slowly falling down

https://youtu.be/iUY8sWQcSHo

@rusefillc
Copy link
Contributor Author

WhatsApp Image 2023-08-19 at 12 29 59

WhatsApp Image 2023-08-19 at 12 30 14

rusefillc pushed a commit to rusefi/libfirmware that referenced this issue Aug 19, 2023
@rusefillc
Copy link
Contributor Author

with magic byte adjustment

WhatsApp Image 2023-08-19 at 12 57 50

WhatsApp Image 2023-08-19 at 12 58 36

@rusefillc
Copy link
Contributor Author

See rusefi/libfirmware#8 for solution

@NOx-z
Copy link
Collaborator

NOx-z commented Aug 19, 2023

This magic byte was the Isense4_high in the microcode dram, which is not exposed to the GDI configuration as it is specific to the DI board and not someone's car. The value was changed from ~3.6Amps to ~6.5Amps.. we should spend a little time in the future really tweaking this setting, but this quick change resolved the issue

@rusefillc
Copy link
Contributor Author

rusefi/libfirmware#9

rusefillc pushed a commit that referenced this issue Aug 20, 2023
@rusefillc rusefillc added the GDI label Aug 23, 2023
@rusefillc rusefillc changed the title GDI4 runs out of juice GDI4 runs out of juice Isense4_high Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants