@@ -91,9 +91,6 @@ public:
std::unique_ptr<CameraConfiguration> generateConfiguration(const StreamRoles &roles);
int configure(CameraConfiguration *config);
- int allocateBuffers();
- int freeBuffers();
-
Request *createRequest(uint64_t cookie = 0);
int queueRequest(Request *request);
@@ -105,7 +102,6 @@ private:
CameraAvailable,
CameraAcquired,
CameraConfigured,
- CameraPrepared,
CameraRunning,
};
@@ -77,8 +77,6 @@ int CameraDevice::open()
void CameraDevice::close()
{
camera_->stop();
-
- camera_->freeBuffers();
camera_->release();
running_ = false;
@@ -690,16 +688,9 @@ void CameraDevice::processCaptureRequest(camera3_capture_request_t *camera3Reque
/* Start the camera if that's the first request we handle. */
if (!running_) {
- int ret = camera_->allocateBuffers();
- if (ret) {
- LOG(HAL, Error) << "Failed to allocate buffers";
- return;
- }
-
- ret = camera_->start();
+ int ret = camera_->start();
if (ret) {
LOG(HAL, Error) << "Failed to start camera";
- camera_->freeBuffers();
return;
}
@@ -42,12 +42,6 @@ int Capture::run(EventLoop *loop, const OptionsParser::Options &options)
return ret;
}
- ret = camera_->allocateBuffers();
- if (ret) {
- std::cerr << "Failed to allocate buffers" << std::endl;
- return ret;
- }
-
camera_->requestCompleted.connect(this, &Capture::requestComplete);
if (options.isSet(OptFile)) {
@@ -67,8 +61,6 @@ int Capture::run(EventLoop *loop, const OptionsParser::Options &options)
writer_ = nullptr;
}
- camera_->freeBuffers();
-
delete allocator;
return ret;
@@ -275,15 +275,13 @@ std::size_t CameraConfiguration::size() const
* \section camera_operation Operating the Camera
*
* An application needs to perform a sequence of operations on a camera before
- * it is ready to process requests. The camera needs to be acquired, configured
- * and resources allocated or imported to prepare the camera for capture. Once
- * started the camera can process requests until it is stopped. When an
- * application is done with a camera all resources allocated need to be freed
- * and the camera released.
+ * it is ready to process requests. The camera needs to be acquired and
+ * configured to prepare the camera for capture. Once started the camera can
+ * process requests until it is stopped. When an application is done with a
+ * camera, the camera needs to be released.
*
* An application may start and stop a camera multiple times as long as it is
- * not released. The camera may also be reconfigured provided that all
- * resources allocated are freed prior to the reconfiguration.
+ * not released. The camera may also be reconfigured.
*
* \subsection Camera States
*
@@ -297,7 +295,6 @@ std::size_t CameraConfiguration::size() const
* node [shape = doublecircle ]; Available;
* node [shape = circle ]; Acquired;
* node [shape = circle ]; Configured;
- * node [shape = circle ]; Prepared;
* node [shape = circle ]; Running;
*
* Available -> Available [label = "release()"];
@@ -307,14 +304,10 @@ std::size_t CameraConfiguration::size() const
* Acquired -> Configured [label = "configure()"];
*
* Configured -> Available [label = "release()"];
- * Configured -> Configured [label = "configure()"];
- * Configured -> Prepared [label = "allocateBuffers()"];
+ * Configured -> Configured [label = "configure(), createRequest()"];
+ * Configured -> Running [label = "start()"];
*
- * Prepared -> Configured [label = "freeBuffers()"];
- * Prepared -> Prepared [label = "createRequest()"];
- * Prepared -> Running [label = "start()"];
- *
- * Running -> Prepared [label = "stop()"];
+ * Running -> Configured [label = "stop()"];
* Running -> Running [label = "createRequest(), queueRequest()"];
* }
* \enddot
@@ -330,19 +323,14 @@ std::size_t CameraConfiguration::size() const
* Configured state.
*
* \subsubsection Configured
- * The camera is configured and ready for the application to prepare it with
- * resources. The camera may be reconfigured multiple times until resources
- * are provided and the state progresses to Prepared.
- *
- * \subsubsection Prepared
- * The camera has been configured and provided with resources and is ready to be
- * started. The application may free the camera's resources to get back to the
- * Configured state or start() it to progress to the Running state.
+ * The camera is configured and ready to be started. The application may
+ * release() the camera and to get back to the Available state or start()
+ * it to progress to the Running state.
*
* \subsubsection Running
* The camera is running and ready to process requests queued by the
* application. The camera remains in this state until it is stopped and moved
- * to the Prepared state.
+ * to the Configured state.
*/
/**
@@ -420,7 +408,6 @@ static const char *const camera_state_names[] = {
"Available",
"Acquired",
"Configured",
- "Prepared",
"Running",
};
@@ -465,8 +452,6 @@ bool Camera::stateIs(State state) const
*
* \todo Deal with pending requests if the camera is disconnected in a
* running state.
- * \todo Update comment about Running state when importing buffers as well as
- * allocating them are supported.
*/
void Camera::disconnect()
{
@@ -474,11 +459,11 @@ void Camera::disconnect()
/*
* If the camera was running when the hardware was removed force the
- * state to Prepared to allow applications to call freeBuffers() and
- * release() before deleting the camera.
+ * state to Configured state to allow applications to free resources
+ * and call release() before deleting the camera.
*/
if (state_ == CameraRunning)
- state_ = CameraPrepared;
+ state_ = CameraConfigured;
disconnected_ = true;
disconnected.emit(this);
@@ -698,53 +683,6 @@ int Camera::configure(CameraConfiguration *config)
return 0;
}
-/**
- * \brief Allocate buffers for all configured streams
- *
- * This function affects the state of the camera, see \ref camera_operation.
- *
- * \return 0 on success or a negative error code otherwise
- * \retval -ENODEV The camera has been disconnected from the system
- * \retval -EACCES The camera is not in a state where buffers can be allocated
- * \retval -EINVAL The configuration is not valid
- */
-int Camera::allocateBuffers()
-{
- if (disconnected_)
- return -ENODEV;
-
- if (!stateIs(CameraConfigured))
- return -EACCES;
-
- if (activeStreams_.empty()) {
- LOG(Camera, Error)
- << "Can't allocate buffers without streams";
- return -EINVAL;
- }
-
- state_ = CameraPrepared;
-
- return 0;
-}
-
-/**
- * \brief Release all buffers from allocated pools in each stream
- *
- * This function affects the state of the camera, see \ref camera_operation.
- *
- * \return 0 on success or a negative error code otherwise
- * \retval -EACCES The camera is not in a state where buffers can be freed
- */
-int Camera::freeBuffers()
-{
- if (!stateIs(CameraPrepared))
- return -EACCES;
-
- state_ = CameraConfigured;
-
- return 0;
-}
-
/**
* \brief Create a request object for the camera
* \param[in] cookie Opaque cookie for application use
@@ -760,14 +698,14 @@ int Camera::freeBuffers()
* The ownership of the returned request is passed to the caller, which is
* responsible for either queueing the request or deleting it.
*
- * This function shall only be called when the camera is in the Prepared
+ * This function shall only be called when the camera is in the Configured
* or Running state, see \ref camera_operation.
*
* \return A pointer to the newly created request, or nullptr on error
*/
Request *Camera::createRequest(uint64_t cookie)
{
- if (disconnected_ || !stateBetween(CameraPrepared, CameraRunning))
+ if (disconnected_ || !stateBetween(CameraConfigured, CameraRunning))
return nullptr;
return new Request(this, cookie);
@@ -839,7 +777,7 @@ int Camera::start()
if (disconnected_)
return -ENODEV;
- if (!stateIs(CameraPrepared))
+ if (!stateIs(CameraConfigured))
return -EACCES;
LOG(Camera, Debug) << "Starting capture";
@@ -883,7 +821,7 @@ int Camera::stop()
LOG(Camera, Debug) << "Stopping capture";
- state_ = CameraPrepared;
+ state_ = CameraConfigured;
pipe_->stop(this);
@@ -92,7 +92,7 @@ FrameBufferAllocator::create(std::shared_ptr<Camera> camera)
*/
int FrameBufferAllocator::allocate(Stream *stream)
{
- if (camera_->state_ != Camera::CameraConfigured && camera_->state_ != Camera::CameraPrepared) {
+ if (camera_->state_ != Camera::CameraConfigured) {
LOG(Allocator, Error)
<< "Camera must be in the configured state to allocate buffers";
return -EACCES;
@@ -136,7 +136,7 @@ int FrameBufferAllocator::allocate(Stream *stream)
*/
int FrameBufferAllocator::release(Stream *stream)
{
- if (camera_->state_ != Camera::CameraConfigured && camera_->state_ != Camera::CameraPrepared) {
+ if (camera_->state_ != Camera::CameraConfigured) {
LOG(Allocator, Error)
<< "Camera must be in the configured state to free buffers";
return -EACCES;
@@ -172,13 +172,6 @@ int MainWindow::startCapture()
adjustSize();
- ret = camera_->allocateBuffers();
- if (ret) {
- std::cerr << "Failed to allocate buffers"
- << std::endl;
- return ret;
- }
-
ret = allocator_->allocate(stream);
if (ret < 0) {
std::cerr << "Failed to allocate capture buffers" << std::endl;
@@ -244,7 +237,6 @@ error:
}
mappedBuffers_.clear();
- camera_->freeBuffers();
return ret;
}
@@ -264,7 +256,6 @@ void MainWindow::stopCapture()
}
mappedBuffers_.clear();
- camera_->freeBuffers();
isCapturing_ = false;
config_.reset();
@@ -171,11 +171,6 @@ protected:
return TestFail;
}
- if (camera_->allocateBuffers()) {
- std::cout << "Failed to allocate buffers" << std::endl;
- return TestFail;
- }
-
Stream *stream = cfg.stream();
BufferSource source;
@@ -241,11 +236,6 @@ protected:
return TestFail;
}
- if (camera_->freeBuffers()) {
- std::cout << "Failed to free buffers" << std::endl;
- return TestFail;
- }
-
return TestPass;
}
@@ -87,11 +87,6 @@ protected:
return TestFail;
}
- if (camera_->allocateBuffers()) {
- cout << "Failed to allocate buffers" << endl;
- return TestFail;
- }
-
Stream *stream = cfg.stream();
int ret = allocator_->allocate(stream);
@@ -158,11 +153,6 @@ protected:
return TestFail;
}
- if (camera_->freeBuffers()) {
- cout << "Failed to free buffers" << endl;
- return TestFail;
- }
-
return TestPass;
}
@@ -29,12 +29,6 @@ protected:
if (camera_->configure(defconf_.get()) != -EACCES)
return TestFail;
- if (camera_->allocateBuffers() != -EACCES)
- return TestFail;
-
- if (camera_->freeBuffers() != -EACCES)
- return TestFail;
-
if (camera_->createRequest())
return TestFail;
@@ -65,12 +59,6 @@ protected:
if (camera_->acquire() != -EBUSY)
return TestFail;
- if (camera_->allocateBuffers() != -EACCES)
- return TestFail;
-
- if (camera_->freeBuffers() != -EACCES)
- return TestFail;
-
if (camera_->createRequest())
return TestFail;
@@ -103,57 +91,6 @@ protected:
if (camera_->acquire() != -EBUSY)
return TestFail;
- if (camera_->freeBuffers() != -EACCES)
- return TestFail;
-
- if (camera_->createRequest())
- return TestFail;
-
- Request request(camera_.get());
- if (camera_->queueRequest(&request) != -EACCES)
- return TestFail;
-
- if (camera_->start() != -EACCES)
- return TestFail;
-
- if (camera_->stop() != -EACCES)
- return TestFail;
-
- /* Test operations which should pass. */
- if (camera_->configure(defconf_.get()))
- return TestFail;
-
- /* Test valid state transitions, end in Prepared state. */
- if (camera_->release())
- return TestFail;
-
- if (camera_->acquire())
- return TestFail;
-
- if (camera_->configure(defconf_.get()))
- return TestFail;
-
- if (camera_->allocateBuffers())
- return TestFail;
-
- return TestPass;
- }
-
- int testPrepared()
- {
- /* Test operations which should fail. */
- if (camera_->acquire() != -EBUSY)
- return TestFail;
-
- if (camera_->release() != -EBUSY)
- return TestFail;
-
- if (camera_->configure(defconf_.get()) != -EACCES)
- return TestFail;
-
- if (camera_->allocateBuffers() != -EACCES)
- return TestFail;
-
Request request1(camera_.get());
if (camera_->queueRequest(&request1) != -EACCES)
return TestFail;
@@ -170,9 +107,6 @@ protected:
delete request2;
/* Test valid state transitions, end in Running state. */
- if (camera_->freeBuffers())
- return TestFail;
-
if (camera_->release())
return TestFail;
@@ -182,9 +116,6 @@ protected:
if (camera_->configure(defconf_.get()))
return TestFail;
- if (camera_->allocateBuffers())
- return TestFail;
-
/* Use internally allocated buffers. */
allocator_ = FrameBufferAllocator::create(camera_);
Stream *stream = *camera_->streams().begin();
@@ -209,12 +140,6 @@ protected:
if (camera_->configure(defconf_.get()) != -EACCES)
return TestFail;
- if (camera_->allocateBuffers() != -EACCES)
- return TestFail;
-
- if (camera_->freeBuffers() != -EACCES)
- return TestFail;
-
if (camera_->start() != -EACCES)
return TestFail;
@@ -236,9 +161,6 @@ protected:
delete allocator_;
- if (camera_->freeBuffers())
- return TestFail;
-
if (camera_->release())
return TestFail;
@@ -276,11 +198,6 @@ protected:
return TestFail;
}
- if (testPrepared() != TestPass) {
- cout << "State machine in Prepared state failed" << endl;
- return TestFail;
- }
-
if (testRuning() != TestPass) {
cout << "State machine in Running state failed" << endl;
return TestFail;