Some cleanup around 'lcd_bootscreen()' and 'lcd_kill_screen()'

Move 'lcd_bootscreen()' from `lcd_init()` to 'setup()' where it is cexecute exactly once. Saves 'bool show_bootscreen'.
Move the call of 'lcd_custom_bootscreen()' to the begin of 'lcd_bootscreen()'.
Move the delays into the related functions.
Move the picture loop around 'lcd_kill_screen()' into the function.
This commit is contained in:
AnHardt 2017-11-03 11:58:20 +01:00
parent 84b93d941f
commit ca0e4b4e96
3 changed files with 41 additions and 63 deletions

View file

@ -816,24 +816,12 @@ void setup() {
lcd_init(); lcd_init();
#ifndef CUSTOM_BOOTSCREEN_TIMEOUT
#define CUSTOM_BOOTSCREEN_TIMEOUT 2500
#endif
#if ENABLED(SHOW_BOOTSCREEN) #if ENABLED(SHOW_BOOTSCREEN)
#if ENABLED(DOGLCD) // On DOGM the first bootscreen is already drawn
#if ENABLED(SHOW_CUSTOM_BOOTSCREEN)
safe_delay(CUSTOM_BOOTSCREEN_TIMEOUT); // Custom boot screen pause
lcd_bootscreen(); // Show Marlin boot screen
#endif
safe_delay(BOOTSCREEN_TIMEOUT); // Pause
#elif ENABLED(ULTRA_LCD)
lcd_bootscreen(); lcd_bootscreen();
#if DISABLED(SDSUPPORT) #if ENABLED(ULTRA_LCD) && DISABLED(SDSUPPORT)
lcd_init(); lcd_init();
#endif #endif
#endif #endif
#endif
#if ENABLED(MIXING_EXTRUDER) && MIXING_VIRTUAL_TOOLS > 1 #if ENABLED(MIXING_EXTRUDER) && MIXING_VIRTUAL_TOOLS > 1
mixing_tools_init(); mixing_tools_init();

View file

@ -715,14 +715,7 @@ void lcd_reset_status() { lcd_setstatusPGM(PSTR(""), -1); }
void kill_screen(const char* lcd_msg) { void kill_screen(const char* lcd_msg) {
lcd_init(); lcd_init();
lcd_setalertstatusPGM(lcd_msg); lcd_setalertstatusPGM(lcd_msg);
#if ENABLED(DOGLCD)
u8g.firstPage();
do {
lcd_kill_screen(); lcd_kill_screen();
} while (u8g.nextPage());
#else
lcd_kill_screen();
#endif
} }
#if ENABLED(ULTIPANEL) #if ENABLED(ULTIPANEL)

View file

@ -278,10 +278,12 @@ void lcd_printPGM_utf(const char *str, uint8_t n=LCD_WIDTH) {
#if ENABLED(SHOW_BOOTSCREEN) #if ENABLED(SHOW_BOOTSCREEN)
bool show_bootscreen = true;
#if ENABLED(SHOW_CUSTOM_BOOTSCREEN) #if ENABLED(SHOW_CUSTOM_BOOTSCREEN)
#ifndef CUSTOM_BOOTSCREEN_TIMEOUT
#define CUSTOM_BOOTSCREEN_TIMEOUT 2500
#endif
void lcd_custom_bootscreen() { void lcd_custom_bootscreen() {
u8g.firstPage(); u8g.firstPage();
do { do {
@ -290,14 +292,15 @@ void lcd_printPGM_utf(const char *str, uint8_t n=LCD_WIDTH) {
( 64 - (CUSTOM_BOOTSCREEN_BMPHEIGHT)) /2, ( 64 - (CUSTOM_BOOTSCREEN_BMPHEIGHT)) /2,
CEILING(CUSTOM_BOOTSCREEN_BMPWIDTH, 8), CUSTOM_BOOTSCREEN_BMPHEIGHT, custom_start_bmp); CEILING(CUSTOM_BOOTSCREEN_BMPWIDTH, 8), CUSTOM_BOOTSCREEN_BMPHEIGHT, custom_start_bmp);
} while (u8g.nextPage()); } while (u8g.nextPage());
safe_delay(CUSTOM_BOOTSCREEN_TIMEOUT);
} }
#endif // SHOW_CUSTOM_BOOTSCREEN #endif // SHOW_CUSTOM_BOOTSCREEN
void lcd_bootscreen() { void lcd_bootscreen() {
#if ENABLED(SHOW_CUSTOM_BOOTSCREEN)
if (show_bootscreen) { lcd_custom_bootscreen();
show_bootscreen = false; #endif
#if ENABLED(START_BMPHIGH) #if ENABLED(START_BMPHIGH)
constexpr uint8_t offy = 0; constexpr uint8_t offy = 0;
@ -320,7 +323,7 @@ void lcd_printPGM_utf(const char *str, uint8_t n=LCD_WIDTH) {
u8g.drawStr(txt2X, u8g.getHeight() - (DOG_CHAR_HEIGHT) * 1 / 2, STRING_SPLASH_LINE2); u8g.drawStr(txt2X, u8g.getHeight() - (DOG_CHAR_HEIGHT) * 1 / 2, STRING_SPLASH_LINE2);
#endif #endif
} while (u8g.nextPage()); } while (u8g.nextPage());
} safe_delay(BOOTSCREEN_TIMEOUT);
} }
#endif // SHOW_BOOTSCREEN #endif // SHOW_BOOTSCREEN
@ -351,20 +354,12 @@ static void lcd_implementation_init() {
#elif ENABLED(LCD_SCREEN_ROT_270) #elif ENABLED(LCD_SCREEN_ROT_270)
u8g.setRot270(); // Rotate screen by 270° u8g.setRot270(); // Rotate screen by 270°
#endif #endif
#if ENABLED(SHOW_BOOTSCREEN)
if (show_bootscreen) {
#if ENABLED(SHOW_CUSTOM_BOOTSCREEN)
lcd_custom_bootscreen();
#else
lcd_bootscreen();
#endif
}
#endif
} }
// The kill screen is displayed for unrecoverable conditions // The kill screen is displayed for unrecoverable conditions
void lcd_kill_screen() { void lcd_kill_screen() {
u8g.firstPage();
do {
lcd_setFont(FONT_MENU); lcd_setFont(FONT_MENU);
u8g.setPrintPos(0, u8g.getHeight()/4*1); u8g.setPrintPos(0, u8g.getHeight()/4*1);
lcd_print_utf(lcd_status_message); lcd_print_utf(lcd_status_message);
@ -372,6 +367,7 @@ void lcd_kill_screen() {
lcd_printPGM(PSTR(MSG_HALTED)); lcd_printPGM(PSTR(MSG_HALTED));
u8g.setPrintPos(0, u8g.getHeight()/4*3); u8g.setPrintPos(0, u8g.getHeight()/4*3);
lcd_printPGM(PSTR(MSG_PLEASE_RESET)); lcd_printPGM(PSTR(MSG_PLEASE_RESET));
} while (u8g.nextPage());
} }
void lcd_implementation_clear() { } // Automatically cleared by Picture Loop void lcd_implementation_clear() { } // Automatically cleared by Picture Loop
@ -477,6 +473,7 @@ inline void lcd_implementation_status_message(const bool blink) {
} }
} }
#else #else
UNUSED(blink);
lcd_print_utf(lcd_status_message); lcd_print_utf(lcd_status_message);
#endif #endif
} }