Various fixes: missed code uses, other updates

Marlin/src/core/utility.h

@@ -86,11 +86,15 @@ constexpr uint8_t ui8_to_percent(const uint8_t i) { return (int(i) * 100 + 127)
 const xyze_char_t axis_codes LOGICAL_AXIS_ARRAY('E', 'X', 'Y', 'Z', AXIS4_NAME, AXIS5_NAME, AXIS6_NAME, AXIS7_NAME, AXIS8_NAME, AXIS9_NAME);
 #if NUM_AXES <= XYZ && !HAS_EXTRUDERS
   #define AXIS_CHAR(A) ((char)('X' + A))
-  #define IAXIS_CHAR AXIS_CHAR
+  #if ANY(HAS_MOTOR_CURRENT_SPI, HAS_MOTOR_CURRENT_I2C, HAS_MOTOR_CURRENT_DAC)
+    #define IAXIS_CHAR AXIS_CHAR
+  #endif
 #else
-  const xyze_char_t iaxis_codes LOGICAL_AXIS_ARRAY('E', 'X', 'Y', 'Z', 'I', 'J', 'K', 'U', 'V', 'W');
+  #if ANY(HAS_MOTOR_CURRENT_SPI, HAS_MOTOR_CURRENT_I2C, HAS_MOTOR_CURRENT_DAC)
+    const xyze_char_t iaxis_codes LOGICAL_AXIS_ARRAY('E', 'X', 'Y', 'Z', 'I', 'J', 'K', 'U', 'V', 'W');
+    #define IAXIS_CHAR(A) iaxis_codes[A]
+  #endif
   #define AXIS_CHAR(A) axis_codes[A]
-  #define IAXIS_CHAR(A) iaxis_codes[A]
 #endif
 
 #if ENABLED(MARLIN_DEV_MODE)

Marlin/src/inc/Conditionals-2-LCD.h

@@ -611,7 +611,6 @@
    *    (calling advance_status_scroll, status_and_len for a scrolling status message)
    */
   #define HAS_DISPLAY 1
-  #define HAS_UI_UPDATE 1
   #define HAS_STATUS_MESSAGE 1
 #endif
 

Marlin/src/lcd/e3v2/proui/dwinui.h

@@ -117,6 +117,7 @@
 #define ICON_SetPreheat10     ICON_SetCustomPreheat
 #define ICON_ShapingX         ICON_MoveX
 #define ICON_ShapingY         ICON_MoveY
+#define ICON_ShapingZ         ICON_MoveZ
 #define ICON_Sound            ICON_Cool
 #define ICON_TMCSet           ICON_PrintSize
 #define ICON_TMCXSet          ICON_MoveX

Marlin/src/lcd/marlinui.h

@@ -208,14 +208,6 @@ class MarlinUI {
 
   static void init();
 
-  static void reinit_lcd() { TERN_(REINIT_NOISY_LCD, init_lcd()); }
-
-  #if HAS_WIRED_LCD
-    static bool detected();
-  #else
-    static bool detected() { return true; }
-  #endif
-
   #if HAS_MULTI_LANGUAGE
     static uint8_t language;
     static void set_language(const uint8_t lang);
@@ -482,8 +474,8 @@ class MarlinUI {
    * @param cstr    A C-string to set as the status.
    */
   static void set_status_no_expire_P(PGM_P const pstr)      { set_status_P(pstr, true); }
-  static void set_status_no_expire(const char * const cstr) { set_status(cstr, true); }
-  static void set_status_no_expire(FSTR_P const fstr)       { set_status(fstr, true); }
+  static void set_status_no_expire(const char * const cstr) { set_status(cstr, true);   }
+  static void set_status_no_expire(FSTR_P const fstr)       { set_status(fstr, true);   }
 
   /**
    * @brief Set a status with a format string and parameters.
@@ -495,16 +487,16 @@ class MarlinUI {
   template<typename... Args>
   static void status_printf(int8_t level, FSTR_P const ffmt, Args... more) { status_printf_P(level, FTOP(ffmt), more...); }
 
-  // Periodic or as-needed display update
-  static void update() IF_DISABLED(HAS_UI_UPDATE, {});
-
   // Tell the screen to redraw on the next call
   FORCE_INLINE static void refresh() {
     TERN_(HAS_WIRED_LCD, refresh(LCDVIEW_CLEAR_CALL_REDRAW));
   }
 
   #if HAS_DISPLAY
 
+    // Periodic or as-needed display update
+    static void update();
+
     static void init_lcd();
 
     // Erase the LCD contents. Do the lowest-level thing required to clear the LCD.
@@ -598,6 +590,10 @@ class MarlinUI {
 
       static void status_screen();
 
+    #else
+
+      static void quick_feedback(const bool=true) {}
+
     #endif // HAS_WIRED_LCD
 
     #if HAS_MARLINUI_U8GLIB
@@ -623,16 +619,16 @@ class MarlinUI {
 
   #else // No LCD
 
+    static void update() {}
     static void init_lcd() {}
     static void clear_lcd() {}
     static void clear_for_drawing() {}
     static void kill_screen(FSTR_P const, FSTR_P const) {}
 
   #endif
 
-  #if !HAS_WIRED_LCD
-    static void quick_feedback(const bool=true) {}
-  #endif
+  static bool detected() IF_DISABLED(HAS_WIRED_LCD, { return true; });
+  static void reinit_lcd() { TERN_(REINIT_NOISY_LCD, init_lcd()); }
 
   static void completion_feedback(const bool good=true);
 

Marlin/src/module/temperature.cpp

@@ -4286,7 +4286,7 @@ void Temperature::isr() {
   // Update lcd buttons 488 times per second
   //
   static bool do_buttons;
-  if ((do_buttons ^= true)) ui.update_buttons();
+  if (FLIP(do_buttons)) ui.update_buttons();
 
   /**
    * One sensor is sampled on every other call of the ISR.

Marlin/src/sd/SdBaseFile.cpp

@@ -643,9 +643,9 @@ bool SdBaseFile::open(SdBaseFile * const dirFile, const uint8_t dname[11]
             emptyIndex = 0,
             reqEntriesNum = useLFN ? getLFNEntriesNum((char*)dlname) + 1 : 1,
             lfnNameLength = useLFN ? strlen((char*)dlname) : 0,
-            lfnName[LONG_FILENAME_LENGTH],
             lfnSequenceNumber = 0,
             lfnChecksum = 0;
+    uint8_t lfnName[LONG_FILENAME_LENGTH];
   #endif
 
   // Rewind this dir

Marlin/src/sd/cardreader.cpp

@@ -87,7 +87,8 @@ IF_DISABLED(NO_SD_AUTOSTART, uint8_t CardReader::autofile_index); // = 0
 
 // private:
 
-MediaFile CardReader::root, CardReader::workDir, CardReader::workDirParents[MAX_DIR_DEPTH];
+MediaFile CardReader::root, CardReader::workDir;
+MediaFile CardReader::workDirParents[MAX_DIR_DEPTH];
 uint8_t CardReader::workDirDepth;
 int16_t CardReader::nrItems = -1;
 
@@ -123,7 +124,7 @@ int16_t CardReader::nrItems = -1;
       #if ENABLED(SDSORT_DYNAMIC_RAM)
         uint8_t *CardReader::isDir;
       #elif ENABLED(SDSORT_CACHE_NAMES) || DISABLED(SDSORT_USES_STACK)
-        uint8_t CardReader::isDir[(SDSORT_LIMIT+7)>>3];
+        uint8_t CardReader::isDir[(SDSORT_LIMIT + 7) >> 3];
       #endif
       #define IS_DIR(n) TEST(isDir[(n) >> 3], (n) & 0x07)
     #endif

Marlin/src/sd/cardreader.h

@@ -271,7 +271,8 @@ class CardReader {
   //
   // Working directory and parents
   //
-  static MediaFile root, workDir, workDirParents[MAX_DIR_DEPTH];
+  static MediaFile root, workDir;
+  static MediaFile workDirParents[MAX_DIR_DEPTH];
   static uint8_t workDirDepth;
   static int16_t nrItems; // Cache the total count
 

buildroot/tests/STM32F103RE_creality

@@ -30,7 +30,7 @@ opt_enable DWIN_LCD_PROUI INDIVIDUAL_AXIS_HOMING_SUBMENU SET_PROGRESS_MANUALLY S
            SOUND_MENU_ITEM PRINTCOUNTER NOZZLE_PARK_FEATURE ADVANCED_PAUSE_FEATURE FILAMENT_RUNOUT_SENSOR \
            BLTOUCH Z_SAFE_HOMING AUTO_BED_LEVELING_UBL MESH_EDIT_MENU LCD_BED_TRAMMING \
            LIMITED_MAX_FR_EDITING LIMITED_MAX_ACCEL_EDITING LIMITED_JERK_EDITING BAUD_RATE_GCODE \
-           CASE_LIGHT_ENABLE CASE_LIGHT_MENU CASE_LIGHT_NO_BRIGHTNESS
+           CASE_LIGHT_ENABLE CASE_LIGHT_MENU CASE_LIGHT_NO_BRIGHTNESS INPUT_SHAPING_X INPUT_SHAPING_Y INPUT_SHAPING_Z
 opt_set PREHEAT_3_LABEL '"CUSTOM"' PREHEAT_3_TEMP_HOTEND 240 PREHEAT_3_TEMP_BED 60 PREHEAT_3_FAN_SPEED 128 BOOTSCREEN_TIMEOUT 1100 CASE_LIGHT_PIN 4
 exec_test $1 $2 "Ender-3 S1 - ProUI (PIDTEMP)" "$3"