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

E-stop (Reset) no pullup? #99

Closed
RealTadango opened this issue Mar 5, 2024 · 5 comments
Closed

E-stop (Reset) no pullup? #99

RealTadango opened this issue Mar 5, 2024 · 5 comments

Comments

@RealTadango
Copy link

Using the generic mapping (pins 2,4,15) pin 2(X) has no pullup active? I only measure 600mv. Tested on 2 ESP32 boards. Can this be a configuration issue or should this pullup always be active?

@terjeio
Copy link
Contributor

terjeio commented Mar 5, 2024

$18 (pullup disable) is not 0?
My dev board works: ~2.5V with pullup enabled, ~0V when disabled.

@RealTadango
Copy link
Author

$18 = 0. If i set it to 1 the X pullup goes to 0... so the pullup is working but very weak. The other 2 pins are at 3.2V and working normally..... strange. I will swap some pins i guess and see if it works correctly as an output

@RealTadango
Copy link
Author

Wait... GPIO02 has a LED? That might interfere.....

But when i use GPIO23 for Probe it works correct with the pullup. I can enable and disable it using $19 and it works correct. If i use GPIO23 for E-Stop (Reset) it does not? I cannot control the pullup using $17 (motor fault pullup works). It always is at 0V... The pin works for Probe with pullup so it is working correctly. Can you explain this?

@RealTadango RealTadango changed the title X limit no pullup? E-stop (Reset) no pullup? Mar 6, 2024
@terjeio
Copy link
Contributor

terjeio commented Mar 6, 2024

Missing pullup for estop is a bug, add this code here for a quick fix:

                case Input_EStop:
                    signal->mode.pull_mode = settings->control_disable_pullup.e_stop ? PullMode_Down : PullMode_Up;
                    signal->mode.inverted = control_fei.e_stop;
                    break;

And P2 as a limit input is a bad choice for the generic map, my dev boards has a pulldown resistor connected, yours has a LED.

@RealTadango
Copy link
Author

Thanks for the bugfix, it works. And i now see the remark for my specific board and indeed 2 is pulled low.... All working now as expected :)

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

2 participants