diff options
| author | Tom Hughes <tomhughes@chromium.org> | 2021-04-06 23:34:48 +0000 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-04-07 07:17:04 +0000 |
| commit | 575d7686274bc97f3b7ddae2e3b5f4171290d6fd (patch) | |
| tree | 05a58fb6dafa421c46a79597cdc32459f1959609 /fuzz/host_command_fuzz.c | |
| parent | 7fba0e12ffa1d48c9f9f6da9d18125adc9aa75b4 (diff) | |
| download | chrome-ec-575d7686274bc97f3b7ddae2e3b5f4171290d6fd.tar.gz | |
Revert "fpsensor: Support building firmware that works for both sensors"
This reverts commit ac08c9d1dbc9f587df3ee6b15d58c0203f7e356e.
Reason for revert: Breaks public build
Original change's description:
> fpsensor: Support building firmware that works for both sensors
>
> This is a refactoring to allow building FPMCU firmware that works for
> one FPC sensor and one ELAN sensor.
>
> 1. When both drivers implement our common functions, e.g. fp_sensor_init(),
> rename them to fp_sensor_init_fpc() and fp_sensor_init_elan().
> 2. There are a few functions implemented not in FPC driver but in FPC
> private library, e.g. fp_sensor_finger_status(). I kept this as-is for
> FPC but renamed the one in ELAN driver to fp_sensor_finger_status_elan()
> 3. If building for ELAN, need to hardcode elan=1 in hatch_fp/board.c
> because the sensor type GPIO always says FPC.
>
> BRANCH=none
> BUG=b:175158241
> TEST=make run-fpsensor; make run-fpsensor_status;
> make run-fpsensor_crypto
> TEST=make -j BOARD=dartmonkey
> TEST=add CONFIG_FP_SENSOR_ELAN515 to board/hatch_fp/board.h;
> make -j BOARD=bloonchipper
> Firmware binary fully works on Dragonair (FPC) and Voema (ELAN)
> TEST=run device tests with http://crrev/c/2750547 and
> http://crrev/i/3654297 on Dragonclaw, all pass
>
> Change-Id: I789090dbdfe35ac6aefd6a629fa4c7bde89dc437
> Signed-off-by: Yicheng Li <yichengli@chromium.org>
> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2727971
> Reviewed-by: Tom Hughes <tomhughes@chromium.org>
> Commit-Queue: Tom Hughes <tomhughes@chromium.org>
Bug: b:175158241, b:184616069
Change-Id: I2a02a6eefc316e7e13aa188f1ae16672dce2babd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2809521
Auto-Submit: Tom Hughes <tomhughes@chromium.org>
Tested-by: Tom Hughes <tomhughes@chromium.org>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: caveh jalali <caveh@chromium.org>
Diffstat (limited to 'fuzz/host_command_fuzz.c')
| -rw-r--r-- | fuzz/host_command_fuzz.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fuzz/host_command_fuzz.c b/fuzz/host_command_fuzz.c index 01a5818b63..4ca94ff616 100644 --- a/fuzz/host_command_fuzz.c +++ b/fuzz/host_command_fuzz.c @@ -12,7 +12,6 @@ #include "console.h" #include "host_command.h" #include "host_test.h" -#include "mock/fp_sensor_mock.h" #include "task.h" #include "test_util.h" #include "timer.h" @@ -135,8 +134,6 @@ static pthread_mutex_t lock; void run_test(int argc, char **argv) { ccprints("Fuzzing task started"); - /* Set the fingerprint driver to the mock to test fp_task. */ - fp_driver = &fp_driver_mock; wait_for_task_started(); while (1) { |