Merge "Remove redundant dependencies."

.clang-format: 100 lines

This allows us to be more like the system/core clang format
and also prevents problems because a lot of hardware/interfaces
code is checked in with 100 lines of code.

Test: pass
Change-Id: I62aeba9deb33ad6ce3d62fada3e0c04139ca3588

Remove redundant dependencies.

android.hidl.base@1.0 and android.hidl.manager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Change-Id: I56655b0214ba147398f9234eb004fba0aceb7e40
Merged-In: I08aaad80f7e2fc262aa3a8b66fe932e8133a928d

Merge "Remove redundant dependencies"

Remove redundant dependencies

android.hidl.base@1.0 and android.hidlmanager@1.0 are built into libhidltransport.

Test: links
Bug: 33276472
Change-Id: I424371932af129bba9d58dcf8652c64297f35f05

nfc: Android.mk -> Android.bp

Test: links
Change-Id: Ia19dafb3ff6bf5772d72133bb2e1687226e0d44f

Merge "Tether Offload HAL"

Merge "configstore-utils: vendor_available"

configstore-utils: vendor_available

Test: with BOARD_VNDK_VERSION := current
Change-Id: I4fe86224d18bc7e7579c13f9273bc4efa25df1ea

Merge "Update .hal file for fixing vts tests"

Merge "Fix Vts errors"

Fix Vts errors

- Add REQUEST_NOT_SUPPORT into General Errors
- Add a util function to check General Errors
- Add a util function to check OEM Errors
- Changes to RIL command api tests based on the QC provided pdf
  to see if the errors from vts-web can be fixed

Test: Manual
Bug: 35671595

(cherry picked from commit 01e7c6ee5fcb1b469cac4a1fc96fc8c7f053c435)
Merged-In: I9500c686a75539dab37f0272349bc2f965cc0a8f
Change-Id: I9500c686a75539dab37f0272349bc2f965cc0a8f

Update .hal file for fixing vts tests

- update 'voice' and 'sms' based on QC pdf file
- remove GENERIC_FAILURE still there
- fix duplicate error code

Test: Manual
Bug: 35671595
(cherry picked from commit 159ec7fbf5e7e1cfd1e76e88c71a72873dc5d8f4)
Merged-In: I83831c222cb0d157d0aa06e48cf0a177ab0ecf31
Change-Id: I83831c222cb0d157d0aa06e48cf0a177ab0ecf31

Merge "stop bluetooth service while VTS test is running"

Tether Offload HAL

Adds new interfaces to perform hardware optimized tethering offload.

Test: as follows
    - built (bullhead)
    - flashed
    - booted
    - "runtest frameworks-net" passes
Bug:34361337
Change-Id: I1ed756306e1efe98bd2703df9c9846a50a87ffcd

Merge "RIL changes to allow the modem to query the framework for the key."

RIL changes to allow the modem to query the framework for the key.

BUG: 35606429
Test: manual
Change-Id: I7687d664c928de2325d50c3e9e7912f10aa37463
Merged-In: Ib1f5ee5f793f53901ad54bb6b6d4cb667dc720ab

stop bluetooth service while VTS test is running

Test: run VTS HAL BT test
Bug: 37153328
Change-Id: I615d96daff496096122cd18158c205073f4623a6
(cherry picked from commit 1965f73bacbb1cd937f97149b29583edb65c2c99)

Change bootHAL to early_hal class

Bug: 37579828
Bug: 37543411
Test: marlin boots with wipe data
(cherry picked from commit 9a4388cf28d5f4804efa4fe0f6cc9485dea6839a)
Merged-In: Ifa508570455289efd58350562fadea5676c3bc67
Change-Id: Ifa508570455289efd58350562fadea5676c3bc67

Merge "Fix test case for setSimCardPower radio api."

Fix test case for setSimCardPower radio api.

Bug: 35800570
Test: vts
Change-Id: I93e78c46971158cd14666931b527d8e268e0a39e
Merged-in: I93e78c46971158cd14666931b527d8e268e0a39e

Remove dependency of libhardware_legacy

It is not used by bluetooth and nfc services.
Bug: 33241851
Bug: 37305530
Test: built and tested on sailfish

(cherry picked from commit 593eeb7a4b0ea3af560dc4ada36237e786b0e039)

Change-Id: I599073296baa5eee72a1fce8caa4eb8352f66218
Merged-In: I599073296baa5eee72a1fce8caa4eb8352f66218

Move bluetooth test to vendor.

The test is still built in data/nativetest64, but it must be marked as
vendor since it depends on vendor modules.

Also proprietary -> vendor.

Test: build with BOARD_VNDK_VERSION := current
Test: run test on internal marlin, passes
Bug: 33241851
(cherry picked from commit 2cfae52a2e479c1dc91caa2b0ae365718bb8e6fc)
Merged-In: I856838fe208a4cd038ea16d47f03f6278224b2ee
Change-Id: I856838fe208a4cd038ea16d47f03f6278224b2ee

Merge "Changed the naming scheme for Treble java libraries (both static and dynamic)"

wifi(interface): Add links to APF docs

Bug: 32119512
Test: Compiles
Merged-In: Iadb68a2a649b120b607735e4d88617b11740832f
Change-Id: I32ec491983d1d9e987a19a00b4fd8e60503c9e7f

Changed the naming scheme for Treble java libraries (both static and dynamic)

android.hardware.foo@1.0-java => android.hardware.foo-V1.0-java

Bug: 37207894
Test: make
Change-Id: I177f4cfd42f635aaf096107af8309d1a82ed41c0
Merged-In: I8b91881dfaf507c0d852d56ebbb53ff0987e5a34

Merge "Mark hidl test libs as vendor."

Mark hidl test libs as vendor.

They must be on the vendor partition for hidl_test to be able to link to
them when the VNDK is in place.

Test: hidl_test
Bug: 33241851
(cherry picked from commit 00556e0a11c4224b31d24e815dd50299c08c61e6)
Merged-In: I23c9e0674aaf33602bc9fde0515eb2f87ce5c60c
Change-Id: I23c9e0674aaf33602bc9fde0515eb2f87ce5c60c

Merge "bluetooth: include what you use"

Merge "Update makefiles."

bluetooth: include what you use

Helper libraries also now on vendor partition.

Test: links
Test: bluetooth works on internal Marlin
Bug: 33241851

Change-Id: Ie99faf9146ce2ce3df83a2b46acfaa36a54c16c2

Merge "Update Radio and Sap service name in VTS."

Update makefiles.

Test: pass
Change-Id: I5cf42c8ff1e11207394e4852ba0bd03e4e3b7d2e

Merge "Add p2 option in OpenLogicalChannel"

Update Radio and Sap service name in VTS.

Bug: 37164398
Test: VTS tests pass.
Change-Id: I71562e5b9709166695d08a96a04928027dcb4ee7
Merged-in: I71562e5b9709166695d08a96a04928027dcb4ee7

Merge "Revert "Weaver HAL""

Revert "Weaver HAL"

This reverts commit 694beec5d73d34cbf1dc56d2b2193f832777e676.

Reason for revert: This will not be in O.

Change-Id: I301f265785815689e9cfc8412db1966a4565b561

Merge changes from topic 'configstore-utils-log'

* changes:
  print log message for values retrieved from configstore
  Handle hidl transaction errors in ConfigStore

Merge "Weaver HAL"

Merge "Empty interface IQuux which is completely unrelated to IBase and IBaz."

Weaver HAL

Weaver is used to harden the user authentication flow by securely
storing secrets which can only be read by presenting the associated key
secret. The key secret is derived from the user's credential.

Bug: 35628284
Test: Implement HAL, boot and call from system_server
Change-Id: Ia72fd537670c08d27320c944a0fe91243137bf6c

print log message for values retrieved from configstore

This CL adds a code for printing log messages in clients-side utility
library functions (getXXX) for configstore so that the developer can
figure out which configuration values are actually retrieved from the configstore.

Bug: 36275627
Test: Built and check the log message appears

Change-Id: I7d5d80ca7a0317816c71016e013f9e73ad23ee08
Merged-In: I2c0895f8afbbb2947b62164acaf62a491c451dc0
(cherry picked from commit 0dc72ecbb798971d6f394d7f4dbc9d1f4006da5f)

Handle hidl transaction errors in ConfigStore

Test: surface flinger not failing, android now bootsup

Bug: b/36445794
Change-Id: I64cc404bec71f5e4eea2e0034f07b86fb60a3e32
Merged-In: I22fa7aab9fa92bc04333aaa0eef45891ebeba8e7
(cherry picked from commit 076b792ade608a7fdf4e600a42550ad17296a39e)

Merge "Added protections for RenderScript VTS tests."

Merge "[AWARE] Legacy HAL update: reduce host wakeup"

Empty interface IQuux which is completely unrelated to IBase and IBaz.

Used to verify that it's not possible to obtain an IQuux interface from an
IBase/IBaz binder, i.e. IHwBinder.asInterface(...) properly respects the
interfaceChain.

Bug: 36749201
Test: hidl_test_java
Change-Id: I84419e78db22acdfcc7d9972d22d139641469bde

Bluetooth HAL: Add to foreground schedtune group

EAS (Energy-Aware Scheduler) tends to cluster tasks together unless
they're marked to prefer an idle CPU (prefer-idle) during placement.
This is to save energy however it increases latencies. This is
noticeable when the Binder driver demotes the RT task to CFS and when
we try to do load balance in this path, the selection path still selects
a busy CPU. Add Bluetooth HAL service to the foreground schedtune group
to fix this as its critical for tasks like bluetooth audio playback.

Test: Boot and verify bluetooth HAL is in the foreground stune group
Bug: 36867985

Change-Id: Ib3c43cca4a2049af3700978b58a05989cc20b6ac
Signed-off-by: Joel Fernandes <joelaf@google.com>

[AWARE] Legacy HAL update: reduce host wakeup

Legacy HAL update to add indications when a solicited publisher
receives a match. Not used in current HAL. However, also added a
control bit to turn off host notification which is now enabled.

Bug: 35195773
Test: integration (sl4a) tests
Change-Id: I8ff378abaae11d4feaa42a9901abb1b48069dc4f

Add p2 option in OpenLogicalChannel

- Adding support for P2 parameter
- Needed to support OMAPI 3.0 defined
  by SIM Alliance

Bug: 27360148
Change-Id: I88134eded001e14c12e1f2a18efd0a2b51bc22fe

Merge "[AWARE] Create true translations between legacy and hidl enums"
am: 24ff2036ce

Change-Id: I00c0d5dce8a57ce76656c8521d93a974308b7c94

Merge "[AWARE] Create true translations between legacy and hidl enums"

Merge "nfc: vts: Correct parameters going for CoreInitialized"
am: 4133401913

Change-Id: Ie205284f3adcfd7f662c1d26dea1a01eb48a12e8

Merge "nfc: vts: Correct parameters going for CoreInitialized"

[AWARE] Create true translations between legacy and hidl enums

Replace implicits translations between legacy and hidl enums (using
casts) with explicit translations.

Bug: 35767620
Test: integration (sl4a) tests
Change-Id: I4ae4cd691226ce29bc0e92f9890fbb343a064505

nfc: vts: Correct parameters going for CoreInitialized

Core Initialized with parameters 1-3 leads the device into recovery
mode. Which causes the test to fail.

Bug: 36751759
Test: Test passes
Change-Id: I1375546cc66c147f26c315a042ca0afb49f0ce06

Merge "Test for getHashChain() in IBase."
am: c8c7ab74a1

Change-Id: I6519da67d36565da408240d2f59a547330971c06

Merge "Test for getHashChain() in IBase."

Merge "Updates bluetooth hal test to use VtsHalHidlTargetTestBase v3"
am: 177171f305

Change-Id: Iafecfba9ace2188d2a21f927b6afe2691ebfcdbc

NFC VTS test use VtsHalHidlCallbackBase for callback wait and notify
am: bfefc5ef10

Change-Id: I0b40ae10c42b0e0a4f0696792c707ba0f990aadf

Merge "Updates bluetooth hal test to use VtsHalHidlTargetTestBase v3"

NFC VTS test use VtsHalHidlCallbackBase for callback wait and notify

VtsHalNfcV1_0TargetTest uses VtsHalHidlCallbackBase.
Callback class extends VtsHalHidlCallbackBase and use waitForCallback and notifyFromCallback

The original NFC test contains the following use cases:
  * Callback notify happens before wait
  * Multiple waiting in serial
The modified version added the following use cases:
  * Wait for specific callback function
  * Passing callback arg data along wait and notify functions

Bug: 36036059
Test: make VtsHalNfcV1_0TargetTest, adb push & adb shell
Change-Id: Ic6668a18254661d07c4ff37309cd96ea5f786d29

Updates bluetooth hal test to use VtsHalHidlTargetTestBase v3

Test: pass
Change-Id: Ia5237a27800010174f2b23a3a4b1157f1c48781f

Test for getHashChain() in IBase.

Each interface has a getHashChain() method that
returns the hash of all .hal files from the rtti
of the interface up to IBase.

Test: hidl_test

Bug: 36602587
Change-Id: I53a79ba69edd109f169c37dbe0fd10e302eb929c

Added protections for RenderScript VTS tests.

Changed many EXPECT_NE to ASSERT_NE for objects that are required later
in the test which would potentially cause a segfault. Additionally updated
a few descriptions.

Bug: 35915961

Test: mm and run on angler
Change-Id: I17c85b2c774688d82b59be2f0b383447e03349e0

Merge "Enable the clang-format pre-submit hook"
am: 3b9e23513e

Change-Id: Idf1651f9e76c8b7e6dfbc70dab8ae567a5f53937

Merge "Enable the clang-format pre-submit hook"

Enable the clang-format pre-submit hook

Bug: 36699008
Test: Try to upload a patch with formatting errors.
Change-Id: I4af7d142ccbbaf39eb236547487c59c5ff8f2cd6

Merge "Change sytle of radio and sap tests to clang google"
am: 7441c791b6

Change-Id: I0aa6ad8fcf24862278705a1f0a9b43cb08370c80

Merge "Change sytle of radio and sap tests to clang google"

Merge "Bluetooth: Replace assert() with CHECK()"
am: b26fea2629

Change-Id: If0285fbce60e230fa947d8400fa0387dbb1bc551

Merge "Bluetooth: Replace assert() with CHECK()"

Change sytle of radio and sap tests to clang google

Test: make VtsHalRadioV1_0TargetTest, make VtsHalSapV1_0TargetTest
Change-Id: Ie13efe0d049ade7839881c14ed2904d8acd99b93

Merge "Add VTS tests for Sap Apis."
am: eb245db746

Change-Id: I6aaeba6da6cb43639e434e2f0ecc10a5731c2b47

Merge "Add VTS tests for Sap Apis."

Bluetooth: Replace assert() with CHECK()

Test: Bluetooth starts/stops, VtsHalBluetoothV1_0TargetTest passes
Change-Id: I1ecdf4ed7507fc8e2f00c35f136cb13e145713e5

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify
am: 51d023c48e

Change-Id: Ifaad9a1c7d0994b43c208ca4c7899daab2baf9ec

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify

VtsHalBluetoothV1_0TargetTest uses VtsHalHidlCallbackBase.
Callback class extends VtsHalHidlCallbackBase and use waitForCallback, notifyFromCallback, and setWaitTimeout

The original Bluetooth test contains the following use cases:
  * Wait for specific callback
  * Set timeout for specific callback waiting
The modified version supports the above use cases.

Bug: 36036059
Test: Test pass locally
Change-Id: I961baaf7f1b09fa9c134629cc7280565a2664ed5

Merge "RenderScript VTS cleanup and fixes"
am: 7b36175068

Change-Id: I03e7a64f197e7b47f7f7dbcd893cc3125fc06850

Merge "Fix RS HIDL server, pass data by bytes instead of by elements."
am: f2b214dc8d

Change-Id: I753fcfd32f0837a099bf157c837b6e7759a9ab45

Merge "RenderScript VTS cleanup and fixes"

Merge "Fix RS HIDL server, pass data by bytes instead of by elements."

Fix RS HIDL server, pass data by bytes instead of by elements.

  Our current stack:
  API->API_TO_HAL_translator->HAL
          ->HAL_TO_Implementation_translator->Implementation

  For most APIs:
  - API passes objectCount.
  - HAL expects objectCount.
  - Implementation expects objectCount.

  For APIs like ScriptGroupCreate:
  - API passes byteCount. And unfortunately, these APIs are part of
    NDK, we could not make them also passing objectCount like others.
  - HAL expects objectCount.
  - Implementation expects byteCount.

  So that both API_TO_HAL_translator and
  HAL_TO_Implementation_translator should correctly convert input
  objectCount/byteCount to byteCount/objectCount.

  This CL only fixes the HAL_TO_Implementation_translator part,
  whereas aosp/356395 fixes the API_TO_HAL_translator part. Both
  parts were mistakenly using byteCount as objectCount, causing
  potential out-of-bound access.

Bug: 36404879
Test: mm on angler
Change-Id: I28541a8926aeafece40e2a3f664bda67e26a34a2

Add VTS tests for Sap Apis.

Bug: 32022332
Test: vts
Change-Id: I9a55978aa485a59510090f46a8e5edfb87bccc0e

Convert all comments into "doxygen-ready" comments.
am: 3e4acb8b3a

Change-Id: I615de8ebda7c08f29e52dc78f65c5408d3a7d58f

Convert all comments into "doxygen-ready" comments.

Bug: 36453077
Test: make
Change-Id: Ic77394d4d2e1ccedba3a28230d870c5d6108bd40

wifi(implementation): Stop loading vendor library statically
am: 34e4bc4dd2

Change-Id: I614a2074fbc2d82618eeb72002724830cf888dab

wifi(implementation): Stop loading vendor library statically

libwifi-hal statically imports the vendor implementation of the legacy
HAL. Since the HIDL daemon is dynamically linking against libwif-hal,
there is no need for it to statically import the vendor implementation
library (one pointed to by: $LIBWIFI-HAL variable).

Bug: 36169790
Test: Compiles & device is able to connect to wifi networks with the
change.

Change-Id: Ie44b58328f9bdde011ba6d4a4d26f786c2d097e7

RenderScript VTS cleanup and fixes

This CL fixes the remaining VTS test cases for the RenderScript
vts-hidl-hal test and cleans up and improves the working test cases.
The goal is to have at least one passing test case per HIDL HAL entry.

Bug: 35915961

Test: mm and run on angler
Change-Id: I4b6e25831ce7d2abb3148c39eb48bde10ed04646

Update makefiles.
am: e90c31f125

Change-Id: I370a9a20b005d7510463594b24e5c4fc8791116a

Update makefiles.

Test: pass
Change-Id: I925baccbe51a32b7b7f3f66cf2918d017ee6b039

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation"
am: 2d0b636b29

Change-Id: I342068f8bbcf6a97c322bcf0049fed83b519bcda

Merge "Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation"

Merge "[WIFI] Restore initialization code"
am: 8f4416aabb

Change-Id: I9f0f828d542e337f185c4a423748f57ef53a00ce

Merge "[WIFI] Restore initialization code"

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)"
am: d5934d06f1

Change-Id: I958c1285dbb5e3c6db8145ac743cedfb61cd544c

Merge "removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)"

[WIFI] Restore initialization code

Restore the usage of WIFI-specific main function. The main
function was inadvertantly disabled by moving it into a static
library - which caused the default main to be used.

Need a custom main since need to stop WIFI before tests start
to enforce a clean starting point.

Bug: 35276551
Test: confirms custom main routine is called
Change-Id: I978827bc0f0f16d9ca4f159fbf42d0bc53f5df99

Merge "Nfc: Null ptr checks in HAL"
am: 3b4d423d42

Change-Id: Icede34b74cd253d6921ce38c5abb0d3f80ee1bd7

Merge "Nfc: Null ptr checks in HAL"

removed unused function (HIDL_FETCH_ISurfaceFlingerConfigs)

HIDL_FETCH_... is required only for passthrough mode, but Configstore
does not provide passthrough mode.

Bug: 35325577
Test: build and check configstore works

Change-Id: I35e916ad4a6c321303a3888dabb4884b17146e16
Merged-In: Iac3ed57568f18c3a8a7f7423a361b0fe45aa54ab