diff --git a/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp b/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp index 31e8014a84..ae56aee9b0 100644 --- a/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp +++ b/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp @@ -32,6 +32,7 @@ #include #include #include +#include using ::aidl::android::hardware::camera::common::CameraDeviceStatus; using ::aidl::android::hardware::camera::common::CameraResourceCost; @@ -1675,9 +1676,9 @@ TEST_P(CameraAidlTest, processMultiCaptureRequestPreview) { BufferStatus::OK, NativeHandle(), NativeHandle()}; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)), + static_cast(halStream.consumerUsage))), halStream.overrideFormat, &buffer_handle); graphicBuffers.push_back(buffer_handle); outputBuffers[k] = { @@ -1896,9 +1897,9 @@ TEST_P(CameraAidlTest, processUltraHighResolutionRequest) { NativeHandle(), NativeHandle()}; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)), + static_cast(halStream.consumerUsage))), halStream.overrideFormat, &buffer_handle); graphicBuffers.push_back(buffer_handle); outputBuffers[k] = { @@ -2064,9 +2065,9 @@ TEST_P(CameraAidlTest, process10BitDynamicRangeRequest) { NativeHandle(), BufferStatus::OK, NativeHandle(), NativeHandle()}; } else { - auto usage = android_convertGralloc1To0Usage( + auto usage = ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)); + static_cast(halStream.consumerUsage))); allocateGraphicBuffer(previewStream.width, previewStream.height, usage, halStream.overrideFormat, &buffer_handle); @@ -2285,9 +2286,9 @@ TEST_P(CameraAidlTest, processCaptureRequestBurstISO) { NativeHandle(), NativeHandle()}; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStreams[0].producerUsage), - static_cast(halStreams[0].consumerUsage)), + static_cast(halStreams[0].consumerUsage))), halStreams[0].overrideFormat, &buffers[i]); outputBuffer = {halStreams[0].id, bufferId + i, ::android::makeToAidl(buffers[i]), BufferStatus::OK, NativeHandle(), NativeHandle()}; @@ -2385,9 +2386,9 @@ TEST_P(CameraAidlTest, processCaptureRequestInvalidSinglePreview) { bufferId = 0; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStreams[0].producerUsage), - static_cast(halStreams[0].consumerUsage)), + static_cast(halStreams[0].consumerUsage))), halStreams[0].overrideFormat, &buffer_handle); } @@ -2508,9 +2509,9 @@ TEST_P(CameraAidlTest, switchToOffline) { } else { // jpeg buffer (w,h) = (blobLen, 1) allocateGraphicBuffer(jpegBufferSize, /*height*/ 1, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)), + static_cast(halStream.consumerUsage))), halStream.overrideFormat, &buffers[i]); outputBuffer = {halStream.id, bufferId + i, ::android::makeToAidl(buffers[i]), BufferStatus::OK, NativeHandle(), NativeHandle()}; @@ -2734,9 +2735,9 @@ TEST_P(CameraAidlTest, flushPreviewRequest) { BufferStatus::OK, NativeHandle(), NativeHandle()}; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStreams[0].producerUsage), - static_cast(halStreams[0].consumerUsage)), + static_cast(halStreams[0].consumerUsage))), halStreams[0].overrideFormat, &buffer_handle); outputBuffer = {halStreams[0].id, bufferId, ::android::makeToAidl(buffer_handle), BufferStatus::OK, NativeHandle(), NativeHandle()}; diff --git a/camera/provider/aidl/vts/camera_aidl_test.cpp b/camera/provider/aidl/vts/camera_aidl_test.cpp index a01fb04dc5..1ad5af92d9 100644 --- a/camera/provider/aidl/vts/camera_aidl_test.cpp +++ b/camera/provider/aidl/vts/camera_aidl_test.cpp @@ -41,6 +41,7 @@ #include #include #include "utils/Errors.h" +#include using ::aidl::android::hardware::camera::common::CameraDeviceStatus; using ::aidl::android::hardware::camera::common::TorchModeStatus; @@ -2340,8 +2341,8 @@ void CameraAidlTest::processCaptureRequestInternal(uint64_t bufferUsage, /* We don't look at halStreamConfig.streams[0].consumerUsage * since that is 0 for output streams */ - android_convertGralloc1To0Usage( - static_cast(halStreams[0].producerUsage), bufferUsage), + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( + static_cast(halStreams[0].producerUsage), bufferUsage)), halStreams[0].overrideFormat, &handle); outputBuffer = {halStreams[0].id, bufferId, ::android::makeToAidl(handle), @@ -2868,9 +2869,9 @@ void CameraAidlTest::processPreviewStabilizationCaptureRequestInternal( /* We don't look at halStreamConfig.streams[0].consumerUsage * since that is 0 for output streams */ - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStreams[0].producerUsage), - GRALLOC1_CONSUMER_USAGE_HWCOMPOSER), + GRALLOC1_CONSUMER_USAGE_HWCOMPOSER)), halStreams[0].overrideFormat, &buffer_handle); outputBuffer = {halStreams[0].id, bufferId, ::android::makeToAidl(buffer_handle), BufferStatus::OK, NativeHandle(), NativeHandle()}; @@ -3851,9 +3852,9 @@ void CameraAidlTest::processColorSpaceRequest( NativeHandle(), BufferStatus::OK, NativeHandle(), NativeHandle()}; } else { - auto usage = android_convertGralloc1To0Usage( + auto usage = ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)); + static_cast(halStream.consumerUsage))); allocateGraphicBuffer(previewStream.width, previewStream.height, usage, halStream.overrideFormat, &buffer_handle); @@ -4009,9 +4010,9 @@ void CameraAidlTest::processZoomSettingsOverrideRequests( NativeHandle(), NativeHandle()}; } else { allocateGraphicBuffer(previewStream.width, previewStream.height, - android_convertGralloc1To0Usage( + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage( static_cast(halStreams[0].producerUsage), - static_cast(halStreams[0].consumerUsage)), + static_cast(halStreams[0].consumerUsage))), halStreams[0].overrideFormat, &buffers[i]); outputBuffer = {halStreams[0].id, bufferId + i, ::android::makeToAidl(buffers[i]), BufferStatus::OK, NativeHandle(), NativeHandle()}; diff --git a/camera/provider/aidl/vts/device_cb.cpp b/camera/provider/aidl/vts/device_cb.cpp index 2c11d3f9be..8a8b925806 100644 --- a/camera/provider/aidl/vts/device_cb.cpp +++ b/camera/provider/aidl/vts/device_cb.cpp @@ -20,6 +20,7 @@ #include #include #include +#include using ::aidl::android::hardware::camera::device::BufferStatus; using ::aidl::android::hardware::camera::device::ErrorMsg; @@ -143,8 +144,8 @@ ScopedAStatus DeviceCb::requestStreamBuffers(const std::vector& b CameraAidlTest::allocateGraphicBuffer( w, h, - android_convertGralloc1To0Usage(static_cast(halStream.producerUsage), - static_cast(halStream.consumerUsage)), + ANDROID_NATIVE_UNSIGNED_CAST(android_convertGralloc1To0Usage(static_cast(halStream.producerUsage), + static_cast(halStream.consumerUsage))), halStream.overrideFormat, &handle); StreamBuffer streamBuffer = StreamBuffer();