OSDN Git Service

Revert "minigbm: rockchip: align NV12 planes to 64 bytes"
authorPin-chih Lin <johnylin@chromium.org>
Tue, 25 Jul 2017 08:06:26 +0000 (08:06 +0000)
committerchrome-bot <chrome-bot@chromium.org>
Tue, 25 Jul 2017 14:00:26 +0000 (07:00 -0700)
commit19643418b4f871b3b31577766953bbfda76d3be4
tree3cb7f7c67e1b735b0403d2c0f4c5635abbf0d947
parent581f3a5e3089cfd5bb38432eb2874d9a5b132a17
Revert "minigbm: rockchip: align NV12 planes to 64 bytes"

This reverts commit 33cca93af658650b655b4b5dcfb18a3235a100cf.

Reason for revert:

Many CTS tests are failed by the change, and video playback is corrupted on Youtube/Netflix app.

Original change's description:
> minigbm: rockchip: align NV12 planes to 64 bytes
>
> When running the mapped_texture_test with the Mali EGL implementation
> compiled with debug symbols, Mali complains that the NV12 planes are
> not sufficiently aligned (specifically, mali_cobj_surface.c line 1806).
> Fix this.
>
> BUG=b:63910596
> TEST=run Youtube App video on Kevin, at fullscreen and at normal size
>
> Change-Id: Ia204360c3aebe2bbd545ac7be0f6d095b0412f9d
> Reviewed-on: https://chromium-review.googlesource.com/556540
> Commit-Ready: Gurchetan Singh <gurchetansingh@chromium.org>
> Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>
> Reviewed-by: Stphane Marchesin <marcheu@chromium.org>

Bug: b:63999659
Test: 1) run CTS on minnie: CtsVideoTestCases android.video.cts.VideoEncoderDecoderTest#testAvcGoog0Qual0720x0480, which fails on ToT and passes with the revert.
2) Play a video (size:426x240) on Youtube app. Video is corrupted on ToT and good with the revert.

Change-Id: I3418d0065733b4e0f99d1ab8c3cd416f7f454bc0
Reviewed-on: https://chromium-review.googlesource.com/583696
Commit-Ready: Pin-chih Lin <johnylin@chromium.org>
Tested-by: Pin-chih Lin <johnylin@chromium.org>
Reviewed-by: Tomasz Figa <tfiga@chromium.org>
rockchip.c