summaryrefslogtreecommitdiff
path: root/driver/fingerprint
Commit message (Collapse)AuthorAgeFilesLines
* include/config.h: Replace CONFIG_FP_SENSOR with CONFIG_FINGERPRINT_MCUTom Hughes2020-10-301-2/+2
| | | | | | | | | | | | | | | | CONFIG_FP_SENSOR was only enabled for RW, but we need a way to know whether the board is a fingerprint MCU when in RO as well. "FINGERPRINT_MCU" is a bit more generic than FP_SENSOR. We still only build the matching-related code for RW. BRANCH=none BUG=b:171370392 TEST=make buildall Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I808faa7a16522791cfb4eb9cc2e163d9eb994804 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2508860 Reviewed-by: Bhanu Prakash Maiya <bhanumaiya@google.com>
* fpsensor: Do not compile driver invocations for public unittest buildsYicheng Li2020-09-141-2/+5
| | | | | | | | | | | | | | | | | | | | | | | If HAVE_FP_PRIVATE_DRIVER is defined, the build will compile fingerprint driver routines when compiling "common/fpsensor/fpsensor.c". Previously we enable HAVE_FP_PRIVATE_DRIVER for both public and private unittest builds. This was because we wanted to test "common/fpsensor/fpsensor.c" with our mocks. However on public unittest builds this will break. This change limits the compilation of driver-related code to private builds. BRANCH=none BUG=b:166807240, b:158580909 TEST=make -j buildall TEST=make -j BOARD=bloonchipper test-rsa (private build) TEST=mv private private-2 (hide private directory) make -j BOARD=bloonchipper clean make -j BOARD=bloonchipper test-rsa (public build) Signed-off-by: Yicheng Li <yichengli@chromium.org> Change-Id: Id144435bf1a60c259f4c1b120b9fc85ef85b7f4c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2406189 Commit-Queue: Tom Hughes <tomhughes@chromium.org> Reviewed-by: Tom Hughes <tomhughes@chromium.org>
* fingerprint: Stopgap to repair public buildsCraig Hesling2020-08-151-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This repairs building bloonchipper and dartmonkey builds when private lib is unavailable. There are two issues this remedies: 1) Redefinition of sensor info macros, like FP_SENSOR_RES_X 2) Undefined reference to fp_sensor_maintenance when building bloonchipper/dartmonkey non-private image. All of these issues stem from the way we handle code when HAVE_FP_PRIVATE_DRIVER is set or faked by a test. This is a fundamental issues that needs some thought and refactoring. BRANCH=none BUG=b:164174822,b:163109916, b:163411883 TEST=# With private time make buildall -j TEST=# Without private time make buildall -j TEST=# With private libs # Cherry-pick https://crrev.com/c/2357870 first, then # this patch. ./util/compare_build.sh -b all -o # This confirms that the all output binaries with private-lib # did not change at all Signed-off-by: Craig Hesling <hesling@chromium.org> Change-Id: If71e3376fb7a904b77f2fc9fc56c8e0daf54db3b Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2357430 Reviewed-by: Andrew de los Reyes <adlr@chromium.org> Reviewed-by: YH Lin <yueherngl@chromium.org>
* driver/fingerprint: Add a common sensor maintenance functionTom Hughes2020-08-065-0/+64
| | | | | | | | | | | | | | | | Add a new console command "fpmaintenance" for testing. BRANCH=none BUG=b:76037094 TEST=With dragonclaw v0.2 connected to Segger J-Trace and servo micro: ./util/flash_jlink.py On FP console: > fpmaintenance Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I7125f8783d3dd7e815612d20742c9d949d00ba71 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2323709 Reviewed-by: Yicheng Li <yichengli@chromium.org>
* driver/fingerprint: Set dead pixels to unknown on init in BEPTom Hughes2020-08-061-0/+2
| | | | | | | | | | | | | This matches the behavior in the libfp driver. BRANCH=none BUG=b:76037094 TEST=make buildall -j Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: Iec52b30be448a40f51baf03ddb9b8e8b0ddeb28f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2321829 Reviewed-by: Yicheng Li <yichengli@chromium.org>
* driver/fingerprint: Add fp_sensor_maintenance functionTom Hughes2020-08-062-0/+61
| | | | | | | | | | | | | | This function has always been in the fingerprint sensor binaries, but somewhere along the way we lost the the declaration. BRANCH=none BUG=b:76037094 TEST=make buildall -j Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: I06899f08c4a626f9d180ecdee6f73af0020b078c Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2321828 Reviewed-by: Yicheng Li <yichengli@chromium.org>
* driver/fingerprint: Exclude header on emulator build, not test buildTom Hughes2020-08-061-1/+1
| | | | | | | | | | | | | | | | Now that we run unit tests on device, we want to be able to include the fingerprint sensor headers in test builds. BRANCH=none BUG=b:76037094 TEST=make buildall -j TEST=With dragonclaw v0.2 connected to Segger J-Trace and servo micro: ./test/run_device_tests.py Signed-off-by: Tom Hughes <tomhughes@chromium.org> Change-Id: Id406fd6039f1136f2ae8743453ead4a951805db5 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2321827 Reviewed-by: Yicheng Li <yichengli@chromium.org>
* driver/fingerprint: Clarify FPC 1145 hardware IDTom Hughes2020-03-091-1/+7
| | | | | | | | | | | | | BRANCH=none BUG=b:150407388 TEST=make buildall -j Change-Id: Id0e7b42affcabf90c94334603a40b87cbc9543a8 Signed-off-by: Tom Hughes <tomhughes@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2092913 Commit-Queue: Craig Hesling <hesling@chromium.org> Reviewed-by: Andrey Perminov <andrey.perminov@fingerprints.com> Reviewed-by: Craig Hesling <hesling@chromium.org>
* bloonchipper: Add transport and sensor detectionTom Hughes2020-01-081-0/+7
| | | | | | | | | | | | | | | | | | | | | | | bloonchipper (aka hatch_fp aka dragonclaw) has a voltage divider that can be used to select the sensor and the transport type. Supported designs: * Dragonclaw rev 0.2 (green with Google logo): go/dragonclaw-schematic-rev-0.2 * Hatch reference v3.0: go/hatch-schematic-rev-3.0 The selection lines are connected to ADC inputs, so a future change will use the ADC to allow more than two transports or sensors. BRANCH=none BUG=b:147113851 TEST=flash dragonclaw rev 0.2 and view console output Change-Id: If2e4b150d34cfe41477be528c70e1645043d4d82 Signed-off-by: Tom Hughes <tomhughes@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1986322 Reviewed-by: Craig Hesling <hesling@chromium.org>
* fpsensor: Clean up headers and MakefilesTom Hughes2019-10-2810-25/+55
| | | | | | | | | | | | | | We no longer need the various levels of indirection since the source files are public. BRANCH=none BUG=b:137848573 TEST=make buildall -j Cq-Depend: chrome-internal:2005128 Change-Id: I7483c233dc54c5dbf2907441365feffc9ae9f0a5 Signed-off-by: Tom Hughes <tomhughes@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1869533
* fpsensor: Add OWNERS for driver/fingerprint directoryTom Hughes2019-10-281-0/+1
| | | | | | | | | | | BRANCH=none BUG=b:137848573 TEST=make buildall -j Change-Id: I1dc902cb74ef902444b43642af52e28394296ae3 Signed-off-by: Tom Hughes <tomhughes@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1869532 Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
* fpsensor: Copy BEP and libfp source files from private repoTom Hughes2019-10-2819-0/+1796
These source files have been approved to be released publicly (see http://b/137848573#comment20). There were no source changes in the move, except for small changes to the Makefiles to adjust for the move. For future "git blame"s refer to the history of the files in the private repo if you need to access the full history. BRANCH=none BUG=b:137848573 TEST=make buildall -j Cq-Depend: chrome-internal:2005127 Change-Id: I1a809ac5a4d95637840ee47712e9c80a40a445c5 Signed-off-by: Tom Hughes <tomhughes@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/1869531
View Lorry Controller Status