Turning off the protection might be beneficial to those who want to do fast and long reversals.

This reverts commit 3682d9bd5b.

This is required when moving the buttons_init() function to the
lcd_init() function

Since the class "MainMenu" was used within a static variable the
initialization of the object (constructor call) was done before Arduino
library startup. It always caused a crash when using AVRStudio with
JTAG debugger (caused from calling the LCD initialization / the lot of
I/O work / the stack used during this calls). By moving the LCD_INIT
out of the constructor and using an explicit call inside of Arduino
setup() implementation immediately fixed all problems and the JTAG
debugger runs fine.

"i" runs from 0 to 4 but "add_homeing" array size is 3 only. On the
other hand the calculated value gets overwritten by either one of the
if choice.

Since a removal or insert of the sd card takes a long time in firmware, just reinitializing the lcd does not hurt.
actually, it solves a problem with the UltiControlle

see https://github.com/ErikZalm/Marlin/issues/148

Changed SLOWDOWN. IF this does not work ok OLD_SLOWDOWN is the old algo.

Fixed small bug in arcs
M105 prints now actual_temp/target_temp