From fe1d23fd6e84d342a254d2e26a455ec9f12be682 Mon Sep 17 00:00:00 2001 From: rusefi Date: Sat, 14 Dec 2019 17:35:11 -0500 Subject: [PATCH] Revert "Fried coil (?) on incompatible firmware change fix #1051" This reverts commit 1eeb92b --- firmware/config/boards/st_stm32f4/board.h | 96 +++++++++++------------ 1 file changed, 45 insertions(+), 51 deletions(-) diff --git a/firmware/config/boards/st_stm32f4/board.h b/firmware/config/boards/st_stm32f4/board.h index b5e7fe62b8..4a100ba869 100644 --- a/firmware/config/boards/st_stm32f4/board.h +++ b/firmware/config/boards/st_stm32f4/board.h @@ -45,12 +45,6 @@ #define EFI_PIN_MODE_DEFAULT PIN_MODE_INPUT #define EFI_DR_DEFAULT PIN_PUPDR_FLOATING -// older Frankenso does not have pull-downs on hi-side drivers and this affects rusEfi primary test mules -// https://github.com/rusefi/rusefi/issues/1051 -#define FRANKENSO_0_4_ISSUE_1051_MODE PIN_MODE_OUTPUT -#define FRANKENSO_0_4_ISSUE_1051_OTYPE PIN_OTYPE_PUSHPULL -#define FRANKENSO_0_4_ISSUE_1051_PUD PIN_PUPDR_PULLDOWN -#define FRANKENSO_0_4_ISSUE_1051_ODR PIN_ODR_LOW /* * Board oscillators-related settings. @@ -550,9 +544,9 @@ EFI_PIN_MODE_DEFAULT(GPIOC_PIN4) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN5) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN6) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOC_PIN7) | \ + EFI_PIN_MODE_DEFAULT(GPIOC_PIN7) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOC_PIN9) | \ + EFI_PIN_MODE_DEFAULT(GPIOC_PIN9) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN10) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN11) | \ EFI_PIN_MODE_DEFAULT(GPIOC_PIN12) | \ @@ -566,9 +560,9 @@ PIN_OTYPE_PUSHPULL(GPIOC_PIN4) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN5) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN6) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOC_PIN7) | \ + PIN_OTYPE_PUSHPULL(GPIOC_PIN7) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOC_PIN9) | \ + PIN_OTYPE_PUSHPULL(GPIOC_PIN9) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN10) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN11) | \ PIN_OTYPE_PUSHPULL(GPIOC_PIN10) | \ @@ -592,21 +586,21 @@ DEFAULT_GPIO_SPEED(GPIOC_PIN14) | \ DEFAULT_GPIO_SPEED(GPIOC_PIN15)) #define VAL_GPIOC_PUPDR (EFI_DR_DEFAULT(GPIOC_OTG_FS_POWER_ON) |\ - EFI_DR_DEFAULT(GPIOC_PIN1) | \ - EFI_DR_DEFAULT(GPIOC_PIN2) | \ - EFI_DR_DEFAULT(GPIOC_PIN3) | \ - EFI_DR_DEFAULT(GPIOC_PIN4) | \ - EFI_DR_DEFAULT(GPIOC_PIN5) | \ - EFI_DR_DEFAULT(GPIOC_PIN6) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOC_PIN7) | \ - EFI_DR_DEFAULT(GPIOC_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOC_PIN9) | \ - EFI_DR_DEFAULT(GPIOC_PIN10) | \ - EFI_DR_DEFAULT(GPIOC_PIN11) | \ - EFI_DR_DEFAULT(GPIOC_PIN12) | \ - EFI_DR_DEFAULT(GPIOC_PIN13) | \ - EFI_DR_DEFAULT(GPIOC_PIN14) | \ - EFI_DR_DEFAULT(GPIOC_PIN15)) + PIN_PUPDR_PULLDOWN(GPIOC_PIN1) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN2) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN3) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN4) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN5) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN6) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN7) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN8) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN9) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN10) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN11) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN12) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN13) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN14) | \ + PIN_PUPDR_PULLDOWN(GPIOC_PIN15)) #define VAL_GPIOC_ODR (PIN_ODR_HIGH(GPIOC_OTG_FS_POWER_ON) | \ PIN_ODR_HIGH(GPIOC_PIN1) | \ PIN_ODR_HIGH(GPIOC_PIN2) | \ @@ -614,9 +608,9 @@ PIN_ODR_HIGH(GPIOC_PIN4) | \ PIN_ODR_HIGH(GPIOC_PIN5) | \ PIN_ODR_HIGH(GPIOC_PIN6) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOC_PIN7) | \ + PIN_ODR_HIGH(GPIOC_PIN7) | \ PIN_ODR_HIGH(GPIOC_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOC_PIN9) | \ + PIN_ODR_HIGH(GPIOC_PIN9) | \ PIN_ODR_HIGH(GPIOC_PIN10) | \ PIN_ODR_HIGH(GPIOC_PIN11) | \ PIN_ODR_HIGH(GPIOC_PIN12) | \ @@ -668,8 +662,8 @@ EFI_PIN_MODE_DEFAULT(GPIOD_OVER_CURRENT) | \ EFI_PIN_MODE_DEFAULT(GPIOD_PIN6) | \ EFI_PIN_MODE_DEFAULT(GPIOD_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOD_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOD_PIN9) | \ + EFI_PIN_MODE_DEFAULT(GPIOD_PIN8) | \ + EFI_PIN_MODE_DEFAULT(GPIOD_PIN9) | \ EFI_PIN_MODE_DEFAULT(GPIOD_PIN10) | \ EFI_PIN_MODE_DEFAULT(GPIOD_PIN11) | \ PIN_MODE_OUTPUT(GPIOD_LED4) | \ @@ -684,8 +678,8 @@ PIN_OTYPE_PUSHPULL(GPIOD_OVER_CURRENT) |\ PIN_OTYPE_PUSHPULL(GPIOD_PIN6) | \ PIN_OTYPE_PUSHPULL(GPIOD_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOD_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOD_PIN9) | \ + PIN_OTYPE_PUSHPULL(GPIOD_PIN8) | \ + PIN_OTYPE_PUSHPULL(GPIOD_PIN9) | \ PIN_OTYPE_PUSHPULL(GPIOD_PIN10) | \ PIN_OTYPE_PUSHPULL(GPIOD_PIN11) | \ PIN_OTYPE_PUSHPULL(GPIOD_LED4) | \ @@ -716,8 +710,8 @@ EFI_DR_DEFAULT(GPIOD_OVER_CURRENT) |\ PIN_PUPDR_PULLUP(GPIOD_PIN6) | \ PIN_PUPDR_PULLUP(GPIOD_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOD_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOD_PIN9) | \ + PIN_PUPDR_PULLUP(GPIOD_PIN8) | \ + PIN_PUPDR_PULLUP(GPIOD_PIN9) | \ PIN_PUPDR_PULLUP(GPIOD_PIN10) | \ PIN_PUPDR_PULLUP(GPIOD_PIN11) | \ EFI_DR_DEFAULT(GPIOD_LED4) | \ @@ -732,8 +726,8 @@ PIN_ODR_HIGH(GPIOD_OVER_CURRENT) | \ PIN_ODR_HIGH(GPIOD_PIN6) | \ PIN_ODR_HIGH(GPIOD_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOD_PIN8) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOD_PIN9) | \ + PIN_ODR_HIGH(GPIOD_PIN8) | \ + PIN_ODR_HIGH(GPIOD_PIN9) | \ PIN_ODR_HIGH(GPIOD_PIN10) | \ PIN_ODR_HIGH(GPIOD_PIN11) | \ PIN_ODR_LOW(GPIOD_LED4) | \ @@ -785,13 +779,13 @@ EFI_PIN_MODE_DEFAULT(GPIOE_PIN5) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN6) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOE_PIN8) | \ + EFI_PIN_MODE_DEFAULT(GPIOE_PIN8) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN9) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOE_PIN10) | \ + EFI_PIN_MODE_DEFAULT(GPIOE_PIN10) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN11) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOE_PIN12) | \ + EFI_PIN_MODE_DEFAULT(GPIOE_PIN12) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN13) | \ - FRANKENSO_0_4_ISSUE_1051_MODE(GPIOE_PIN14) | \ + EFI_PIN_MODE_DEFAULT(GPIOE_PIN14) | \ EFI_PIN_MODE_DEFAULT(GPIOE_PIN15)) #define VAL_GPIOE_OTYPER (PIN_OTYPE_PUSHPULL(GPIOE_PIN0) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN1) | \ @@ -801,13 +795,13 @@ PIN_OTYPE_PUSHPULL(GPIOE_PIN5) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN6) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOE_PIN8) | \ + PIN_OTYPE_PUSHPULL(GPIOE_PIN8) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN9) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOE_PIN10) | \ + PIN_OTYPE_PUSHPULL(GPIOE_PIN10) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN11) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOE_PIN12) | \ + PIN_OTYPE_PUSHPULL(GPIOE_PIN12) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN13) | \ - FRANKENSO_0_4_ISSUE_1051_OTYPE(GPIOE_PIN14) | \ + PIN_OTYPE_PUSHPULL(GPIOE_PIN14) | \ PIN_OTYPE_PUSHPULL(GPIOE_PIN15)) #define VAL_GPIOE_OSPEEDR (DEFAULT_GPIO_SPEED(GPIOE_PIN0) | \ DEFAULT_GPIO_SPEED(GPIOE_PIN1) | \ @@ -833,13 +827,13 @@ EFI_DR_DEFAULT(GPIOE_PIN5) | \ EFI_DR_DEFAULT(GPIOE_PIN6) | \ EFI_DR_DEFAULT(GPIOE_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOE_PIN8) | \ + EFI_DR_DEFAULT(GPIOE_PIN8) | \ EFI_DR_DEFAULT(GPIOE_PIN9) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOE_PIN10) | \ + EFI_DR_DEFAULT(GPIOE_PIN10) | \ EFI_DR_DEFAULT(GPIOE_PIN11) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOE_PIN12) | \ + EFI_DR_DEFAULT(GPIOE_PIN12) | \ EFI_DR_DEFAULT(GPIOE_PIN13) | \ - FRANKENSO_0_4_ISSUE_1051_PUD(GPIOE_PIN14) | \ + EFI_DR_DEFAULT(GPIOE_PIN14) | \ EFI_DR_DEFAULT(GPIOE_PIN15)) #define VAL_GPIOE_ODR (PIN_ODR_HIGH(GPIOE_PIN0) | \ PIN_ODR_HIGH(GPIOE_PIN1) | \ @@ -849,13 +843,13 @@ PIN_ODR_HIGH(GPIOE_PIN5) | \ PIN_ODR_HIGH(GPIOE_PIN6) | \ PIN_ODR_HIGH(GPIOE_PIN7) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOE_PIN8) | \ + PIN_ODR_HIGH(GPIOE_PIN8) | \ PIN_ODR_HIGH(GPIOE_PIN9) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOE_PIN10) | \ + PIN_ODR_HIGH(GPIOE_PIN10) | \ PIN_ODR_HIGH(GPIOE_PIN11) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOE_PIN12) | \ + PIN_ODR_HIGH(GPIOE_PIN12) | \ PIN_ODR_HIGH(GPIOE_PIN13) | \ - FRANKENSO_0_4_ISSUE_1051_ODR(GPIOE_PIN14) | \ + PIN_ODR_HIGH(GPIOE_PIN14) | \ PIN_ODR_HIGH(GPIOE_PIN15)) #define VAL_GPIOE_AFRL (PIN_AFIO_AF(GPIOE_PIN0, 0U) | \ PIN_AFIO_AF(GPIOE_PIN1, 0U) | \