Skip to content

STM32: Prevent possible crashes before HAL init #22508

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thinkyhead merged 12 commits into from
Aug 9, 2021
Merged

STM32: Prevent possible crashes before HAL init #22508

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
6612f8d
STM32: Prevent possible crashes before HAL init
tpruvot Aug 4, 2021
c59cc8a
Brackets to avoid warnings
tpruvot Aug 4, 2021
6d399ca
Handle special if/else case
tpruvot Aug 4, 2021
1fcb6db
Disable global FASTIO_PREINIT, tested ok
tpruvot Aug 6, 2021
047fcad
STM32: FastIO_init first + HAL_STM32 env define
tpruvot Aug 6, 2021
7f08383
Remove last refs to STM32GENERIC in F1 HAL
tpruvot Aug 8, 2021
93c48a7
Simplify code inclusion for STM32 HAL
tpruvot Aug 8, 2021
1d77ebd
Define HAL_STM32 in HAL/platforms.h
thinkyhead Aug 8, 2021
7baad5b
ibid.
thinkyhead Aug 8, 2021
75ec003
Tweak ini comments
thinkyhead Aug 9, 2021
875e0d8
Merge remote-tracking branch 'upstream/bugfix-2.0.x' into pr/22508
thinkyhead Aug 9, 2021
9b4485d
Define FASTIO_INIT for HAL/STM32
thinkyhead Aug 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Marlin/src/HAL/STM32/HAL.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ TERN_(POSTMORTEM_DEBUGGING, extern void install_min_serial());

// HAL initialization task
void HAL_init() {
FastIO_init();
//FastIO_init(); // Moved in setup()

// Ensure F_CPU is a constant expression.
// If the compiler breaks here, it means that delay code that should compute at compile time will not work.
Expand Down
2 changes: 1 addition & 1 deletion Marlin/src/HAL/STM32/fastio.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@

#include "../../inc/MarlinConfig.h"

GPIO_TypeDef* FastIOPortMap[LastPort + 1];
GPIO_TypeDef* FastIOPortMap[LastPort + 1] = { 0 };

void FastIO_init() {
LOOP_L_N(i, NUM_DIGITAL_PINS)
Expand Down
4 changes: 4 additions & 0 deletions Marlin/src/MarlinCore.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1131,6 +1131,10 @@ inline void tmc_standby_setup() {
* - Set Marlin to RUNNING State
*/
void setup() {
#ifdef HAL_STM32
FastIO_init();
#endif

#ifdef BOARD_PREINIT
BOARD_PREINIT(); // Low-level init (before serial init)
#endif
Expand Down
9 changes: 1 addition & 8 deletions Marlin/src/pins/stm32f1/pins_LONGER3D_LK.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -99,19 +99,12 @@
// Avoid nozzle heat and fan start before serial init
#define BOARD_OPENDRAIN_MOSFETS

#define BOARD_INIT_OD_PINS() { \
#define BOARD_PREINIT() { \
OUT_WRITE_OD(HEATER_0_PIN, 0); \
OUT_WRITE_OD(HEATER_BED_PIN, 0); \
OUT_WRITE_OD(FAN_PIN, 0); \
}

#ifdef MAPLE_STM32F1
// Only Maple Framework allow that early
#define BOARD_PREINIT BOARD_INIT_OD_PINS
#else
#define BOARD_INIT BOARD_INIT_OD_PINS
#endif

//
// PWM for a servo probe
// Other servo devices are not supported on this board!
Expand Down
2 changes: 1 addition & 1 deletion ini/stm32-common.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
platform = ststm32@~12.1
board_build.core = stm32
build_flags = ${common.build_flags}
-std=gnu++14
-std=gnu++14 -DHAL_STM32
-DUSBCON -DUSBD_USE_CDC
-DTIM_IRQ_PRIO=13
-DADC_RESOLUTION=12
Expand Down