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

Pin C4 not behaving correctly? Need testing? #50

Closed
ElDominio opened this issue Apr 19, 2022 · 17 comments
Closed

Pin C4 not behaving correctly? Need testing? #50

ElDominio opened this issue Apr 19, 2022 · 17 comments
Assignees

Comments

@ElDominio
Copy link
Collaborator

I have both a revA and a revB with issues on the Launch Control input where it stays engaged no matter what. I haven't been able to test much, but Tercel and Corolla both stay with launch control engaged if I enable launch control. Will update with more tests, but if any bench tests can be done it would be great

@andreika-git
Copy link
Collaborator

andreika-git commented Apr 19, 2022

  1. What's the value of the setting 'boardUse2stepPullDown'? Is it "With Pull Down" or "With Pull Up"?

  2. And what about revC? Does it work on revC?

@ElDominio
Copy link
Collaborator Author

  1. Pretty sure I have it as "With Pullup", since we wired it to a switch to ground

  2. Have not tested revC yet

@andreika-git
Copy link
Collaborator

  1. Does the revB board have the required fix of Q21 transistor? (see C4&C7 pull down mode does not work on rev B #40)

  2. Did you set the Launch Control settings correctly?
    a) "Activation Mode" should be set to "Switch Input"
    b) "Switch Input" pin should be set to "C4 - 2STEP"

Currently there is no option for "Switch Input inverted" in the firmware, although we already have "Clutch Input inverted". That's why your switch may act in inverted mode, and launch control may be engaged all the time except when activated. Maybe we should ask @rusefillc to add the inverting pin mode option...

  1. Can you please try the "readpin PF11" command in the rusEFI Console? Please run it twice with the switch enabled and disab led. This will check the voltage on the input pin without any Launch Control logic.

rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
@rusefillc
Copy link
Contributor

@ElDominio I've made the change we can now adjust Launch control pin mode

Please update to latest firmware, please select inverted pin mode, please report back.

@rusefillc
Copy link
Contributor

@andreika-git reminded me that we need actual code for that not just UI :)

@rusefillc rusefillc assigned rusefillc and unassigned ElDominio Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
@ElDominio
Copy link
Collaborator Author

I updated, used it as clutch input and now it works fine
image

@andreika-git
Copy link
Collaborator

It would be great if you could test with the new "Switch input inverted" option too...

@ElDominio
Copy link
Collaborator Author

image

it disables as an option if i use switch input mode

@andreika-git
Copy link
Collaborator

We'll fix it, thanks!

rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
@ElDominio
Copy link
Collaborator Author

image

That is fixed, but in switch mode it still doesn't listen to what I select. It is active when 5v is present, inactive when GND, and I need the opposite

@ElDominio
Copy link
Collaborator Author

ElDominio commented Apr 19, 2022

Separate issue rusefi/rusefi#4085

Separate issue rusefi/rusefi#4086

@rusefillc
Copy link
Contributor

its greyed but still selected. you need to fully unselect when changing mode

@rusefillc
Copy link
Contributor

OK so original problem is that we launch button "inverted" is missing!

should be
image

and we have

image

rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 19, 2022
rusefillc pushed a commit to rusefi/rusefi that referenced this issue Apr 20, 2022
@rusefillc
Copy link
Contributor

@ElDominio just added logic to hand the new "inverted" flag.

Overall we have three things for each switch:

  1. pin
  2. inverted or not
  3. physical pull ups

@rusefillc
Copy link
Contributor

@ElDominio how does it look with latest changes?

@rusefillc
Copy link
Contributor

closing for now?

@ElDominio
Copy link
Collaborator Author

New update is great and works great 😁

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