diff --git a/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp b/camera/provider/aidl/vts/VtsAidlHalCameraProvider_TargetTest.cpp index e335853c28..c1bffb4164 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; @@ -1622,9 +1623,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] = { @@ -1838,9 +1839,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] = { @@ -2004,9 +2005,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); @@ -2223,9 +2224,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()}; @@ -2323,9 +2324,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); } @@ -2446,9 +2447,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()}; @@ -2672,9 +2673,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 f70e93c7c1..ed34b191e3 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; @@ -2263,8 +2264,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), @@ -2775,9 +2776,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()}; @@ -3757,9 +3758,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); @@ -3913,9 +3914,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();