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

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

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

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"

Bluetooth  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: Icc006a71b41bcd03fe4a04ee92503e76ff07cd77

Merge "Adding all nfc vts related things back."

supplicant(vts): Add gtest framework

Bug: 33457575
Test: adb push
out/target/product/angler/data/nativetest/supplicant_hidl_test/supplicant_hidl_test
/data/ &&  adb shell /data/supplicant_hidl_test
Change-Id: I8b560483166822a33b8d81433178fdbc6077ec11

bluetooth: Add initialization completed callback

Test: bluetooth_hidl_hal_test
Bug: 31972505
Change-Id: I64c19a7fe7299079c5daf74e7b238b0d71d65db2

Adding all nfc vts related things back.

Test: none
Merged-In: I84a42ae27d36a5479f2ca859849bb1b4536bdbad
Change-Id: I84a42ae27d36a5479f2ca859849bb1b4536bdbad

wifi(vts): Generate the .vts files

Generates the .vts files for all wifi HIDL interfaces.

hidl-gen -o hardware/interfaces/wifi/1.0/vts/ -L vts -r
android.hardware:hardware/interfaces -r
android.hidl:system/libhidl/transport android.hardware.wifi@1.0
hidl-gen -o hardware/interfaces/wifi/supplicant/1.0/vts/ -L vts -r
android.hardware:hardware/interfaces -r
android.hidl:system/libhidl/transport
android.hardware.wifi.supplicant@1.0

Bug: 32022435
Test: Compiles
Change-Id: Idb037dafb72e645e5f6618bc909ba02ae7232e77

wifi(interface): Use the "bitfield" type for masks

Change all the variables representing masks to the new bitfield type
in all the Wifi HIDL interfaces.
The generated code will still produce the underlying type of enum as
param. So, no changes are needed in the implementation.

Bug: 33358724
Test: Compiles
Change-Id: Id2e5dee2ce1f1ec8e744aa1c81d3cb9c66e7fc39

supplicant(interface): Add sta network idstr & ft key mgmt

Bug: 34284015
Bug: 32571829
Test: Compiles
Change-Id: Id4d08657a4ed5a76f727e89c19991a65ce7f91e5

supplicant(interface): Add missing STA callbacks

Add a few missing callbacks in ISupplicantStaIfaceCallbacks &
ISupplicantCallbacks.

Bug: 34274091
Test: Compiles
Change-Id: I5d7819f9708f6caa4da80ed481408e376a5636f8

supplicant(interface): Add various WPS device params

Add setters for the various params used for WPS operations.

While there,
Add missing error code in IWifiStaIface method docstrings.

Bug: 34198758
Test: Compiles
Change-Id: I3b4709083975320210c9ed2dff59852372778483

supplicant(interface): Add P2P iface methods/cbs

Add the missing P2P iface methods and callbacks.

Bug: 34221586
Test: Compiles
Change-Id: I9bace8ac5304dd65d81bb255dac023657c9460cc

Merge "wifi: Add capabilitiy flag for ND offload"

Merge "supplicant(interface): Add interface for WPS"

Merge "Add test for type resolution."

wifi: Add capabilitiy flag for ND offload

Bug: 34224734
Test: Compiles
Change-Id: If4e6303fdff792a9dd576551fd84a741d8183798

supplicant(interface): Add interface for WPS

Add the methods/callbacks required for WPS functionality through
wpa_supplicant.

Bug: 34198758
Test: Compiles
Change-Id: Iff1450621a4b039307ad6098f98326c35e53d4c6

Merge "Testing struct that has vector of other struct"