Merge "bluetooth: include what you use" into oc-dev

Merge "Change drm/crypto service names to "default"" into oc-dev

Merge "configstore: add startGraphicsAllocatorService" into oc-dev

Merge changes from topic 'hidl-vendor-available' into oc-dev

* changes:
  Update makefiles (2/2).
  Update makefiles.

Merge "Add p2 option in OpenLogicalChannel" into oc-dev

Merge "Add PRESENT_FENCE_IS_NOT_RELIABLE" into oc-dev

Merge "Mark hidl test libs as vendor." into oc-dev

Merge "libgrallocmapperincludes: as header lib and vendor" into oc-dev

Merge "Add IOmxStore and remove IOmxNode::setQuirks." into oc-dev

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

Add PRESENT_FENCE_IS_NOT_RELIABLE

Test: adb shell /data/nativetest/libgui_test/libgui_test
  --gtest_filter=*GetFrameTimestamps*
Bug: 36730849, 36887025

Change-Id: Ide992e4dda7840f7982237943256ea3019eda434

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

Add p2 option in OpenLogicalChannel

- Adding support for P2 parameter
- Needed to support OMAPI 3.0 defined
  by SIM Alliance

Test: Basic telephony sanity
Bug: 27360148
Merged-In: I6a274195065ab1ef8406849228f60859ee826e53
Change-Id: I88134eded001e14c12e1f2a18efd0a2b51bc22fe

bluetooth: include what you use

Helper libraries also now on vendor partition.

Test: links
Test: bluetooth works on internal Marlin
Bug: 33241851

(cherry picked from commit 96510c80f84e192f106678d269e793ade8510b1c)
Merged-In: Ie99faf9146ce2ce3df83a2b46acfaa36a54c16c2
Change-Id: Ie99faf9146ce2ce3df83a2b46acfaa36a54c16c2

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

libgrallocmapperincludes: as header lib and vendor

Bug: 33241851
Test: links
Test: try build with BOARD_VNDK_VERSION current, no errors related to
  this module
Test: (sanity) YouTube

Change-Id: Ia2c3922bf8bf1ab51396a8c8c5fa8d909c442994

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

configstore: add startGraphicsAllocatorService

Add ISurfaceFlingerConfigs::startGraphicsAllocatorService to control
whether SurfaceFlinger should start IAllocator service.

Bug: 36462585
Test: boots
Change-Id: I421413673246cc7c552bc9eeeb7a9ce2783faf9e

Mark hidl test libs as vendor.

They must be on the vendor partition for hidl_test to be able to link to
them when the VNDK is in place.

Test: hidl_test
Bug: 33241851
Change-Id: I23c9e0674aaf33602bc9fde0515eb2f87ce5c60c

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

Change drm/crypto service names to "default"

bug: 36371166

Test: vts drm hidl test

Change-Id: Iae98708c4f9b58dc697fb54c9f9b67d99e7ed0cf

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

Update makefiles (2/2).

Test: pass
Change-Id: Ic7e6289c7b69785491708ce0acbb657f4c704229

Update makefiles.

Test: pass
Change-Id: I5cf42c8ff1e11207394e4852ba0bd03e4e3b7d2e

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)

Add IOmxStore and remove IOmxNode::setQuirks.

IOmxStore is used for passing results from parsing media_codecs.xml and
related XML files to the framework. Once XML parsing is moved to the HAL
side, IOmxNode::setQuirks will not be needed anymore.

Test: Media post-submit tests on Pixel phone.
Test: Manual use of Camera, Photos, Play Movies and YouTube apps.
Bug: 36952714
Change-Id: I6b24c486c6f8afcbef6ee88a13b2b5bb08d3e656

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