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

Add test for fixing 0XFF (upper X)

Bug: 31983995

Test: make android.hardware.tests.expression@1.0
Change-Id: I05bed0164c5cc988843a60d1efc8f5aeb4986b9b

Merge "Revert "Revert "Revert "nfc: reflect style guide.""""

Revert "Revert "Revert "nfc: update default implementation to match style."""

This reverts commit 22be21e8647a587ee8f2effb57f38c793d90c2c5.

Change-Id: I0534b0eb87623091685fa0e2d94d744d2aeaa4ca

Revert "Revert "Revert "nfc: reflect style guide."""

This reverts commit 106bc9d27a271b316ce3b4237d59a908c23a4ec0.

Change-Id: Ie2e0ad4aeaa58c45bab05234615bd7fa02b84c1a

Merge "Add test for missing package name for FQName lookup."

Merge changes from topic 'rename_wifi_hal'

* changes:
  wifi(interface): Rename interface to iface
  wifi(interface): Add/Correct docstring params
  wifi: Add implementation of driver/firmware memory dumps
  wifi: Add Implementation of IWifiChip.requestChipDebugInfo
  wifi: Use hal_tool & if_tool
  wifi: Restructure wifi legacy HAL implementation
  wifi: Change namespace
  wifi: Move legacy HAL implementation

Add test for missing package name for FQName lookup.

Bug: 31705664

Test: `cd system/tools/hidl && mma`
Change-Id: If3568f1d1e77ad97ccfce5eac5938048f3852560

Add test for vec<vec<T>> as a return value.

Bug: 31955917

Test: hidl_test
Change-Id: I9ae7f9b04126261210cb4621266676727877d92f

wifi(interface): Rename interface to iface

The HIDL objects are called "interface", so rename the wifi interface to
iface to distinguish between them better.

This is especially needed in the follow up CL.

Bug: 31943042
Test: mmm -j32 hardware/interfaces/wifi/1.0/
Change-Id: I3bc34930f75159d26321c0e071af9084d0585508

wifi(interface): Add/Correct docstring params

Changes in the CL:
1. Add missing docstring params in the HIDL interface.
2. Reword some method docs. Occurences of "will" is replaced
by "must" (based on comments received on supplicant HIDL interface:
go/aog/275115).
3. Add onFailure callbacks for all the methods and added docstrings for
them.

Bug: 31352200
Test: mmma -j32 hardware/interfaces/wifi/1.0
Change-Id: Ib7152625851023d6244f742bb77a8fdc0829e00d

Merge "Revert "Revert "nfc: update default implementation to match style."""

wifi: Add implementation of driver/firmware memory dumps

Bug: 31352200
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: I25d42cfd4c2b1ea7e63bdc036f37f81aed93c565

wifi: Add Implementation of IWifiChip.requestChipDebugInfo

Bug: 31352200
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: Id0c02e37dac66de6f830785881cb67f113c0fb19

Revert "Revert "nfc: update default implementation to match style.""

This reverts commit 3768be324bc81b04f4a3a9dfbbe3324dc13cc5f0.

Change-Id: Ifecd3ac227af14b49eba83c1126d51996ce3b8f5

Revert "Revert "nfc: reflect style guide.""

This reverts commit 69bb15e06eea6a0e437d388273c4810682393162.

Change-Id: Icd6ac819606c83e1518c4b6252f6416f55e618fd

wifi: Use hal_tool & if_tool

Changes in the CL:
1. Currently |WifiNative.cpp|, uses |hal_tool| to initialize the
function table and |if_tool| to set the interface up when |startHAL|
method is invoked. Use the same sequence in the HIDL'ized HAL.
2. Remove the assertion if the function table initialization fails. This
will result in a failure indication on starting the HAL now.

Bug: 31352200
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: I268845ed62158b6a2ff36659f0bb15c4100a7222

wifi: Restructure wifi legacy HAL implementation

Restructured the existing code to create a new class called
|WifiLegacyHal|. This class will be used as a wrapper to invoke
all the legacy HAL functions and handle the "C" style callbacks.

Bug: 31936700
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: I63e8543f49886f8446101320a97d1e96e30d1035

wifi: Change namespace

Move the implementation to "android::hardware::wifi:V1_0::implementation"
namespace.
This is following the style guidelines in NFC HIDL
implementation(go/aog/279421).

While there,
Run checkstyle to correct formatting issues.

Bug:31936700
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: I5600a60a0041b3318ed9289823ec335a8ed8a83d

wifi: Move legacy HAL implementation

Changes in the CL:
1. Move the legacy HAL implementation from
"frameworks/opt/net/wifi/wifi_hal_legacy" to
"hardware/interfaces/wifi/1.0/default".
2. Renamed the .rc file and the module name in makefile to
conform to the HIDL style guide.

Files renamed from:
wifi_hal_service.cpp/h -> wifi.cpp/h
wifi_chip_service.cpp/h -> wifi_chip.cpp/h
main.cpp -> service

Bug: 31821133
Test: mmma -j32 hardware/interfaces/wifi/1.0/default
Change-Id: I5e65e2fdb5596346bde6963588031dcea42d633a

Add test for referring to nested types.

Bug: 31821285

Test: `make hidl_test && adb sync && adb shell hidl_test`
Change-Id: Iae030baa5bbe6e890512b494a0b657163c6a99a6

Merge "Revert "nfc: reflect style guide.""

Revert "nfc: update default implementation to match style."

This reverts commit 6b6381cb58aba593951f0718f9beecef0b49e284.

Change-Id: I2891a4ea774ad0047d9f99074fe773808dae0f7c

Revert "nfc: reflect style guide."

This reverts commit e0c416a6f247992089663095f29a9e50cef2ef99.

Change-Id: I3fec772343d88a29225aef61bb549fef8ac9b797

Merge changes from topic 'nfc-style'

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

Auto hardware/interfaces/Android.bp

Bug: 31909444
Test: make various interfaces, mma -j
Change-Id: Ie2de39572e482415b5279110586a0cb7e5220c8e

nfc: update default implementation to match style.

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

nfc: reflect style guide.

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

Convert to Android.bp

See build/soong/README.md for more information.

Bug: 31742855
Test: mma -j
Change-Id: Icb93f3b1b2caeb5e9c00f88ca35841b927ad075d

Tests for Reference Support.

Bug: 31300815 pointer support
Bug: 31349114 nested types
Bug: 31757611 Parcel ref# cap

Test: `make hidl_test && adb sync && adb shell hidl_test`
      Only works with a kernel patch.

Change-Id: I48351b1bf861a4d2d77f823f7242524156cde1e7

Merge "Use MQDescriptorSync instead of MQdescriptor in interface"

Merge "Default binderized implementation of android.hardware.nfc@1.0"

Merge "wifi: Move supplicant HIDL interfaces to subfolder"

Default binderized implementation of android.hardware.nfc@1.0

Provide a default binderized HAL implementation.  The implementation
uses android.hardware.nfc@1.0-impl.so, which in turn loads the
conventional HAL (nfc_nci.default.so).  If a device includes package
android.hardware.nfc@1.0-service, the device will automatically use the
binderized implementation of the NFC HAL.  If a device includes package
android.hardware.nfc@1.0-impl instead, it will use the default
implementation in hardware/interfaces/nfc/1.0/default.

b/31688271
Test: pass

Change-Id: I31cd93d28705d287975b485092cffd7661599d7c
Signed-off-by: Iliyan Malchev <malchev@google.com>

Verify that method arguments and results can share the same name.

Bug: 30778622
Test: hidl_test, hidl_test_java
Change-Id: Idb35a03337010c1fb63c2896f216392398e03e0c

wifi: Move supplicant HIDL interfaces to subfolder

Move all the supplicant HIDL interfaces to a separate subfolder.

Bug: 31800508
Test: Compiles
Change-Id: I09bfb17b626312e8be819238ab2edbb83ca04322

wifi(hidl): Add supplicant HIDL interface

Convert the existing AIDL binder interface of wpa_supplicant to HIDL.

Summary of changes:
1. Every HIDL method returns an instance of |SupplicantStatus|
along with any others params (for getters). This is needed to return the
equivalent of |Binder::Status| to indicate errors in the args passed,
stale proxy, etc.
2. All constants are changed to enums. There were some constants which
should have been enums in the first place, but wasn't because AIDL did
not support enums. But, there are others which should be standalone
constants, but are enums now because constants are not supported in HIDL.
3. Conform to HIDL style guide.

Bug: 31365276
Test: `mmm -j32 hardware/interfaces/wifi/1.0/`
Change-Id: Iba753e279fd260788d8628ea2f5c2281b5844095

Merge "Verify that vectors of arrays within structures are properly marshaled"

Use MQDescriptorSync instead of MQdescriptor in interface

Add comments and cleanup code as per HIDL style guidelines.

Bug: 31550963
Test: Built and ran existing FMQ unit tests and benchmarks.

Change-Id: I0a09ba1fcb520b46e5e6299b473c962c93d815eb

Verify that vectors of arrays within structures are properly marshaled

in the Java backend.

Change-Id: I70baa4ac55d8b0e06b31b2e50662be14b8528ff3
Bug: 31682327
Test: hidl_test_java

Updated indentation to 4 spaces for Android.mk.

Test: `cd system/tools/hidl && mma`
Change-Id: I088893c3c8b080239f0f32e3434faf3da3fefc7a

Updated tests for constant expressions.

Test: `make android.hardware.tests.expression@1.0` compiles
      and generates enum class Color with autofilled values.
Test: `make hidl_test_java` succeeded.
Test: `make hidl_test` succeeded.

Bug: 31628863 Add test for autofilling enum values.
Bug: 31592132 allow constant expressions be array sizes.

Change-Id: I79f35365f1d37843922725ad5a405848f3e8ec08

Updated makefiles that also add static java library targets.

Bug: 31684178
Test: hidl_test and hidl_test_java
Change-Id: Ib53d1be2b1535837ca97cb61796e335141fc7611

android.hardware.nfc@1.0: provide a default implementation

b/31524912
Test: pass

Change-Id: Id9d34f62f4a2b92bdc3beb3e62c89c82743c0ca0
Signed-off-by: Iliyan Malchev <malchev@google.com>

update auto-generated makefiles

b/31524912
Test: pass

Change-Id: Iafea8f87e93181041f42820df39ec43b3c785410
Signed-off-by: Iliyan Malchev <malchev@google.com>

[Java] Add a test to verify that typedefs in types.hal does not mistakenly

generate a makefile rule.

Bug: 31678014
Test: regenerate makefiles using hidl-gen.
Change-Id: I72ceaedd33288269b74e2aa233043c741994cd1a

Merge "Verify that structs with multiple array fields now successfully compile for Java"

Verify that structs with multiple array fields now successfully compile for Java

Previously "_hidl_array_offset_*" was redeclared multiple times in the same
scope.

Bug: 31675085
Test: run hidl_test, hidl_test_java
Change-Id: I370956ae3008c05bda66dc5cdc44d33c9883376f

Initial pieces of WiFi HAL interface definition

Test: make -j48 android.hardware.wifi@1.0 android.hardware.wifi@1.0-java
Bug: 31350762
Change-Id: I1b598be397e08165fc9fd607888e064b139e8007

Merge "Additional test cases to verify multi-dimensional arrays work properly."

Additional test cases to verify multi-dimensional arrays work properly.

Bug: 31438033
Change-Id: I1fc58cbbe6f161f7f71a43fa43d743843ba22f76
Test: run updated "hidl_test" and "hidl_test_java"

Add dependency on libcutils.

* Used for get system properies.

Test: pass test hidl_test.
Change-Id: I3b6d2deb442cd627a6f30312f0c204177d410ecb

Merge "Remove @hal_type from .hal files."

Remove @hal_type from .hal files.

Test: ran make android.hardware.nfc@1.0 successfully
      compiles.

Bug: 31527823

Change-Id: Ib2c2f40ef6a8939a8344ade6138a7b0f32f14ba9

fix update-makefiles.sh to not need Android.mk

When auto-generating Android.mk for HIDL packages, update-makefiles.sh
now just looks for all directories hosting .hal files, converts them to
HIDL package names, and invokes hidl-gen on these package names to
generate the appropriate Android.mk files.

Change-Id: I8a824672f33be58508ae7c07de463f4f589a9a73
Signed-off-by: Iliyan Malchev <malchev@google.com>

Add tests to verify that multi-dimensional arrays in Java work (inside structs)

Bug: 31438033
Change-Id: Ic27868fb0f603ca4acb2a42f144493d6ec26044f

Adds tests to verify proper C++ code generation for multi-dimensional arrays

and vectors of arrays.

Bug: 31438033
Change-Id: Ia7c2198de31bdb4705687726abe3d0371625e56a

Merge "interfaces: regenerate makefiles"

Merge "Java test interface extended to verify vec<STRUCT-TYPE> and STRUCT-TYPE[]"

interfaces: regenerate makefiles

b/31372383

Change-Id: I4fd9d3f174a22f29bc3beb9aeb7c30fc6bd4acd8
Signed-off-by: Iliyan Malchev <malchev@google.com>

Java test interface extended to verify vec<STRUCT-TYPE> and STRUCT-TYPE[]

argument/result passing.

Bug: 31380705
Change-Id: I17fd03a10f9740f68a6fb318a5c2ef9847f4c050

WireMQDescriptor => MQDescriptor

b/31091383 Adding WireMQDescriptor as a hidl type

Change-Id: Iff23c833e8ab232ad37fd9bfbfe4d49fbc06b369

android.hardware.tests.msgq@1.0: modify ITestMsgQ

Modify the configure() method so that the implementation does not have
to rely on status_t to return service errors.

b/31367910
b/31348667

Change-Id: I0bc88c396ce4e55ed38b534b7e3ae2c0a4faad62
Signed-off-by: Iliyan Malchev <malchev@google.com>

interfaces: update makefiles

b/31015466
b/31045569

Change-Id: I07d54fd956fedc4b94622ace1c8f2be3f2f93b2b
Signed-off-by: Iliyan Malchev <malchev@google.com>

Merge "Updated makefiles (includes java-library rules for applicable packages)."

Updated makefiles (includes java-library rules for applicable packages).

Change-Id: I178faf37dddb5149c1bdc217fa1395b2566194d7
Bug: 31221188

Revert "package android.hardware.tests.* as part of VTS (auto-generated)"

This reverts commit afba0364c6fbf7e2fb5018b35b904c968f90ebd0.

Change-Id: I9d143d97ef7857502d88fef92b9ae336ee824bcd

add test for constant expression evaluation output

Simply run:

    # hidl_gen converts .hal to .h correctly
    make hidl_gen_test
    # converted .h can compile
    make android.hardware.tests.expression@1.0

to run the tests.

b/30951879: int8_t is same as char, and when emitting output the
    character corresponding to that number is emitted instead of
    the number itself.

Change-Id: Icacba6b6262cc0026a94f64527b80e2c0e35e72e

Merge "Apache2 module license."

Apache2 module license.

Bug: 31110244
Change-Id: I7ce63f940199e8cd7d193bdde39fc6cd448db802

Interface 'IBaz' and friends used by "hidl_test_java".

Change-Id: If49d996e34ba5aa48488e92e71234208001bfddb

build 32- and 64-bit libs (auto-generated)

Change-Id: Iccb63cafc5db17abd423d589416d50ff21c89efa

android.hardware.tests.foo@1.0::IFooCallback: add instrumentation

Add two instrumentation methods to IFooCallback, to help with timing
measurements in gTest.

The first one, reportResults(), blocks the caller for a given length of
time while waiting for all the other methods to execute, then reports
whether these methods were executed within this length of time, for how
long their caller was blocked, and also how long the execution of the
bodies of each of them took.  Note that the execution time for a
method's body is independent of whether it is a oneway or a blocking
method.  For example, heyItsTheMeaningOfLife() is a oneway method, so it
returns immediately to the caller (so the caller-blocked time should be
very small).  However, if its body takes several seconds to run, then
this is the information that will be returned for
heyItsTheMeaningOfLife() as well.

The second instrumentation method, youBlockedMeFor(), is used by the
caller of IFooCallback to report how long the caller was blocked for.
This information is saved and passed by reportResults() as described
above.

b/30855757 Convert hidl test to gTest

Change-Id: I35ac708e424bcb143fce959609fcc747f1ec37fb
Signed-off-by: Iliyan Malchev <malchev@google.com>

Add test aidl file for benchmark binder performance.

Bug: 31068579
Change-Id: Idf7d5da902eb17f166ac07dc966c009bc1d9c7ae

package android.hardware.tests.* as part of VTS (auto-generated)

this is to run the benchmarks continuously in the lab

Change-Id: Ia45c440a77ccc16db2cb1066ecde3b6521991ec9

regenerate Android.mk files

Change-Id: I6a8b48943df469c9f345c05fb00c671f6f8dd393

Updated makefiles linking against libhidl.

Bug: 30839546
Change-Id: I8c68d0b8db0816f9bfe58fc3898114fb9e151ddc