diff options
| author | Ankit Goyal <layog@google.com> | 2023-03-01 16:52:36 -0800 |
|---|---|---|
| committer | Android Build Coastguard Worker <android-build-coastguard-worker@google.com> | 2023-03-02 20:34:36 +0000 |
| commit | 6a433f083ca509e634580da958c32fb46ddb160c (patch) | |
| tree | f80fcfbfd3ed22d31fc90fed9693c662c624eb9f | |
| parent | c46c08205bfcc2021714296e1906dc2ce0c38bef (diff) | |
Revert "gralloc4: Warn about multifd PlaneLayout once"
This reverts commit 370e40a3db70425e6bae5dc9c22fc4bd6e0fa3b4.
Bug: 269705937
Test: No display artifacts after camera shot
Change-Id: I8a044c10cf4e01786e30813cc703569ee2024baa
(cherry picked from commit 955a611bb3a2ee61a706f22deb71cde7b0dbe96a)
Merged-In: I8a044c10cf4e01786e30813cc703569ee2024baa
| -rw-r--r-- | gralloc4/src/hidl_common/MapperMetadata.cpp | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/gralloc4/src/hidl_common/MapperMetadata.cpp b/gralloc4/src/hidl_common/MapperMetadata.cpp index fb79930..1123270 100644 --- a/gralloc4/src/hidl_common/MapperMetadata.cpp +++ b/gralloc4/src/hidl_common/MapperMetadata.cpp @@ -305,15 +305,11 @@ static android::status_t get_plane_layouts(const private_handle_t *handle, std:: int64_t sample_increment_in_bits = format_info.bpp[plane_index]; int64_t offset = handle->plane_info[plane_index].offset; - static bool warn_multifd = true; - if (warn_multifd) { - uint8_t fd_count = get_exynos_fd_count(base_format); - if (fd_count != 1) { - warn_multifd = false; - MALI_GRALLOC_LOGW("Offsets in plane layouts of multi-fd format (%s %" PRIu64 - ") are not reliable. This can lead to image corruption.", - format_name(base_format), handle->alloc_format); - } + uint8_t fd_count = get_exynos_fd_count(base_format); + if (fd_count != 1) { + MALI_GRALLOC_LOGW("Offsets in plane layouts of multi-fd format (%s %" PRIu64 + ") are not reliable. This can lead to image corruption.", + format_name(base_format), handle->alloc_format); } PlaneLayout layout = {.offsetInBytes = offset, |