From 721f4b03d80fc7b65cfb4c7ecebe545ef1fe0565 Mon Sep 17 00:00:00 2001 From: Laurent Pinchart Date: Wed, 7 Jul 2021 03:34:28 +0300 Subject: cam: Add camera index to file name of capture frames To prepare for multi-camera support, extend the file naming scheme for captured frames to include the camera index in addition to the stream name and frame number. Signed-off-by: Laurent Pinchart Reviewed-by: Kieran Bingham --- src/cam/camera_session.cpp | 9 ++++++--- src/cam/camera_session.h | 4 +++- src/cam/main.cpp | 4 ++-- 3 files changed, 11 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/cam/camera_session.cpp b/src/cam/camera_session.cpp index f2383567..90261a8c 100644 --- a/src/cam/camera_session.cpp +++ b/src/cam/camera_session.cpp @@ -22,9 +22,11 @@ using namespace libcamera; CameraSession::CameraSession(CameraManager *cm, const std::string &cameraId, + unsigned int cameraIndex, const OptionsParser::Options &options) - : options_(options), last_(0), queueCount_(0), captureCount_(0), - captureLimit_(0), printMetadata_(false) + : options_(options), cameraIndex_(cameraIndex), last_(0), + queueCount_(0), captureCount_(0), captureLimit_(0), + printMetadata_(false) { char *endptr; unsigned long index = strtoul(cameraId.c_str(), &endptr, 10); @@ -152,7 +154,8 @@ int CameraSession::start() streamName_.clear(); for (unsigned int index = 0; index < config_->size(); ++index) { StreamConfiguration &cfg = config_->at(index); - streamName_[cfg.stream()] = "stream" + std::to_string(index); + streamName_[cfg.stream()] = "cam" + std::to_string(cameraIndex_) + + "-stream" + std::to_string(index); } camera_->requestCompleted.connect(this, &CameraSession::requestComplete); diff --git a/src/cam/camera_session.h b/src/cam/camera_session.h index b3136383..b0f50e7f 100644 --- a/src/cam/camera_session.h +++ b/src/cam/camera_session.h @@ -28,7 +28,7 @@ class CameraSession { public: CameraSession(libcamera::CameraManager *cm, - const std::string &cameraId, + const std::string &cameraId, unsigned int cameraIndex, const OptionsParser::Options &options); ~CameraSession(); @@ -60,6 +60,8 @@ private: std::map streamName_; std::unique_ptr writer_; + unsigned int cameraIndex_; + uint64_t last_; unsigned int queueCount_; diff --git a/src/cam/main.cpp b/src/cam/main.cpp index 7688fa55..96f55831 100644 --- a/src/cam/main.cpp +++ b/src/cam/main.cpp @@ -136,7 +136,7 @@ int CamApp::parseOptions(int argc, char *argv[]) "If the file name ends with a '/', it sets the directory in which\n" "to write files, using the default file name. Otherwise it sets the\n" "full file path and name. The first '#' character in the file name\n" - "is expanded to the stream name and frame sequence number.\n" + "is expanded to the camera index, stream name and frame sequence number.\n" "The default file name is 'frame-#.bin'.", "file", ArgumentOptional, "filename", false, OptCamera); @@ -201,7 +201,7 @@ int CamApp::run() if (options_.isSet(OptCamera)) { const OptionValue &camera = options_[OptCamera]; session = std::make_unique(cm_.get(), - camera.toString(), + camera.toString(), 0, camera.children()); if (!session->isValid()) { std::cout << "Failed to create camera session" << std::endl; -- cgit v1.2.1