Merge "audiohal: Fix incorrect parameter handling in IDevicesFactory::openDevice" into oc-dev

audiohal: Fix incorrect parameter handling in IDevicesFactory::openDevice

Check for possibly invalid values of IDevicesFactory::Device enum.

Bug: 37592306
Change-Id: I4a2300d3be9ab38e6c51ba8a3b5bbbb606a11c2b
Test: VTS
(cherry picked from commit 8c16d85122472ef01a9e023c9568968faac1aff8)

add more hals to framework compatibility matrix

Bug: 37321309

Test: marlin builds (PRODUCT_FULL_TREBLE=true)
Test: bullhead builds (PRODUCT_FULL_TREBLE=false)

Change-Id: Ib92183a0749d972d9e89f38e158d32080c8023c6

Merge "Fix Keymaster HAL crash." into oc-dev

Merge "Revert "audiohal: Get rid of multiple inheritance in IDevice implementation"" into oc-dev

Revert "audiohal: Get rid of multiple inheritance in IDevice implementation"

This reverts commit e4228e7aa683044e17e6592c9014b5d4691445e7.

Further analysis of the bug has shown that the problem is not related
to the use of multiple inheritance.

Bug: 36225019
Change-Id: Icdcaf84061c0ecf29f891521bd961c3ee89f1cfc

Merge "Omx VTS tests" into oc-dev

Omx VTS tests

Bug: 32023356
Test: make vts.

preload elementary stream information during decoding

instead of using extractor to get the frame boundaries, frame
flags and timestamp information, they are read from a metadata
file in the resources directory on demand. Instead of doing it
on demand, we load the entire file information and use it. This
modification comes in handy for future test fixtures

Change-Id: I3a434018111eb66008f17c24b4c2c16d0db22ed1

bug fix: wait on input consumption

after queuing all the frames that need to be processed by
the component, the application continues its job of deinitializing
the component before ensuring all the inputs were decoded.
This is corrected.

Change-Id: I09a11fd5badabe2889ba243383145be07be51953

add end of stream test for audio decoder

component is provided an input with nFlags field set OMX_BUFFERFLAG_EOS.
the application expects this flag to be returned via one of its output
buffers. this flag is passed to the component at different timings to test
the robustness.

Change-Id: I00e3fdd5ef50d93f2620d8b0e9fc3c358de428d7

add timestamp deviation test for decoder components

the list of timestamps received from the component is
compared with the list provided to the component for any
differences

Change-Id: Ifcc16b7a00f58e1c1dfdba9f538d307ddcbfcc71

add option to configure resource files path

default location of resource files is /sdcard/media/. client can provide
choice of this location via the option -P.

example: VtsHalMediaOmxV1_0TargetVideoDecTest -C <component> -R <role> -P /sdcard/res/

Change-Id: Ib8d3af03c19a171e3bbb9839f7e331cbe576b24b

add flush test for audio & video dec components

while decoding a group of frames, flush call is made and it is
checked if all the i/o buffers are sent back to client. we then
seek to the next keyframe and test for decoding. monotonous
increase of timestamps is ensured.

Change-Id: Id48c76c157ebdfdefd3003e57519da3bd9715973

Add functional vts test for Health HAL

Bug: 35915444
Test: Boot and run the test successfully on sailfish

Change-Id: I5965d1ffbc4f1a89454f962698ce67adc4c35293
Signed-off-by: Sandeep Patil <sspatil@google.com>

Merge "Add YUV support in EVS VTS tests" into oc-dev

Merge "Avoid crash when setting buffer on deleted layer" into oc-dev

Avoid crash when setting buffer on deleted layer

When setLayerBuffer() was called on a layer previously destroyed by
destroyLayer() the code would crash. Instead, return an error.

Bug: 37159844

Test: Ran vr flinger in a way that would trigger the crash, and
confirmed that I now get error logs instead of crashing. Unfortunately
the error code is consumed by the Composer wrapper class and not
propagated back to the caller, but that's a separate issue (b/37901601).

Change-Id: I75a5b954d47a1deac44d03851f60f347085eca89

Add YUV support in EVS VTS tests

Additionally adds log markers at test start and ends tests early which
fail necessary preconditions.
Test:  Run against default and sample EVS drivers
bug:  37687125

Change-Id: Ie34674df5fe099dd4f380df485801ab0288147e9

Merge "Add framework compatibility matrix" into oc-dev

Add framework compatibility matrix

Test: builds and verify that system/compatibility_matrix.xml exist
Bug: 37321309
Change-Id: Ic336e7660a19a4c2b5104739e0b7ace11096fcf7

Merge "graphics: polish Transform docs and indent ColorMode" into oc-dev

Merge "audiohal: Get rid of multiple inheritance in IDevice implementation" into oc-dev

Merge "Fixed the hasCarrierFrequency flag in GnssStatus(part 2)" into oc-dev

Merge "Fix RssiMonitoring VTS tests" into oc-dev

Merge "graphics: remvoe -Wno-unused-parameter" into oc-dev

audiohal: Get rid of multiple inheritance in IDevice implementation

We still not sure what causes crashes in Device::get|setParam*,
but it seems that it is somehow caused by the fact that the parameters
code is in a separate class with virtual methods, from which
Device class inherits along with IDevice interface.

The workaround is to substitute multiple inheritance with
delegation in Device class. Hopefully this will either eliminate
crashes or make the underlying reasons more clear.

Some of the code got reformatted by clang-format as a presubmit
requirement.

Bug: 36225019
Test: make
Change-Id: Id785c3565bbebd5acc26ca46472961698d9c6208

Merge "Prefix ENABLE_MEDIADRM_64 with TARGET_" into oc-dev

Fixed the hasCarrierFrequency flag in GnssStatus(part 2)

Bug: 37757180
Fixes: 37757180
Test: cts-tradefed run cts -m CtsLocationTestCases --test
android.location.cts.GnssStatusTest

Change-Id: I817931ee2c19d0a770c4d7cedfbd69bc85db6978
Chang-Id: Icf4240ccad7ff367d96001973c0be36308ffd9b3

graphics: remvoe -Wno-unused-parameter

Bug: 34611192
Test: builds
Change-Id: I8d6802ee6c618ea506fcca8e7d74e3b7026f29be

graphics: polish Transform docs and indent ColorMode

Note, current.txt change (as opposed to addition) is only allowed
because this is an unreleased interface.

Bug: 32593292
Test: builds
Change-Id: I95381d1a3a456aeadbc472e2dbbffde19cb43e6c

Merge "audiohal: Support stateful downmixer effects" into oc-dev

Merge "Camera: use framework header for VideoNativeHandleMetadata" into oc-dev

Prefix ENABLE_MEDIADRM_64 with TARGET_

This build-time flag should be prefixed.

Bug: 37039525
Test: Built OS
Change-Id: If6f6cd1ad2df5f000a4264cabeb7d9eb77e194ea

Merge "Move sensor hal service to class 'hal' from main." into oc-dev

Camera: use framework header for VideoNativeHandleMetadata

Test: build
Bug: 35625849
Change-Id: I8159dd571b6b390fa089d497d45b76c1ec5bda31

Merge "Camera: use gralloc mapper for handle import" into oc-dev

Merge changes from topic 'device_version_cleanup' into oc-dev

* changes:
  CameraProvider: Check camera device version
  Camera: Insert default RAW boost if needed
  Camera: Override AE pre-capture trigger cancel
  Camera: Map stream dataspaces

Camera: use gralloc mapper for handle import

Bug: 37540361
Test: marlin, smoke test GCA
Change-Id: I9d58d9c6af20b37c016f036bce2dcf55817c59aa

Merge "Fix KM3.0 deleteKey behavior inconsistent with VTS test." into oc-dev

audiohal: Support stateful downmixer effects

Fraunhofer's downmixer has additional buffering, this is incompatible
with how DownmixerBufferProvider used to process incomplete buffers.

Now the effects HIDL wrapper delivers frameCount updates in
audiobuffers to the server side. This fixes playback of multichannel
sound streams.

Bug: 36181621
Test: play 5.1 track from Play Music
Change-Id: Iffe3327c6eedef85151eec2b41e0aedfb7ffa6dc

Move sensor hal service to class 'hal' from main.

Moving sensor hal service to hal class.

Bug: 36576271
Test: Ensure that sensors still stream.
Change-Id: I1ab984063c48401ee535b9be4df75d554d91578f
(cherry picked from commit 5d0b2e7c373d19eeffc915e5bcb0aaa9187f5a8c)

Merge "Clean up after RTT operations finished. (1/2)" into oc-dev

graphics: remove non-HAL pixel formats

HALs are not expected to see YCBCR_422_888, YCBCR_444_888,
FLEX_RGB_888, and FLEX_RGBA_8888.  JPEG is purely wrong.

Note, current.txt change (as opposed to addition) is only allowed
because this is an unreleased interface.

Bug: 32593292
Test: builds
Change-Id: I432e6cb75cfc347a41e8e1632ae1898eed3350c8

graphics: clean up PixelFormat docs

Make the docs more self-contained, not referring to stuff not
defined by the HIDL interfaces.

Mark JPEG, YCBCR_422_888, YCBCR_444_888, FLEX_RGB_888,
FLEX_RGBA_8888 as "MUST NOT USE".  These format are never supposed
to be seen by gralloc and do not belong to the HIDL interfaces.

Note, current.txt change (as opposed to addition) is only allowed
because this is an unreleased interface.

Bug: 32593292
Test: builds
Change-Id: Ia7607cc73dd2f58dfa27e71b0004b21c7ca6904d

Merge "graphics: reorder PixelFormat" into oc-dev

graphics: reorder PixelFormat

Sort the enum values in their numerical order. There is no other
change.

Note, current.txt change (as opposed to addition) is only allowed
because this is an unreleased interface.

Bug: 32593292
Test: builds
Change-Id: I47914a7971f922f5602f982b41dd7602842e853b

Merge "Basic audio/video encoder/decoder test." into oc-dev

Fix KM3.0 deleteKey behavior inconsistent with VTS test.

Keymaster 3.0 VTS test required that deleteKey returns
ErrorCode::OK even if the key blob parameter is invalid or garbage.
The rationale is that deleteKey shall have the invariant that
key blobs are unusable after the deleteKey call. If it was unusable
before, this invariant is upheld.

This patch makes the legacy wrapper for the Keymaster HAL translate
an ErrorCode::INVALID_KEY_BLOB retuned by the legacy delete_key to
ErrorCode::OK.

Bug: 37351644
Test: Manually run VtsHalKeymasterV3_0TargetTest with legacy keymaster
      HAL installed (tested with sailfish)

Change-Id: Ib22c8b8e10334770a1d4a5570acf16c2c52a6c60

Merge "Add onAcquired log back" into oc-dev

Clean up after RTT operations finished. (1/2)

Bug: 36185358

Test: Manual with RTT app.

Change-Id: Ic9c9d020acaf384664a9881ec2806f941b7a0c1c

Revert "Add capabilities for wifi_hal_legacy"

This reverts commit 1addbeaa4672adae37ade6784d7a99b6bf1e0604.

It breaks WiFi connectivity on Fugu.

Bug: 37713566
Test: Able to connect to GoogleGuest on Fugu.

Merge changes from topic 'fix_configure_chip' into oc-dev

* changes:
  Fix configureChip() VTS test
  Return ERROR_NOT_SUPPORTED when configureChip() is used for reconfiguration

Merge "stop bluetooth service while VTS test is running" into oc-dev

Merge "Update .hal file for fixing vts tests" into oc-dev

Merge "Fix Vts errors" into oc-dev

Merge "audiohal: Fix handling of invalid arguments in 'debugDump' methods" into oc-dev

Basic audio/video encoder/decoder test.

Bug: 32023356
TODO: cleanup the test code b/37719510

ComponentTest :-
1. allocatePortBuffers cleanup
2. More than one test fixture performs State transitions
(Loaded <-> Idle <-> Execute). So, this is moved to a routine and
invoked when needed.
3. Dispatch input/output buffers code section is moved to a routine
4. Seperated Test fixtures StateTransitions_Flush
5. Added Doc string
6. Test component port population

AudioDecTest :-
1. Backport Cleanup changes from ComponentTest to AudioDecTest
2. Added elementary streams for the following codecs mp3, amr-nb &
Updated paths to elementary streams in audio decoder test

Test: Tested ComponentTest and AudioDecTest for mp3

Change-Id: I90decde4dfab55c3d456dfd88b474dd5abe8eebc

List Available components (roles) in android media framework

Change-Id: I210665843e997e60b167f2d680c87b1cc4f90c58

add README.md

Change-Id: I3a2a057d10b18f4f374928a996b8b4ea4b084770

add basic video decoder tests

for a given component test for successful decoding of an input clip

Test: tested for softavc decoder

Change-Id: I2f4c034c34a456e1ed0c2d32ef381ca48aeda8ff

add basic audio encoder tests

for a given component, test for successful encoding of an input clip

Test: tested on softaac, softamrnb, softamrwb & softflac components

Change-Id: I05f5646ffed37143a21ea72f0fd782c2bb5c4d82

add basic video encoder tests

For a given component test successful encoding of nframes of a
raw video clip

Test: tested on softavc encoder.
Change-Id: Ia1d754061e8362f6c08a2be3d6384c95589a36de

Fix Vts errors

- Add REQUEST_NOT_SUPPORT into General Errors
- Add a util function to check General Errors
- Add a util function to check OEM Errors
- Changes to RIL command api tests based on the QC provided pdf
  to see if the errors from vts-web can be fixed

Test: Manual
Bug: 35671595
Change-Id: I9500c686a75539dab37f0272349bc2f965cc0a8f

Fix configureChip() VTS test

Bug: 36562856
Bug: 37446050
Test: compile, vts test pass

Change-Id: I99ee0336d9cc11329de560daf26216ac15e5a0a9

Update .hal file for fixing vts tests

- update 'voice' and 'sms' based on QC pdf file
- remove GENERIC_FAILURE still there
- fix duplicate error code

Note, current.txt updated only because this interface
has not been released as of the time of this CL.

Test: Manual
Bug: 35671595
Change-Id: I83831c222cb0d157d0aa06e48cf0a177ab0ecf31

Return ERROR_NOT_SUPPORTED when configureChip() is used for reconfiguration

Bug: 36562856
Bug: 37446050
Test: compile, VTS test fail as expected
Change-Id: I895dd0d6e96b0d0a2b429c3a68be1f7c7e32a7e3

Merge "Make BT HAL audio threads RT" into oc-dev

Make BT HAL audio threads RT

Bluetooth threads that are used in audio have deadline
requirements for glitchless playback.  Those threads need to be
scheduled as RT tasks to ensure that they can meet the deadline even if
there is high system load.  Use schedulerservice to request RT
scheduling.

Bug 37518404
Test:  play Bluetooth audio, check for RT with systrace
Change-Id: I043a97df3d2801922b54dcbf7c700ae11a48c217

Merge "audiohal: Fix typos in comments, add missing comments" into oc-dev

Merge changes from topic 'hals-should-use-mapper' into oc-dev

* changes:
  graphics: use mapper from the composer
  graphics: clarify importBuffer and passthrough HALs

Add onAcquired log back

It was in fingerprintd, which is replaced now

Fixes: 37677260
Test: touch FP when enrolling/authenticating and make sure
onAcquired log is there

Change-Id: Iadecdfa1c18b0aaa083de18dbe7f701c78465ace

stop bluetooth service while VTS test is running

Test: run VTS HAL BT test
Bug: 37153328
Change-Id: I615d96daff496096122cd18158c205073f4623a6

audiohal: Fix handling of invalid arguments in 'debugDump' methods

Add checks for validity of the passed in native handle.

Bug: 37590777
Change-Id: If906f0086c3442819e1844cc521215d5b74af015
Test: VTS
(cherry picked from commit 7bae6a0b635b142cfba9d4c4612e1e114f85b842)

graphics: use mapper from the composer

We must use the mapper HAL instead of gralloc0/gralloc1 from the
composer.

Bug: 37540361
Test: boots on marlin, angler, and ryu
Change-Id: I5a3ff6a025bf51a3507a4f33fa77e9506a6f1ec9

graphics: clarify importBuffer and passthrough HALs

A buffer handle recieved from a HAL is by definition raw and needs
to be imported.  But because of passthrough HALs, such a raw handle
may have been imported already.  Explicitly specify that an
implementation must accept such a raw handle.

Bug: 37540361
Test: boots on angler, ryu and marlin
Change-Id: I5ecf526e59b27cc4a8f7f5d5ec27477da0946ece

CameraProvider: Check camera device version

Make sure we are working with supported devices.

Bug: 34392075
Test: Manual using application
Change-Id: I834612303f3e02e79924efbdcc25cff890defefb

Camera: Insert default RAW boost if needed

RAW boost key inside result and request metadata could be
absent depending on CameraHal. In case RAW boost range is
supported, the RAW boost value must always be present.

Bug: 34392075
Test: runtest -x
cts/tests/camera/src/android/hardware/camera2/cts/DngCreatorTest.java
Change-Id: I4c27b3fe5228ff604e3f2796abc670be5c17da67

Camera: Override AE pre-capture trigger cancel

CameraHal version 3.2 doesn't support cancel AE pre-capture
trigger. Camera request metadata that contains this value
needs to be overriden respectively. The corresponding results
also require the same kind of modification.

Bug: 34392075
Test: runtest -x
cts/tests/camera/src/android/hardware/camera2/cts/StillCaptureTest.java
-m testAePrecaptureTriggerCancelJpegCapture
Change-Id: If8bcf7e97ca8691cf890d5d6306d49dc3ddd087a

Camera: Map stream dataspaces

Older CameraHal versions are still using deprecated dataspace
definitions. Mapping between these dataspaces is needed to avoid
discrepancies.

Bug: 34392075
Test: Manual using camera application
Change-Id: I811dec879494445736b4e3731b2d516e396a9058

Merge "Fix VtsHalDrmV1_0Target test failure" into oc-dev

audiohal: Fix typos in comments, add missing comments

These changes are in comments only, no ABI breaking changes.

Bug: 32238101
Test: make
Change-Id: Iead4ba01dae1a1c05d8fbed7f194182128e9e6ad

Merge "Adding missing NI_TYPE for emergency SUPL" into oc-dev

Merge changes from topic 'fuguWifi' into oc-dev

* changes:
  Improve logger feature set handling
  Add capabilities for wifi_hal_legacy

Merge "Finish adding manufacturer and model to device ID attestation" into oc-dev

Merge changes from topic 'o-usage64-2' into oc-dev

* changes:
  automotive: Adjust tests for libui gralloc flags change
  camera: Adjust for libui gralloc flags change

Merge "wifi(implementation): Turn on DFS flag for STA" into oc-dev

wifi(implementation): Turn on DFS flag for STA

DFS flag needs to be turned on in the legacy HAL for STA operations. So,
initialize this flag on creation of STA iface.
PS: Not adding an explicit HIDL API for this since there is no need to
toggle this from framework and since the HIDL API's are frozen now.

Bug: 36169790
Test: Verified the fix with regression tests: b/37528903.
Change-Id: I214097fcb399cef7fc0cbd2c278622fac259a3e3

Fix VtsHalDrmV1_0Target test failure

DrmHalVendorFactoryTest.EmptyPluginUUIDNotSupported
was failing on 64 bit builds. The test was using
an uninitialized hidl_array as the empty UUID which
was incorrect. This change initializes the array
contents to null values.

Test: drm hal tests pass

bug: 37172151
Change-Id: I0acb86921f0daceba30fa7879980ce819db64a74

Change bootHAL to early_hal class

Bug: 37579828
Bug: 37543411
Test: marlin boots with wipe data
Change-Id: Ifa508570455289efd58350562fadea5676c3bc67

Merge "Fix failing VTS test cases" into oc-dev

Fix Keymaster HAL crash.

Test: VTS test validates this fix
Change-Id: I5996c2166cf4141c97424df64a380f45b001d52e

Adding missing NI_TYPE for emergency SUPL

Bug: 37409476
Test: Builds, and on Pixel, basic GPS & XTRA tests pass.
Partners & Pixel team running the SUPL emergency tests.

Change-Id: I6b23608e82dc1682d61a516ab5045d3b78b53cae

automotive: Adjust tests for libui gralloc flags change

Change-Id: Ia88d8fc664196eae7153ccfb755ea9c3d135fd0c
Bug: b/33350696
Test: Bullhead: booted to launcher, take photo, watch youtube

camera: Adjust for libui gralloc flags change

Change-Id: Ia09737c28a78a8a50a9f751ed2774299ecc3d58c
Bug: b/33350696
Test: Bullhead: booted to launcher, take photo, watch youtube

Fix failing VTS test cases

Also, increase number of binder threads in VHAL for performance testing

Fix: b/37543801, b/36778517
Bug: b/36510399

Test: make vts BUILD_GOOGLE_VTS=true -j32 && vts-tradefed run commandAndExit vts --skip-all-system-status-check --primary-abi-only --skip-preconditions --module VtsHalAutomotiveVehicleV2_0Host -l INFO && adb shell /data/nativetest64/android.hardware.automotive.vehicle@2.0-manager-unit-tests/android.hardware.automotive.vehicle@2.0-manager-unit-tests
Change-Id: I850c6be328f1a07ae3fda898a18137c76afe3115

Merge "Clarifying comments and todos, update current.txt." into oc-dev

Merge "audio hal: initialize audio patch handle" into oc-dev

Merge "Make setTimeLapseConfig take frequencies instead of periods." into oc-dev

Merge "media: Add getStopTimeOffsetUs in GraphicBufferSource." into oc-dev

Make setTimeLapseConfig take frequencies instead of periods.

Test: Manual use of Camera, Movies, Photos and YouTube apps.

Test: With CtsMediaTestCases.apk installed,
adb shell am instrument -e size small -w
'android.media.cts/android.support.test.runner.AndroidJUnitRunner'

Bug: 37302530
Change-Id: I63ba807955c2e2a7ec7f6ecc2d8a832f252c0c1b

media: Add getStopTimeOffsetUs in GraphicBufferSource.

Test: Compiles
Bug: 35925528
Change-Id: Ie469e42c986211540880f04a5e47efc090817ba8

audio hal: initialize audio patch handle

Initialize audio patch handle to avoid
uninitialized value causing validation
failure in vendor hal audio patch.

Bug: 36994757
Test: Initiate voice call, no audio heard from other end

Change-Id: I4cf57210432430af89109f605813645ff332bfbb

Add missing document for supplicant hal, current.txt.

**Note, hashes are being modified in place ONLY
because this interface has not yet been released
in a release of Android.

This adds additional information for setEapPhase2Method().
This also adds the missing error type for find(), stopFind(), and
reject().

Bug: 37254796
Test: compile
Change-Id: I35735a7ec6ca5d490219bb176f1d6c727432113f

Improve logger feature set handling

Some devices don't support querying logger feature set. In that
case set the logger feature set value to 0 instead of failing
the query for regular WiFi feature set.

Bug: 37105075
Bug: 37483427
Test: get battery stats on Fugu.
Change-Id: I3e162f8ddeaaa1965b41518592a253c6c90ddfa7

Add capabilities for wifi_hal_legacy

Bug: 37105075
Bug: 37483427
Test: Fugu Wifi VendorHal initializes successfully
Change-Id: If30aa641e42a9a4be435db97f36af4d172eb8ebe

Clarifying comments and todos, update current.txt.

**Note, hashes are being modified in place ONLY
because this interface has not yet been released
in a release of Android.

Updated comments for clarity nearby
some removed todo's on telephony APN
and GNSS demodulated satellite data bits.

Fixes: 34108918
Test: Built & run on Pixel, GPS & XTRA pass.

Change-Id: Ia67be4c4c095479bbbf001987c1629b57ca864a5

Finish adding manufacturer and model to device ID attestation

This is a follow-up to change I5a9fd839497976cdb1e44cbe4a2d5b7730732b4c,
where manufacturer and model were added to the set of attestable device
IDs.

Bug: 37522655
Test: GTS com.google.android.gts.security.DeviceIdAttestationHostTest

Change-Id: Ied4246f4fc490feb2093f04c268aab83c8e1326d

Merge "graphics: keep mapper valid during process termination" into oc-dev

Merge "Add current.txt" into oc-dev

graphics: keep mapper valid during process termination

GraphicBufferMapper is valid during process termination.  IMapper
must stay valid as well.  It should not rely on global/static
objects that may have been destructed.

Bug: 37480313
Test: libgui_test
Change-Id: Icb8079153306f2465c26c0f1ce812895ad83f21b