wifi: Make methods deliver status synchronously (3/3)

Make all the |IWifiIface|/|IWifiRttController| HIDL interface
methods return a synchronous status code. Change from using the event
callbacks to the synchronous callbacks for delivering status.

While there,
Use the default std::string to hidl_string constructor in status
callbacks.

Bug: 32056230
Bug: 32061909
Test: Compiles
Change-Id: Ifa06a11afb085bfd6684f7b997fb730d192018ea

wifi: Make methods deliver status synchronously (2/3)

Make all the |IWifiChip| HIDL interface methods return a synchronous
status code. Change from using the event callbacks to the synchronous
callbacks for delivering status.

While there,
1. Use std::tie to retrive values out of the legacy HAL functions returning
a pair.
2. Use the std::vector to hidl_vec constructor for returning vector of
ifnames.

Bug: 32056230
Bug: 32061909
Test: Compiles
Change-Id: Iac27521be17cd9852df04ad7d412e09160a08d33

wifi: Make methods deliver status synchronously (1/3)

Make the following |IWifi| HIDL interface methods return a synchronous
status code:
a) start()
b) stop()
The other methods in this interface do not have a failure case and hence
not returning a status code.

This changes the nature of event callbacks registered for each
interface. Previously, every operation's status was sent to all the
registered event callbacks. Now, only the caller is notified of the
operation's status via the passed synchronous callbacks. The event
callbacks are now used to broadcast only important state changes/events.

Bug: 32056230
Bug: 32061909
Test: Compiles
Change-Id: I95dc3fa139ac3ac7500d81e9e0dbc4f4de04e127

wifi: Rename failure_reason_util to wifi_status_util

The HIDL interface now returns a |WifiStatus| instance to indicate the
status of any operation. This is replacing the existing asynchronous
delivery of success or failure (using |FailureReason| instance).

Rename the existing util class to |wifi_status_util| and add a couple of
methods to create a |WifiStatus| instance with empty description.

Bug: 32056230
Test: Compiles
Change-Id: I8488f7cd7d6ad6bd7a0c3c82a7ef83299d877d45

wifi(interface): Add status for all methods

Add missing status returns for a few HIDL methods. This is to keep the
interface consistent.

Bug: 32146455
Test: update_makefile.sh
Change-Id: Ia66fe4d00e884e5ce1e1906db77bb5ffdaebffdd

wifi(interface): Add wifi host debug wake up reason stats

These stats are used to determine if the device is being woken up
frequently by the wlan chipset. The stats structure describe all the
various resons for which the host was woken up by the wlan chipset
(firmware).

Bug: 32221997
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: Id52915348fef0283affcd834c6f1db5055e54ace

wifi(interface): Add wifi debug packet fate related interface

The packet fate debug mechanism is used to track the state of all the
packets transmitted/received during the association process.

Also,
Add the various debug capabilities in the respective interfaces.
All ring buffer, driver/firmware dump, etc related capabilities are in
IWifiChip object.
The packet fate capability is exposed in IWifiStaIface object.

Bug: 32221997
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: Ic6bf49d682b70a1cdcd18c69fc25a544921bd548

wifi(interface): Add wifi debug ring buffer related interface

The debug ring buffers is a purely debug mechanism to let the driver
report debug info like connection events, power events, etc to the
framework.
The framework used to previously dump out the raw byte stream in
the bugreport and the vendors had some tools to parse out the data.
This is now being changed to provide the framework with the
internal ring buffer structs to ease parsing this data in framework
itself. This will eventually be used in the new wifilogd daemon.

Bug: 32221997
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I2c90662cfa9d07ae6fc72198a286338dbaacbfc2

wifi(interface): Move all StaIface types to types.hal

Some of these structs are used in the debug framework and hence needs to
live in types.hal. Add a |Sta| prefix to differentiate them from other
types similar to what is done for Nan and Rtt types.

Bug: 31991459
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: Idc49b28447609c4334b0a086481c404c06a63fea

wifi(interface): Fix compilation failure

Changes in the CL:
a) Unions are not allowed in the HIDL generated code that is going to be
used in java.
b) Few RTT constants were defined twice in the legacy rtt.h. One set
as enums, the other set as #define's. Remove the duplicated values in
the HIDL interface.

Bug: 31991076
Bug: 31991232
Test: mmm -j32 hardware/interfaces/wifi/1.0/
Change-Id: I03db70c7b89c0be53a7754ab8b34a19ad2c6e536

wifi(interface): Rtt controller HIDL interface

Bug: 31991232
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I53b95e85ee8a71bcb0494420913c9511f87908c0

wifi(interface): NAN HIDL interface

Interface is mostly a replication of the wifi_nan.h HAL header file.
1. All the methods are in |IWifiNanIface|.
2. Moved all the callbacks to |IWifiNanIfaceEventCallback.hal|.
2. Moved all the data types to |types.hal|.
3. Changed all the variable size arrays to vecs.

Bug: 31991076
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I2af0b0003cf30a0f2bfdcb56b69c492cf831745b

wifi(interface): Link layer stats collection

Add HIDL methods for STA iface link layer stats.

While there,
Add the missing gscan API for fetching a list of channels for a given
band.

Bug:31991459
Test: Compiles
Change-Id: I5d448eb823faae4e8f5c25f746cf59e70df454cf

wifi(interface): Make methods synchronous

Having all the HIDL methods asynchronous, makes it hard for the
calling code to keep track of the operation status/result.
There are some operations which will generate asynchronous results (like
bgscan), convert all the other methods to synchronous methods.

The |EventCallback| objects will now just broadcast important events
(needed for other clients to listen for state changes). This will no
longer be used to send responses to every command sent to the HAL.

Bug: 32061909
Test: `./hardware/interfaces/update-makefiles.sh`
Change-Id: Id2433f4c8e028268dd027cdeb239ba4082b157b5

wifi(interface): Add status for every method

Add a status parameter for every HIDL interface object method which can
possibly become invalid. This should help inform the caller that the
object being used is stale/invalid now.

While there,
Rename |CommandFailureReson| to |FailureReasonCode|.

NOTE: |FailureReason| will continue to indicate any errors during the
processing of the command via the corresponding |onFailure| callback.

Bug: 32056230
Test: Compiles
Change-Id: I2ec5af3075221e483579410f344bcedd6bf17a93

wifi(interface): Add gscan/apf related API's

Changes in the CL:
1. Add gscan/APF related API's to |IWifiStaIface|.
2. Add a new callback HIDL interface (|IWifiStaIfaceEventCallback)
for all callbacks received from |IWifiStaIface|.

Bug: 31991459
Test: Compiles
Change-Id: Id9f2ded9e20bee393ab53d84efa814d52704cd2c

supplicant(interface): Fix compilation failure

The generated code uses the variable |code| internally.

Bug: 31116047
Test: mmm -j32 hardware/interfaces/wifi/supplicant/1.0/
Change-Id: Ic6d0c9a198a026460f67fa4920b8003a304f6727

Merge "supplicant(interface): Add support for Hotspot 2.0"

supplicant(interface): Add support for Hotspot 2.0

HIDL interface to
a) Initiate ANQP queries & icon queries for hotspot 2.0
b) Add callbacks for the reception of WNM frames (hotspot
remediation, deauth imminent notice).

Bug: 31116047
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: Ib3353f04b493f50c6415d1f3475efa957d921540

Merge "Run boot HAL service as root"

Merge "Extend Telephony HIDL interface to include unsolicited RIL commands."

Extend Telephony HIDL interface to include unsolicited RIL commands.

Bug:32020264
Test: No test, only interface.
Change-Id: I242cfd3cffb43531025da37d079d7a68b55d7ca3

Merge "supplicant(interface): Support P2P operations"

Update Android.bp to reexport dependent packages.

b/32710416
Test: mma
Change-Id: Iaca7300c366c2c9e860a29e6a111248d315cc5b4

Run boot HAL service as root

The service must run as root to access block devices required by
the existing implementation.

Test: bootctl is-slot-bootable 1
Change-Id: I882cba8ad24943781d5c447b67518acc03efc9a8
Signed-off-by: Connor O'Brien <connoro@google.com>

supplicant(interface): Support P2P operations

Add methods for all the P2P related operations. This is a subset of P2P
functionalities exposed by wpa_supplicant used in the Android codebase
currently.

While there,
Add missing |FAILURE_ARGS_INVALID| status code in the setter methods in
ISupplicantStaNetwork.hal

Bug: 31497295
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I657ef6deaa6203bcf890c43f98a32f6230962d04

Merge "Support genrules with multiple tools"

Support genrules with multiple tools

To allow genrules with more than one tool, rename the tool property to
tools and make it an array, replace $tool with $(location <label>),
and use $() for other variables for consistency.

Bug: 31948427
Test: compare build.ninja
Change-Id: I3d714f70a2af0dc60faeee10e09b6ed166601f1d
(cherry picked from commit b785f5b82ef2efa62ca9d7288cfff64c178080b0)

Merge "supplicant(interface): Remove "wpa_supplicant""

Merge "Extend HIDL interface for Radio (RIL)."

Extend HIDL interface for Radio (RIL).

+ All remaining solicited requests to radio covered.

Test: No test for the interface.
Bug: 32020264
Change-Id: I1ee21c01ec9676919a0e88dadc3be2e21a3459a5

Merge "Fix test for array size orders."

supplicant(interface): Remove "wpa_supplicant"

sed -i "s/wpa_supplicant/the supplicant/g" *.hal

Bug: 32602594
Test: Compiles
Change-Id: Id905cabba29359db8727c131fb64357dfbb104cd

hidl_test: do not use Return for app errors.

Test: hidl_test
Bug: 31348667
Change-Id: Icc7e926bbb886b040fdea0ff9dee8bedad0860e7

Fix test for array size orders.

Test: hidl_test
Test: hidl_test_java
Bug: 31438033
Change-Id: I100d7ed6dac24005fa1a5c46a3e09aa062d957f7

wpa_supplicant(interface): Remove create/remove iface methods from HIDL

Use a new struct(IfaceInfo) to list and get iface objects from the root.

Bug: 32577464
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I912d37f62767980e7f88de5b4439425276de50f8

wpa_supplicant(interface): Add iface/network type

Add separate types of Iface/Network object for STA (station mode) and
P2P mode ifaces and associated networks. These expose very different
functionality and exposing 2 different types of objects makes the
interface cleaner.
Although, they're still represented via the same struct for both types
in the wpa_supplicant core.

Bug: 32553421
Test: ./hardware/interfaces/update-makefiles.sh
Change-Id: I41601a650709429c65014cb7adaf6cb9ab03787e

Merge "Running update-makefiles for fixed spacing."

Add test for null native_handle_t*.

Test: hidl_test

Bug: 30814137
Change-Id: Ic52a4c588e2fde3859a0ae8c098e5cb60a87efe8

Running update-makefiles for fixed spacing.

Test: pass
Change-Id: I4dcaf70b1c014f3ecf685d4884601be6f17ee51d

Add method to IFoo for testing NULL interfaces.

Bug: 32410513
Test: mma, hidl_test
Change-Id: I791c3a95e670a859e5e494ead987d2a93f5047a5

Merge "Initial version of BT SAP hal."

Add interface methods for testing vec<handle>.

Test: hidl_test
Bug: 32160974
Change-Id: I7eda1650fcf8d58778b2aad97a5bd62afc95bc36

Merge "add test for binder inheritance."

Merge "NFC HAL doesn't need readproc."

Initial version of BT SAP hal.

Test: No build issues. Interface only; nothing to test yet.

Bug: 32020264
Change-Id: I8ed46b5e39c37a9efe1593206f782a591f47c9b7

NFC HAL doesn't need readproc.

Bug: 31928447
Test: NFC still works
Change-Id: I9f43192c33f3fdf91e4aa97d4c5e5fa49a01a85e

Develop HIDL interface for radio (RIL).

Test: Only built the hidl interface as there isn't anything else to test.

Bug: 32020264
Change-Id: Ibf62feb9c562992cffb9ba6d1ec85a7e9c22f359

add test for binder inheritance.

Test: hidl_test
Test: `make hidl_test_java` compiles
Change-Id: Ie9217b79e369f2a1e7a7c9ed02524b17e3fc8b7f

Merge "Add methods to test flavor of FMQ supporting unsynchronized writes."

Stop all these merge conflicts.

Test: mma -j64
Change-Id: I319198af32adf1fef180c5c2c22ba4b360fa7609

Use hidl_string conversion constructor as test.

Test: hidl_test
Change-Id: I626c40e1a3121e4f41ba9f612efc7321f7c03f85

Merge "boot: use defaultPassthroughServiceImplementation"

hidl_test: move duplicated code to shared lib

Test: hidl_test

Change-Id: Ia68b78d44d68a6b254c686afe64faf7cc89510d7

Merge "Add implementation for hwbinder benchmark service."

Update headers (for export shared lib headers)

Test: mma -j64
Bug: 32337293
Change-Id: I93e7cab90efac407525a1a5f2ad30f816a530e17

Merge "wifi: Removing reference to IServiceManager.h."

Removed overloaded functions from IFoo.hal.

Bug: 31758541
Test: hidl_test
Change-Id: Ic92cf9996357a8aa9b3785540784c686887032e0

Add implementation for hwbinder benchmark service.

Bug:32279499
Test: make android.hardware.tests.libhwbinder@1.0-impl
Change-Id: I75056afa32b8189020608779ec495659ea35ed48

boot: use defaultPassthroughServiceImplementation

Bug: 32282345
Test: make
Change-Id: Ifb479c665aa4e37881c86152d43f4f1f88983133

wifi: Removing reference to IServiceManager.h.

Bug: 32313592
Test: make

Change-Id: If1b842c563b5934b58fb328c2f2315c42ae6fc06

Merge "Adds tests to verify proper marshaling of vectors of interface types."

Merge "Use defaultPassthroughServiceImplementation for nfc."

Merge changes from topic 'boot-control-hal'

* changes:
  Add android.hardware.boot_control@1.0 service.
  Add boot_control HIDL default implementation
  boot_control: Define the boot_control HIDL interface.

Adds tests to verify proper marshaling of vectors of interface types.

Bug: 30570663
Test: hidl_test
Change-Id: I1e4d1f826773abbde6b9019eb6af90ac64007e38

Add methods to test flavor of FMQ supporting unsynchronized writes.

Test: Built and ran the unit tests.
Bug: 31223183

Change-Id: Ib0c52c4d17d31a24f4c5f5cd96547bcfae673725

Use defaultPassthroughServiceImplementation for nfc.

Bug: 32282345
Test: nfc
Change-Id: I4f45a3d4b91ccee2fcd0ca8fe599e090129a02e3

Add android.hardware.boot_control@1.0 service.

Bug: 31864052
Change-Id: I982077d71a81c0bc8d081f0cc7e83180b97f2b87
Signed-off-by: Connor O'Brien <connoro@google.com>

Add boot_control HIDL default implementation

Create basic implementation that passes commands
through to the old HAL implementation.

Bug: 31864052
Test: Ran and compared output to old implementation
Change-Id: I01f4450dc3a1893e13b8fb325ea40cf9c98297be
Signed-off-by: Connor O'Brien <connoro@google.com>

boot_control: Define the boot_control HIDL interface.

Bug: 31864052
Merged-In: I9222b140ff51c4d3846b86b03719e2c8e9b7fe47
Change-Id: I9222b140ff51c4d3846b86b03719e2c8e9b7fe47

Merge "Adds a test to verify proper marshaling of vectors of arrays in Java"

Adds a test to verify proper marshaling of vectors of arrays in Java

Bug: 32180328
Test: hidl_test_java
Change-Id: I756ce1d2ce35b7a3a63c6dbafba90e9765d61f64

hidl_test: Fix failing tests.

Test: The following succeeds.
croot && cd hardware/interfaces
mma
croot && cd system/tools/hidl
mma
adb shell /data/nativetest64/hidl_test/hidl_test

Bug: 31819198

Change-Id: I40c2338d09d5e80b9bad2d918b7b8c3f2709f434

add default implementation to android.hardware.tests.foo/bar/pointer@1.0

Test: hidl_test
Test: mma

Merged-In: I808423c040099354513db0ad108210fcec17c8a7
Change-Id: I808423c040099354513db0ad108210fcec17c8a7

Merge "Revert "Remove test for overloaded function.""

Revert "Remove test for overloaded function."

This reverts commit 93800287fb5cf07657d9056bbcdd3f96ce432433.

Change-Id: I95b0aeb64eaf9938c6558e5817bd79c3f2e5496e

Merge "Remove test for overloaded function."

Remove test for overloaded function.

Bug: 31758541
Test: hidl_test
Change-Id: Ib02d9e8531ea2b7ec36bf8785e6827cfa45b53ba

Add test for single letter IDs.

Test: mma

Bug: 32068694
Change-Id: I56659715beaadd8ebb3b9f36558e7ebcdb3e4d27

Merge changes from topic 'nfc-style'

* changes:
  nfc: update default implementation to match style.
  nfc: reflect style guide.

Modify return values of read/write/config methods to bool

Modify return values of read/write/config methods to bool
for the FMQ unit tests and benchmarks.

Bug: 31780470
Test: Built and ran existing FMQ unit tests and benchmarks

Change-Id: I2b7c9cf073f0c1a2908d48552e258ce21240ae64

nfc: update default implementation to match style.

Test: make android.hardware.nfc@1.0-impl
Change-Id: I00357d40df1a6b2289f86b33d7a5833e358884d6

nfc: reflect style guide.

Test: make android.hardware.nfc@1.0
Change-Id: I19b5c5a8b7e6345152441be149349933774b3de0

Merge "Cleanup nfc service code."

Add Bluetooth HAL definition

Test: compiles
Bug: 31972505
Change-Id: I7608b0814c9ca697b21c9e115cfcd81ab1364892

Cleanup nfc service code.

Test: mma
Change-Id: Ie65c5bfde292107bec567a297e86eebf5cd7fa41

Merge "Added test for overloading."

Added test for overloading.

Test: make, hidl_test
Bug: 31758541
Change-Id: Ia0fdad2f0b0155065a7dec5526ab5d14a9ec1f52

wifi: Invoke failure callbacks

Invoke the failure callbacks for |requestChipDebugInfo|,
|requestDriverDebugDump| and |requestFirmwareDebugDump| methods.

Bug: 32014425
Test: Compiles
Change-Id: I49b8ae93fe846df6d9663b10d2a70eb89bd00931

wifi: Add WifiRttController object

Create the WifiRttController object and add the createRttController
method in WifiChip.

Bug: 31991232
Test: Compiles
Change-Id: I34649097c96488c660bef5acceac1a6478f80dca

wifi: Add Iface related method implementations in WifiChip

Add the iface creation/retrieval/list method implementations in WifiChip.
WifiChip does not yet support the chip mode configuration. So, all
|createIface| method calls succeed today. Once the mode configuration
changes are in place, we will deny iface creation requests depending on
the iface combination supported in the current mode.

The iface objects are all invalidated and deleted on mode switches and
when HAL is stopped.

While there,
Change the |callbacks_| member to std::vector instead of std::set. We
don't really need a set to store the list of callbacks.

Bug: 31943042
Test: Compiles
Change-Id: Ic94864c8b11aabbe569b5157d68d6056364a3c4c

wifi: Add the various iface object implementation

This CL adds the implementation classes for the various iface HIDL
interfaces defined. These classes implement the
|getName| & |getType| methods in the Iface HIDL interface.

Integration of these objects with WifiChip class is implemented in the
next CL.

Bug: 31943042
Test: Compiles
Change-Id: I219afd6441e15c76cf7c61f7bcd2b3568dba2b98

wifi: Remove "Wlan" prefix from wifi_legacy_hal public methods

All the legacy HAL API's in the function table uses the "wlan0"
interface handle for the various operations. But, this is an internal
detail that should be abstracted inside WifiLegacyHal class. So, rename
the public methods to remove the "Wlan" prefix from them.

Also, add methods to fetch the iface names to use for the various types
of HAL.

Bug: 31943042
Test: Compiles
Change-Id: I35a6cdea0ad7cff295d33c0245953258129fba43

wifi: Add support for multiple chips in IWifi

The modified HIDL interface supports multiple chip instances on the
device. Modify the |IWifi| interface implementation to support the new
methods.

NOTE: The legacy HAL implementation will continue to only expose 1 chip.

While there,
Change the |callbacks_| member to std::vector instead of std::set. We
don't really need a set to store the list of callbacks.

Bug: 32003988
Test: Compiles
Change-Id: I31e704100f716e223095890279bdf93d2a04377e

wifi(interface): Add RTT Controller object

Create a RTT controller object to use for initiating all the RTT related
HAL opertations.
Since we don't want to fix the iface on which to initiate RTT operations,
these methods will be rooted in a new standalone object |RttController|.
Framework can decide to intiate an instance of |RttController| on a
specified iface or let the implementation pick one.

Bug: 31991232
Test: Interface compiles.
Change-Id: I65f7a7babd72db26ce6549f572abd9ef73700c82

wifi(interface): Add Iface objects

Create a child object under IWifiChip to represent each interface
within the chip. Each iface object has a |type| & |ifname| which should
help us uniquely identify them. This should help us expose methods that
are applicable only to a specific interface type.

While there,
Assign a unique id to every chip on the device.
Add IWifi.listChipIds() to retrieve the list of chip Id's avaiable on the
device. IWifi.getChip() will now use the provided Id to retrieve the
corresponding IWifiChip object(because HIDL language doesn't support
vec<HIDL objects>).

Bug: 31943042
Bug: 32003988
Test: Interface compiles (not implementation)
Change-Id: I723007566ca4220362c02d0f452753fee4e31fce

Merge "Initial version of radio hal"

Initial version of radio hal

Test: No build issues. Unused interface; nothing to test yet.

Bug: 32020264
Change-Id: Ie57eb43b777dadbc0b0350f2e33227be3f1d65f1

Merge "Add test for importing a single type."

Add test for importing a single type.

Test: `cd system/tools/hidl && mma`

Bug: 31821285

Change-Id: I0be97b16b84f76b64c09f9dc52709416601cbbdf