[libcamera-devel] [PATCH 08/11] android: camera_device: Use abortRequest() instead of open-coding it

[Umang Jain]

Hi Laurent,

Thank you for the patch,

On 10/18/21 6:59 PM, Umang Jain wrote:
> From: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
>
> Call abortRequest() in CameraDevice::requestComplete() instead of
> open-coding it.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>


Reviewed-by: Umang Jain<umang.jain at ideasonboard.com>

> ---
>   src/android/camera_device.cpp | 21 ++++++---------------
>   1 file changed, 6 insertions(+), 15 deletions(-)
>
> diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp
> index cd9e9fe2..216f29c2 100644
> --- a/src/android/camera_device.cpp
> +++ b/src/android/camera_device.cpp
> @@ -831,6 +831,11 @@ void CameraDevice::abortRequest(Camera3RequestDescriptor *descriptor) const
>   	notifyError(descriptor->frameNumber_, nullptr, CAMERA3_MSG_ERROR_REQUEST);
>   
>   	for (auto &buffer : descriptor->buffers_) {
> +		/*
> +		 * Signal to the framework it has to handle fences that have not
> +		 * been waited on by setting the release fence to the acquire
> +		 * fence value.
> +		 */
>   		buffer.buffer.release_fence = buffer.buffer.acquire_fence;
>   		buffer.buffer.acquire_fence = -1;
>   		buffer.buffer.status = CAMERA3_BUFFER_STATUS_ERROR;
> @@ -1111,21 +1116,7 @@ void CameraDevice::requestComplete(Request *request)
>   				<< " not successfully completed: "
>   				<< request->status();
>   
> -		notifyError(descriptor->frameNumber_, nullptr,
> -			    CAMERA3_MSG_ERROR_REQUEST);
> -
> -		for (auto &buffer : descriptor->buffers_) {
> -			/*
> -			 * Signal to the framework it has to handle fences that
> -			 * have not been waited on by setting the release fence
> -			 * to the acquire fence value.
> -			 */
> -			buffer.buffer.release_fence = buffer.buffer.acquire_fence;
> -			buffer.buffer.acquire_fence = -1;
> -			buffer.buffer.status = CAMERA3_BUFFER_STATUS_ERROR;
> -		}
> -
> -		descriptor->status_ = Camera3RequestDescriptor::Status::Error;
> +		abortRequest(descriptor);
>   		sendCaptureResults();
>   
>   		return;