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

VR discrete issues #245

Closed
mck1117 opened this issue Jan 3, 2023 · 14 comments
Closed

VR discrete issues #245

mck1117 opened this issue Jan 3, 2023 · 14 comments
Labels
bug Something isn't working IMPORTANT

Comments

@mck1117
Copy link

mck1117 commented Jan 3, 2023

rusefi/rusefi#4874
rusefi/rusefi#4922
and maybe rusefi/alphax-4chan#84

Seems that TLV3202 instead of MAX9142 has slightly different behavior, and is susceptible to oscillation on the falling edge that can cause an early falling edge.

In bench testing, adding a 10nF capacitor across pins 4-5 (pin 5 to ground) solves the issue, but further testing required. @ElDominio is going to try it on his real engines that have this issue.

@mck1117
Copy link
Author

mck1117 commented Jan 3, 2023

The bodge I did on the bench:
B02B3E39-ACCA-4FF2-B069-012EC05C4233

@ElDominio
Copy link

Would I not have to put on at pin3 too? Since I see we're putting a cap across 2IN+ and GND, wouldn't one be needed between 1IN+ and GND too? just for my info

@rusefillc rusefillc added bug Something isn't working IMPORTANT labels Jan 5, 2023
@rusefillc
Copy link
Collaborator

rusefillc commented Jan 5, 2023

@Qwerty-OFF can you help with ^^^?

WhatsApp Image 2023-01-05 at 11 30 51

@ElDominio
Copy link

Yaris with 10nf cap across 4 and 5 still has issues

@ElDominio
Copy link

changingtocamactivewithinvertyaaris.zip

I changed from not using cam to using cam

@ElDominio
Copy link

I'm a fucking idiot, forgot to add -360

New issue: cam reads slow af and sometimes crazy
camcrazy.zip

@rusefillc
Copy link
Collaborator

@ElDominio please clarify

"VR discrete issue" not resolved
or
"VR discrete issue" impossible to test because of CAM issue
or
"VR discrete issue" resolved, now separate issue?

@mck1117
Copy link
Author

mck1117 commented Jan 5, 2023

he forgot to add a second cap to the cam channel, and is going to test on a rev E so he can get an SD log

@ElDominio
Copy link

Yaris
RevE with 10nF capacitors on TLV3202

OsciLogandNormalLogwithcapsRevE.zip
re_15.zip

@ElDominio
Copy link

Uploading q4{840dbaa} (dc2579a1-796f-45ee-a709-273b7efd75f1 id=0x7f0a0401 tag=BaseCommentFragment) …

@mck1117
Copy link
Author

mck1117 commented Jan 6, 2023

Cap seems to have made an improvement to crank signal:

before on left, after on right

Left has very not-50% duty cycle, falling edge comes very soon after rising edge. Right has near 50/50 duty, which is expected for VR signal with many teeth (like a 36-2 crank)

image

@rusefillc
Copy link
Collaborator

#247 for actual change

let's start by improving newer 2 sided version? we might end up making improved one sided version as well I guess

@ElDominio
Copy link

We might need another cap according to @mck1117 , @rusefillc . Pending testing

@mck1117
Copy link
Author

mck1117 commented Jan 22, 2023

We might need another cap according to @mck1117 , @rusefillc . Pending testing

the updated schematic has both caps, like the original little test board I did

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working IMPORTANT
Projects
None yet
Development

No branches or pull requests

3 participants