Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gralloc: unify gralloc frontends to use cros_gralloc_convert_format #74

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

chenyanxzhu
Copy link
Contributor

No description provided.

@sysopenci
Copy link

Improper Commit Message
Tracked on not found in commit message,
make sure Tracked-On: Jira-ticket is present.

The previous ANDROID_VERSION is also checked against the wrong version
because the fp16 and 10bit rgba formats were released in 8.0. So this
change also fixes to check against api level 26 instead.

BUG=b:199524294
TEST=emerge-$BOARD arc-cros-gralloc

Tracked-On: OAM-111243
Change-Id: I16c1dba4901b374c5aecbc0e0f859638d08895aa
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3158977
Tested-by: Yiwei Zhang <zzyiwei@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
Commit-Queue: Yiwei Zhang <zzyiwei@chromium.org>
Signed-off-by: chenyanxzhu <chenyanx.zhu@intel.com>
Reordered the cases to align with actual enum value for the hal formats,
and added 3 more formats support to align with gralloc4 support:
1. Y8
2. Y16
3. YCBCR_P010

BUG=b:199524294
TEST=CtsNativeHardwareTestCases
TEST=gralloc4 builds in aosp

Tracked-On: OAM-111243
Change-Id: I7a6743524c024d46ab2a7a6d973f431ea2721c49
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/minigbm/+/3160131
Tested-by: Yiwei Zhang <zzyiwei@chromium.org>
Auto-Submit: Yiwei Zhang <zzyiwei@chromium.org>
Reviewed-by: Jason Macnak <natsu@google.com>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>
Commit-Queue: Yiwei Zhang <zzyiwei@chromium.org>
Signed-off-by: chenyanxzhu <chenyanx.zhu@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Stale Stale label for inactive open prs Valid commit message
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants