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 pin issue #36

Closed
lakeroe opened this issue Feb 2, 2018 · 6 comments
Closed

Spindle enable pin issue #36

lakeroe opened this issue Feb 2, 2018 · 6 comments

Comments

@lakeroe
Copy link

lakeroe commented Feb 2, 2018

Hello,

for my application I need a separate spindle enable pin. With default settings PB13 is not working as expected. For this reason I uncommented following line in config.h
#define USE_SPINDLE_DIR_AS_ENABLE_PIN
This leads to an compilation error and I had to add "&& !defined(CPU_MAP_STM32F103)" in line 123 in grbl.h
Now it compiles fine and PB13 works as expected.
I just don't know if this is the intended way to do ...

Best regards,
lakeroe

@SailWithChips
Copy link

I have same problem like you. But also with your lines I have not PB13 working as expected.

@lakeroe
Copy link
Author

lakeroe commented Mar 7, 2018

My working BIN-File is attached (rename *.txt to *.bin). Try it if you like ...
grbl.txt

@SailWithChips
Copy link

Thanks for sharing, works your bin file. PB13 as expected, PA15 as probe input also fine now I would like to know in your BIN file were it's the PWM signal for spindle rpm's?.
Would be really appreciatted if you share also the CooIde project or at least the cpu_map.h ;)

Thanks a lot!!

@lakeroe
Copy link
Author

lakeroe commented Mar 7, 2018

PWM signal is PA8.
The complete project is attached. I don't use any IDE, compile project by running _MAKE.bat

GRBL.zip

@SailWithChips
Copy link

Thanks a lot lakeroe. With your bin and CPU_MAP, everything is working so nice... thanks a lot for your help!

@robomechs
Copy link

robomechs commented Jun 22, 2018

hmm, yeah, strange condition.
Why the issue is closed?

#if defined(USE_SPINDLE_DIR_AS_ENABLE_PIN) && !defined(CPU_MAP_ATMEGA328P)
  #error "USE_SPINDLE_DIR_AS_ENABLE_PIN may only be used with a 328p processor"
#endif

Based on this condition, we will never be able to use this functionality with STM32F103C8T6 board. However, in the rest of the code this is assumed. I suggest to comment it out at all)))

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

4 participants