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 1 commit
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
8 changes: 7 additions & 1 deletion Marlin/src/MarlinCore.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1188,7 +1188,13 @@ void setup() {

#if HAS_SUICIDE
SETUP_LOG("SUICIDE_PIN");
OUT_WRITE(SUICIDE_PIN, !SUICIDE_PIN_INVERTING);
SET_OUTPUT(SUICIDE_PIN);
#ifdef ARDUINO_ARCH_STM32
// We are before FastIO_init()!
digitalWrite(SUICIDE_PIN, !SUICIDE_PIN_INVERTING);
#else
WRITE(SUICIDE_PIN, !SUICIDE_PIN_INVERTING);
#endif
#endif

#ifdef JTAGSWD_RESET
Expand Down
7 changes: 4 additions & 3 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,10 +99,11 @@
// Avoid nozzle heat and fan start before serial init
#define BOARD_OPENDRAIN_MOSFETS

// STM32 framework cannot use WRITE macros before HAL_init()
#define BOARD_PREINIT() { \
OUT_WRITE_OD(HEATER_0_PIN, 0); \
OUT_WRITE_OD(HEATER_BED_PIN, 0); \
OUT_WRITE_OD(FAN_PIN, 0); \
_SET_OUTPUT_OD(HEATER_0_PIN); digitalWrite(HEATER_0_PIN, 0); \
_SET_OUTPUT_OD(HEATER_BED_PIN); digitalWrite(HEATER_BED_PIN, 0); \
_SET_OUTPUT_OD(FAN_PIN); digitalWrite(FAN_PIN, 0); \
}

//
Expand Down
1 change: 0 additions & 1 deletion ini/stm32-common.ini
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ board_build.core = stm32
build_flags = ${common.build_flags}
-std=gnu++14
-DUSBCON -DUSBD_USE_CDC
-DFASTIO_PREINIT
-DTIM_IRQ_PRIO=13
-DADC_RESOLUTION=12
build_unflags = -std=gnu++11
Expand Down