Minimizes intermittent failures when a device is in sniff mode.
Fixes: 245881714
Test: enable/disable with spatial audio feature enabled
Change-Id: I5fb5b9ff37a6f8a245678c87ee82e2032dc845f2
The new 6.0 kernel headers changed all variable length
structures from [0] to []. This causes a clang warning to
trigger, so rewrite the affected data structure using a
union to avoid having a variable sized array in the middle
of the structure.
Test: Builds.
Change-Id: I60c6d31a93566c4aa9f7501f39f88543ee76ce91
specifically, interpret HID data value as unsigned if minValue is
positive
Fix: 238391386
Test: See "Testing Done" in comments in ag/19889364
Change-Id: I715c2189ad90ee71e022ae131354354157694556
Bug: 228369656
Test: Used uhid-sample to add a dynamic sensor and verified correct
min/max sample rate.
Change-Id: I88a1bec26a03a97ee7a3c590d2601151b89ee545
Bug: 226012902
Test: Used uhid-sample to add a dynamic sensor and verified it can be
sampled.
Test: Simulated start sampling ioctl timeout and verified operation
completes with a timeout and eventually cleans up.
Test: Simulated start sampling ioctl timeout with device removal before
completion and verified operation completes with a timeout and
eventually cleans up.
Change-Id: I9a957bc1e4000b2c587278146ecbe6e903010616
Also, don't require 16-bit strings for HID name, manufacturer, and
unique ID. Also change dump of HID unique ID to be in hex.
Bug: 213483369
Test: Verified that the standard Android head tracker sensor uses the
HID unique ID for the sensor UUID.
Test: Verified that custom Android sensors use a fabricated UUID.
Change-Id: I125f84187868543dccab758e22dce744c013578e
Bug: 207008609
Test: Verified dynamic sensor sampling with proposed standard Android
head tracker.
Change-Id: I163b79c2c99f02961adebcb5dcb7e39603944e24
Bug: 205041452
Test: Verified that multiple subscriptions can be made with sensor_test
to a dynamic sensor without hanging.
Change-Id: Ie7671df8294bee04a0ebe1940242c2a83105f897
Bug: 195964858
Test: Paired a Sony PS4 controller and verified it can be sampled as an
accelerometer. The first subscription always returns all 0's, but
subsequent subscriptions return proper accel samples.
Change-Id: I4bbc4695c988f600a62502f9fccabaa68f42c72b
Bug: 195964858
Test: Paired a Sony PS4 controller and verified it is present in sensor
list as an accelerometer. It can't be sampled, but the LSM6DSR
accelerometer may still be sampled.
Change-Id: I182291cebdf84e5f6cd44d16debd6a07c4bc5281
Bug: 195964858
Test: Verified dynamic sensor manager is present in sensor list and that
no SELinux violations occur.
Test: See details in testing done comment in
https://googleplex-android-review.git.corp.google.com/15874906 .
Change-Id: I5d587dc46bdec66c3162bf222d36b285b8d2ca3d
Bug: 195964858
Test: Verified dynamic sensor manager is present in sensor list.
Test: Test: See details in testing done comment in
https://googleplex-android-review.git.corp.google.com/15831086 .
Change-Id: Ia34596d79f8a6c6985b35dcae75126e456f6009e
Bug: 195964858
Test: Verified the dynamic sensor sub-HAL initializes and non-dynamic
sensors can sample.
Test: See details in testing done comment in
https://googleplex-android-review.git.corp.google.com/15806307 .
Change-Id: I59934814cc61c7319731eb840ff2132a8c5ce241
C++20 is stricter about member ordering in designated initializers than
C99.
Bug: 139945549
Test: mm
Change-Id: I045da46821280e720286a25a0c96b5750d1ad05e
Signed-off-by: Nick Desaulniers <ndesaulniers@google.com>
-Wimplicit-fallthrough was printing warnings for these, they look
like real errors. Fix them and re-enable the error.
Test: treehugger
Change-Id: I818e18a995760f17b85b4c72ee577e531fa2a601
It will be a global error by default.
Test: make checkbuild
Bug: 112564944
Change-Id: Ifdcaca758036cf9624f79befb65a1cbbd2f9b010
Exempt-From-Owner-Approval: do not block on new warnings
* Remove unused local variables.
* Remove or comment out unused static functions.
* Fix trivial bugs found by unused value warnings.
Bug: 66996870
Test: build with WITH_TIDY=1
Change-Id: I99389b883c89551850180d25241a35a40bb77b26
This section of code uses PRODUCT_FULL_TREBLE, but
what PRODUCT_FULL_TREBLE means is too fuzzy, and
it is being broken up into multiple flags.
Also, this is an unnecessary restriction (HALs can
be implemented anyway people want).
Fixes: 67975863
Test: multiproduct_kati
Change-Id: I0de36d1ee23052c1fadcdf9a8b1ce85781dd88a0
