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

Spindle enable Bit #1

Closed
cncPan opened this issue Dec 30, 2018 · 5 comments
Closed

Spindle enable Bit #1

cncPan opened this issue Dec 30, 2018 · 5 comments

Comments

@cncPan
Copy link

cncPan commented Dec 30, 2018

Hello
I am sending M3 , M5 commands but i can not see PB1 bit (spindle enable bit) to toggle.

@Alfrederson
Copy link

@yaroslavVl I told you it was going to be a problem

@cncPan check this: usbcnc/grbl#63

@cncPan
Copy link
Author

cncPan commented Dec 30, 2018

Thank you for the comment . I configured $30 = $31 = 1000 , min and max spindle speed the same value and used spindle_pwm_bit as enable bit.

@robomechs
Copy link
Owner

robomechs commented Dec 30, 2018

Ok, I'll fix it.
Now we have next logic:

  • PWM+direction (def VARIABLE_SPINDLE, undef USE_SPINDLE_DIR_AS_ENABLE_PIN) or
  • PWM+enable (def VARIABLE_SPINDLE, def USE_SPINDLE_DIR_AS_ENABLE_PIN) or
  • enable+direction ((doesn't work now. bug) undef VARIABLE_SPINDLE, undef USE_SPINDLE_DIR_AS_ENABLE_PIN) or
  • just enable (undef VARIABLE_SPINDLE, def USE_SPINDLE_DIR_AS_ENABLE_PIN)

Is it ok if we will use PWM+enable+direction instead of PWM+enable?

@Alfrederson
Copy link

@yaroslavVl I guess so, since it doesn't leave either pin non-functional

@robomechs
Copy link
Owner

Ok, now PWM, Direction and Enable are avalaible by default. You can disable PWM (VARIABLE_SPINDLE) or Enable (USE_SPINDLE_ENABLE_PIN) or both, but direction will always run.

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