Commit graph

310 commits

Author SHA1 Message Date
Teo Georgescu
29254e1e23 Revert "vib example client on product"
This reverts commit f2ff92f3e0.

Reason for revert: DroidMonitor: Potential culprit for Bug b/265255626 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted.

Change-Id: I30f87a33803d00c03f9472a77ba73479c06044b0
2023-01-12 09:43:12 +00:00
Steven Moreland
f2ff92f3e0 vib example client on product
Bug: partner question
Test: # PRODUCT_PACKAGES += ExampleVibratorJavaProductClient
    adb root
    adb setenforce 0 # for find permission
    adb shell am start -n example.vib/example.vib.MyActivity
    # vibrator works

Change-Id: Ie70f4cfe0f546a43a5d11bbdd1f1143d817aab50
2023-01-11 01:41:48 +00:00
Steven Moreland
581e0135aa vibrator: clean OWNERS
I wrote a script for it.

Bug: N/A
Test: N/A
Change-Id: I459d19c9ef798755ab5c2d9085986ba04c4a2d9d
2023-01-03 23:53:54 +00:00
Steven Moreland
89c4d48e09 Merge "easy vib. instance name" am: 92f223a9cc
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2322764

Change-Id: I10c5e0a739cdf0eb6ac40bd3c80e13694c88a6f0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-12-01 22:02:04 +00:00
Steven Moreland
92f223a9cc Merge "easy vib. instance name" 2022-12-01 21:33:56 +00:00
Steven Moreland
b5e0adbeb2 easy vib. instance name
Bug: 257537714
Test: vts_treble_vintf_vendor_test after using this
Change-Id: Id0c56b2df59eb854a5f62d8adc65d0a6b38fcd3c
2022-12-01 01:25:15 +00:00
Steven Moreland
aebd7e6bb7 Merge "example vendor app" am: e49b3684a1
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2308365

Change-Id: Ibd9e0647157da27d64c041e14e97635422415ad2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-11-21 18:26:08 +00:00
Steven Moreland
0385dac9fc example vendor app
see comments added in code and Android.bp

We don't have examples of a common Android ecosystem usecase, so
I'm adding one here.

Bug: 227289255
Test: after sepolicy + PRODUCT_PACKAGES:
    11-19 03:14:27.460   488   488 I android.hardware.vibrator-service.example: Vibrator on for timeoutMs: 100
Change-Id: Iee430d85e23c6c8f771dc5aa77bff9be1a91f080
2022-11-19 03:35:37 +00:00
TreeHugger Robot
62bee20d8e Merge "Fix VtsHalVibratorManagerTargetTest test fail" into tm-qpr-dev am: d39f11cbb0
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/19992878

Change-Id: I5055ea6c6fad217f9ede8af0ce7e30372bdedc0a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-09-21 21:30:18 +00:00
Chase Wu
8fb96d255e Fix VtsHalVibratorManagerTargetTest test fail
Adding the cancelSynced() once prepareSynced() was called and return ok.

Fix: 247687286
Test: atest VtsHalVibratorManagerTargetTest
Signed-off-by: Chase Wu <chasewu@google.com>
Change-Id: I91e08e92cb8e1399eb6acdbedf95958fbf44c393
2022-09-21 11:48:40 +08:00
Pawan
70066b2383 Updating cc_fuzz with default config am: 8e13f2ac2c am: a9974359b4 am: 3753e1d8e7 am: 01c2ea1f3a
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2190582

Change-Id: I3eb09ead9b7edf9c4f254df90ae75250f845b6c3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-23 23:04:17 +00:00
Pawan
8e13f2ac2c Updating cc_fuzz with default config
Using service_fuzzer_defaults in
	- android.hardware.health-service.aidl_fuzzer
	- android.hardware.vibrator-service.example_fuzzer

Bug: 240318902
Test: m android.hardware.health-service.aidl_fuzzer
Test: m android.hardware.vibrator-service.example_fuzzer
Change-Id: I3fd1cf37a5835ebef14658ed40332132cec07e49
2022-08-22 22:16:41 +00:00
Donghong Weng
aca6491feb Merge "Explicit conversion" am: 160a22f49e am: 3c745ce814 am: 33e9c08107 am: 3aa5895a2f
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2186215

Change-Id: Idaad6f2b39142fe7202cd5a903b0e67f5c97f4d0
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-08-22 03:01:50 +00:00
Donghong Weng
160a22f49e Merge "Explicit conversion" 2022-08-22 01:45:35 +00:00
donghongweng
af442b177f Explicit conversion
Test: m tests
Change-Id: Ie0c00a51dee1d97239b0ef6f29334d176ddec32a
2022-08-17 06:59:51 +00:00
Jooyung Han
fcef58a22a Merge "Remove AIDL-libs from VNDK" into tm-dev-plus-aosp am: 212b1e7610 am: fdc3418cc9
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/18869738

Change-Id: I7079ebd194212fd82891b2210f6a4df8d1c8bfa8
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-16 01:54:37 +00:00
Jooyung Han
d81d6a12be Remove AIDL-libs from VNDK
Some are still in VNDK because they are used in other VNDK libs.

Bug: 234181591
Test: m
Merged-In: If999df9c78a20df931177da11742b1c5de19bc08
Change-Id: If999df9c78a20df931177da11742b1c5de19bc08
2022-06-14 16:42:42 +09:00
Jooyung Han
c0b24f4f13 Remove AIDL-libs from VNDK
Some are still in VNDK because they are used in other VNDK libs.

Ignore-AOSP-First: some libs are still in internal master only.

Bug: 234181591
Test: m
Merged-In: If999df9c78a20df931177da11742b1c5de19bc08
Change-Id: If999df9c78a20df931177da11742b1c5de19bc08
(cherry picked from commit 5527adfd7f)
2022-06-14 14:08:59 +09:00
Treehugger Robot
55a93f6c79 Merge "vibrator: turn off vibrator after testing ChangeVibrationAmplitude" am: 05eb1a279b am: fa0ac0f30c am: 70d2180d62 am: 8cc45e8b18
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2104024

Change-Id: Ia4f8c02105a85075f97dddbdc826921edc406e55
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-08 15:45:50 +00:00
Treehugger Robot
70d2180d62 Merge "vibrator: turn off vibrator after testing ChangeVibrationAmplitude" am: 05eb1a279b am: fa0ac0f30c
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2104024

Change-Id: Ie43d991e7d0a79be679b8bf64ae1922c499d6921
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-06-08 14:53:56 +00:00
Treehugger Robot
05eb1a279b Merge "vibrator: turn off vibrator after testing ChangeVibrationAmplitude" 2022-06-08 13:48:44 +00:00
Simon Bowden
608655b450 Use a shared ref for Vibrator when passed to a worker thread.
To try to protect against this easy mistake, I removed the implicit
capture from thread lambdas.

Bug: 228523213
Test: atest
Change-Id: I8fa37dacb4acf89848a8b17db8db317a988809d4
2022-06-01 12:15:42 +00:00
Fenglin Wu
a464fb4db6 vibrator: turn off vibrator after testing ChangeVibrationAmplitude
Turn off vibrator explicitly after testing ChangeVibrationAmplitude.

Bug: 232319176
Test: VtsHalVibratorTargetTest
Change-Id: I482ea8266378636340c614eda832f4237d22e005
2022-05-24 09:02:58 +08:00
Lais Andrade
f1b4dd3f1d Check primitive duration is positive for supported primitives
Bug: 203629838
Fix: 231486866
Test: VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I2e534f4589c9f594884385a99051c06f518103f7
Merged-In: I2e534f4589c9f594884385a99051c06f518103f7
2022-05-16 16:14:46 +08:00
chasewu
22cb901d1a Correct the PWLE valid duration
Getting the maximum duration from HAL to avoid
different supported duration under different devices.

Fix: 231486866
Test: atest VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I02e96869d9a8e6d5143ae64e3a077bf90dcdb43e
Merged-In: I02e96869d9a8e6d5143ae64e3a077bf90dcdb43e
2022-05-11 15:58:46 +08:00
Treehugger Robot
942f0c579b Merge "Build with upstream lld: Fix incorrect static dependencies" am: 9f18cbd7f6 am: 67a2c6d6f1 am: 8fea839883
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2061768

Change-Id: I70b59b3a2197a73415c1ca91b84df1848b14a380
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-13 22:46:33 +00:00
Pirama Arumuga Nainar
3be0ec611f Build with upstream lld: Fix incorrect static dependencies
Bug: http://b/197965342

Remove unnecessary `static_libs` dependencies or move them to
`shared_libs` to build with upstream LLD.  See b/197965342#comment1
(internal) for rationale.  Some info is available externally at
https://github.com/llvm/llvm-project/issues/42899.

Per go/android-lld-static-lib-fix, OWNERS are added for visibility.  No
action is needed if the change looks good.  This change will be merged
after two business days with Global Approvers.

Test: Build modules with aosp/2036867 in addition to presubmit
Change-Id: Id3ec5d188d7d0ae364e52b98fb9cf7221274665a
2022-04-11 20:04:20 +00:00
chasewu
a266b2c8f2 Correct the PWLE valid duration
Getting the maximum duration from HAL to avoid
different supported duration under different devices.

Fix: 227606057
Test: atest VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I02e96869d9a8e6d5143ae64e3a077bf90dcdb43e
2022-03-31 23:59:40 +08:00
chasewu
70da3cc0a1 Fix the maximum frequency value calculation
Fix: 205250672
Test: VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I908ab93eaee17de2af163c0fe3c50feea567ee48
Merged-In: I908ab93eaee17de2af163c0fe3c50feea567ee48
2022-03-29 19:32:42 +08:00
chasewu
d1a57eaac5 Fix the default BandwidthAmplitudeMap contant
Fix: 225286612
Test: atest VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I5c550426b7cbbdf6e4e9ad90d6248b6ed0523399
Merged-In: I5c550426b7cbbdf6e4e9ad90d6248b6ed0523399
2022-03-29 19:25:42 +08:00
chasewu
8af5e8479a Fix the default BandwidthAmplitudeMap contant
Fix: 225286612
Fix: 224650717
Test: forrest test
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I5c550426b7cbbdf6e4e9ad90d6248b6ed0523399
2022-03-23 07:17:29 +00:00
Chase Wu
be3069df36 Merge "Fix the maximum frequency value calculation" into tm-dev 2022-03-18 04:10:42 +00:00
Steven Moreland
0347ea978c Merge "Fuzzer for default vibrator service." am: 5ceed8997b am: 4b572a4d5a am: 650468afcc
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2025684

Change-Id: Iae65dca0d8370dce428f97311c30143cbb22d83d
2022-03-15 17:22:03 +00:00
chasewu
2456a147a5 Fix the maximum frequency value calculation
Fix: 205250672
Test: VtsHalVibratorTargetTest
Signed-off-by: chasewu <chasewu@google.com>
Change-Id: I908ab93eaee17de2af163c0fe3c50feea567ee48
2022-03-15 21:21:39 +08:00
Steven Moreland
2e96b9dce1 Fuzzer for default vibrator service.
As an example for a fuzzer of an AIDL service.

Bug: 224646709
Test: run fuzzer for a few minutes (doesn't get very
    deep in the fuzzer - will follow-up with a dictionary
    or a corpus if the fuzzer can't get past the header
    and interface token)
Change-Id: Iac02a1f02500a0098d67823ac22de1706778927b
2022-03-15 01:48:16 +00:00
Chih-hung Hsieh
34593d4e2a Merge "Add timed out test files to tidy_timeout_srcs" am: ff54f73b4f am: fa83970b3a am: 581616e658
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1992232

Change-Id: I2eac13fe3b5ab0b4f15d6ceadf376e5e74124a81
2022-02-24 01:32:09 +00:00
Chih-Hung Hsieh
ad1bf31b78 Add timed out test files to tidy_timeout_srcs
* Timed out runs do not show any warning messages.
* These test files cannot finish clang-tidy runs with
  the following settings:
    TIDY_TIMEOUT=90
    WITH_TIDY=1
    CLANG_ANALYZER_CHECKS=1
* When TIDY_TIMEOUT is set, in Android continuous builds,
  tidy_timeout_srcs files will not be compiled by clang-tidy.
  When developers build locally without TIDY_TIMEOUT,
  tidy_timeout_srcs files will be compiled.
* Some of these test modules may be split into smaller ones,
  or disable some time consuming checks, and then
  enable clang-tidy to run within limited time.

Bug: 201099167
Test: make droid tidy-hardware-interfaces_subset
Change-Id: I1de28f1572fff368f67eab512fffec9f2e5c2a9b
2022-02-18 17:25:41 -08:00
Tai Kuo
a90257463d vibrator: aidl: update OWNERS file
Remove leungv@ and add taikuo@ into the OWNERS file.

Bug: 219384042
Test: Presubmit
Change-Id: I191fdf2500a1bec54f2f55671c99b7b1120aee74
2022-02-14 12:33:12 +08:00
Tai Kuo
af798d4d2c vibrator: aidl: update OWNERS file
Remove leungv@ and add taikuo@ into the OWNERS file.

Bug: 219384042
Test: Presubmit
Change-Id: I191fdf2500a1bec54f2f55671c99b7b1120aee74
2022-02-14 12:27:15 +08:00
Steven Moreland
77b74ffc0a Merge "CHECK -> CHECK_EQ copy pasta" am: a8138bfbf6 am: dd39b5c9d0 am: f711c9d406 am: 53c7517b02
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1921302

Change-Id: I3f32bf37c1c025e82e14ee6c9f0ffde87f6bcbd6
2021-12-14 18:40:32 +00:00
Steven Moreland
ffb03995e6 CHECK -> CHECK_EQ copy pasta
Hey did I start this? We can take a little bit better errors. :)

Bug: N/A
Test: N/A
Change-Id: I0e96f7f49dae9968904333da9b8269694058c11c
2021-12-14 01:45:47 +00:00
Lais Andrade
1c8a5a1b6a Check primitive duration is positive for supported primitives
Fix: 203629838
Test: VtsHalVibratorTargetTest
Change-Id: I2e534f4589c9f594884385a99051c06f518103f7
2021-11-09 16:40:51 +00:00
Treehugger Robot
a99d927ae5 Merge "Backfill owner information for VtsHalVibrator*." am: ad56c830f3 am: 7253b8fdcc am: 612c47898d am: b88a0cf1d5 am: dce5d18766
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1814058

Change-Id: Ic9ce332f201618e494e8696b4ef137b9c215d0b2
2021-10-18 19:20:18 +00:00
Yiming Pan
f3f344a134 Backfill owner information for VtsHalVibrator*.
Owner information is backfilled for the following modules:
VtsHalVibratorTargetTest
VtsHalVibratorV1_0TargetTest
VtsHalVibratorV1_1TargetTest
VtsHalVibratorV1_2TargetTest
VtsHalVibratorV1_3TargetTest

Each VTS module is required to have OWNERS file. The ownership is based on
go/vts-owners. For more information about ownership policy, please visit
go/xts-owners-policy.

Test: Tree Hugger
Bug: 143903671
Change-Id: I72b6f2312cd1863e24a05160d688bac86378e37a
2021-10-15 16:29:59 -07:00
Lais Andrade
a2f4ca07c6 Update haptic owners to include frameworks file am: 157418535e am: b2ef6e4be5 am: a8e821bbe2 am: 3776ab3910 am: 729f23a381
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1854183

Change-Id: Ib058bb3c5b0158e9a2ebf6fce91182468f3f9735
2021-10-12 17:14:56 +00:00
Lais Andrade
157418535e Update haptic owners to include frameworks file
Change-Id: I4c089df77dc2b8cf5d43535fbc9dc5f80cd4ca9c
2021-10-12 14:44:23 +00:00
Daniel Norman
e98099957e Merge "Merge "Creates a vendor APEX for the default vibrator HAL service." am: dca9f88513" into stage-aosp-master am: 9e0ab80bb6 am: 2fdcd81183 am: f13bf1a153 am: a252ec3844
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15925131

Change-Id: I32ac331e4671876e071b680b399fed1a21d23b6e
2021-09-28 16:46:01 +00:00
Daniel Norman
7a17de7333 Merge "Creates a vendor APEX for the default vibrator HAL service." am: dca9f88513
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1828079

Change-Id: I580ded8f6f8a19ece2769398aef92bd79936a01e
2021-09-27 23:00:26 +00:00
Daniel Norman
938b862996 Creates a vendor APEX for the default vibrator HAL service.
Bug: 200168006
Test: Ensure vibrator service is running on device.
Change-Id: I218b79f589cfae4f36cdc964ef2f2d0deaf7254e
2021-09-24 16:16:19 +00:00
TreeHugger Robot
3a90a1e4b6 Merge "Update ActivePwle description for amplitude values" into stage-aosp-master am: 1f81aeea7b am: 8d9b5150ea am: b1205f9cd0 am: 234838c23c
Original change: https://googleplex-android-review.googlesource.com/c/platform/hardware/interfaces/+/15827549

Change-Id: I3364e9d6cb5538001fab27af498e7dad0f64312d
2021-09-21 20:34:01 +00:00