Skip to content

Commit

Permalink
Revert "Only download & compile required libraries (MarlinFirmware#18699
Browse files Browse the repository at this point in the history
)"

This reverts commit 4a55bdb.
  • Loading branch information
zvoniimiir committed Jul 20, 2020
1 parent 11fca02 commit 66d8593
Show file tree
Hide file tree
Showing 9 changed files with 87 additions and 285 deletions.
7 changes: 0 additions & 7 deletions Marlin/src/core/drivers.h
Original file line number Diff line number Diff line change
Expand Up @@ -176,13 +176,6 @@
#define HAS_TMC_SPI 1
#endif

//
// TMC26XX Stepper Drivers
//
#if HAS_DRIVER(TMC26X)
#define HAS_TMC26X 1
#endif

//
// L64XX Stepper Drivers
//
Expand Down
5 changes: 1 addition & 4 deletions Marlin/src/feature/touch/xpt2046.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,7 @@

#include "xpt2046.h"
#include "../../inc/MarlinConfig.h"
#if ENABLED(FSMC_GRAPHICAL_TFT)
#include "../../lcd/dogm/ultralcd_DOGM.h" // for LCD_FULL_PIXEL_WIDTH, etc.
#endif

#include "../../lcd/dogm/ultralcd_DOGM.h" // for LCD_FULL_PIXEL_WIDTH, etc.

/*
* Draw and Touch processing
Expand Down
5 changes: 1 addition & 4 deletions Marlin/src/inc/SanityCheck.h
Original file line number Diff line number Diff line change
Expand Up @@ -2205,15 +2205,12 @@ static_assert(hbm[Z_AXIS] >= 0, "HOMING_BUMP_MM.Z must be greater than or equal
+ ENABLED(MKS_12864OLED_SSD1306) \
+ ENABLED(U8GLIB_SH1106_EINSTART) \
+ ENABLED(OVERLORD_OLED) \
+ ENABLED(FYSETC_242_OLED_12864) \
+ ENABLED(DGUS_LCD_UI_ORIGIN) \
+ ENABLED(DGUS_LCD_UI_FYSETC) \
+ ENABLED(DGUS_LCD_UI_HIPRECY) \
+ ENABLED(MALYAN_LCD) \
+ ENABLED(TOUCH_UI_FTDI_EVE) \
+ ENABLED(FSMC_GRAPHICAL_TFT) \
+ ENABLED(TFT_LVGL_UI) \
+ ENABLED(SPI_GRAPHICAL_TFT)
+ ENABLED(FSMC_GRAPHICAL_TFT)
#error "Please select no more than one LCD controller option."
#endif

Expand Down
4 changes: 4 additions & 0 deletions Marlin/src/lcd/HD44780/lcdprint_hd44780.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@

#include <string.h>

#ifndef LCD_CLASS
#include <LiquidCrystal.h>
#define LCD_CLASS LiquidCrystal
#endif
extern LCD_CLASS lcd;

int lcd_glyph_height() { return 1; }
Expand Down
4 changes: 2 additions & 2 deletions Marlin/src/module/stepper/TMC26X.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
//
// TMC26X Driver objects and inits
//
#if HAS_TMC26X
#if HAS_DRIVER(TMC26X)

#include "TMC26X.h"

Expand Down Expand Up @@ -141,4 +141,4 @@ void tmc26x_init_to_defaults() {
#endif
}

#endif // HAS_TMC26X
#endif // TMC26X
6 changes: 3 additions & 3 deletions Marlin/src/pins/pins.h
Original file line number Diff line number Diff line change
Expand Up @@ -507,7 +507,7 @@
#elif MB(MKS_ROBIN_MINI)
#include "stm32f1/pins_MKS_ROBIN_MINI.h" // STM32F1 env:mks_robin_mini
#elif MB(MKS_ROBIN_NANO)
#include "stm32f1/pins_MKS_ROBIN_NANO.h" // STM32F1 env:mks_robin_nano
#include "stm32f1/pins_MKS_ROBIN_NANO.h" // STM32F1 env:mks_robin_nano env:mks_robin_nano35
#elif MB(MKS_ROBIN_LITE)
#include "stm32f1/pins_MKS_ROBIN_LITE.h" // STM32F1 env:mks_robin_lite
#elif MB(BTT_SKR_MINI_V1_1)
Expand Down Expand Up @@ -541,9 +541,9 @@
#elif MB(CCROBOT_MEEB_3DP)
#include "stm32f1/pins_CCROBOT_MEEB_3DP.h" // STM32F1 env:STM32F103RC_meeb
#elif MB(CHITU3D_V5)
#include "stm32f1/pins_CHITU3D_V5.h" // STM32F1 env:chitu_f103 env:chitu_v5_gpio_init
#include "stm32f1/pins_CHITU3D_V5.h" // STM32F1 env:chitu_f103 env:chitu_v5_gpio_init env:chitu_f103_lvgl
#elif MB(CHITU3D_V6)
#include "stm32f1/pins_CHITU3D_V6.h" // STM32F1 env:chitu_f103
#include "stm32f1/pins_CHITU3D_V6.h" // STM32F1 env:chitu_f103 env:chitu_f103_lvgl
#elif MB(CREALITY_V4)
#include "stm32f1/pins_CREALITY_V4.h" // STM32F1 env:STM32F103RET6_creality
#elif MB(TRIGORILLA_PRO)
Expand Down
50 changes: 0 additions & 50 deletions buildroot/share/PlatformIO/scripts/common-features-dependencies.h

This file was deleted.

135 changes: 0 additions & 135 deletions buildroot/share/PlatformIO/scripts/common-features-dependencies.py

This file was deleted.

Loading

5 comments on commit 66d8593

@rhapsodyv
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the build problem is already solved. We can try this PR: MarlinFirmware#18726

@zvoniimiir
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried it but I still had the same issue.

@rhapsodyv
Copy link

@rhapsodyv rhapsodyv commented on 66d8593 Jul 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you do one more try?

This PR: MarlinFirmware#18728

@rhapsodyv
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now all the fixes were merged. I think you will not have any more problems! Let me know!

Thanks!

@zvoniimiir
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed everything is working now.
Thanks for being proactive!

Please sign in to comment.