grblHAL / iMXRT1062

grblHAL driver for NXP iMXRT1062 (Teensy 4.x)
Other
50 stars 37 forks source link

I have run the compile both with Arduino IDE and PlatformIO with MCP3221 Enabled. #68

Closed sntlewis closed 1 year ago

sntlewis commented 1 year ago

I have run the compile both with Arduino IDE and PlatformIO with MCP3221 Enabled.

The following compile time errors occur on lines 129 and 386 in ioports.c

src\ioports.c: In function 'wait_on_input': src\ioports.c:129:20: error: 'analog_n_in' undeclared (first use in this function) else if(port < analog_n_in) ^ src\ioports.c:129:20: note: each undeclared identifier is reported only once for each function it appears in src\ioports.c: In function 'ioports_init': src\ioports.c:386:34: error: 'analog_n_in' undeclared (first use in this function) hal.port.num_analog_in = analog_n_in = 1;

I have found a MCP3221 "daughter card" on Phil-Barrett's github.

Originally posted by @sntlewis in https://github.com/grblHAL/iMXRT1062/discussions/67

terjeio commented 1 year ago

Oops, a regression - can you try with changing these lines to:

#if MCP3221_ENABLE
static xbar_t analog_in;
static uint_fast8_t analog_n_in;
static enumerate_pins_ptr on_enumerate_pins;
#endif

I'll fix this in the next commit.

sntlewis commented 1 year ago

Thank fixed it. Now onto the next level of hardware development, order parts for the mcp3221 bob.