Commit graph

375 commits

Author SHA1 Message Date
Treehugger Robot
2734d2f2bb Merge "Remove the active deletion action of callbacks" into main 2024-01-22 10:03:12 +00:00
Yifan Hong
4cab108176 health: OWNERS -elsk +dvander.
Test: N/A
Change-Id: I8f5b3baf649c85a0af8c6c935ea1e847c08afea9
2024-01-17 10:49:07 -08:00
Treehugger Robot
7535e95430 Merge "health: Remove 2.0 HAL implementation." into main 2024-01-09 00:28:28 +00:00
David Anderson
8b413eb4ab health: Query new properties from BatteryMonitor.
Bug: 309792384
Test: VtsHealthTargetTest
Change-Id: I3e0117cf0e98b4861ede92623994877374cd2e23
2023-12-11 11:01:52 -08:00
David Anderson
85b3b0321a Add android.hardware.health AIDL HAL V3.
Bug: 309792384
Test: m
Change-Id: I5e9183b840f523f4d5b81dec813d35983882f71d
2023-12-11 11:01:52 -08:00
Yifan Hong
06343d55fe health: Remove 2.0 HAL implementation.
Remove HIDL 2.0 service implementation and unused
helper libraries.

Test: TH
Bug: 308450739
Change-Id: I8e1eb743d978315ae9b5b516d3e2e9dc6151d2c2
2023-11-02 13:42:21 -07:00
Treehugger Robot
69f5242fba Merge "Don't depend on String8 cast to C string [health]" into main am: d937584b22
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2748301

Change-Id: I1c913e1f157c5270904b2d92d729b85977b5189d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-09-26 19:15:18 +00:00
Tomasz Wasilczyk
4f5fdf09a9 Don't depend on String8 cast to C string [health]
Bug: 295394788
Test: m checkbuild
Change-Id: I67f0468dc21dfb0baab58361078f4f7bc750bdbd
2023-09-13 04:25:40 +00:00
Treehugger Robot
3d5c48819a Merge "Split out health.storage OWNERS file" into main am: 5a3ecbaaf6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2747143

Change-Id: Icda20bc02328cb60e0356b59e1aba84273504d19
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-09-11 18:59:36 +00:00
Treehugger Robot
6fca0ecdec Merge "Merge OWNERS files for health" into main am: a2a5939168
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2746942

Change-Id: Ifc1af121db0c6e6fd15161f9745268a525dbe2cb
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-09-11 18:56:21 +00:00
Devin Moore
0fe0782761 Split out health.storage OWNERS file
Test: none
Bug: 296950573
Change-Id: Ib91206406a35cc76d228d6351dcc785cb3d00002
2023-09-11 17:36:30 +00:00
Devin Moore
4d235eed2b Merge OWNERS files for health
Test: none
Bug: 296950573
Change-Id: I568017d621f9c539ab53c9fc48cf5f5844126f24
2023-09-08 21:22:23 +00:00
Hang Lu
cef01a32a1 Remove the active deletion action of callbacks
The active deletion action will cause binder blocked in some corner
cases. As serviceDied will take care of its deletion, remove the active
deletion has no side effect and could fix the issue.

Test: none
Bug: 296817256
Change-Id: Ib30b3910202e6bbd86ac59e0e69fbeb0a890dd38
2023-09-04 10:52:04 +08:00
Tomasz Wasilczyk
abb05b95e3 Merge "Use String8/16 c_str [health]" into main am: e76430d6f8
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2727996

Change-Id: I4e389c3a74abfbcc992d45ce3241bf7a5284aad7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-29 22:26:26 +00:00
Treehugger Robot
142866d7be Merge "Move OWNERS from health aidl HAL to general health folder" into main am: 8d9bd17981
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2731254

Change-Id: I174e3ff139dd6545b52a9629fcd98ecd1ebf3455
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-08-29 22:25:27 +00:00
Tomasz Wasilczyk
e76430d6f8 Merge "Use String8/16 c_str [health]" into main 2023-08-29 22:02:56 +00:00
Tomasz Wasilczyk
4336fc5995 Move OWNERS from health aidl HAL to general health folder
Test: N/A
Change-Id: I01d4b52042d4f9f7b68898e83cb2058e9877a1f0
2023-08-29 21:11:05 +00:00
Xin Li
879960bdce Merge Android U (ab/10368041)
Bug: 291102124
Merged-In: I7b6fffac2ada0e039f79bad8cc9b4d954e9c3460
Change-Id: I9466127d8d0fa38df36ca99f704853b2db871e67
2023-08-25 13:29:30 -07:00
Tomasz Wasilczyk
fcaba4b69c Use String8/16 c_str [health]
Bug: 295394788
Test: make checkbuild
Change-Id: I6671af787ca8c687d30d0db8dbb4f639157b5e81
2023-08-25 19:39:33 +00:00
Ryan Prichard
c8fc80c47b Include <functional> for std::function and std::placeholders
Bug: b/175635923
Test: treehugger
Change-Id: I3dabb8c44334713be9e461dd81f437c3da1ea936
2023-07-21 16:18:11 -07:00
Yifan Hong
70197263f0 health: Check return value of LinkedCallback::Make.
If LinkedCallback::Make returns nullptr, do not put it
in Health::callback_. Otherwise, OnHealthInfoChanged
crashes later because the linked callback objects are
not null checked before accessing.

Test: android.hardware.health-service.aidl_fuzzer (with
  a special corpus)
Fixes: 289599278
Change-Id: I8bad41dbcfbefeb54744059baffd4eef1ae7ec42
2023-07-06 17:46:25 -07:00
Yihan Dong
818d990a5e Create OWNERS for VtsHalHealthStorageTargetTest am: 36f5abe27c am: fa91cb1e25 am: ff4059824a
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2566351

Change-Id: I10e5143d9c5910ada1c96f50b18476fd6ec1f0ed
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-04-27 22:02:00 +00:00
Yihan Dong
36f5abe27c Create OWNERS for VtsHalHealthStorageTargetTest
Bug: 278494006
Change-Id: I950548feb2673ba8f6c646a4916a4730ef6ce763
2023-04-27 09:11:30 +00:00
Alex Buynytskyy
ec1198d941 UpsideDownCake is now 34
Ignore-AOSP-First: UpsideDownCake Finalization
Bug: 275409981
Test: build
Change-Id: I832ab7a8b5445ba85e27bbcf5be64906bfa0e5be
2023-04-07 23:41:20 +00:00
Jack Wu
2d6557c0f7 health: fix vts setChargingPolicy failing
Bug: 274308314
Test: m android.hardware.health-update-api
Change-Id: I0f1a41d86d6998f061a29a51b9d1cf43a47949c2
Signed-off-by: Jack Wu <wjack@google.com>
2023-03-23 12:01:22 +00:00
Jack Wu
cbbf24f15d health: get batteryStateOfHealth from healthd
Bug: 251427118
Test: m android.hardware.health-update-api
Change-Id: I556570782b0a9698fd3530be86f8ede33b11afe2
Signed-off-by: Jack Wu <wjack@google.com>
2023-02-23 01:24:00 +00:00
AleX Pelosi
39c564112b health: batteryStateOfHealth should be a property
Move batteryStateOfHealth to HealthData

Bug: 251427118
Test: m android.hardware.health-update-api
Change-Id: I8835489f5c3d75ffc48e198a63eafc52b555ccd4
Signed-off-by: AleX Pelosi <apelosi@google.com>
2023-02-23 01:23:51 +00:00
AleX Pelosi
8a1f34f1b7 health: update owners
+apelosi

Bug: N/A
Test: N/A
Change-Id: Ia698145eda18b8a6e34e80303ac14524c036b1fe
Signed-off-by: AleX Pelosi <apelosi@google.com>
2023-02-17 13:19:32 +00:00
Yifan Hong
8bde9fedfc health: update owners.
-stayfan, +wjack

Test: N/A
Bug: N/A
Change-Id: I31f4f21efc30c90bc8e46567e7d08dfdabf717d1
2023-02-03 21:43:04 +00:00
Jack Wu
9beec7e3af health: fix vts test failed in aidl v2
Add a check before executing the test to determine whether the
health HAL is indeed the V2 version.

Bug: 266665472
Test: vts test
Change-Id: I43c7d0139037701f0ef2e8f3925272470684a50e
Signed-off-by: Jack Wu <wjack@google.com>
2023-02-02 16:49:43 +08:00
Jack Wu
3356161d6c health: expose battery health data and update version to V2
1. add the new propertes
  - batteryHealthData
  - batteryStateOfHealth
  - chargingState
  - chargingPolicy
2. add new types to health status
3. add new methods
  - setChargingPolicy
  - getChargingPolicy
  - getBatteryHealthData

Bug: 251425963
Test: m android.hardware.health-update-api
Change-Id: Ie2339c50e1f9bfc28427e4108a95b1a16c739205
Signed-off-by: Jack Wu <wjack@google.com>
2023-01-21 07:33:51 +00:00
Jack Wu
0edf57e04b health: fix depending on multiple versions of the same aidl_interface
When health aidl_interface version updraged, some libraries are
depending on V1 and android.hardware.health-translate-ndk at the
same time. It will cause multiple versions depending if
android.hardware.health-translate-ndk is depending on new version.
version.

Fork android.hardware.health-translate-ndk generate two versions,
so that other libraries can depend on the same aidl_interface,
to avoid multiple versions depending.

Bug: 251425963
Test: m
Change-Id: I0641e9f6c1445ee2549b956012ebb35eb1f8ac97
Signed-off-by: Jack Wu <wjack@google.com>
2023-01-11 09:45:19 +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
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
Hamzeh Zawawy
25c5d2e804 Merge "Add an AIDL fuzzer for health service" am: 5c04494c60 am: b1165847fb
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2094863

Change-Id: Ibd4de8004027e98123fb4ad586766a3063af5722
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-05-12 21:11:03 +00:00
hamzeh
34b47964d5 Add an AIDL fuzzer for health service
Bug: 228862320
Test: SANITIZE_TARGET=hwaddress make android.hardware.health-service.aidl_fuzzer
Change-Id: I917441738a7a9d18d7c4fcbe0212bdaf77fe265e
2022-05-10 12:23:11 -07:00
Jiyong Park
70f5837e1f Freeze AIDL APIs for TM
Bug: 225941299
Test: m
Change-Id: I61eb51c0334eb99489c6f1570110d7e18c350c99
2022-04-16 07:03:24 +09:00
Yifan Hong
2b1a7769ff health: Add stayfan and smoreland as owners. am: c99d0553c6 am: 2cceaa983b am: d54f7ca3f6
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2064636

Change-Id: I27836e06f118478dbdc3c8be5b0a87be991be281
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2022-04-14 22:15:20 +00:00
Yifan Hong
c99d0553c6 health: Add stayfan and smoreland as owners.
Test: none
Change-Id: I4c79b1aba1172b990cae55009b4cc095d35acab7
2022-04-14 12:55:51 -07:00
Yifan Hong
ad4577c05e Merge "health: batterymonitor uses AIDL HealthInfo." am: e9cb1bedfd am: 577ea5663a am: 84a60fff99
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2004612

Change-Id: Ic4841cf2375aed018557b776ff7344d8f0ff97a2
2022-03-02 20:26:16 +00:00
Yifan Hong
e9cb1bedfd Merge "health: batterymonitor uses AIDL HealthInfo." 2022-03-02 19:20:31 +00:00
Yifan Hong
bc84a7903a health: batterymonitor uses AIDL HealthInfo.
As a side effect, chargerDockOnline is properly set.

Test: TH
Bug: 222120465
Change-Id: I660e8ac0d1897ce30e3ae5d4f402a0b9ccac3c5c
2022-03-01 12:27:13 -08: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
Treehugger Robot
d32b40676c Merge "libhealthloop: Remove the POWER_SUPPLY_SUBSYSTEM macro" am: 652b89bd0a am: cb27f8cccb am: ae389f2b16
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1864853

Change-Id: I07a100c875a4a735175276d06034d8ae51f53919
2022-02-12 01:06:40 +00:00
Treehugger Robot
652b89bd0a Merge "libhealthloop: Remove the POWER_SUPPLY_SUBSYSTEM macro" 2022-02-12 00:23:15 +00:00
Treehugger Robot
0a1fb8c0d3 Merge "health: vendor.charger-* -> vendor.charger." am: 71af5c2da9 am: 91484f8aef am: 5c41d85749 am: 23356e6123
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1962399

Change-Id: Ia5007f2cea75cf446299fb0bc489b4111645f96f
2022-01-28 02:22:49 +00:00
Treehugger Robot
71af5c2da9 Merge "health: vendor.charger-* -> vendor.charger." 2022-01-28 01:20:11 +00:00
Yifan Hong
64345cd628 health: vendor.charger-* -> vendor.charger.
Devices may have actions triggered by
init.svc.vendor.charger. If the name is changed,
these actions are not triggered properly.

The name is usually vendor.charger, so the default
service will also use vendor.charger as the name.

Test: none
Bug: 215152892
Change-Id: Icb296ae82a6f3472730fdb3abc35dcede22889e7
2022-01-26 15:53:11 -08:00
Treehugger Robot
4ce8a75899 Merge "health: support Dock charging" am: a366ec1a33 am: 3ab4cb982d am: eb9d901dfd am: 3a288e41bc
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/1946230

Change-Id: I394eb34371d275b35444d0b7f0eacce22a0f8868
2022-01-25 03:04:31 +00:00