Libva header fix #197
Merged
Libva header fix #197
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Observed build error due to missing assert header. Added assert header. Tests: Prepared EB, there was no build error. Tracked-On: NA Signed-off-by: Ankit Agarwal <[email protected]>
Updated android.hardware.media.c2 hal from hidl V1.0 to aidl V1. The contents of this file was copied from AOSP frameworks/av/media/codec2/hal/services/vendor.cpp and modified then. Tests: Prepared EB and its booted successfully. Tracked-On: OAM-122196 Signed-off-by: Ankit Agarwal <[email protected]>
Start with Android U, the get function of IMapper4 will check whether the buffer handle is reserved. 1. Call importBuffer to preserve handle before getting the buffer's info. 2. Call freeBuffer at the end. 3. Switch to use gralloc4. Tracked-On: OAM-112928 Signed-off-by: zhangyichix <[email protected]>
As IMapper4 is deprecated from Android15, changes done to use IMapper5 stable-c mapper. Tests done: - Android boot in GVT-d and BM config - H/W accelerated video playback - adb reboot Tracked-On: OAM-123348 Signed-off-by: Jeevaka Prabu Badrappan <[email protected]>
Signed-off-by: Shi, Qiming <[email protected]>
As libmfx_c2_components_hw depends on libva headers, add libva_headers to header_libs. Tests done: - Android boot in GVT-d - Camera preview and capture - Video playback and recording - adb reboot Tracked-On: OAM-126508 Signed-off-by: JeevakaPrabu <[email protected]>
sysopenci
added
Valid commit message
Pending Developer Approval
sysopenci
requested review from
dyang23,
feijiang1,
gkdeepa,
TianmiChen,
YuanjunHuang and
yyao3
TianmiChen
reviewed
Oct 22, 2024
|
|
||
| name: "[email protected]", | ||
| defaults: ["libcodec2-hidl-defaults", "mfx_c2_exe_defaults"], | ||
| name: "android.hardware.media.c2-service.intel", |
There was a problem hiding this comment.
hi Jeeva, could you please share why we rename the service name here? Is there any naming rule defined in Android 15?
There was a problem hiding this comment.
aidl naming convention removed the version information. Thats the reason behind the name change to android.hardware.media.c2-service.intel
TianmiChen
approved these changes
Oct 24, 2024
sysopenci
removed request for
yyao3,
gkdeepa,
YuanjunHuang,
feijiang1 and
dyang23
|
Android CI has started MERGE Build for this pr ,Please check the linked Tracked-On issue/Android CI Web for more details. |
|
Android CI has completed MERGE Build for this pr, build is SUCCESS. Please check the linked Tracked-On issue/Android CI Web for more details. For Binaries: /cactus-absp-or-local/celadon-merge/793 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.