android_convertGralloc1To0Usage() is used to convert gralloc1_usage(uint64_t) to gralloc_usage (int32_t),
then passed as a para in allocateGraphicBuffer(). But definition of allocateGraphicBuffer() is as
void CameraAidlTest::allocateGraphicBuffer(uint32_t width, uint32_t height, uint64_t usage, PixelFormat format, buffer_handle_t* buffer_handle).
The type of "usage" is uint64_t. So the convert (uint64_t -> int32_t -> uint64_t) will change usages(31 bit is 1)
such as 0x0000000080020b00 to 0xffffffff80020b00. It's unexpected.
Use ANDROID_NATIVE_UNSIGNED_CAST to fix the issue.
Bug: https://issuetracker.google.com/issues/330051542
Change-Id: I23dfd603be0da6864d1021a57c3c3ce76cb78bda
Signed-off-by: Fang Hui <hui.fang@nxp.com>
We need to make sure there is no duplicate binder call to HAL layer.
This patch fix this.
Test: atest VtsHalThreadNetworkTargetTest
Bug: 338556737
Change-Id: Ic414ec1099049ba5e7a0c17baffcd467aa910034
Aconfig flagging storage is migrating to a new underlying storage, which
has new dependencies. Add the dependencies as a defaults.
Bug: 328444881
Test: m
Change-Id: Idca333a4f4198ac5c8724a9696a9f68433a5f39b
This commit enables support of dynamic radio URLs, so that the radio
URL can be runtime generated.
Bug: 329188649
Test: presubmit
Change-Id: I11a0d3de47609d59ba50c8b5317c8238f09e18dd
Make it clearer that the default implementation is non-secure in
various ways, most notably because the TA code is running in-process
in the HAL service.
Bug: 314513765
Test: VtsAidlKeyMintTargetTest
Change-Id: I83509110c32a4df8df6c0b288d14659e022442ff
Stable-c mapper instance should be declared in two ways:
- vintf manifest
- service_contexts
The latter part was missing.
Bug: 336010345
Change-Id: I7bd776bf7d943049ef319c90739dae10071409ab
Test: documentation
am skip reason: Merged-In I75414ba18be93707310cace21b52508edfc33a20 with SHA-1 8877018da4 is already in history
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2884867
Change-Id: I0db01ce7eb446bdf07f1e80e07ef8381236ca220
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
am skip reason: Merged-In I75414ba18be93707310cace21b52508edfc33a20 with SHA-1 8877018da4 is already in history
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2884867
Change-Id: I22ca2323fb6943e0910b344fc38070d64c1f9c74
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
relaxing several parameter checking to align with HIDL
- no need to have stage in use to set bands/channels
- band settings no need to be sorted by frequency
Bug: 328012516
Test: atest VtsHalDynamicsProcessingTargetTest
Change-Id: If12d978ee69ee7f087a7e8758513a9c6bacf817f
Merged-In: If12d978ee69ee7f087a7e8758513a9c6bacf817f
Some audio outputs use A/V sync and requre mode bursts
in order to start reporting the presentation position.
Bug: 300735639
Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icad0942f2ba1dcd6f030a7dc4f37e22fdbd6dd71
According to the definition of the 'PAUSED' state
in StreamDescriptor.aidl, s/w (the client) stops
writing once the buffer is filled up. That means, it is
allowed for an output stream not to consume data
from the MQ while in the paused state, so allow that
in the test.
Also, update the state transition sequence in the test
to flush any data after making a burst while in the 'PAUSED'
state.
Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icb5fd02ca4ede63d7ae33613ab66cb96f3e6df29
The test was using '0' as an "invalid" patch ID value, however
this value is valid in the context of 'IModule.setAudioPatch'
method and means "create a new patch and allocate and ID for it".
Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: Icd33f3cbd1602ec5aa162fa72fc3ddd59ccffbef
This is aligned with the HIDL implementation VTS. The echo reference
device can't provide any input until certain preconditions are met,
and modeling these preconditions in the test is not trivial.
Also, add the information into the mix port into the trace scope
for easier identification on test failure.
Bug: 328010709
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I737479d8ef1961791ac3bd82aeb779453d2e49f4
am skip reason: Merged-In Idd78c0dae944e3d39a6828b43db5a608adbaf1d9 with SHA-1 056aa140ca is already in history
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3047257
Change-Id: I04959cdfd8c40c3c289ffb3e675db9678286cdec
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
A callback should be executed every time the NFC interface is opened,
even in OpenAfterOpen scenarios. If a callback is not executed, a
crash may occur where the NfcService hangs while waiting for an
OPEN_CPLT event.
Bug: 327582461
Merged-In: Idd78c0dae944e3d39a6828b43db5a608adbaf1d9
Change-Id: Idd78c0dae944e3d39a6828b43db5a608adbaf1d9
Signed-off-by: Bruno Carneiro <bccunha@motorola.com>
(cherry picked from commit 056aa140ca)
am skip reason: Merged-In Idad4127ce26188145c7b07d60d5e6ce40a80aa5b with SHA-1 73e55bdbcf is already in history
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/3044892
Change-Id: I6b5eba0db4184caf5ffe4822d3dc6b9e4c6de974
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Since "active" means "used by the stream for acquiring data,"
it was unreasonable to expect the list of active microphones
to be non-empty prior to actually starting data acquisition.
This change adds running of 'burst' commands before calling
'getActiveMicrophones'.
To reuse existing code some refactorings have been made.
Added 'AudioInputFlags::HOTWORD_TAP' to the list of port
config flags for which I/O testing is not performed.
Bug: 328010709
Bug: 328362233
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I876c0b6d7365e104ec9ed8cf5033a83f822006b6
