From b6809a06ad090807d1b0fe2e44d6aef06eb5d79d Mon Sep 17 00:00:00 2001 From: Marissa Wall Date: Thu, 13 Feb 2020 11:41:26 -0800 Subject: [PATCH] gralloc4-vts: update sampleIncrementInBits sampleIncrementInBits is per sample not per component. An RGBA sample is 32 bits not 8. Test: VtsHalGraphicsMapperV4_0 Bug: 149310539 Change-Id: If4bd5aac87cada7040f52ee40159a72fa72a97be --- .../4.0/vts/functional/VtsHalGraphicsMapperV4_0TargetTest.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/graphics/mapper/4.0/vts/functional/VtsHalGraphicsMapperV4_0TargetTest.cpp b/graphics/mapper/4.0/vts/functional/VtsHalGraphicsMapperV4_0TargetTest.cpp index aa45e3bb8b..726df9357d 100644 --- a/graphics/mapper/4.0/vts/functional/VtsHalGraphicsMapperV4_0TargetTest.cpp +++ b/graphics/mapper/4.0/vts/functional/VtsHalGraphicsMapperV4_0TargetTest.cpp @@ -143,7 +143,7 @@ class GraphicsMapperHidlTest EXPECT_EQ(24, offsetInBitsA); EXPECT_EQ(0, planeLayout.offsetInBytes); - EXPECT_EQ(8, planeLayout.sampleIncrementInBits); + EXPECT_EQ(32, planeLayout.sampleIncrementInBits); // Skip testing stride because any stride is valid EXPECT_EQ(mDummyDescriptorInfo.width, planeLayout.widthInSamples); EXPECT_EQ(mDummyDescriptorInfo.height, planeLayout.heightInSamples); @@ -1384,7 +1384,7 @@ TEST_P(GraphicsMapperHidlTest, SetPlaneLayouts) { planeLayouts.push_back(planeLayoutA); planeLayoutRGB.offsetInBytes = 0; - planeLayoutRGB.sampleIncrementInBits = 32; + planeLayoutRGB.sampleIncrementInBits = 24; planeLayoutRGB.strideInBytes = info.width + 20; planeLayoutRGB.widthInSamples = info.width; planeLayoutRGB.heightInSamples = info.height;