TFT Presets, Generic options, Sanity checks

Author: thinkyhead

Marlin/Configuration.h

@@ -2205,22 +2205,83 @@
 //=============================================================================
 
 /**
- * TFT Type - Select your Display type
- *
- * Available options are:
- *   MKS_TS35_V2_0,
- *   MKS_ROBIN_TFT24, MKS_ROBIN_TFT28, MKS_ROBIN_TFT32, MKS_ROBIN_TFT35,
- *   MKS_ROBIN_TFT43, MKS_ROBIN_TFT_V1_1R
- *   TFT_TRONXY_X5SA, ANYCUBIC_TFT35, LONGER_LK_TFT28
- *   TFT_GENERIC
- *
- * For TFT_GENERIC, you need to configure these 3 options:
- *   Driver:     TFT_DRIVER
- *               Current Drivers are: AUTO, ST7735, ST7789, ST7796, R61505, ILI9328, ILI9341, ILI9488
- *   Resolution: TFT_WIDTH and TFT_HEIGHT
- *   Interface:  TFT_INTERFACE_FSMC or TFT_INTERFACE_SPI
+ * Specific TFT Model Presets. Enable one of the following options
+ * or enable TFT_GENERIC and set sub-options.
  */
+
+//
+// 480x320, 3.5", SPI Display From MKS
+// Normally used in MKS Robin Nano V2
+//
+//#define MKS_TS35_V2_0
+
+//
+// 320x240, 2.4", FSMC Display From MKS
+// Normally used in MKS Robin Nano V1.2
+//
+//#define MKS_ROBIN_TFT24
+
+//
+// 320x240, 2.8", FSMC Display From MKS
+// Normally used in MKS Robin Nano V1.2
+//
+//#define MKS_ROBIN_TFT28
+
+//
+// 320x240, 3.2", FSMC Display From MKS
+// Normally used in MKS Robin Nano V1.2
+//
+//#define MKS_ROBIN_TFT32
+
+//
+// 480x320, 3.5", FSMC Display From MKS
+// Normally used in MKS Robin Nano V1.2
+//
+//#define MKS_ROBIN_TFT35
+
+//
+// 480x272, 4.3", FSMC Display From MKS
+//
+//#define MKS_ROBIN_TFT43
+
+//
+// 320x240, 3.2", FSMC Display From MKS
+// Normally used in MKS Robin
+//
+//#define MKS_ROBIN_TFT_V1_1R
+
+//
+// 480x320, 3.5", FSMC Stock Display from TronxXY
+//
+//#define TFT_TRONXY_X5SA
+
+//
+// 480x320, 3.5", FSMC Stock Display from AnyCubic
+//
+//#define ANYCUBIC_TFT35
+
+//
+// 320x240, 2.8", FSMC Stock Display from Longer/Alfawise
+//
+//#define LONGER_LK_TFT28
+
+//
+// Generic TFT with detailed options
+//
 //#define TFT_GENERIC
+#if ENABLED(TFT_GENERIC)
+  // :[ 'AUTO', 'ST7735', 'ST7789', 'ST7796', 'R61505', 'ILI9328', 'ILI9341', 'ILI9488' ]
+  #define TFT_DRIVER AUTO
+
+  // Interface. Enable one of the following options:
+  //#define TFT_INTERFACE_FSMC
+  //#define TFT_INTERFACE_SPI
+
+  // TFT Resolution. Enable one of the following options:
+  //#define TFT_RES_320x240
+  //#define TFT_RES_480x272
+  //#define TFT_RES_480x320
+#endif
 
 /**
  * TFT UI - User Interface Selection. Enable one of the following options:

Marlin/src/core/macros.h

@@ -170,7 +170,7 @@
 #define _DO_12(W,C,A,V...) (_##W##_1(A) C _DO_11(W,C,V))
 #define __DO_N(W,C,N,V...) _DO_##N(W,C,V)
 #define _DO_N(W,C,N,V...)  __DO_N(W,C,N,V)
-#define DO(W,C,V...)       _DO_N(W,C,NUM_ARGS(V),V)
+#define DO(W,C,V...)       (_DO_N(W,C,NUM_ARGS(V),V))
 
 // Macros to support option testing
 #define _CAT(a,V...) a##V
@@ -186,6 +186,7 @@
 #define _DIS_1(O)           NOT(_ENA_1(O))
 #define ENABLED(V...)       DO(ENA,&&,V)
 #define DISABLED(V...)      DO(DIS,&&,V)
+#define COUNT_ENABLED(V...) DO(ENA,+,V)
 
 #define TERN(O,A,B)         _TERN(_ENA_1(O),B,A)    // OPTION converted to '0' or '1'
 #define TERN0(O,A)          _TERN(_ENA_1(O),0,A)    // OPTION converted to A or '0'
@@ -200,6 +201,7 @@
 #define ALL(V...)           ENABLED(V)
 #define BOTH(V1,V2)         ALL(V1,V2)
 #define EITHER(V1,V2)       ANY(V1,V2)
+#define MANY(V...)          (COUNT_ENABLED(V) > 1)
 
 // Macros to support pins/buttons exist testing
 #define PIN_EXISTS(PN)      (defined(PN##_PIN) && PN##_PIN >= 0)

Marlin/src/inc/Conditionals_LCD.h

@@ -311,31 +311,25 @@
   #define IS_ULTIPANEL
 #endif
 
-// Compatibility
-#if ENABLED(FSMC_GRAPHICAL_TFT)
-  #define TFT_CLASSIC_UI
-  #define TFT_INTERFACE_FSMC
-  #define TFT_GENERIC
-#elif ENABLED(SPI_GRAPHICAL_TFT)
-  #define TFT_CLASSIC_UI
-  #define TFT_INTERFACE_SPI
+// TFT Compatibility
+#if ANY(FSMC_GRAPHICAL_TFT, SPI_GRAPHICAL_TFT, TFT_320x240, TFT_480x320, TFT_320x240_SPI, TFT_480x320_SPI, TFT_LVGL_UI_FSMC, TFT_LVGL_UI_SPI)
+  #define IS_LEGACY_TFT
   #define TFT_GENERIC
-#elif EITHER(TFT_320x240, TFT_480x320)
-  #define TFT_COLOR_UI
+  #warning "Don't forget to update your TFT settings in Configuration.h."
+#endif
+
+#if ANY(FSMC_GRAPHICAL_TFT, TFT_320x240, TFT_480x320, TFT_LVGL_UI_FSMC)
   #define TFT_INTERFACE_FSMC
-  #define TFT_GENERIC
-#elif EITHER(TFT_320x240_SPI, TFT_480x320_SPI)
-  #define TFT_COLOR_UI
+#elif ANY(SPI_GRAPHICAL_TFT, TFT_320x240_SPI, TFT_480x320_SPI, TFT_LVGL_UI_SPI)
   #define TFT_INTERFACE_SPI
-  #define TFT_GENERIC
-#elif ENABLED(TFT_LVGL_UI_FSMC)
-  #define TFT_LVGL_UI
-  #define TFT_INTERFACE_FSMC
-  #define TFT_GENERIC
-#elif ENABLED(TFT_LVGL_UI_SPI)
+#endif
+
+#if EITHER(FSMC_GRAPHICAL_TFT, SPI_GRAPHICAL_TFT)
+  #define TFT_CLASSIC_UI
+#elif ANY(TFT_320x240, TFT_480x320, TFT_320x240_SPI, TFT_480x320_SPI)
+  #define TFT_COLOR_UI
+#elif EITHER(TFT_LVGL_UI_FSMC, TFT_LVGL_UI_SPI)
   #define TFT_LVGL_UI
-  #define TFT_INTERFACE_SPI
-  #define TFT_GENERIC
 #endif
 
 // FSMC/SPI TFT Panels (LVGL)
@@ -837,83 +831,85 @@
 #if ENABLED(MKS_TS35_V2_0)
   // Most common: ST7796
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY)
-  #define TFT_WIDTH  480
-  #define TFT_HEIGHT 320
+  #define TFT_RES_480x320
   #define TFT_INTERFACE_SPI
-  #define GRAPHICAL_TFT_UPSCALE 3
 #elif ENABLED(MKS_ROBIN_TFT24)
   // Most common: ST7789
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_Y)
-  #define TFT_WIDTH  320
-  #define TFT_HEIGHT 240
+  #define TFT_RES_320x240
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif ENABLED(MKS_ROBIN_TFT28)
   // Most common: ST7789
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_Y)
-  #define TFT_WIDTH  320
-  #define TFT_HEIGHT 240
+  #define TFT_RES_320x240
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif ENABLED(MKS_ROBIN_TFT32)
   // Most common: ST7789
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_Y)
-  #define TFT_WIDTH  320
-  #define TFT_HEIGHT 240
+  #define TFT_RES_320x240
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif ENABLED(MKS_ROBIN_TFT35)
   // Most common: ILI9488
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_X | TFT_INVERT_Y)
-  #define TFT_WIDTH  480
-  #define TFT_HEIGHT 320
+  #define TFT_RES_480x320
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 3
 #elif ENABLED(MKS_ROBIN_TFT43)
   #define TFT_DEFAULT_ORIENTATION 0
   #define TFT_DRIVER SSD1963
-  #define TFT_WIDTH  480
-  #define TFT_HEIGHT 272
+  #define TFT_RES_480x272
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif ENABLED(MKS_ROBIN_TFT_V1_1R)
   // ILI9328 or R61505
-  #define TFT_DEFAULT_ORIENTATION (TFT_INVERT_X | TFT_INVERT_Y | TFT_EXCHANGE_XY)
-  #define TFT_WIDTH  320
-  #define TFT_HEIGHT 240
+  #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_X | TFT_INVERT_Y)
+  #define TFT_RES_320x240
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif EITHER(TFT_TRONXY_X5SA, ANYCUBIC_TFT35)
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_X | TFT_INVERT_Y)
   #define TFT_DRIVER ILI9488
-  #define TFT_WIDTH  480
-  #define TFT_HEIGHT 320
+  #define TFT_RES_480x320
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 3
 #elif ENABLED(LONGER_LK_TFT28)
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_X | TFT_INVERT_Y)
-  #define TFT_WIDTH  320
-  #define TFT_HEIGHT 240
+  #define TFT_RES_320x240
   #define TFT_INTERFACE_FSMC
-  #define GRAPHICAL_TFT_UPSCALE 2
 #elif ENABLED(TFT_GENERIC)
   #define TFT_DEFAULT_ORIENTATION (TFT_EXCHANGE_XY | TFT_INVERT_X | TFT_INVERT_Y)
+  #if NONE(TFT_RES_320x240, TFT_RES_480x272, TFT_RES_480x320)
+    #define TFT_RES_320x240
+  #endif
+  #if NONE(TFT_INTERFACE_FSMC, TFT_INTERFACE_SPI)
+    #define TFT_INTERFACE_SPI
+  #endif
+#endif
+
+#if ENABLED(TFT_RES_320x240)
+  #define TFT_WIDTH  320
+  #define TFT_HEIGHT 240
+  #define GRAPHICAL_TFT_UPSCALE 2
+#elif ENABLED(TFT_RES_480x272)
+  #define TFT_WIDTH  480
+  #define TFT_HEIGHT 272
+  #define GRAPHICAL_TFT_UPSCALE 2
+#elif ENABLED(TFT_RES_480x320)
+  #define TFT_WIDTH  480
+  #define TFT_HEIGHT 320
+  #define GRAPHICAL_TFT_UPSCALE 3
 #endif
 
 // FSMC/SPI TFT Panels using standard HAL/tft/tft_(fsmc|spi).h
 #if ENABLED(TFT_INTERFACE_FSMC)
   #define HAS_FSMC_TFT 1
   #if ENABLED(TFT_CLASSIC_UI)
-    #define FSMC_GRAPHICAL_TFT
+    #define HAS_FSMC_GRAPHICAL_TFT 1
   #elif ENABLED(TFT_LVGL_UI)
-    #define TFT_LVGL_UI_FSMC
+    #define HAS_TFT_LVGL_UI_FSMC 1
   #endif
 #elif ENABLED(TFT_INTERFACE_SPI)
   #define HAS_SPI_TFT 1
   #if ENABLED(TFT_CLASSIC_UI)
-    #define SPI_GRAPHICAL_TFT
+    #define HAS_SPI_GRAPHICAL_TFT 1
   #elif ENABLED(TFT_LVGL_UI)
-    #define TFT_LVGL_UI_SPI
+    #define HAS_TFT_LVGL_UI_SPI 1
   #endif
 #endif
 

Marlin/src/inc/Conditionals_adv.h

@@ -148,7 +148,7 @@
 
 #if ANY(MARLIN_BRICKOUT, MARLIN_INVADERS, MARLIN_SNAKE, MARLIN_MAZE)
   #define HAS_GAMES 1
-  #if (1 < ENABLED(MARLIN_BRICKOUT) + ENABLED(MARLIN_INVADERS) + ENABLED(MARLIN_SNAKE) + ENABLED(MARLIN_MAZE))
+  #if MANY(MARLIN_BRICKOUT, MARLIN_INVADERS, MARLIN_SNAKE, MARLIN_MAZE)
     #define HAS_GAME_MENU 1
   #endif
 #endif

Marlin/src/inc/Conditionals_post.h

@@ -383,7 +383,7 @@
 
 #endif
 
-#if ANY(HAS_GRAPHICAL_TFT, LCD_USE_DMA_FSMC, FSMC_GRAPHICAL_TFT, SPI_GRAPHICAL_TFT) || !PIN_EXISTS(SD_DETECT)
+#if ANY(HAS_GRAPHICAL_TFT, LCD_USE_DMA_FSMC, HAS_FSMC_GRAPHICAL_TFT, HAS_SPI_GRAPHICAL_TFT) || !PIN_EXISTS(SD_DETECT)
   #define NO_LCD_REINIT 1  // Suppress LCD re-initialization
 #endif