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

add tests for parsing constant expressions.

Change-Id: Ic1f524c604fc72b9d1b7b330b608ff38ce287921

"import types" should never be necessary, it is implicit (if types.hal exists)

Change-Id: I63e6a2104415a5a71bf3eb93c646f9bf32f1b079

Merge "android.hardware.tests.foo@1.0::IFooCallback: use booleans"

Merge "android.hardware.tests.foo@1.0: add oneway methods"

Merge "Move .hal for FMQ benchmark code to hardware/interfaces."

Merge "Move .hal file for FMQ tests to harware/interfaces"

android.hardware.tests.foo@1.0::IFooCallback: use booleans

b/30853785 auto-generation for bools is broken

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

android.hardware.tests.foo@1.0: add oneway methods

b/30843534: oneway methods seem broken

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

Merge "Added mock interfaces"

Move .hal for FMQ benchmark code to hardware/interfaces.

This is required to convert to the new hidl-gen.

Change-Id: Ia4df37d993be45ac6cb69836fd159dceef10d75c

Move .hal file for FMQ tests to harware/interfaces

This is required to convert to the new hidl-gen.

Change-Id: I87a5d3d006fe80ad16d4abb63d400b3c52a2bae9

Merge "libhwbinder benchmark interface."

libhwbinder benchmark interface.

Change-Id: I2863272033bfe986f70f75c51412b8c1a55a84f3

Add annotation of hal_type for Nfc/NfcCallback interface.

Change-Id: I5bb53c396ea11a94fa556cf561b584b6f12f476c

Added mock interfaces

Change-Id: I227bb450e714e392a3959c0d314842b9879f8b99

script to auto-generate makefiles for HALs

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

android.hardware.nfc@1.0: regenerate Android.mk

hidl-gen \
-Lmakefile \
-randroid.hardware:hardware/interfaces \
android.hardware.nfc@1.0

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

fix the fix

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

fix build

Add hidl-gen as a dependency on HAL modules, and set the -r path
properly in the absence of $TOP being defined.

Proper fix will include auto-generating the makefile correctly form
hidl-gen.

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

nfc interfaces in the new style.

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

Initial empty repository