Merge "bug fix: configure input port buffer size" into oc-mr1-dev

Merge "[AWARE] Fix maxNdpSessions documentation" into oc-mr1-dev

Merge "wifi(implementation): Reduce logging severity" into oc-mr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev
am: 92d3bc1e1a  -s ours

Change-Id: I1a648056655a978d867b50a53d929c646fc6d035

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev am: e2dae8c258
am: 5d18c87e88  -s ours

Change-Id: Ib03c1512099ec88e1c65672dadbe29ef3b9d9086

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-mr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dr1-dev

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev
am: e2dae8c258

Change-Id: I396a157f3522843c3d36870719cb3e5176754dcc

Merge "Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'" into oc-dev

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO
Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

wifi(implementation): Reduce logging severity

IE elements should not be present in the batched scan results, but it
being there doesn't cause any problems. The IE elements will be ignored
when we send it over the HIDL interface, so reduce the log level to
prevent log spamming.

Bug: 64448186
Test: None
Change-Id: Ia0d0312a113c907bb0992bb8f41e734fe9d05114

bug fix: configure input port buffer size

In video decoders, size of a single input frame (elementary-stream)
is dependent on width, height, color format, profile settings, ...
Most of this information is part of sps, pps and requires parsing.
But as the max size of the stream is known ahead, use it to configure
the input buffer size requirements

Bug: 63875287
Bug: 63796949

Change-Id: Ib760c4f55b094260a0abd120f852dcf1899df4e0

Merge "Tetheroffload tests static link to HAL def libs." into oc-mr1-dev

[AWARE] Fix maxNdpSessions documentation

Clarify meaning of maxNdpSessions capability. Total, not per interface.

Bug: 64685830
Test: builds
Change-Id: I24fc0a97172a4ca161f432b6c5c1dc55a44dfb12

Merge "Widen dummy band range to cover all regions." into oc-mr1-dev

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev am: 52af675d14
am: fc73b0fcbd  -s ours

Change-Id: I4f9fcf22f98f9c40290d321fabe05a082524da19

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev
am: 52af675d14

Change-Id: Ic5080cb6b22cd3984d0eb39d87cabf7c8a30178d

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-dev

Tetheroffload tests static link to HAL def libs.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadControlV1_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalTetherOffloadConfigV1_0Target
Change-Id: Ib771c07646f3e219c7b9ba4be5515552d7fd6348

Update broadcastradio HAL 1.0 VTS tests.

This is a cherry-pick of the following VTS changes from O MR1 to O:
 - ag/2462390: Fix Broadcast radio 1.0 VTS to not fail on
   other radio types than AM/FM.
 - ag/2477726: Make tuned field description more precise and
   fix VTS tests to obey it.
 - ag/2549733: Implement out-of-band metadata images (a test for in-band).

Additionally, it fixes config callback handling in OpenTunerTwice test.

Bug: b/64324703
Test: VTS
Change-Id: I590b9b4d927fb0cd41f92e919343b5b94ad0e246
Merged-In: I590b9b4d927fb0cd41f92e919343b5b94ad0e246

Merge "Update broadcastradio HAL 1.0 VTS tests." into oc-mr1-dev

Merge "configstore: sandbox with seccomp filter" into oc-mr1-dev

Camera: Switch 'cancelPictureFail' to 'cancelPictureNOP'

Calls to 'cancelPicture' should be no-ops in case image capture
is not active.

Merged-In: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1
Bug: 64569316
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --primary-abi-only --module
VtsHalCameraProviderV2_4Target -l INFO

Change-Id: I5fad0a5b7aafd208c3a1b2f62a2b9046cde858d1

Update broadcastradio HAL 1.0 VTS tests.

This is a cherry-pick of the following VTS changes from O MR1 to O:
 - ag/2462390: Fix Broadcast radio 1.0 VTS to not fail on
   other radio types than AM/FM.
 - ag/2477726: Make tuned field description more precise and
   fix VTS tests to obey it.
 - ag/2549733: Implement out-of-band metadata images (a test for in-band).

Additionally, it fixes config callback handling in OpenTunerTwice test.

Bug: b/64324703
Test: VTS
Change-Id: I590b9b4d927fb0cd41f92e919343b5b94ad0e246

Merge "Mark diagnostic values as either being numerically equivalent to OBD2 values, or providing the equivalent OBD2 value" into oc-mr1-dev

Merge "Keymaster test statically links to dependencies." into oc-mr1-dev

Widen dummy band range to cover all regions.

Bug: b/64115131
Test: VTS, instrumentation
Change-Id: I34e9aac440a003226200243c835e6116ba7eae90

Merge "Neural networks test static links to HAL def libs." into oc-mr1-dev

Keymaster test statically links to dependencies.

This test now statically links to libs not guaranteed to be on the
device.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalKeymasterV3_0Target
Change-Id: I6a7b8c116153f18f61a71e5b5bef98343a4de43b

Merge "VTS tests now static link to HAL def libs. #2" into oc-mr1-dev

VTS tests now static link to HAL def libs. #2

And use VtsHalTargetTestDefaults.

Bug: 64040096
Test: crop out all non-affected tests from vts-hal-hidl.xml and run
vts-tradefed run commandAndExit vts-hal-hidl --skip-all-system-status-check
--skip-preconditions
2 failures, same as without this change:
GatekeeperHidlTest.DeleteAllUsersTest
GatekeeperHidlTest.DeleteUserTest

Change-Id: I8f6995e9536a9aefe283ee3effec9f5a7f03b620

configstore: sandbox with seccomp filter

Configstore HAL is accessible to third party apps and thus requires
a tight sandbox that reflects the limited system access this HAL
needs.

We use two primary mechanisms to sandbox configstore, selinux and
seccomp, with the goal of restricting its access to userspace and
the kernel. The addition of a seccomp filter is primarily aimed
at reducing the kernel's attack surface that is reachable by
configstore HAL.

Seccomp filters are architecture dependent, so filters need to be
added for each architecture. This change adds a seccomp filter for
arm64 and issues a non-fatal runtime warning for other architectures
which still require a seccomp filter.

Bug: 36453956
Test: boot Marlin and Angler. Verify that configstore is not aborting
    due to seccomp violations.
Test: "cat proc/<configstore pid>/status | grep seccomp " returns:
    seccomp: 2
    Which indicates that configstore is using seccomp-bpf.

Change-Id: Iab014ff357b7329085a5e18a92f51838d2c72371
(cherry picked from commit ed95043d6433c5900984f24dbb101bba78109775)

Merge "Graphics tests statically link to HAL definition libs." into oc-mr1-dev

Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev
am: 269d9ba09a

Change-Id: I1260d97229e5330481178c8f802e2c7f6b651582

Merge "Bluetooth: Check the number of file descriptors" into oc-dr1-dev

Merge "Renderscript test static link to HAL def libs." into oc-mr1-dev

Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev
am: 5397fa8175

Change-Id: I1587c5451950a89c6aac991a0d8b7dacea16fe21

NFC: Add OWNERS file for default/
am: a4d1a74700

Change-Id: I43572572901298d1e915c9753798361bdd5d38c2

Merge "Bluetooth: Add OWNERS file for default/" into oc-dr1-dev

Merge "Media omx tests static link to dependencies." into oc-mr1-dev

NFC: Add OWNERS file for default/

Test: build
Change-Id: I6e854923fdd66991bcc7ed3446106d148abea982
(cherry picked from commit 31439b1d24d49fcb431dfa48bdaa0b1d57deded4)

Bluetooth: Add OWNERS file for default/

Test: build
Change-Id: Ib4cbd5502ac0791d4bc700f28de68d0548256de6
(cherry picked from commit 7c127dc4cbf90713bcdbce89fa166228c912847d)

Merge changes I51b1e596,I3aece23a into oc-mr1-dev

* changes:
  bug fix: output colorformat configuration fixed
  bug fix: android list management in dequeueMsg()

Bluetooth: Check the number of file descriptors

Opening the serial port(s) returns -1 when it fails, check
that the number of file descriptors is greater than 0 and
less than the size of the array.

Test: VtsHalBluetoothV1_0TargetTest, Bluetooth start/stop
Bug: 64420262
Change-Id: Icbffa5bd8c02030e389bac03aa5e1e78cc3c6057
(cherry picked from commit 89316af23eb7aa300015a3d80bc97891cfe6031e)

Merge "Skip direct report test if sensor is not available" into oc-dev am: 63524f9fcc
am: 7bea962c9c

Change-Id: I99d4eb05d1348e24910c5ebe2e87b6b334138b31

Merge "Skip direct report test if sensor is not available" into oc-dev
am: 63524f9fcc

Change-Id: Iffe683356ed925142ef877ecb5289463d8d4ac1e

Merge "Skip direct report test if sensor is not available" into oc-dev

wifi(vts): Remove a tight loop of toggling wifi state
am: 0d1bb97447

Change-Id: Icec63b9ffd5d12003da44bdde73edde04e55f11a

wifi(vts): Remove a tight loop of toggling wifi state

The tight loop of HAL start/stop exposes a race condition within the HAL.
Adding a fix for preventing this race would need a fix which would be
pretty risky at this point of the release. The issue itself is unlikely
to happen in real use cases because wifi toggles (user initiated or test
scripts initiated) via framework goes through a series of processing
before it invokes the HAL start/stop.

Bug: 64195190
Test: `make vts -j30 BUILD_GOOGLE_VTS=true TARGET_PRODUCT=aosp_arm64 &&
vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l
INFO`

Change-Id: I4e4d65f8b6e2e423a3a5f26e5a97a78b7e99c3e8

Merge "Fix typo in in offload control VTS test" into oc-dr1-dev
am: 5d77efddbd

Change-Id: I2be944da8800040540bf98aeb197a933d1da0e42

Merge "Fix typo in in offload control VTS test" into oc-dr1-dev

Fix typo in in offload control VTS test

Test: built, flashed, run
Bug: 38220415
Bug: 64479437
Change-Id: I3aa4afa3b52297dd92e37b4f39a5bd25b129ba04

Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev
am: 3a57cca59c

Change-Id: I6a1aeea07e64b4748151f62ae4827132eba32781

Merge "Add the error code "OPERATION_NOT_ALLOWED" and log the error." into oc-dr1-dev

Merge "Add OWNERS for broadcast radio HAL 1.1." into oc-mr1-dev

Add the error code "OPERATION_NOT_ALLOWED" and log the error.

Test: Telephony sanity tests, run vts -m VtsHalRadioV1_1Target
Bug: 64426938
Change-Id: Id4f06842186b10a52640372c23bc2b368cc1e9ab

wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE
am: 8c216524a3

Change-Id: I1bf9be99d4e673209edb893040483bf772ee3ca0

wifi(vts): Ignore IWifi.stop() returning ERROR_NOT_AVAILABLE

When IWifi.stop() is invoked back to back (happens in the ConfigureChip
vts test), the HAL would return ERROR_NOT_AVAILABLE if the previous stop
is still being processed. This is not an error that needs to fail the test,
but a legitimate status for stop. We have a retry mechanism to handle
this in both the VTS test and framework for the case where IWifi.start()
is invoked while the previous stop is being processed.

While there, corrected a few log messages emitted by the HAL to debug
such startup/stop issues better.

Bug: 63971806
Test: `vts-tradefed run commandAndExit vts --skip-all-system-status-check
--primary-abi-only --skip-preconditions --module VtsHalWifiV1_0Target -l
INFO`

Change-Id: I5e3470ac97541a6ea10aceec9b737e5d03ed5206

Merge "Update IRadioResponse.hal to add new error codes." into oc-mr1-dev

Merge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev
am: 6d5cf73b75

Change-Id: I004e81a764313fdaf23f9580b144e1ee2ba79b27

Merge "Tetheroffload Control and Config VTS" into oc-dr1-dev
am: a11e731b6e

Change-Id: I24712d426e1227b1f96cfa35f9f7b320e27fc901

Merge "Refactor and expand tetheroffload Control and Config VTS" into oc-dr1-dev

Merge "Tetheroffload Control and Config VTS" into oc-dr1-dev

Media omx tests static link to dependencies.

These tests now statically link to libs not guaranteed to be on the
device, which include HAL definition libs.

Also, remove global include paths.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalMediaOmxV1_0Host

Change-Id: Iea1dc549704f61b1416357d72dc1bf26864978d4

Merge "thermal: IThermal V1.1, add methods to send thermal throttling events" into oc-mr1-dev

bug fix: handle OMX_EventBufferFlag events am: c8df8940e1
am: 2921996e7a  -s ours

Change-Id: I660fb164aa2f1ed1420e1cb5f81df462200e49bd

bug fix: handle OMX_EventBufferFlag events
am: c8df8940e1

Change-Id: Ifa9493d869e42a76100011d5af20c5062bc44891

Add OWNERS for broadcast radio HAL 1.1.

Test: not necessary
Change-Id: I73881db46382276f56963c9d3a40cb0bdc1a2070

Merge "Change VENDOR program types to range." into oc-mr1-dev

Merge "Remove unused variables." into oc-mr1-dev

thermal: IThermal V1.1, add methods to send thermal throttling events

Add IThermalCallback and IThermal::registerThermalCallback() method.
Frameworks code calls this method to register a callback used by the
IThermal HAL implementation to send thermal events to the framework
ThermalService.

Bug: 30982366
Test: VtsHalThermalV1_1Target on marlin
      manual test on marlin using marlin 1.1 HAL with modified
      thermal-engine.conf and temporary debug code for notification
Change-Id: Ib49ad93a9495e3af515fced4e46f20186661fe07
(cherry picked from commit cf964d79c1d41bba999e58cb476af099191a6634)

bug fix: handle OMX_EventBufferFlag events

OMX_EventBufferFlag event is sent when the component has processed a buffer
with its EOS flag set. This event is not sent by soft omx components.
Vendor components can send this. From IOMX point of view, this event is
not sent for processing

bug:64102197
Merged-In: I3a978a885b1e4446f82f2356ae677f70ea6f8150
Change-Id: I3a978a885b1e4446f82f2356ae677f70ea6f8150

Renderscript test static link to HAL def libs.

And use VtsHalTargetTestDefaults.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalRenderscriptV1_0Target
Change-Id: Id6177663b441b5b0971143309401d73395ab51db

Graphics tests statically link to HAL definition libs.

These tests now statically links to HAL def libs which are
not guaranteed to be on the device.

Bug: 64040096
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalGraphicsMapperV2_0Target
Test: vts-tradefed run commandAndExit vts --skip-all-system-status-check
--skip-preconditions --module VtsHalGraphicsComposerV2_1Target
Change-Id: I83b68dafbd161f88394f67dbda14f9957ddf7232

Refactor and expand tetheroffload Control and Config VTS

Test: as follows
    - built
    - flashed
    - booted
    - adb sync && adb shell ...$test
Bug: 38220415
Change-Id: Iab715c45ad0457f3eea43ee408fa947ffc01c4aa

Tetheroffload Control and Config VTS

Basic VTS for Tetheroffload Config and Control  HALs.

Bug: 38220415
Change-Id: I6d0867e019aca58d8a6bd1404b8b85ed98274f11
CRs-fixed: 2083182

Merge "Guard against racy ComposerClient reconnection" into oc-mr1-dev

Change VENDOR program types to range.

Bug: b/64331420
Test: VTS, instrumentation
Change-Id: If91d8e6835ed3acaf429bb05b96f3d3a7985d19b

Merge "Add me into owners of radio vts" into oc-mr1-dev

Merge "Update comments for VtsHalTargetTestDefaults." into oc-mr1-dev

Merge "Pass ProgramInfo over currentProgramInfoChanged and drop afSwitch_1_1." into oc-mr1-dev

Merge changes from topic 'hidl-vndk-enabled-oc-mr1' into oc-mr1-dev

* changes:
  Mark hidl utils as vndk
  Manually update makefile for vndk-sp hidl libs.
  Update make file for vndk enabled.

Merge "Wifi supplicant test statically links to dependencies." into oc-mr1-dev

Add me into owners of radio vts

Test: no need
Bug: 64220334
Change-Id: I2a9b82ee1562859827a844e729451f6d03c4e592

Merge "Pass vendor-specific info as map." into oc-mr1-dev

Merge "Implement comments from API review" into oc-mr1-dev

Pass ProgramInfo over currentProgramInfoChanged and drop afSwitch_1_1.

Bug: b/64229617
Test: VTS, instrumentation
Change-Id: Id2d209a31d801f85e197312673eb6b7ee63ada46

Neural networks test static links to HAL def libs.

Bug: 64040096
Test: compiles
Change-Id: I87d26a93fa28d7cd93176489a0d0556b7350f1ed

Wifi supplicant test statically links to dependencies.

This test now statically links to libs not guaranteed to be on the
device.

Bug: 64040096
Test: compiles
Change-Id: I986e61835e641e15bdad0ff9571ee8ffa59b2a46

Mark hidl utils as vndk

As a VNDK module, Android.bp must have 'vndk' tag as well as
'vendor_available: true'.

The 'vndk' tag for VNDK module is formated as below:
vndk: {
    enabled: true,
},

VNDK modules will be installed both in system/lib(64) as normal and
in system/lib(64)/vndk as a vendor variant.

Bug: 63866913
Test: build and boot with BOARD_VNDK_VERSION=current
Change-Id: If0eb0c1bddfa5bdc7ea0ca4635d4e53b59836582

Manually update makefile for vndk-sp hidl libs.

vndk-sp is not automatically tagged by hidl-gen.
For vndk-sp libs, "support_system_process: true" is manually added
in "vndk" property.

Bug: 63866913
Test: build and boot with BOARD_VNDK_VERSION=current
Change-Id: I2b18d691411e58dc55bcdfa39ecb3659242c8437

Update make file for vndk enabled.

Update the Android.bp generated with hidl-gen.

Test: build with and without BOARD_VNDK_VERSION=current
Bug: 63866913
Change-Id: I1a9db1df49e0f13c5790da2b118ae9ec63ba34a7

Implement comments from API review

Bug:  64372008
Test:  Fix comments
Change-Id: Ia2bc0ae4c45015e7965b544f5d46f7e2a3ff0e2a

Pass vendor-specific info as map.

This solves a problem of identifying, if a given application is talking
to a hardware that can handle format of these strings.

Bug: b/64229617
Test: VTS, instrumentation
Change-Id: If1a5f9900800b8537ef158beefcbaf3c1008c3da

Merge "Update makefiles for vndk enabled." into oc-mr1-dev

Remove unused variables.

Test: compiles
Change-Id: Idf7631f98ad4e04f4a43e66161ba27e93d180241

Merge "Implement missing broadcastradio 1.1 VTS tests." into oc-mr1-dev

Mark diagnostic values as either being numerically equivalent to OBD2 values, or providing the equivalent OBD2 value

Test: build
Bug: 64024685
Change-Id: I24ef13514a94e10c599605e6970a023168774049

Update comments for VtsHalTargetTestDefaults.

Bug: 64040096
Test: only comments added
Change-Id: I4a38aa0ed339c65aa4ffbe5c46489f27c745d5d6