apballard / Marlin

Optimized firmware for RepRap 3D printers based on the Arduino platform.
http://www.marlinfw.org/
GNU General Public License v3.0
20 stars 12 forks source link

Not working on latest Marlin #4

Open MechanicallyDev opened 6 years ago

MechanicallyDev commented 6 years ago

When I compile I get an error on the serial communication: I already changed the serial3 to serial2 on nexconfig.h. Nextion board is connected to serial 2 on ramps (pin 16 and 17).

Is this fork still working? Thank you

sketch\Experimental_LCD.cpp: In function 'void processBuffer(const char*)':

Experimental_LCD.cpp:92: error: 'SERIAL_ECHO_START' was not declared in this scope

   SERIAL_ECHO_START;

   ^

Experimental_LCD.cpp:94: error: 'SERIAL_EOL' was not declared in this scope

   SERIAL_EOL;

   ^

sketch\Experimental_LCD.cpp: In function 'void ProcessPage(char*, uint8_t)':

Experimental_LCD.cpp:229: error: 'SERIAL_ECHO_START' was not declared in this scope

  SERIAL_ECHO_START;

  ^

Experimental_LCD.cpp:231: error: 'SERIAL_EOL' was not declared in this scope

  SERIAL_EOL;

  ^

sketch\Experimental_LCD.cpp: In function 'void processMessage(const char*)':

Experimental_LCD.cpp:248: error: 'SERIAL_ECHO_START' was not declared in this scope

  SERIAL_ECHO_START;

  ^

Experimental_LCD.cpp:250: error: 'SERIAL_EOL' was not declared in this scope

  SERIAL_EOL;

  ^

exit status 1
'SERIAL_ECHO_START' was not declared in this scope
weeelfi commented 6 years ago

Just put open and close parenthesis on all of those lines

MechanicallyDev commented 5 years ago

Thanks weeelfi. Now I'm having an issue with "lcd_refresh(); was not declared in this scope" This function doesn't exist elsewhere in the code. Should I just comment it out?

If I comment it the compile completes, but there is no feedback between the ramps and the nextion display.

GuusvanMarle commented 5 years ago

Hello, I've had the same issue with "lcd_refresh(); was not declared in this scope", after fixing loads of errors before that, and indeed it does not talk to the display. Any progress on this error yet?

GuusvanMarle commented 5 years ago

Lcd_refresh() is also called in UBL_G29.cpp But i haven't figured out the difference between the two.

neocogent commented 5 years ago

Any progress on this issue? I may want to update this to work with recent marlin for my own use. If solved, did you create a fork with fixes that may allow building on going forward?