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

Rev C test log #63

Open
rusefillc opened this issue Sep 20, 2022 · 18 comments
Open

Rev C test log #63

rusefillc opened this issue Sep 20, 2022 · 18 comments
Assignees

Comments

@rusefillc
Copy link
Contributor

ETB works

@rusefillc
Copy link
Contributor Author

@ElDominio have you received any rev C? Do they crank/start real vehicle? Looking to confirm if adjusted crank/cam conditioner is happy.

@rusefillc
Copy link
Contributor Author

rusefillc commented Sep 24, 2022

@ElDominio says "i put on forte, forte turned on without problem. but let me recheck just in case"

@ElDominio
Copy link
Collaborator

DanielRiveraGenesisrevC.zip

No RPM :(
Will upload to rusefi online just in case

https://rusefi.com/online/view.php?log=1052

@rusefillc
Copy link
Contributor Author

@ElDominio
Copy link
Collaborator

ElDominio commented Oct 5, 2022

IMG_20221004_201724.jpg

No RPM reading

@Qwerty-OFF
Copy link
Collaborator

Qwerty-OFF commented Oct 5, 2022

Looks like we've lost crank pull-up somehow :(

Try adding external 4.7K ish resistor

image

@ElDominio
Copy link
Collaborator

Did the mod, still no RPM anywhere, TS, console or LED.

@ElDominio
Copy link
Collaborator

IMG_20221013_190011.jpg

@ElDominio
Copy link
Collaborator

the two on cam were disconnected for the test

i only had 2k resistors to install no 4.7; i normally use 1k pullup so i fugred 2k was closer to 4.7 lol

leds stay off all the time

rusefillc pushed a commit to rusefi/rusefi that referenced this issue Oct 16, 2022
@rusefillc
Copy link
Contributor Author

image

@rusefillc
Copy link
Contributor Author

@ElDominio looks like part of the issue was software not adjusted to rev C changes

With original "remove six parts" #63 (comment)

but without "add resistors" I have Hall sensor working on the bench, worth trying on real vehicle with latest firmware.

@ElDominio
Copy link
Collaborator

ElDominio commented Oct 18, 2022

Updated to latest, still not reading
image

Fan is now on?
Console reads nothing either

@rusefillc
Copy link
Contributor Author

All I need is a photo of the board in current state and a log of cranking attempt to confirm if we got any counter activity or not.

@ElDominio
Copy link
Collaborator

Will do tomorrow

@ElDominio
Copy link
Collaborator

ElDominio commented Oct 20, 2022

crankingrevc.zip

WhatsApp Image 2022-10-20 at 18 39 22

WhatsApp Image 2022-10-20 at 18 41 19

@rusefillc
Copy link
Contributor Author

@ElDominio can you hack a pull-up to 5v externally or internally?

to begin with I suggest we just focus on Crank

@Qwerty-OFF was a bit wrong at
image

in reality 5v is on top side of the other resistor array, the one closer to LEDs. left-bottom corner

image

another option is waiting for me to land in PR

@rusefillc
Copy link
Contributor Author

actually please remove the wrong resistor lead from

image

note how it's connecting things which were not supposed to be connected

@ElDominio
Copy link
Collaborator

STill no RPM

WhatsApp Image 2022-10-20 at 19 08 07

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