summaryrefslogtreecommitdiff
path: root/common/fpsensor.c
diff options
context:
space:
mode:
authorVincent Palatin <vpalatin@chromium.org>2018-02-14 12:24:16 +0100
committerchrome-bot <chrome-bot@chromium.org>2018-02-16 18:47:47 -0800
commit59c68a1d4bf82362062f5185214f7604c55b7efa (patch)
tree83ecb5c1ef2fceef3552bbff5431506eff4e8940 /common/fpsensor.c
parent074acb0cd7f35ae4d3058f5453fbb84cadf2082f (diff)
downloadchrome-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.c10
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) ||