graphics: Update RAW16 format comments
am: cb0b46654d

Change-Id: Id5ff9b719351c5f7c1d08d3398dd0e01f20ec149

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
am: e13455b32a

Change-Id: Id6f15287394bb2531316add0a4111f70d0e736d6

Merge "[AWARE][VTS] Update WifiChipHidlTest NAN tests" into oc-dev
am: 24e1601315

Change-Id: I5c763c6d5c894f9c1ff2e637859ea24fd1f0835f

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
am: 03d65ff3ef

Change-Id: I27853ae03a52dc4e044fbece1edbe449d809e7ea

Merge "Log errno" into oc-dev

[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

[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
am: 5ed473340c

Change-Id: Ifa644f4630408fbad1d5587a7df5da1317caea01

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

Merge "Adds default keymaster service to the early_hal class" into oc-dev
am: 8080d70564

Change-Id: I45857b74581c2e7b4c321cc13b8ebc77546fb7a2

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

Merge "Fix transitive includes." into oc-dev
am: 6f5d1945a3

Change-Id: I372e8f95f2cf02137c6140719ab59b08351cae2d

Merge "Fix transitive includes." into oc-dev

Merge changes from topic 'configstore-utils-log' am: e8600fe892 am: 725cbd866c
am: cb1953b3a7  -s ours

Change-Id: I6fb40f95ed793c08e9d53be61b26856cf0d088cb

Merge changes from topic 'configstore-utils-log' am: e8600fe892
am: 725cbd866c

Change-Id: Ia3379660fc8f21170c09ec98d95ac6f223b566d3

Merge changes from topic 'configstore-utils-log'
am: e8600fe892

Change-Id: I28f7348da2c6656db4c0ac27bb3fa066d1396d9e

Merge changes from topic 'configstore-utils-log'

* changes:
  print log message for values retrieved from configstore
  Handle hidl transaction errors in ConfigStore

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

Change-Id: I736607036a29fd59279d3c31c1c786a79fcea727

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 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 "Weaver HAL" am: 2f5a5e10f7 am: e3f5387b42
am: 4ab649b502

Change-Id: I9ef7d44793117b24cfdf601b2fe9a9f9be88cf74

Merge "Weaver HAL" am: 2f5a5e10f7
am: e3f5387b42

Change-Id: Iec3b2d31c58862819c0a1db9d8051f509731617b

Merge "Fix hidl_string copy in boot VTS test" into oc-dev
am: 7dfa9bdf1f

Change-Id: I4b3833f791ff5f58777c005c77cf928dcd6bb013

Merge "Weaver HAL"
am: 2f5a5e10f7

Change-Id: Ibe908202b5f86478dcea71b4eb75d9a1fca6f825

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

Merge "Weaver HAL"

Merge "Empty interface IQuux which is completely unrelated to IBase and IBaz." am: 931ca54208 am: 9af34a0c10
am: 388aad8cdb

Change-Id: I6b323384a81fd9f6371f4d2303cab819f41dfcb9

Merge "Empty interface IQuux which is completely unrelated to IBase and IBaz." am: 931ca54208
am: 9af34a0c10

Change-Id: I29cc2b5225bc24f364f9ca399f8a728d7078c953

Merge "Empty interface IQuux which is completely unrelated to IBase and IBaz."
am: 931ca54208

Change-Id: I3746b93cae3231117cd00ed84735aaf0b754a930

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

Merge "Add additional drm hal tests" into oc-dev
am: 69571ba233

Change-Id: I98013b34a2e58f34daeb8cd1eed0cdc4fdc862fe

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

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

Change-Id: I8a67043f4eea44e9c45b434ab0d2da433ab2c3a1

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
am: 7ba8df9b7c

Change-Id: Ib5fc2666b258978b76710d2998d20e7a03fb9bff

Merge "Correct comment on getProvisionRequest" into oc-dev

Weaver HAL

Weaver is used to harden the user authentication flow by securely
storing secrets which can only be read by presenting the associated key
secret. The key secret is derived from the user's credential.

Bug: 35628284
Test: Implement HAL, boot and call from system_server
Change-Id: Ia72fd537670c08d27320c944a0fe91243137bf6c

Add Flag for Enabling 64-bit Legacy DRM Plugins
am: fcde9ff3a0

Change-Id: I9dd764999131ca95b01adf33dc3ba60330100ce6

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
am: ccc1d2e736

Change-Id: If3ebe7e1c54a2bc68b4ba0065ba3b62eb158b02e

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
am: 163dc91b11

Change-Id: I39cf8b158eb5d18cc0b5dfc692e2224d9757ac3d

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

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: I7d5d80ca7a0317816c71016e013f9e73ad23ee08
Merged-In: I2c0895f8afbbb2947b62164acaf62a491c451dc0
(cherry picked from commit 0dc72ecbb798971d6f394d7f4dbc9d1f4006da5f)

Handle hidl transaction errors in ConfigStore

Test: surface flinger not failing, android now bootsup

Bug: b/36445794
Change-Id: I64cc404bec71f5e4eea2e0034f07b86fb60a3e32
Merged-In: I22fa7aab9fa92bc04333aaa0eef45891ebeba8e7
(cherry picked from commit 076b792ade608a7fdf4e600a42550ad17296a39e)

Merge "Added protections for RenderScript VTS tests." am: 792ec065e0 am: f251028320
am: 3db05819ae

Change-Id: I8c709868b1a2ff5b3078b44bf740f4a1cd7899d5

Merge "Added protections for RenderScript VTS tests." am: 792ec065e0
am: f251028320

Change-Id: I60ad212ce37b90e65ae034e5080cee8bc44e9185

Merge "Added protections for RenderScript VTS tests."
am: 792ec065e0

Change-Id: Icc2e3fca4a3f9d9368ec22303588bec0d6653532

Merge "Added protections for RenderScript VTS tests."

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

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
am: 7b63265377

Change-Id: Ie8ec0292375064385b3786d79625cb3583e88531

Merge "[AWARE] Legacy HAL update: reduce host wakeup" am: 6ae199efa4 am: c6e4705e9d
am: 7f4335b30a

Change-Id: I90570e1f80662d0557eb1f30623e609c5bed8716

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

Merge "[AWARE] Legacy HAL update: reduce host wakeup" am: 6ae199efa4
am: c6e4705e9d

Change-Id: Id7c2cf6440b21f14d4fd766cc82edc8a92a76e99

Merge "[AWARE] Legacy HAL update: reduce host wakeup"
am: 6ae199efa4

Change-Id: I5a868d9c738e4c826bfeb5d6bc9e5c329bcb932e

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 "[AWARE] Legacy HAL update: reduce host wakeup"

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

Change-Id: I687a2bcfb8d401fcc936ebfeda9bfab56dfee83a

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

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

Change-Id: I7babaf3520fb9719c38a86f58fda86675cd52475

Merge "wifi(implementation): Print out the cb object on failure" 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

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

Change-Id: Ib8cb9c7cca60817256c425ff4c89d60a9109ccb5

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
am: 4094107835

Change-Id: I723ff4fc7e2352e31adb3fc11079f0b88832fd40

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

Change-Id: Iea0ad9868ed0c4347e8614c7bc858c56938fafb7

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)

Bluetooth HAL: Add to foreground schedtune group am: 1b8f6f7c77 am: 887448f949
am: 93aa1c0f94

Change-Id: I84f46cedb3ac0c480f6c542dcbf8a705355559b8

Bluetooth HAL: Add to foreground schedtune group am: 1b8f6f7c77
am: 887448f949

Change-Id: I5d8af23d7fae64e9ebb385e3f2fceb37ac6adc3c

Bluetooth HAL: Add to foreground schedtune group
am: 1b8f6f7c77

Change-Id: I04b1ba8e8d6151b4421d63171ab12cb46e71c793

removed log print statements for configstore from SurfaceFlingerConfigs.cpp
am: 0481c3f717

Change-Id: I9cf1135fb289a4d8a0b3b23f48782934962f7a28

print log message for values retrieved from configstore
am: 0dc72ecbb7

Change-Id: I426cae83b86fa45efbb55ecaa014e8be900a5dea

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
am: 0dff6286eb

Change-Id: I00514c09d2fe5b6de27b3c0597641acb831c9930

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

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>

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

Change-Id: Ic82a3c73e6de95c5f4dfc53a0cb530e499e2a377

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

[AWARE] Legacy HAL update: reduce host wakeup

Legacy HAL update to add indications when a solicited publisher
receives a match. Not used in current HAL. However, also added a
control bit to turn off host notification which is now enabled.

Bug: 35195773
Test: integration (sl4a) tests
Change-Id: I8ff378abaae11d4feaa42a9901abb1b48069dc4f

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
am: 90b4baef52

Change-Id: If3c80873ef9da35e57254d1411ad127cac833a04

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

Change-Id: Id64c09796add93d4cd9a3504051256c898e3f3e8