From 8e716be52900389215344577112e88df0d8616df Mon Sep 17 00:00:00 2001
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: Mon, 5 Jul 2021 19:05:23 +0300
Subject: cam: Move CameraConfiguration creation to CameraSession class

Creating a configuration for a camera is an operation that logically
belongs to the CameraSession class. Move it there.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/cam/camera_session.h | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'src/cam/camera_session.h')

diff --git a/src/cam/camera_session.h b/src/cam/camera_session.h
index 31e8d6db..39dbbdf3 100644
--- a/src/cam/camera_session.h
+++ b/src/cam/camera_session.h
@@ -26,7 +26,10 @@ class CameraSession
 {
 public:
 	CameraSession(std::shared_ptr<libcamera::Camera> camera,
-		      libcamera::CameraConfiguration *config);
+		      const OptionsParser::Options &options);
+
+	bool isValid() const { return config_ != nullptr; }
+	libcamera::CameraConfiguration *config() { return config_.get(); }
 
 	int start(const OptionsParser::Options &options);
 	void stop();
@@ -41,7 +44,7 @@ private:
 	void processRequest(libcamera::Request *request);
 
 	std::shared_ptr<libcamera::Camera> camera_;
-	libcamera::CameraConfiguration *config_;
+	std::unique_ptr<libcamera::CameraConfiguration> config_;
 
 	std::map<const libcamera::Stream *, std::string> streamName_;
 	std::unique_ptr<BufferWriter> writer_;
-- 
cgit v1.2.1