diff options
author | Vincent Palatin <vpalatin@chromium.org> | 2018-02-14 12:24:16 +0100 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-02-16 18:47:47 -0800 |
commit | 59c68a1d4bf82362062f5185214f7604c55b7efa (patch) | |
tree | 83ecb5c1ef2fceef3552bbff5431506eff4e8940 /common/fpsensor.c | |
parent | 074acb0cd7f35ae4d3058f5453fbb84cadf2082f (diff) | |
download | chrome-ec-59c68a1d4bf82362062f5185214f7604c55b7efa.tar.gz |
fpsensor: add quality test capture type
Add support for an additional finger image capture type
used for quality testing.
Signed-off-by: Vincent Palatin <vpalatin@chromium.org>
BRANCH=none
BUG=b:72360575, b:71770455
TEST=On Meowth, run 'ectool --name=cros_fp fpmode capture qual'
then 'ectool --name=cros_fp fpframe raw > finger_mq.bin'
Change-Id: I1b9525dc2adf0b91aef2f7124803c90d6a3bb0ca
Reviewed-on: https://chromium-review.googlesource.com/924124
Commit-Ready: Vincent Palatin <vpalatin@chromium.org>
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Nicolas Norvez <norvez@chromium.org>
Diffstat (limited to 'common/fpsensor.c')
-rw-r--r-- | common/fpsensor.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/common/fpsensor.c b/common/fpsensor.c index b1016e217d..bf342f3cce 100644 --- a/common/fpsensor.c +++ b/common/fpsensor.c @@ -67,6 +67,14 @@ static inline int is_test_capture(uint32_t mode) || capture_type == FP_CAPTURE_PATTERN1); } +static inline int is_raw_capture(uint32_t mode) +{ + int capture_type = FP_CAPTURE_TYPE(mode); + + return (capture_type == FP_CAPTURE_VENDOR_FORMAT + || capture_type == FP_CAPTURE_QUALITY_TEST); +} + static void send_mkbp_event(uint32_t event) { atomic_or(&fp_events, event); @@ -245,7 +253,7 @@ static int fp_command_frame(struct host_cmd_handler_args *args) void *out = args->response; uint32_t offset = params->offset; - if (FP_CAPTURE_TYPE(sensor_mode) != FP_CAPTURE_VENDOR_FORMAT) + if (!is_raw_capture(sensor_mode)) offset += FP_SENSOR_IMAGE_OFFSET; if (offset + params->size > sizeof(fp_buffer) || |