From patchwork Wed Oct 21 15:41:41 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kieran Bingham X-Patchwork-Id: 10185 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 15BE7BDB13 for ; Wed, 21 Oct 2020 15:42:02 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id C1DBD60F3E; Wed, 21 Oct 2020 17:42:01 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="DONrD6FI"; dkim-atps=neutral Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 1A8AF60C21 for ; Wed, 21 Oct 2020 17:41:56 +0200 (CEST) Received: from Q.local (cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id A99BFBB5; Wed, 21 Oct 2020 17:41:55 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1603294915; bh=hfSr/lTm/td/TXGUPuLj4tL0PxxXw6o1IR2NExokVx4=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=DONrD6FIUO3tTjsDNHbgSE8heWjuydKkO1FfF5fqxqEt2NDVdpejfek4TOQjt9Z9w rwIbCdNHWOa8XfL6rUE8m5y7OqenR8pAAuAZPiIJ7uAFRE0SytcHhA//Cf9SFimNyg OZluLRrF1yxbeWQOOUVytETAV+pK+dp6HP9othho= From: Kieran Bingham To: libcamera devel Date: Wed, 21 Oct 2020 16:41:41 +0100 Message-Id: <20201021154148.511505-9-kieran.bingham@ideasonboard.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20201021154148.511505-1-kieran.bingham@ideasonboard.com> References: <20201021154148.511505-1-kieran.bingham@ideasonboard.com> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v4 08/15] android: camera_device: use member style on Camera3RequestDescriptor X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Use the postfixed '_' member variable naming style for the Camera3RequestDescriptor structure, which in turn ensures that variable shadowing does not occur on the constructor. Signed-off-by: Kieran Bingham Reviewed-by: Laurent Pinchart --- src/android/camera_device.cpp | 56 +++++++++++++++++------------------ src/android/camera_device.h | 10 +++---- 2 files changed, 33 insertions(+), 33 deletions(-) diff --git a/src/android/camera_device.cpp b/src/android/camera_device.cpp index 5272b9ecded7..9cf1c98410f9 100644 --- a/src/android/camera_device.cpp +++ b/src/android/camera_device.cpp @@ -170,28 +170,28 @@ MappedCamera3Buffer::MappedCamera3Buffer(const buffer_handle_t camera3buffer, CameraDevice::Camera3RequestDescriptor::Camera3RequestDescriptor( Camera *camera, unsigned int frameNumber, unsigned int numBuffers) - : frameNumber(frameNumber), numBuffers(numBuffers) + : frameNumber_(frameNumber), numBuffers_(numBuffers) { - buffers = new camera3_stream_buffer_t[numBuffers]; + buffers_ = new camera3_stream_buffer_t[numBuffers]; /* * FrameBuffer instances created by wrapping a camera3 provided dmabuf * are emplaced in this vector of unique_ptr<> for lifetime management. */ - frameBuffers.reserve(numBuffers); + frameBuffers_.reserve(numBuffers); /* * Create the libcamera::Request unique_ptr<> to tie its lifetime * to the descriptor's one. Set the descriptor's address as the * request's cookie to retrieve it at completion time. */ - request = std::make_unique(camera, - reinterpret_cast(this)); + request_ = std::make_unique(camera, + reinterpret_cast(this)); } CameraDevice::Camera3RequestDescriptor::~Camera3RequestDescriptor() { - delete[] buffers; + delete[] buffers_; } /* @@ -1393,8 +1393,8 @@ int CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reques camera3Request->num_output_buffers); LOG(HAL, Debug) << "Queueing Request to libcamera with " - << descriptor->numBuffers << " HAL streams"; - for (unsigned int i = 0; i < descriptor->numBuffers; ++i) { + << descriptor->numBuffers_ << " HAL streams"; + for (unsigned int i = 0; i < descriptor->numBuffers_; ++i) { camera3_stream *camera3Stream = camera3Buffers[i].stream; CameraStream *cameraStream = static_cast(camera3Buffers[i].stream->priv); @@ -1403,8 +1403,8 @@ int CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reques * Keep track of which stream the request belongs to and store * the native buffer handles. */ - descriptor->buffers[i].stream = camera3Buffers[i].stream; - descriptor->buffers[i].buffer = camera3Buffers[i].buffer; + descriptor->buffers_[i].stream = camera3Buffers[i].stream; + descriptor->buffers_[i].buffer = camera3Buffers[i].buffer; std::stringstream ss; ss << i << " - (" << camera3Stream->width << "x" @@ -1435,7 +1435,7 @@ int CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reques * lifetime management only. */ buffer = createFrameBuffer(*camera3Buffers[i].buffer); - descriptor->frameBuffers.emplace_back(buffer); + descriptor->frameBuffers_.emplace_back(buffer); LOG(HAL, Debug) << ss.str() << " (direct)"; break; @@ -1458,12 +1458,12 @@ int CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reques return -ENOMEM; } - descriptor->request->addBuffer(cameraStream->stream(), buffer, - camera3Buffers[i].acquire_fence); + descriptor->request_->addBuffer(cameraStream->stream(), buffer, + camera3Buffers[i].acquire_fence); } /* Queue the request to the CameraWorker. */ - worker_.queueRequest(descriptor->request.get()); + worker_.queueRequest(descriptor->request_.get()); return 0; } @@ -1489,13 +1489,13 @@ void CameraDevice::requestComplete(Request *request) * pipeline handlers) timestamp in the Request itself. */ FrameBuffer *buffer = buffers.begin()->second; - resultMetadata = getResultMetadata(descriptor->frameNumber, + resultMetadata = getResultMetadata(descriptor->frameNumber_, buffer->metadata().timestamp); /* Handle any JPEG compression. */ - for (unsigned int i = 0; i < descriptor->numBuffers; ++i) { + for (unsigned int i = 0; i < descriptor->numBuffers_; ++i) { CameraStream *cameraStream = - static_cast(descriptor->buffers[i].stream->priv); + static_cast(descriptor->buffers_[i].stream->priv); if (cameraStream->camera3Stream().format != HAL_PIXEL_FORMAT_BLOB) continue; @@ -1511,7 +1511,7 @@ void CameraDevice::requestComplete(Request *request) * separate thread. */ - MappedCamera3Buffer mapped(*descriptor->buffers[i].buffer, + MappedCamera3Buffer mapped(*descriptor->buffers_[i].buffer, PROT_READ | PROT_WRITE); if (!mapped.isValid()) { LOG(HAL, Error) << "Failed to mmap android blob buffer"; @@ -1535,19 +1535,19 @@ void CameraDevice::requestComplete(Request *request) /* Prepare to call back the Android camera stack. */ camera3_capture_result_t captureResult = {}; - captureResult.frame_number = descriptor->frameNumber; - captureResult.num_output_buffers = descriptor->numBuffers; - for (unsigned int i = 0; i < descriptor->numBuffers; ++i) { - descriptor->buffers[i].acquire_fence = -1; - descriptor->buffers[i].release_fence = -1; - descriptor->buffers[i].status = status; + captureResult.frame_number = descriptor->frameNumber_; + captureResult.num_output_buffers = descriptor->numBuffers_; + for (unsigned int i = 0; i < descriptor->numBuffers_; ++i) { + descriptor->buffers_[i].acquire_fence = -1; + descriptor->buffers_[i].release_fence = -1; + descriptor->buffers_[i].status = status; } captureResult.output_buffers = - const_cast(descriptor->buffers); + const_cast(descriptor->buffers_); if (status == CAMERA3_BUFFER_STATUS_OK) { - notifyShutter(descriptor->frameNumber, + notifyShutter(descriptor->frameNumber_, buffer->metadata().timestamp); captureResult.partial_result = 1; @@ -1561,8 +1561,8 @@ void CameraDevice::requestComplete(Request *request) * is here signalled. Make sure the error path plays well with * the camera stack state machine. */ - notifyError(descriptor->frameNumber, - descriptor->buffers[0].stream); + notifyError(descriptor->frameNumber_, + descriptor->buffers_[0].stream); } callbacks_->process_capture_result(callbacks_, &captureResult); diff --git a/src/android/camera_device.h b/src/android/camera_device.h index 86f2b8974b53..fd08738a5351 100644 --- a/src/android/camera_device.h +++ b/src/android/camera_device.h @@ -79,11 +79,11 @@ private: unsigned int numBuffers); ~Camera3RequestDescriptor(); - uint32_t frameNumber; - uint32_t numBuffers; - camera3_stream_buffer_t *buffers; - std::vector> frameBuffers; - std::unique_ptr request; + uint32_t frameNumber_; + uint32_t numBuffers_; + camera3_stream_buffer_t *buffers_; + std::vector> frameBuffers_; + std::unique_ptr request_; }; struct Camera3StreamConfiguration {