Merge "Add more VTS tests for ISupplicantP2pIface" into oc-dev

Merge "Fix failure to load power HAL implementation" into oc-dev

Merge "Add more VTS tests for ISupplicantNetwork" into oc-dev

Merge "sound trigger HAL: remove dummy Union" into oc-dev

Merge "Fix VTS test for SupplicantStaNetwork::setGetEapPhase2Method()" into oc-dev

Merge "Add VTS tests for ISupplicantIface" into oc-dev

Merge "Add more VTS tests for ISupplicantNetwork" into oc-dev

Merge "Added protections for RenderScript VTS tests." into oc-dev

Add more VTS tests for ISupplicantNetwork

This covers the following APIs by adding tests to
ISupplicantStaNetwork:
ISupplicantStaNetwork::setPsk(),
ISupplicantStaNetwork::getPsk(),
ISupplicantStaNetwork::setEapSubjectMatch(),
ISupplicantStaNetwork::getEapSubjectMatch(),
ISupplicantStaNetwork::setUpdateIdentifier,
ISupplicantStaNetwork::SetProactiveKeyCaching
ISupplicantStaNetwork::SendNetworkEapSimUmtsAutsResponse,
ISupplicantStaNetwork::SendNetworkEapSimGsmAuthFailure,
ISupplicantStaNetwork::SendNetworkEapSimUmtsAuthFailure,
ISupplicantStaNetwork::GetWpsNfcConfigurationToken

Bug: 33457575
Test: VTS tests pass

Change-Id: I7d990471aadc46e7d553f2179d6d234530cb1df6

Merge "Fix VTS test for SupplicantP2pIface::stopFind()" into oc-dev

Add more VTS tests for ISupplicantP2pIface

This covers the following APIs by adding tests to
ISupplicantP2pIface:
ISupplicantP2pIface::SetGroupIdle(),
ISupplicantP2pIface::SetPowerSave()

Bug: 33457575
Test: VTS tests pass
Change-Id: I8291735e74cf88c0e1ee4b5297b39aa37c33d17c

Fix failure to load power HAL implementation

Some working implementations of the libhardware power HAL do not
implement an open method. Change the HIDL implementation to not rely
on that method being available.

Test: Flashed marlin with modified power.marlin that omits an open()
implementation; power HAL loads successfully and passes VTS
Bug: 37245218
Signed-off-by: Connor O'Brien <connoro@google.com>
Change-Id: Icc77b3011a0e44330062aa51ae49e39ad349a2a5

Merge "Copy wifi_hidl_call_util.h for supplicant VTS test" into oc-dev

Add more VTS tests for ISupplicantNetwork

This covers the following APIs by adding tests to
ISupplicantStaNetwork:
ISupplicantNetwork::getInterfaceName(),
ISupplicantNetwork::getType()

Bug: 33457575
Test: VTS tests pass
Change-Id: I9a24195cefefcc23b7603ff33b66a497eba07148

Add VTS tests for ISupplicantIface

This covers the following APIs by adding tests to both
ISupplicantStaIface and ISupplicantP2pIface:
ISupplicantIface::getName(),
ISupplicantIface::getType()

Bug: 33457575
Test: VTS tests pass

Change-Id: Ibbc4be2e367fc6f179005696724cccbcca58d10f

Copy wifi_hidl_call_util.h for supplicant VTS test

This copies wifi_hidl_call_util.h to supplicant cts directory
so supplicant vts tests can use this to simplify code.

Bug: 33457575
Test: compile, manual test
Change-Id: I696475062d04d54fafae5c3323818ce1d1ff7d9d

sound trigger HAL: remove dummy Union

Remove dummy Union definition added as work around for JAVA build
problem.

Bug: 34109137
Test: make
Change-Id: I5a8107de2deab1f4f62c3e0ae654dd768cf0e307

Merge "Update Radio and Sap service name in VTS." into oc-dev

Fix VTS test for SupplicantStaNetwork::setGetEapPhase2Method()

EAP method needs to be set to construct the eap phase 2 method string.

Bug: 37220169
Test: vts tests pass
Change-Id: Ie69aaac0826d4bde784f864631a718242dd5af2a

Fix VTS test for SupplicantP2pIface::stopFind()

stopFind() returns SUCCESS even when there is no ongoing
find() operation.

Bug: 37220169
Test: vts tests pass
Change-Id: Ia3f3b4e0c4e7fb7263c1bcbfda1a40b6e5f1cba2

Merge "graphics: revise gralloc interfaces" into oc-dev

wifi: add VTS tests for IWifiApIface

Bug: 32022435
Bug: 36813739
Test: run VtsHalWifiV1_0Target test
Change-Id: I31db3b1bfe5deed5691e0d342762298b4b0d3620

Merge "Support binderized and passthrough vendor services" into oc-dev

Merge "wifi: add VTS tests for IWifiStaIface" into oc-dev

wifi: add VTS tests for IWifiStaIface

This will cover all APIs except:
- startBackgroundScan
- stopBackgroundScan
- startSendingKeepAlivePackets
- stopSendingKeepAlivePackets

Bug: 32022435
Bug: 36813739
Test: run VtsHalWifiV1_0Target test
Change-Id: I71a1a97e4b0f1145657b249f3eb20305f810962b

graphics: revise gralloc interfaces

Revise IAllocator and IMapper to reduce IPC and to support gralloc0
devices.

Specifically, IAllocator is trimmed down to have essentially only

    allocate(BufferDescriptor descriptor, uint32_t count)
        generates (Error error,
                   uint32_t stride,
                   vec<handle> buffers);

The ability to allocate buffers with shared backing store is
removed.  ProducerUsage and ConsumerUsage are moved to the
graphics.common package and are merged and renamed to BufferUsage.
BufferUsage's bits follow gralloc0.

IMapper gains

    typedef vec<uint32_t> BufferDescriptor;
    createDescriptor(BufferDescriptorInfo descriptorInfo)
          generates (Error error,
                     BufferDescriptor descriptor);

where BufferDescriptor is an implementation-defined blob.  lockFlex
is replaced by lockYCbCr.  All getters are removed.

Reference counting with retain/release is replaced by
importBuffer/freeBuffer.

Most if not all gralloc1 features are not used by the runtime yet.
There is also not too much test written for them.  As such, they
tend to behave differently between implementations and cannot be
used reliably.

Bug: 36481301
Test: builds and boots on Pixel
Change-Id: I1d31105120517ea2c128c7a19297acf3bfd312bb

Merge "Fix event listener timeout issues" into oc-dev

Update Radio and Sap service name in VTS.

Bug: 37164398
Test: VTS tests pass.
Change-Id: I71562e5b9709166695d08a96a04928027dcb4ee7

Merge "audiohal: init and send all offloadinfo related element" into oc-dev

graphics: Update RAW16 format comments

RAW16 format could be used together with depth dataspace.
The buffer contents in this case will be phase detection
statistics data.

Bug: 36015382
Test: testPDStats
Change-Id: Ied1c179193abef62a87fcec6f0906528ed58ea2b
(cherry picked from commit 23699d63143fd9d47f8584c6627e5c38838b333f)

Merge "[WIFI][VTS] Don't call RequestDriverDebugDump API if not supported" into oc-dev

Merge "[AWARE][VTS] Update WifiChipHidlTest NAN tests" into oc-dev

Merge "Log errno" into oc-dev

Support binderized and passthrough vendor services

Previously only the vendor service name was used
to get the drm and crypto services, which would work
either for binderized or passthrough devices, but not
both. With this change, the vendor service name is
tried first, and failing that the default passthrough
service is used.

Change-Id: I05e9552da992e8d9dbba0f8dde485406262410a9
related-to-bug:34178477
Test: drm vts test

[WIFI][VTS] Don't call RequestDriverDebugDump API if not supported

The RequestDriverDebugDump API has a capability bit indicating if
supported on device. Calling if not supported should additionally
return ERROR_NOT_SUPPORTED - but does not today. This is arguably
not a valid issue - should not call API if already known not to be
supported.

Remove test for this (questionable) condition until decide on proper
semantics.

Bug: 36562856
Test: test passes
Change-Id: Ic8a1c8b5f77a1b1767c0efc9735731084dfde7cc

audiohal: init and send all offloadinfo related element

- init all offloadInfo element in audioOffloadInfoFromHal

Bug: 37158321
Test: make
Change-Id: I59d2f6dcd63f95a44465ee8ce8767351fae80521

[AWARE][VTS] Update WifiChipHidlTest NAN tests

NAN is not enabled on O - update all WifiChip NAN tests to expect
failure.

These tests will start failing again on any platforms/code where
NAN is enabled. Will fix more comprehensively at that point.

Bug: 36562856
Test: build + run vts
Change-Id: I0eaf7934d327fb91bff59dd33bd4a8069888c967

Merge "Add command line strictness to VTS" into oc-dev

Merge "Adds default keymaster service to the early_hal class" into oc-dev

Merge "Fix transitive includes." into oc-dev

Log errno

Bug 36790737

Test: ensure errno is logged
Change-Id: Ifa9a0b6ee0baebfdfce894622ef40888dd83dddf

Merge "Empty interface IQuux which is completely unrelated to IBase and IBaz." into oc-dev

Fix event listener timeout issues

The drm vts event listener tests would hang if event
callouts failed to work as expected. Use the vts test
callback template instead.

Change-Id: I5234f5760d90c1e884badc25cd251da27c2c672d
related-to-bug:34178477

Fix transitive includes.

Files relying on transitive include of utils/Log.h (and things that it
includes) from MQDescriptor.h

Test: pass
Merged-In: Iff316b21bef556bb026378b7f89e97ded3febef4
Change-Id: Iff316b21bef556bb026378b7f89e97ded3febef4

Merge "Fix hidl_string copy in boot VTS test" into oc-dev

Merge "Add additional drm hal tests" into oc-dev

Merge "Use explicit .c_str() for hidl_string" into oc-dev

Add command line strictness to VTS

Was compile time, making command line for
easier test team usage of the stronger
test you can do with GNSS signal available.

Also ensuring location calculated correctly after
a delete all (when -agps present), and faster tests.

Bug: 36356863
Test: Verified command line switches work as expected
  on a Pixel

Change-Id: I4aa92ff2ca664b1c93bbc9ac154d27063160acf2

Merge "Correct comment on getProvisionRequest" into oc-dev

Add Flag for Enabling 64-bit Legacy DRM Plugins

We are ready for devices to begin migrating to a 64-bit Legacy DRM
Loader. However, not all devices are ready to make this jump yet. A
device needs to have all of its Legacy DRM Plugins ready to run as
64-bit before it can do this. This includes having 64-bit hardware
integrations.

So that devices can turn on 64-bit support as they get ready, I am
adding flags that can be set in a device's device.mk file. There are two
parts:

* ENABLE_MEDIADRM_64 enables building the Legacy DRM Plugin HAL as
  64-bit.

* Setting drm.64bit.enabled=true in PRODUCT_PROPERTY_OVERRIDES switches
  the Legacy DRM Plugin HAL to searching for DRM Plugins in the 64-bit
  directory.

Test: Validated that turning on these flags enables a 64-bit Legacy DRM
      Plugin HAL that loads 64-bit Legacy DRM Plugins from the lib64
      directory.
Bug: 36076017
Change-Id: I8d21fc7e7c0357eb6755839d41caa0f37ca20aff

Merge changes from topic 'vhal-def-refactor' into oc-dev

* changes:
  Extract logic of storing values to separate class
  Fix an issue where Vehicle HAL would crash when trying to update a continuous property without an initial floating-point value
  Refactor default values in VHAL

Add additional drm hal tests

Test: VTS tests passing

bug:34178477
Change-Id: Ie8fa5f2ad193b717b0564bb3046de8a64ccd8d85

Merge "Bluetooth HAL: Add to foreground schedtune group" into oc-dev

Extract logic of storing values to separate class

Also rename DefaultVehicleHal to EmulatedVehicleHal and
extract emulator related code to VehicleEmulator class

Test: mm -j32 ; verified Car Service worked

Change-Id: I34361fdec6f94629cf7ef6c35ff56ef9ce78b855

Fix an issue where Vehicle HAL would crash when trying to update a continuous property without an initial floating-point value

Bug: 36858992
Test: build, flash and boot

(cherry picked from commit 945d0bbc866ce21f262a1715c5a2560898c8aca1)

Merged-In: I9906abbf386cf75e838102299e3e97de3475db63
Change-Id: I19ef534fde190a3a1adaca377e8f09760c0ccf4e

Refactor default values in VHAL

Move default values closer to config declration to add properties
with less pain in future.

Test: mm -j32; verified if Car Service works

(cherry picked from commit 24ade1751811742948b8ce5177f57f873a4b1b96)
Merged-In: I2f18837658149eb657f44d81e34eb8e6e497a25c
Change-Id: Ie4285581fa9d871b366b0ed2f08fa8073739a0a9

Adds default keymaster service to the early_hal class

The keymaster service is required by vold which starts very early
during boot up. This leads to the dependency loop:
init->installkey->vdc->vold->keymaster-service->init

This patch resolves this issue by adding service keymaster to the class
early-hal which will get started right after hwservicemanager on devices
that get treble.

Bug: 35764921
Test: Device boots. And lshal shows that service keymaster has two
      clients, keystore and vold.

Change-Id: Id0b6722de900b5513d46adb81d76b02fa6e155df

Added protections for RenderScript VTS tests.

Changed many EXPECT_NE to ASSERT_NE for objects that are required later
in the test which would potentially cause a segfault. Additionally updated
a few descriptions.

Bug: 35915961

Test: mm and run on angler
Change-Id: I17c85b2c774688d82b59be2f0b383447e03349e0
(cherry picked from commit b2039ad7643d024a5541af33da574068f194ee81)

Fix hidl_string copy in boot VTS test

hidl_strings returned through a callback need to be copied; the
current approach just saves a pointer to the underlying buffer, which
is incorrect and sometimes causes the test to fail.

Test: the GetSuffix test now passes reliably
Change-Id: I4bb143be98b00b20f9da4aebef3b5ab4980b7690
Signed-off-by: Connor O'Brien <connoro@google.com>

Merge "CameraProvider: Refactor legacy name regex code" into oc-dev

Correct comment on getProvisionRequest

There is no session ID in the parameter list.

Test: Compiles, it's a comment change

bug: 36735099
Change-Id: I0eccfcb3aa312a694b391c0b5a76e3b8425a7d36

Merge "USB: HIDL: check the status of registerAsService" into oc-dev

Merge "wifi(implementation): Print out the cb object on failure" into oc-dev

Merge "Fix DRM prebuilt vendor module handling" into oc-dev

wifi(implementation): Print out the cb object on failure

This should help us figure out if there are multiple callback objects in
wifi_chip when we see the IPC storm from
onDebugRingBufferDataAvailable().

Bug: 36599054
Test:  ACTS wifi tests.
Change-Id: I8c67032c59ecdd45824e131b59d890ce6c94fe25

wifi(implementation): Ignore IE parsing error

The IE blob may not always be fully formed in gscan full scan results.
So, log errors when we hit a parsing error and return the IE elements we
could parse so far.

Bug: 36455164
Test: ACTS wifi gscan tests.
Change-Id: I804937ea5e69f51696aaabf3fe3f53850c145ad3

wifi(implementation): Return const-ref for callback set

Bug: 36792787
Test: Compiles
Change-Id: Ibd19c6f41801d0fa1ea7dc9c3334213865d649ec

CameraProvider: Refactor legacy name regex code

The match will use iterators that will be invalidated when the
used string input goes out of scope. Refactor the API to return
the two components of the match.

Bug: 36871785
Test: m && m SANITIZE_TARGET=address
Change-Id: I2418541234a09e53856ea61e0fbc4b67d8036308

Bluetooth HAL: Add to foreground schedtune group

EAS (Energy-Aware Scheduler) tends to cluster tasks together unless
they're marked to prefer an idle CPU (prefer-idle) during placement.
This is to save energy however it increases latencies. This is
noticeable when the Binder driver demotes the RT task to CFS and when
we try to do load balance in this path, the selection path still selects
a busy CPU. Add Bluetooth HAL service to the foreground schedtune group
to fix this as its critical for tasks like bluetooth audio playback.

Test: Boot and verify bluetooth HAL is in the foreground stune group
Bug: 36867985

Change-Id: Ib3c43cca4a2049af3700978b58a05989cc20b6ac
Signed-off-by: Joel Fernandes <joelaf@google.com>
(cherry picked from commit 1b8f6f7c771a7ff04790833674705972009a3c7d)

Fix DRM prebuilt vendor module handling

bug:36817233
Change-Id: I93659a9901198d05fd43bf95804067a244533808

removed log print statements for configstore from SurfaceFlingerConfigs.cpp

Log messages for the retrieved values from configstore will be
automatically printed at the client side (by configstore-utils).

Bug: 36275627
Test: Built and check the log messages
Change-Id: Id5033a74ef58778e93e39be6ac5b9e10ee28c5cf

print log message for values retrieved from configstore

This CL adds a code for printing log messages in clients-side utility
library functions (getXXX) for configstore so that the developer can
figure out which configuration values are actually retrieved from the configstore.

Bug: 36275627
Test: Built and check the log message appears

Change-Id: I2c0895f8afbbb2947b62164acaf62a491c451dc0

Merge "Add DeleteKey and DeleteAllKeys test to Keymaster VTS tests" into oc-dev

Merge "Revert "camera: Map between legacy gralloc0 usage, and HIDL gralloc1 usages"" into oc-dev

Empty interface IQuux which is completely unrelated to IBase and IBaz.

Used to verify that it's not possible to obtain an IQuux interface from an
IBase/IBaz binder, i.e. IHwBinder.asInterface(...) properly respects the
interfaceChain.

Bug: 36749201
Test: hidl_test_java
Change-Id: I84419e78db22acdfcc7d9972d22d139641469bde

USB: HIDL: check the status of registerAsService

registerAsService call could fail. Hence check the return value.

Bug: 36704362
Test: Manually checked to see if usb hal service is running.
Change-Id: I46dcd74dbee6aba72cf344f10854e5d4b1b6de18

Revert "camera: Map between legacy gralloc0 usage, and HIDL gralloc1 usages"

This reverts commit 2d80c0d66f9ca869d6102f07bd236b7c4c94c205.

Test: fix b/36876770
Bug: b/36876770
Change-Id: Ia251ec97273d71fd5bc759aa1161960f6266358a

Merge "[AWARE] Create true translations between legacy and hidl enums" into oc-dev

Merge "Camera: HAL1: remove libbinder usage" into oc-dev

Merge "Enable the clang-format pre-submit hook" into oc-dev

Merge "Clarifying broadcast vs. predicted orbits" into oc-dev

Use explicit .c_str() for hidl_string

hidl_string no longer will provide an implicit cast to const char* as it
interfers with other expected behaviors of the class.  It now emulated
std::string in requiring a call to .c_str() to get the same behavior.

Bug:  36532780
Test:  Build the tree
Change-Id: I265590c7c18f425e44863df062a56b67519a932a

[AWARE] Create true translations between legacy and hidl enums

Replace implicits translations between legacy and hidl enums (using
casts) with explicit translations.

(cherry-pick of commit 284511779ca6c425a4f701ca8098484d954e13c4)

Bug: 35767620
Test: integration (sl4a) tests
Merged-In: I4ae4cd691226ce29bc0e92f9890fbb343a064505
Change-Id: I4ae4cd691226ce29bc0e92f9890fbb343a064505

Merge "nfc: vts: Correct parameters going for CoreInitialized" into oc-dev

Merge "Remove dependency to libbinder in VHAL" into oc-dev

Merge "Test for getHashChain() in IBase." into oc-dev

nfc: vts: Correct parameters going for CoreInitialized

Core Initialized with parameters 1-3 leads the device into recovery
mode. Which causes the test to fail.

Bug: 36751759
Change-Id: If2b91237a445ce050b7da6be5f97dd8eea30046e
Merged-In: I1375546cc66c147f26c315a042ca0afb49f0ce06
Test: Test passes

Merge "Camera: add interface to evict obsolete buffer caches" into oc-dev

Merge "add components necessary for bootanim to animation class" into oc-dev

Remove dependency to libbinder in VHAL

Also, removed per property ACL as it was based on client UID

Test: mm -j ; verified Car Service works

Bug: b/36649684
Change-Id: I7625a013dda8e3ebce5b86a253a8c1befe267c11
(cherry picked from commit 72aebeb3ec070b6c334fab68b2c46f94868d014a)

Merge "camera: Map between legacy gralloc0 usage, and HIDL gralloc1 usages" into oc-dev

Camera: add interface to evict obsolete buffer caches

Test: fix CTS ReprocessCaptureTest
Bug: 34461678
Change-Id: Icde654b0c8423c31d7d39d180913ffa374e7de3c

Test for getHashChain() in IBase.

Each interface has a getHashChain() method that
returns the hash of all .hal files from the rtti
of the interface up to IBase.

Test: hidl_test

Bug: 36602587
Change-Id: I53a79ba69edd109f169c37dbe0fd10e302eb929c

Merge "Camera: add HAL1 recording batching support" into oc-dev

Updates bluetooth hal test to use VtsHalHidlTargetTestBase v3

Test: pass
Change-Id: Ia5237a27800010174f2b23a3a4b1157f1c48781f
(cherry picked from commit b8012eb907639990f4cbfe04ca1490d1ab270168)

NFC VTS test use VtsHalHidlCallbackBase for callback wait and notify

VtsHalNfcV1_0TargetTest uses VtsHalHidlCallbackBase.
Callback class extends VtsHalHidlCallbackBase and use waitForCallback and notifyFromCallback

The original NFC test contains the following use cases:
  * Callback notify happens before wait
  * Multiple waiting in serial
The modified version added the following use cases:
  * Wait for specific callback function
  * Passing callback arg data along wait and notify functions

Bug: 36036059
Test: make VtsHalNfcV1_0TargetTest, adb push & adb shell
Change-Id: Ic6668a18254661d07c4ff37309cd96ea5f786d29
(cherry picked from commit bfefc5ef104b63d0644ec3b4c766131987824535)

Bluetooth VTS test use VtsHalHidlCallbackBase for callback wait and notify

VtsHalBluetoothV1_0TargetTest uses VtsHalHidlCallbackBase.
Callback class extends VtsHalHidlCallbackBase and use waitForCallback, notifyFromCallback, and setWaitTimeout

The original Bluetooth test contains the following use cases:
  * Wait for specific callback
  * Set timeout for specific callback waiting
The modified version supports the above use cases.

Bug: 36036059
Test: Test pass locally
Change-Id: I961baaf7f1b09fa9c134629cc7280565a2664ed5
(cherry picked from commit 51d023c48ec6cf709e756ceed62c11b25fb0c085)

Camera: HAL1: remove libbinder usage

Test: Angler HAL1 preview/takePicture/record
Bug: 35887419
Change-Id: I6cb37fd78a2fad1ee3b24e4cd61e3a3b2bce4744

Camera: add HAL1 recording batching support

Test: Angler HAL1 + set mBatchSize to 4 locally
Bug: 35997432
Change-Id: I6baf112f9da56534ae81008bad818fba1e76a294

Merge "Change sytle of radio and sap tests to clang google" into oc-dev