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

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

Merge "[AWARE] Add Passphrase data-path security"
am: 9e71d7d44f

Change-Id: I4f0e514d2f49156942c6818379347c369c79889b

Merge "Bluetooth: Protect pointers and callbacks"
am: 54906b6ccf

Change-Id: Idb2e6f6fc796c95552b112060b9866a81073dc37

Merge "[AWARE] Add Passphrase data-path security"

Merge "Bluetooth: Protect pointers and callbacks"

Nfc: Null ptr checks in HAL

Adds null pointer checks in the default implementation.

Test: compiles
Bug: 36149477
Change-Id: I6195ce92f5cefd4b82874866d08014830f400192
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "use toString instead of to_string"
am: 6a73f61b96

Change-Id: Ia28bfdf70b7a79c96822aa3f0402bc0290ad7b3d

Merge "use toString instead of to_string"

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs."
am: 60449e68d1

Change-Id: I0bcc97f30baf408e6eb6b96f7f37b3b536953fc7

Bluetooth: Protect pointers and callbacks

Test: kill com.android.bluetooth
Bug: 36150800
Change-Id: I9533731c9ce3776c84e6c1d3fdf3e266b328a084

Merge "Add VTS for couple of missing APIs and some cleanup for earlier APIs."

use toString instead of to_string

Bug: 35703683
Test: hidl_test_java passes

Change-Id: Ib7d6807f6b9ddcd7b0271b17a0a52be0ee07ec6c

Remove GENERIC_FAILURE and INVALID_ARGUMENT for Api documentation

Remove GENERIC_FAILURE for the documentation of the apis
in the IRadioResponse.hal, which should not be
returned in any case.

Remove INVALID_ARGUMENT for the documentation of the apis
in IRadioResponse.hal if it is not present in the ril.h file.

Bug: 36385785
Test: No change to behavior
Change-Id: Iab0976ddc6b10863cdbdbfb8a8f213d7d9b17cb5

Merge "disabled passthrough mode for Configstore"
am: 7f8b76a322

Change-Id: Ib7e37189c90752d8c536cb2e5bdf0a2518865981

Merge "disabled passthrough mode for Configstore"

[AWARE] Add Passphrase data-path security

Add mechanism to configure data-path security using Passphrase
(as an alternative to PMK).

Bug: 35866810
Test: integration (sl4a) tests passing.
Change-Id: If7094a44af1996d3caf5b776d492797f14a7be11

Add VTS for couple of missing APIs and some cleanup for earlier APIs.

Bug: 32022332
Test: vts
Change-Id: I361af4a37985e5bc686a2d7302917afa134ebc24

supplicant(interface): Add an event for bssid changes
am: c314841614

Change-Id: I587c0c479708448492dce329c267f2b3761911d8

wifi(implementation): Return proper status code on initialize
am: a1c76e4d53

Change-Id: Ia6e18ee9f9543fb4059dd349c8a0f9ef286ad14d

supplicant(interface): Add an event for bssid changes

This is useful in debugging (& collecting metrics) when the active bssid changes.

Bug: 36451639
Test: Compiles & manual tests using the framework changes.
Change-Id: I5676dfb9ec9c13ae900c315eb8dfacb857e70fc1

disabled passthrough mode for Configstore

This change removes target "android.hardware.configstore@1.0-impl" to
disable passthrough mode.

Bug: 35325577
Test: android.hardware.configstore@1.0-impl.so not exist in outputs and
    configstore works in binderized mode

Change-Id: Ieedb6fea179d50ac35fc32b0e21752b2334558ae

wifi(implementation): Return proper status code on initialize

The fallback HAL will return |WIFI_ERROR_NOT_SUPPORTED| when
|init_wifi_vendor_hal_func_table| is invoked.  This should be sent as is
to the HIDL interface instead of sending |WIFI_ERROR_UNKNOWN|.

Bug: 34859006
Test: Compiles
Change-Id: I5fd132368715bd158e617ad3cf2e6f88d147cef7

Merge "Use simplified update-makefiles.sh script."
am: 04ff4a07eb

Change-Id: I947819188949b34426b4766d64943f8d5ce96742

Merge "Use simplified update-makefiles.sh script."

Merge "supplicant(interface): Correct the WFD info length"
am: d51c817675

Change-Id: I05c077fbdfb2c0b085e869101c5548318ade3eaf

Merge "supplicant(interface): Support for P2P client list and save config"
am: 91c2cb3284

Change-Id: Id18d9ad5427703f3b975691d8995ea641b9405d2

Merge "supplicant(interface): Correct the WFD info length"

Merge "supplicant(interface): Support for P2P client list and save config"

Merge "configstore: respect service registration status."
am: e8e87df57f

Change-Id: I528fc4914ab9b1da49825b425bd5672e0b117add

Merge "configstore: respect service registration status."

Versioning, add additional package import.
am: a233c182be

Change-Id: I5d84ccd57530fa7086f50e469ab75a05d40ad800

supplicant(interface): Correct the WFD info length

Error logs from test:
03-17 18:34:34.303  1781  1781 E wpa_supplicant: Unexpected WFD device
info len: 6

Bug: 36401779
Test: Compiles
Change-Id: I65805599f8f3242a5c1f6ba8c0a1b9da05df79b4

supplicant(interface): Support for P2P client list and save config

Bug: 36042785
Test: Compiles
Change-Id: Ib47b47762c05e6252f692c1456e9b8d5dd8c051a