Merge "update-makefiles.sh"

Merge "Modify method to get MQDescriptor for unsynchronized FMQ."

wifi(vts): simplify HIDL calls

Presently, the core of the Wifi test logic is
obscured by the boilerplate required to create
a callback.

This CL provides some utilities to simplify
the creation of a HIDL result callback, and
migrates existing Wifi code to use the new
utilities.

Along the way: add a .clang-format file, so
that I don't misformat code with 2-space
indents (the Google default).

Bug: 34817351
Test: vts-tradefed run commandAndExit vts --module=HalWifiHidlTargetTest
Change-Id: Id2c728f96c3369c74adc8dfce7228b0a15a0781e

update-makefiles.sh

There have been some changes to the way hidl-gen generates these and
some CLs got out of sync with hidl-gen.

Test: pass
Change-Id: I33c573499d5dd37de447902fd93929b80a5df371

Merge "[AWARE] Correct comment on Discovery Window usage"

Merge "Remove --coverage in Android.bp"

[AWARE] Correct comment on Discovery Window usage

Clarify that DW period specification on discovery session
does not override the overall system DW period specification
(if one is provided).

Bug: 35040192
Test: none - comment fix
Change-Id: Ia7c495f1cd50a4bce0ffbaf242e52bc4f110eef2

Merge "Bluetooth: Add CHECK and move ASSERT"

Modify method to get MQDescriptor for unsynchronized FMQ.

The modified method adds a new parameter specifying
if a new unsynchronized FMQ needs to be set up by the server
or if it is to return an MQDescriptor to
an unsynchronized FMQ that has already been configured
by the server(if one exists)

Bug: 35319190
Test: FMQ unit tests
Change-Id: I64a5b356a2761faaeeea6fdd2c3a7da9b60f0016

Move hidl shims to the vendor partition.

We need google shims on the vendor partition because they are providing
an implementation of a vendor defined interface. They were written by
google just as a courtesy/to make the transition easier. They're
basically a set for vendors to assemble their hal implementations
from.

Bug: 34135607
Test: marlin persist.hal.binderization on/off

Merged-In: I3f86c77c147514fe470f08088a66de312ae32d4e
Change-Id: I3f86c77c147514fe470f08088a66de312ae32d4e

Merge changes Ieee26013,I275bba11,I7e7258e7,I88111181

* changes:
  supplicant(interface): Remove redundant callbacks
  wifi(implementation): Copy the user complete callback by value
  wifi(implementation): Add status check for all cb invocation
  wifi(implementation): Add global lock

supplicant(interface): Remove redundant callbacks

The information broadcast via these callbacks are already available via
other callbacks.

1. All of the onProvisionDiscovery* callback data can be retrieved via the
|onProvisionDiscoveryCompleted| callback.
2. OnAssociated & onConnected callbacks data can retrieved via the
onStateChanges callback.

Bug: 34221586
Test: ./hardware/interfaces/update-makefiles.sh

Change-Id: Ieee26013dfda7a9be41a80e98ff1537da16b3466

wifi(implementation): Copy the user complete callback by value

The callback object may go out of scope by the time it's invoked. So,
make a copy of it. We're doing the same for the other user callbacks
received in this class.

Bug: 34930275
Test: Compiles
Change-Id: I275bba11ae8735a21514d871e8aa473f92f0fd7d

Bluetooth: Add CHECK and move ASSERT

Test: bluetooth_hidl_hal_test passes
Change-Id: Ib857265204d39ffb54afc6f589fdc1b852362a97

Merge "Update vts files on aosp."

Merge "Add missing response function."

Update vts files on aosp.

* Based on the latest hidl-gen and .hal files.

Test: mma
      make vts.

Change-Id: I4765450d985afd552fe720d91b9ab3422f5b7a47

wifi(implementation): Add status check for all cb invocation

Bug: 34820508
Test: Compiles
Change-Id: I7e7258e782142bae7373fa0543e6c17184b043a9

wifi(implementation): Add global lock

Add a global lock to address synchronization issues between the main
HIDL method servicing thread and the legacy HAL's event loop thread.

Also, added some documentation for the threading model used.

Bug: 34261034
Test: Compiles
Change-Id: I881111814ff5ebd601d6a4c85cf284b30ae47ed3

Merge "[WIFI] Update VTS generated file"

[WIFI] Update VTS generated file

Large churn in files - seems to be mostly driven
by changes in generated code as opposed to HAL
changes.

Bug: 33459306
Test: build + vts-tradefed run commandAndExit vts --module=HalWifiHidlTargetTest
Change-Id: I1762a24108f9119dbc9c6807d6a2870f016a26d9

Add missing response function.

Test: None; only interface change.
Bug: 32020264
Merged-in: I28638f9371a6b09d72862a13a15d5f7e319d9337
Change-Id: I28638f9371a6b09d72862a13a15d5f7e319d9337

Merge "[AWARE] Update capabilties"

Merge "Add tests for minor version upgrade enforcement."

Merge "[AWARE] Add support for Subscribe Service ID List Attribute"

Merge "[AWARE] Add support for SDEA.ServiceInfo field"

Merge "[AWARE] Update HIDL to match spec and add comments"

[AWARE] Update capabilties

Update capability configuration to reflect added/remove fields:
- Added extendedServiceSpecificInfo
- Remove vendor-specific attribute (VSA)

Bug: 35193423
Bug: 35193235
Test: unit tests and integration (sl4a) tests
Change-Id: I0dad08ff9d16ea54f5db7303cc5f90e942f48037

[AWARE] Add support for Subscribe Service ID List Attribute

Add HAL definition to support Subscribe Service ID List Attribute
inclusion in beacons. Until support is added in legacy HAL will not
be functional.

Rationale: enhances likelihood of being able to find the right service.

Bug: 35195516
Bug: 35040192
Test: unit tests & integration (sl4a) tests
Change-Id: I343266e3277c7351f29f6851cb1a9c1746cda6e8

[AWARE] Add support for SDEA.ServiceInfo field

Add support for SDEA.ServiceInfo as extended service
specific information field in all SDF-related APIs:
discovery (configure & match), message sending and
receiving.

Rationale: SDEA.ServiceInfo will form the basis for
upper layer discovery protocols on NAN (e.g. mDNS).
Even if not exposed to user-facing API will need to
have handle to be able to create adaptation layers.

Bug: 35193423
Bug: 35040192
Test: unit test and integration (sl4a) tests.
Change-Id: I3fc225952e93a134d5e58795281238829ed4b1ea

[AWARE] Update HIDL to match spec and add comments

Update some HIDL names to match more closely the spec
naming convention. Add comments - whenever possible
provide reference to NAN spec.

Bug: 34985658
Bug: 35040192
Test: unit tests and integration (sl4a) tests
Change-Id: I957fb8a9814b3acaf8b710d397fe18b39f6a8c77

Remove --coverage in Android.bp

It's no longer allowed now that Soong supports code coverage.

Bug: 32749731
Test: None
Change-Id: I09ef27983e0744543c3a6e1c0c73d69217dd582c

Merge "[AWARE] Modify the discovery session handle to uint8_t"

Add tests for minor version upgrade enforcement.

Test: compiles
Test: add folder test/versioning/2.0 will make update-makefiles.sh to
fail.
Test: removing @2.3::IBar will let update-makefiles.sh to pass.
Test: Change @2.3::IBar to extend from @2.2::IFoo will let
update-makefiles.sh to fail.

Bug: 28825532
Change-Id: Iad1138b784505c903ab1049f08f6c07c38b777e1

Use filegroup as genrule srcs.

Test: mmma hardware/interfaces -j64
Change-Id: If282ed5f4a87045d169fcea7ad648b046a5f1cfb

Updated makefiles

Java intermediates are now created in the common rather than the device-
specific generated sources subdirectory.

Bug: 33842609
Test: make
Change-Id: I16af4cd19bf0b24baa0ad9ebfdb86b4d959c8672

[AWARE] Modify the discovery session handle to uint8_t

Match the NAN protocol.

Bug: 35040192
Test: unit tests + integration (sl4a) tests
Change-Id: I9b39d184708c6e000bd4d84f97659f20d7ec75fd

[AWARE] Remove support for vendor-specific attributes

Vendor-specific attribute support API is not fully functional or
thought out. The Tx portion is working but the Rx portion requires
additional configuration to prevent spamming the framework with all
vendor-specific attributes.

Remove until full solution is designed.

Bug: 35193235
Test: unit tests and integration (sl4a) tests.
Change-Id: I31a601af3b2c1f9f736130ce00abf73c55ac086a

Bluetooth: Disable low power managerment on Hal->close()

This ensures the vendor library can release any wakelocks held and
perform any additional cleanup steps as necessary.

Change-Id: Ibfeac161109162ac375eef9c20f147f0506de602
Fixes: 35097918
Test: manual; disable Bluetooth, monitor wakelocks

Merge "Remove vts driver/profiler build rules."

HIDL change for Android O RIL update

1. Added two new API for screen state replacement.
2. Data call API refactoring work.

bug: 20125037, 32224135, 32223897, 32249926, 30282096, 30173958,
33653220
Test: Telephony sanity tests and unit tests

Merged-In: I6b266637914a85bd57c6a055c52c8ff4ced17e1f
Change-Id: I342b99bed8663d107052e791ac97949b34f7dd99

Remove vts driver/profiler build rules.

Generate filegroups for .hal files so that driver/profiler build rules
can migrate to another project.

Bug: 34893894
Test: mmma hardware/interfaces -j32
Change-Id: Iefc98a6df16706fbb798bc32405fe8ff0c3de450

Merge "Bluetooth: Generate a random address"

Bluetooth: Generate a random address

If a device does not have a factory address set, generate one.

Since this is a bug, add a warning.

Bug: 34952761
Test: Pixel C Bluetooth turns on and off
Change-Id: Icd2ef5c40f0ef6ad93effd9dd662b4b79d3eb0f6

move VTS stub files and non-functional tests (aosp)

Test: mma
Bug: 34976004
Merged-In: Id9c88a9778fba49f0841a474aa067d271f14e207
Change-Id: Id9c88a9778fba49f0841a474aa067d271f14e207

Merge changes Ic4fcbb22,I1be8a18d

* changes:
  Bluetooth: Use AsyncFdWatcher for power management
  Bluetooth: AsyncFdWatcher: Refactor timeout lock

Merge "Bluetooth: Free memory from transmit_cb commands"

Merge "Support driver/profiler for all hals."

Bluetooth: Use AsyncFdWatcher for power management

Implement low power mode using timeouts from the interface.

Test: Boots, Bluetooth turns on/off,
      Instrumented bt_vendor library shows power management
      calls.
Change-Id: Ic4fcbb222f24b66c13f3797b14475d278c49fbc6

Bluetooth: AsyncFdWatcher: Refactor timeout lock

Allow timeouts to change the timeout.
Add a unit test for this.

Test: Unit tests pass, Bluetooth starts and stops.
Change-Id: I1be8a18dd74a312175ec9c27de37213272650e8e

Bluetooth: Free memory from transmit_cb commands

Test: Bluetooth starts and stops
Change-Id: I540fcc77f9bae0178db325b7014f2b839d4ad77d

Merge "wifi: Add scanning mac oui setter"

Support driver/profiler for all hals.

* Update the make file to generate driver/profiler for hals.

Test: mma
Change-Id: I2a91d913b05beb8e1ecf28a205852f848ef1f5ee

vts files for radio hal

Merged-In:I8bce0b3ce4874442638e4967b4b88fcd60497729
Test: mma
Change-Id: I8bce0b3ce4874442638e4967b4b88fcd60497729
(cherry picked from commit 689c6fb1b542dad521272d8c2b8ff823c5b7c062)

wifi: Add scanning mac oui setter

Bug: 34886995
Test: Compiles
Change-Id: I55d4cf55d1832f27de975f97c2e739012ddacc28

Merge "Wi-Fi/NAN HAL/service fixups"

Wi-Fi/NAN HAL/service fixups

Fixes related to debugging during integration with
framework. Changes are due to errors in original
definition.

Bug: 32315992
Test: integration (sl4a) tests
Change-Id: I5c427e4d0f8677a611eb6d9abfa641a7f8406e31

Bluetooth: Move AsyncFdWatcher tests to the device

Darwin builds were breaking.

Test: compile, unit tests pass
Change-Id: Ia00f590fe56211e25dff8d3138ebd93a306cf2c6

Bluetooth: Add a timeout in async_fd_watcher

Add a timeout to the select call and a callback.
Add unit tests for async_fd_watcher.

Test: unit tests pass
Change-Id: I0076fd52e79aac0d2a9fcceb90aae318d5f0757b

Merge "wifi: Fixing Nits"

Bluetooth: Make Send() send a complete packet

Add the packet type to Send() so that it is called once per packet.

Test: Bluetooth starts/stops and scans
Change-Id: I2ef9c5f9a85d3227d4ff181b6a6931f239f75049

wifi: Fixing Nits

Changes in the CL to fix bugs found while integrating
with framework:
1. Added IWifiApIface.setCountryCode()
Bug: 34682168
2. Fixed documentation nits in .hal files.
Bug: 34468218
3. Changed roaming state enum values.
Bug: 34715231
4. Added log for EBUSY in legacyErrorToString()
Bug: 34715469
5. Remove "-Wno-unused parameter" from Android.mk
Bug: 34611289
6. Remove the erroneous |onFailure| callback invocation in wifi.cpp.
7. Add null terminations to strings passed to legacy HAL API's
8. Remove unused |IWifiStaIface.stopDebugPacketFateMonitoring|.

Test: Compiles
Change-Id: If3a3e2f360a378c59dc56b8bbe3a0c6d400b3bd8

Bluetooth: Check for waiting HAL callbacks

Test: Bluetooth boots, turns on/off
Change-Id: Iad2f359a78c9acbf0fe44740ed0813de462ce57c

Bluetooth: Use fixed-size preambles

Test: Bluetooth turns on/off finds devices in Settings
Change-Id: Id640f3dbde3f53b31ce62eccf59bbc8d25130388

Bluetooth: Clean up some messages

Use Info for low-frequency messages.
Print hex for characters.

Test: Bluetooth turns on/off
Change-Id: If1fe20a71287cc91c2a7225488e5d44c7ee4ecee

Merge "Include what you use."

Fixed typos and incorrect types

Fixed typos, incorrect types for bitmap, and comment
alignment.

Test: Build
Merged-In: I51a99fa71877cfe168ee2cf31de702f36f57d013
Change-Id: I51a99fa71877cfe168ee2cf31de702f36f57d013

Merge "Removed TODOs from radio hal."

Include what you use.

Test: mm
Bug: http://b/34740564
Change-Id: I75cf88b79faf68764f1223e39595d0320d7b682a

Merge "Increase the timeout for NfcHidlTargetProfilingTest."

Increase the timeout for NfcHidlTargetProfilingTest.

* Given there's still test failue due to timeout:
  https://android-vts-internal.googleplex.com/show_table?testName=NfcHidlTargetProfilingTest,
  increased the limit to 25m.

Test: run vts -m NfcHidlTargetProfilingTest
Change-Id: Ie5fee6a399281ec538e6724207916908bb8da4d0

Merge "supplicant(interface): Add the new radio work methods"

Merge "NFC: Update VTS test"

use the default binder/passthrough mode in vts tests

Test: mma
Change-Id: I9146d599c623aae1196fcf7e37b7b2715796c748

Merge "update the used precondition var name for IR"

supplicant(interface): Add the new radio work methods

These radio work methods can be used by external programs (wificond in
our case) to request wpa_supplicant to stop doing any offchannel
operations.

Also,
1. Add the setUpdateIdentifier method in ISupplicantStaNetwork.
2. Add the setProactiveKeyCaching method in ISupplicantStaNetwork.
3. Add sendNetworkEapSimGsmAuthFailure &
sendNetworkEapSimUmtsAuthResponse in ISupplicantStaNetwork.
4. Add the setWpsDeviceType() method in ISupplicantIface.

Bug: 34454675
Bug: 32581374
Test: /hardware/interfaces/update_makefiles.sh
Change-Id: I83fe577f6bd39d2daef2cb203ad9a8a93657c988

Removed TODOs from radio hal.

+ Opened new bug b/34683858 for addressing one of the TODO in P.

Bug: 34109705
Test: Removed only comments.
Change-Id: Ie78eacbc80def2df3ec448f14a89fbac185bf2af

update the used precondition var name for IR

Test: mma
Change-Id: Ibec34c82e41d1f93568c29e390a1fc031fa3d3bc

NFC: Update VTS test

- Order of data and CORE_CONN_CREDITS_NTF is not defined, check for the
order in which they were received before checking for contents
- Configuration status is added to WriteCoreReset test
- Add WriteCoreResetConfigReset test

Test: VTS test passes
Change-Id: Ifb730bbb51c366807028fe7e6757c0f8eae943c3
Signed-off-by: Ruchi Kandoi <kandoiruchi@google.com>

Merge "Add host-side code for wifi HAL test"

Merge "bootctrl HAL uses "default" service name"

Add host-side code for wifi HAL test

Test: make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module HalWifiHidlTargetTest
-l INFO

Bug: 34179464
Change-Id: I70c9495a5f2cb0c726373969ada5e33a00cc4c18

Merge "bluetooth: Configure VTS to run the HAL gtest"

Merge "wifi HAL uses "default" service name"

Merge "wpa_supplicant HAL uses "default" service name"

bootctrl HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

In support of b/33844934

Change-Id: I2c0aa199b5649f720710bfd564c077b6ce3acc9c
Merged-In: I43f282ffedf18e4b0817b49fc4860ac39b127d04

Merge "Don't memset &ptr"

IR HAL Target-side test

Test: make vts; vts-tradefed run -m HalIrHidlTargetTest
Change-Id: I6b750f66a0bc207056abcb493693baebc8af81a5
Signed-off-by: Connor O'Brien <connoro@google.com>

bluetooth: Configure VTS to run the HAL gtest

Configure VTS to run target-side functional test

Test: run the test

Change-Id: I1d41113d422ca21ba539b43d22ae5e47facf5fcd

Merge "use the default binder/passthrough mode in NFC VTS test."

Merge changes from topics 'id_str_supplicant_hidl', 'p2p_wps_supplicant_hidl'

* changes:
  wifi(vts): Generate the .vts files
  wifi(interface): Use the "bitfield" type for masks
  supplicant(interface): Add sta network idstr & ft key mgmt
  supplicant(interface): Add missing STA callbacks
  supplicant(interface): Add various WPS device params
  supplicant(interface): Add P2P iface methods/cbs

wpa_supplicant HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

In support of b/33844934

Change-Id: Ib80edecf53adf442e935a2535b8cc6e2e5aefd1e

Don't memset &ptr

Bug caught by running clang-FORTIFY on Android.

Bug: None
Test: Builds
Change-Id: I23ed01cf23bdba74fb141d0b0d61c0da4c750a42

use the default binder/passthrough mode in NFC VTS test.

Test: mma
Bug: 34179753
Change-Id: Id9ac440a53b6256e33d1bfdedcf14caa38670467

Merge "consumerir HAL uses "default" service name"

wifi HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

In support of b/33844934

Change-Id: Ia086509b3433976524bc4a0ac4b24385840ce565

consumerir HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

In support of b/33844934

Change-Id: Ie7394569d20afba0228f01830385eb598795c831
Merged-In: I1b3b021aaa98ce99560126ddd277caa427ddf151

Nfc HAL uses "default" service name

The getService() and registerAsService() methods of interface objects
now have default parameters of "default" for the service name. HALs
will not have to use any service name unless they want to register
more than one service.

Test: builds; verify HAL still works

In support of b/33844934

Change-Id: Id7b76e1df98e1ecacb530a1b5fec76bf14ba6eb7

Merge "Bluetooth  HAL uses "default" service name"

Merge "supplicant(vts): Add gtest framework"