stm32f4: Enable gpio port clocks that are used

We can reduce the power draw of the MCU by limiting which
GPIO port clocks are enabled.
This CL uses the gpio.inc to determine the minimal set of
GPIO ports that need to be enabled/clocked. Only these
ports have their clocks enabled at boot.
Other stm32 chip variants in EC simply enabled all GPIO
port clocks at boot init.

Thank you to Ravi Chandra Sadineni for identifying this optimization
and validating.
For more context, see crrev.com/c/1888116, which this CL replaces.

BRANCH=nocturne,hatch
BUG=b:130561737
TEST=make buildall -j
TEST=make BOARD=bloonchipper
     # Connect dragonclaw dev board over servo micro
     sudo servod --board=bloonchipper --config bloonchipper_rev0.1.xml &
     ./util/flash_ec --board=bloonchipper
     minicom -D $(dut-control raw_fpmcu_uart_pty | cut -d: -f2)
     > sysinfo
     # Check that we are in RW

Signed-off-by: Craig Hesling <hesling@chromium.org>
Change-Id: Iad51b11eb5959b5d502320560b9ebda7e614d97e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1890924
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>

1 files changed, 13 insertions, 1 deletions

diff --git a/chip/stm32/system.c b/chip/stm32/system.c
index ddba03958a..58b77fba92 100644
--- a/chip/stm32/system.c
+++ b/chip/stm32/system.c
@@ -9,6 +9,7 @@
 #include "console.h"
 #include "cpu.h"
 #include "flash.h"
+#include "gpio_chip.h"
 #include "host_command.h"
 #include "registers.h"
 #include "panic.h"
@@ -567,6 +568,17 @@ int system_set_bbram(enum system_bbram_idx idx, uint8_t value)
 
 int system_is_reboot_warm(void)
 {
+	/*
+	 * Detecting if the system is warm is relevant for a
+	 * few reasons.
+	 * One such reason is that some firmwares transition from
+	 * RO to RW images. When this happens, we may not need to
+	 * restart certain clocks. On the flip side, we may need
+	 * to restart the clocks if the RW requires a different
+	 * set of clocks. Thus, the clock configurations need to
+	 * be checked for a perfect match.
+	 */
+
 #if defined(CHIP_FAMILY_STM32F0) || defined(CHIP_FAMILY_STM32F3)
 	return ((STM32_RCC_AHBENR & 0x7e0000) == 0x7e0000);
 #elif defined(CHIP_FAMILY_STM32L)
@@ -576,7 +588,7 @@ int system_is_reboot_warm(void)
 			== STM32_RCC_AHB2ENR_GPIOMASK);
 #elif defined(CHIP_FAMILY_STM32F4)
 	return ((STM32_RCC_AHB1ENR & STM32_RCC_AHB1ENR_GPIOMASK)
-			== STM32_RCC_AHB1ENR_GPIOMASK);
+			== gpio_required_clocks());
 #elif defined(CHIP_FAMILY_STM32H7)
 	return ((STM32_RCC_AHB4ENR & STM32_RCC_AHB4ENR_GPIOMASK)
 			== STM32_RCC_AHB4ENR_GPIOMASK);