Compiling Grbl about grbl HOT 9 CLOSED

edit included file motion_control.c

void mc_dwell(uint32_t milliseconds)
{
st_synchronize();
delay_ms(milliseconds);
}

/* function for long delay */
void delay_ms(uint32_t ms) {
while ( ms )
{
_delay_ms(1);
ms--;
}
}

I Think, probe :-/

from grbl.

Take a look at my grbl fork. I've effectively picked up the torch, as Simen has been busy with other things. A lot of lingering bugs have been fixed and improvements have been made to v0.7, which is now master status.

FYI, a new v0.8 version with run-time commands (decelerating feedhold, resume, reset, jogging(TBD)) will be released later tonight. This is an alpha release.

from grbl.

I just checked out your fork, but it seems to have the same problem. I looked into it a bit more, and it seems that it is due to a change in avr-libc. As of v1.7.1 it is not possible to use variables with the delay functions. Apparently you weren't supposed to do this with previous versions anyways, but now it is being enforced due to a change in how delays are implemented.
You're probably using an older version of the library, so you don't get this error. It's really quite trivial to fix, I just added a couple of inline functions to nuts_bolts to mimic the old functionality of _delay_ms() and _delay_us(). The function that Identy suggested above does the trick, and it seems to be what most people are using.

from grbl.

:)

from grbl.

Good to know! I haven't run into this problem, as I pretty much exclusively use the avrdude compiler that comes the Arduino software. It sounds like it still supports the variable in the delay. I'll try to work a fix in when I get some time.

from grbl.

I am having the exact same problem with the latest versions in Master and Edge.

avr-gcc -Wall -Os -DF_CPU=16000000 -mmcu=atmega328p -I. -ffunction-sections -c motion_control.c -o motion_control.o
avr-gcc -Wall -Os -DF_CPU=16000000 -mmcu=atmega328p -I. -ffunction-sections -c limits.c -o limits.o
In file included from limits.c:21:0:
/usr/lib/gcc/avr/4.5.3/../../../avr/include/util/delay.h: In function ‘homing_cycle’:
/usr/lib/gcc/avr/4.5.3/../../../avr/include/util/delay.h:230:28: error: __builtin_avr_delay_cycles expects an integer constant.
/usr/lib/gcc/avr/4.5.3/../../../avr/include/util/delay.h:230:28: error: __builtin_avr_delay_cycles expects an integer constant.
make: *** [limits.o] Error 1

from grbl.

Thanks! Looks like I missed the _delay_us() calls when I had fixed it for the _delay_ms(). I'll issue a fix for this later today. If you want to fix it immediately, look in nut_bolts.c and repeat the same procedure as did with _delay_ms().

from grbl.

Thanks that fixed it. For completeness I changed it to this in limits.c:
delay_us(settings.pulse_microseconds);
STEPPING_PORT ^= out_bits & STEP_MASK;
delay_us(step_delay);

and I added this function to nuts_bolts.h:
void delay_us(uint16_t us)
{
while (us--) { _delay_us(1);}
}

from grbl.

Good to hear. Sorry that I didn't post the fix when I stated. I've been sidetracked with other things. When I finally get this next edge push out, I'll make sure to update the master branch with this fix.

from grbl.