diff options
| author | Yi Chou <yich@google.com> | 2023-03-16 18:57:23 +0800 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-25 13:23:52 +0000 |
| commit | fa58ebaa77b02f9e2b523d6d67458700d835ff15 (patch) | |
| tree | 30fbe4072ea691285491e25f46c777b3e824c9b9 | |
| parent | 80e13ae8a591cd2ab68a77a5b1526d933ec02d33 (diff) | |
| download | chrome-ec-fa58ebaa77b02f9e2b523d6d67458700d835ff15.tar.gz | |
fpsensor: Switch to the ScopedFastCpu
The RAII ScopedFastCpu is more convenient than the C API.
BUG=b:248508087
TEST=make V=1 BOARD=bloonchipper -j
BRANCH=none
Change-Id: I2f71008ac130759126e3b1fcccda99d93d757ee6
Signed-off-by: Yi Chou <yich@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4344623
Reviewed-by: Tom Hughes <tomhughes@chromium.org>
| -rw-r--r-- | common/fpsensor/fpsensor.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/common/fpsensor/fpsensor.cc b/common/fpsensor/fpsensor.cc index e3d96cde16..c824f5c11a 100644 --- a/common/fpsensor/fpsensor.cc +++ b/common/fpsensor/fpsensor.cc @@ -31,6 +31,7 @@ extern "C" { #include "fpsensor_detect.h" #include "fpsensor_state.h" #include "fpsensor_utils.h" +#include "scoped_fast_cpu.h" #if !defined(CONFIG_RNG) #error "fpsensor requires RNG" @@ -212,8 +213,9 @@ static void fp_process_finger(void) res = spi_transaction_flush(&spi_devices[0]); if (res) CPRINTS("Failed to flush SPI: 0x%x", res); + /* we need CPU power to do the computations */ - clock_enable_module(MODULE_FAST_CPU, 1); + ScopedFastCpu fast_cpu; if (sensor_mode & FP_MODE_ENROLL_IMAGE) evt = fp_process_enroll(); @@ -223,9 +225,6 @@ static void fp_process_finger(void) sensor_mode &= ~FP_MODE_ANY_CAPTURE; overall_time_us = time_since32(overall_t0); send_mkbp_event(evt); - - /* go back to lower power mode */ - clock_enable_module(MODULE_FAST_CPU, 0); } else { timestamps_invalid |= FPSTATS_CAPTURE_INV; } |