From patchwork Thu Jun 20 15:31:39 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1484 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 158A960BA1 for ; Thu, 20 Jun 2019 17:30:41 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id A402124000A; Thu, 20 Jun 2019 15:30:40 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:39 +0200 Message-Id: <20190620153144.5394-2-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 1/6] libcamera: Add V4L2Controls X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:41 -0000 Add Libcamera V4L2 control support, implemented using the V4L2 Extended Control APIs. This patch defines the types used to define and manage controls. Signed-off-by: Jacopo Mondi --- src/libcamera/include/v4l2_controls.h | 86 ++++++++ src/libcamera/meson.build | 1 + src/libcamera/v4l2_controls.cpp | 282 ++++++++++++++++++++++++++ 3 files changed, 369 insertions(+) create mode 100644 src/libcamera/include/v4l2_controls.h create mode 100644 src/libcamera/v4l2_controls.cpp diff --git a/src/libcamera/include/v4l2_controls.h b/src/libcamera/include/v4l2_controls.h new file mode 100644 index 000000000000..acd23dabd831 --- /dev/null +++ b/src/libcamera/include/v4l2_controls.h @@ -0,0 +1,86 @@ +/* SPDX-License-Identifier: LGPL-2.1-or-later */ +/* + * Copyright (C) 2019, Google Inc. + * + * v4l2_controls.h - V4L2 Controls Support + */ + +#ifndef __LIBCAMERA_V4L2_CONTROLS_H__ +#define __LIBCAMERA_V4L2_CONTROLS_H__ + +#include +#include + +#include + +#include +#include + +namespace libcamera { + +class V4L2ControlInfo +{ +public: + V4L2ControlInfo(const struct v4l2_query_ext_ctrl &ctrl); + + unsigned int id() const { return id_; } + unsigned int type() const { return type_; } + size_t size() const { return size_; } + const std::string &name() const { return name_; } + +private: + unsigned int type_; + std::string name_; + unsigned int id_; + size_t size_; +}; + +class V4L2Control +{ +public: + V4L2Control(unsigned int id, int value = 0) + : id_(id), value_(value) {} + + int64_t value() const { return value_; } + void setValue(int64_t value) { value_ = value; } + + unsigned int id() const { return id_; } + +private: + unsigned int id_; + int64_t value_; +}; + +class V4L2Controls +{ +public: + V4L2Controls &operator=(const V4L2Controls &) = delete; + + using iterator = std::vector::iterator; + using const_iterator = std::vector::const_iterator; + + iterator begin() { return controls_.begin(); } + const_iterator begin() const { return controls_.begin(); } + iterator end() { return controls_.end(); } + const_iterator end() const { return controls_.end(); } + + bool empty() const { return controls_.empty(); } + size_t size() const { return controls_.size(); } + + void reset() { controls_.clear(); } + void add(unsigned int id, int64_t value = 0); + + V4L2Control *operator[](unsigned int index) + { + return &controls_[index]; + } + + V4L2Control *getControl(unsigned int id); + +private: + std::vector controls_; +}; + +} /* namespace libcamera */ + +#endif /* __LIBCAMERA_V4L2_CONTROLS_H__ */ diff --git a/src/libcamera/meson.build b/src/libcamera/meson.build index f26ad5b2dc57..985aa7e8ab0e 100644 --- a/src/libcamera/meson.build +++ b/src/libcamera/meson.build @@ -23,6 +23,7 @@ libcamera_sources = files([ 'stream.cpp', 'timer.cpp', 'utils.cpp', + 'v4l2_controls.cpp', 'v4l2_device.cpp', 'v4l2_subdevice.cpp', 'v4l2_videodevice.cpp', diff --git a/src/libcamera/v4l2_controls.cpp b/src/libcamera/v4l2_controls.cpp new file mode 100644 index 000000000000..abf596e7f1ef --- /dev/null +++ b/src/libcamera/v4l2_controls.cpp @@ -0,0 +1,282 @@ +/* SPDX-License-Identifier: LGPL-2.1-or-later */ +/* + * Copyright (C) 2019, Google Inc. + * + * v4l2_controls.cpp - V4L2 Controls Support + */ + +#include "v4l2_controls.h" + +/** + * \file v4l2_controls.h + * \brief Support for V4L2 Controls using the V4L2 Extended Controls APIs. + * + * The V4L2 defined "Control API" allows application to inspect and modify set + * of configurable parameters on the video device or subdevice of interest. The + * nature of the parameters an application could modify using the control + * framework depends on what the driver implements support for, and on the + * characteristics of the underlying hardware platform. Generally controls are + * used to modify user visible settings, such as the image brightness and + * exposure time, or non-standard parameters which cannot be controlled through + * the V4L2 format negotiation API. + * + * Controls are identified by a numerical ID, defined by the V4L2 kernel headers + * and have an associated type. Each control has a value, which is the data + * that can be modified with V4L2Device::setControls() or retrieved with + * V4L2Device::getControls(). + * + * The control's type along with the control's flags defines the type of the + * control's value content. Controls might transport a single data value + * stored in variable inside the control, or they might as well deal with more + * complex data types, such as arrays of matrices, stored in a contiguous + * memory locations associated with the control and called 'the payload'. Such + * controls are called 'compound controls' and are currently not supported by + * the libcamera V4L2 control framework. + * + * libcamera implements support for controls using the V4L2 'Extended Control' + * API, which allows future handling of controls with payloads of arbitrary + * sizes. + * + * The libcamera V4L2 Controls framework operates on lists of controls, + * wrapped by the V4L2Controls class, to match the V4L2 extended controls API. + * The interface to set and get control is implemented by the V4L2Device + * class, and this file only provides the data type definitions. + * + * \todo Add support for compound controls + */ + +namespace libcamera { + +/** + * \class V4L2ControlInfo + * \brief Information on a V4L2 control + * + * The V4L2ControlInfo class represent all the information related to + * a V4L2 control, such as its ID, its type, its user-readable name and the + * expected size of its value data. + * + * V4L2ControlInfo instances are created by inspecting the fieldS of a struct + * v4l2_query_ext_ctrl structure, after it has been filled by the device + * driver as a consequence of a VIDIOC_QUERY_EXT_CTRL ioctl call. + * + * This class does not contain the control value, but only static information + * on the control, which shall be cached by the caller at initialisation time + * or the first time the control information is accessed. + */ + +/** + * \brief Construct a V4L2ControlInfo from a struct v4l2_query_ext_ctrl + * \param ctrl The struct v4l2_query_ext_ctrl as returned by the kernel + */ +V4L2ControlInfo::V4L2ControlInfo(const struct v4l2_query_ext_ctrl &ctrl) +{ + id_ = ctrl.id; + type_ = ctrl.type; + name_ = static_cast(ctrl.name); + size_ = ctrl.elem_size * ctrl.elems; +} + +/** + * \fn V4L2ControlInfo::id() + * \brief Retrieve the control ID + * \return The V4L2 control ID + */ + +/** + * \fn V4L2ControlInfo::type() + * \brief Retrieve the control type as defined by V4L2_CTRL_TYPE_* + * \return The V4L2 control type + */ + +/** + * \fn V4L2ControlInfo::size() + * \brief Retrieve the control value data size (in bytes) + * \return The V4L2 control value data size + */ + +/** + * \fn V4L2ControlInfo::name() + * \brief Retrieve the control user readable name + * \return The V4L2 control user readable name + */ + +/** + * \class V4L2Control + * \brief A V4L2 control value + * + * The V4L2Control class represent the value of a V4L2 control. + * The class stores values that have been read from or will be applied to + * a V4L2 device. + * + * The value stored in the class instances does not reflect what is actually + * applied to the hardware but is a pure software cache optionally initialized + * at control creation time and only modified by a control read operation. + * + * The write and read controls the V4L2Control class instances are not meant + * to be directly used but are instead intended to be grouped in + * V4L2Controls instances, which are then passed as parameters to + * V4L2Device::setControls() and V4L2Device::getControls() operations. + */ + +/** + * \fn V4L2Control::V4L2Control + * \brief Construct a V4L2 control with ID \a id and value \a value + * \param id The V4L2 control ID + * \param value The control value + */ + +/** + * \fn V4L2Control::value() + * \brief Retrieve the value of the control + * + * It is important to notice that the returned value is not read from the + * hardware at the time this operation is called, but it's the cached value + * obtained the most recent time the control has been read with + * V4L2Device::getControls() or the value of the control has been set to by the + * user with the add() operation. + * + * \return The V4L2 control value. The value might be 0 if the control has + * never been read from the device before this operation is called. + */ + +/** + * \fn V4L2Control::setValue() + * \brief Set the value of the control + * \param value The new V4L2 control value + * + * It is important to notice that the value is not applied to the + * hardware at the time this operation is called, but it will only + * be actually applied by calling V4L2Device::setControls(). + */ + +/** + * \fn V4L2Control::id() + * \brief Retrieve the control ID this instance refers to + * \return The V4L2Control ID + */ + +/** + * \class V4L2Controls + * \brief Container of V4L2Control instances + * + * The V4L2Controls class works as a container for a list of V4L2Control + * instances. The class provides operations to add a new control to the list, + * get back a control value, and reset the list of controls it contains. + * + * In order to set and get controls, user of the libcamera V4L2 control + * framework should operate on instances of the V4L2Controls class, and use + * them as argument for the V4L2Device::setControls() and + * V4L2Device::getControls() operations, which write and read a list of + * controls from or to a V4L2 device (a video device or a subdevice). + * + * Controls are added to a V4L2Controls instance with the add() method, with + * or without an initial value. + * + * To write controls to a device, the controls of interest shall be added with + * an initial value by calling V4L2Controls::add(unsigned int id, int64_t + * value) to prepare for a write operation. Once the values of all controls of + * interest have been added, the V4L2Controls instance is passed to the + * V4L2Device::setControls(), which sets the controls on the device. + * + * To read controls from a device, the desired controls are added with + * V4L2Controls::add(unsigned int id) to prepare for a read operation. The + * V4L2Controls instance is then passed to V4L2Device::getControls(), which + * reads the controls from the device and updates the values stored in + * V4L2Controls. + * + * V4L2Controls instances can be reset to remove all controls they contain and + * prepare to be re-used for a new control write/read sequence. + */ + +/** + * \typedef V4L2Controls::iterator + * \brief Iterator on the V4L2 controls contained in the instance + */ + +/** + * \typedef V4L2Controls::const_iterator + * \brief Const iterator on the V4L2 controls contained in the instance + */ + +/** + * \fn iterator V4L2Controls::begin() + * \brief Retrieve an iterator to the first V4L2Control in the instance + * \return An iterator to the first V4L2 control + */ + +/** + * \fn const_iterator V4L2Controls::begin() const + * \brief Retrieve a constant iterator to the first V4L2Control in the instance + * \return A constant iterator to the first V4L2 control + */ + +/** + * \fn iterator V4L2Controls::end() + * \brief Retrieve an iterator pointing to the past-the-end V4L2Control in the + * instance + * \return An iterator to the element following the last V4L2 control in the + * instance + */ + +/** + * \fn const_iterator V4L2Controls::end() const + * \brief Retrieve a constant iterator pointing to the past-the-end V4L2Control + * in the instance + * \return A constant iterator to the element following the last V4L2 control + * in the instance + */ + +/** + * \fn V4L2Controls::empty() + * \brief Verify if the instance does not contain any control + * \return True if the instance does not contain any control, false otherwise + */ + +/** + * \fn V4L2Controls::size() + * \brief Retrieve the number on controls in the instance + * \return The number of V4L2Control stored in the instance + */ + +/** + * \fn V4L2Controls::reset() + * \brief Removes all controls in the instance + */ + +/** + * \brief Add control with ID \a id and optional \a value to the instance + * \param id The V4L2 control ID (V4L2_CID_*) + * \param value The V4L2 control value + * + * V4L2Control can be added with an explicit value, or without one. In that + * case the control's value is defaulted to 0. + */ +void V4L2Controls::add(unsigned int id, int64_t value) +{ + controls_.emplace_back(id, value); +} + +/** + * \fn V4L2Controls::operator[](unsigned int index) + * \brief Access the control at index \a index + * \param index The index to access + * \return The V4L2 control at index \a index + */ + +/** + * \brief Get a pointer the control with ID \a id + * \param id The V4L2 control ID (V4L2_CID_xx) + * \return A pointer to the V4L2Control with ID \a id or nullptr if the + * control ID is not part of this instance. + */ +V4L2Control *V4L2Controls::getControl(unsigned int id) +{ + for (V4L2Control &ctrl : controls_) { + if (ctrl.id() == id) + return &ctrl; + } + + return nullptr; +} + +} /* namespace libcamera */ From patchwork Thu Jun 20 15:31:40 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1485 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 9C95060BA1 for ; Thu, 20 Jun 2019 17:30:41 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 3D466240007; Thu, 20 Jun 2019 15:30:41 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:40 +0200 Message-Id: <20190620153144.5394-3-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 2/6] libcamera: v4l2_device: List valid controls at open X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:42 -0000 Enumerate all the valid controls a device supports at open() time. A control is valid only if its type is supported. Store the control informations in a map inside the device to save querying the control when setting or getting its value from the device and provide an operation to retrieve information by control ID. Signed-off-by: Jacopo Mondi --- src/libcamera/include/v4l2_device.h | 9 ++++ src/libcamera/v4l2_device.cpp | 72 +++++++++++++++++++++++++++++ 2 files changed, 81 insertions(+) diff --git a/src/libcamera/include/v4l2_device.h b/src/libcamera/include/v4l2_device.h index 75f23a05b903..91a72fcecbcc 100644 --- a/src/libcamera/include/v4l2_device.h +++ b/src/libcamera/include/v4l2_device.h @@ -7,18 +7,23 @@ #ifndef __LIBCAMERA_V4L2_DEVICE_H__ #define __LIBCAMERA_V4L2_DEVICE_H__ +#include #include #include "log.h" namespace libcamera { +class V4L2ControlInfo; + class V4L2Device : protected Loggable { public: void close(); bool isOpen() const { return fd_ != -1; } + const V4L2ControlInfo *getControlInfo(unsigned int id) const; + const std::string &deviceNode() const { return deviceNode_; } protected: @@ -32,6 +37,10 @@ protected: int fd() { return fd_; } private: + void listControls(); + int validateControlType(const V4L2ControlInfo *info); + + std::map controls_; std::string deviceNode_; int fd_; }; diff --git a/src/libcamera/v4l2_device.cpp b/src/libcamera/v4l2_device.cpp index 99621a724b96..b113ff77e3cf 100644 --- a/src/libcamera/v4l2_device.cpp +++ b/src/libcamera/v4l2_device.cpp @@ -13,6 +13,7 @@ #include #include "log.h" +#include "v4l2_controls.h" /** * \file v4l2_device.h @@ -82,6 +83,8 @@ int V4L2Device::open(unsigned int flags) fd_ = ret; + listControls(); + return 0; } @@ -107,6 +110,21 @@ void V4L2Device::close() * \return True if the V4L2 device node is open, false otherwise */ +/** + * \brief Retrieve information about a control + * \param[in] id The control ID + * \return A pointer to the V4L2ControlInfo for control \a id, or nullptr + * if the device doesn't have such a control + */ +const V4L2ControlInfo *V4L2Device::getControlInfo(unsigned int id) const +{ + auto it = controls_.find(id); + if (it == controls_.end()) + return nullptr; + + return &it->second; +} + /** * \brief Perform an IOCTL system call on the device node * \param[in] request The IOCTL request code @@ -137,4 +155,58 @@ int V4L2Device::ioctl(unsigned long request, void *argp) * \return The V4L2 device file descriptor, -1 if the device node is not open */ +/* + * \brief List and store information about all controls supported by the + * V4L2 device + */ +void V4L2Device::listControls() +{ + struct v4l2_query_ext_ctrl ctrl = {}; + + /* \todo Add support for menu and compound controls. */ + ctrl.id = V4L2_CTRL_FLAG_NEXT_CTRL; + while (ioctl(VIDIOC_QUERY_EXT_CTRL, &ctrl) == 0) { + if (ctrl.type == V4L2_CTRL_TYPE_CTRL_CLASS || + ctrl.flags & V4L2_CTRL_FLAG_DISABLED) { + ctrl.id |= V4L2_CTRL_FLAG_NEXT_CTRL; + continue; + } + + V4L2ControlInfo info(ctrl); + if (validateControlType(&info)) + continue; + + controls_.insert(std::pair + (ctrl.id, info)); + ctrl.id |= V4L2_CTRL_FLAG_NEXT_CTRL; + } +} + +/* + * \brief Make sure the control type is supported + * \param[in] info The V4L2ControlInfo to inspect type of + * \return 0 on success or a negative error code otherwise + * \retval -EINVAL The control type is not supported + */ +int V4L2Device::validateControlType(const V4L2ControlInfo *info) +{ + /* \todo Support compound controls. */ + switch (info->type()) { + case V4L2_CTRL_TYPE_INTEGER: + case V4L2_CTRL_TYPE_BOOLEAN: + case V4L2_CTRL_TYPE_MENU: + case V4L2_CTRL_TYPE_BUTTON: + case V4L2_CTRL_TYPE_INTEGER64: + case V4L2_CTRL_TYPE_BITMASK: + case V4L2_CTRL_TYPE_INTEGER_MENU: + break; + default: + LOG(V4L2, Error) << "Control type '" << info->type() + << "' not supported"; + return -EINVAL; + } + + return 0; +} + } /* namespace libcamera */ From patchwork Thu Jun 20 15:31:41 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1486 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 38D1160BA1 for ; Thu, 20 Jun 2019 17:30:42 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id C4140240006; Thu, 20 Jun 2019 15:30:41 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:41 +0200 Message-Id: <20190620153144.5394-4-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 3/6] libcamera: v4l2_device: Implement get and set controls X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:42 -0000 Implement getControls() and setControls() operations in V4L2Device class. Both operations take a V4L2Controls instance and read or write the V4L2 controls on the V4L2 device. Signed-off-by: Jacopo Mondi --- src/libcamera/include/v4l2_device.h | 3 + src/libcamera/v4l2_device.cpp | 193 ++++++++++++++++++++++++++++ 2 files changed, 196 insertions(+) diff --git a/src/libcamera/include/v4l2_device.h b/src/libcamera/include/v4l2_device.h index 91a72fcecbcc..3af27c9f7af5 100644 --- a/src/libcamera/include/v4l2_device.h +++ b/src/libcamera/include/v4l2_device.h @@ -15,6 +15,7 @@ namespace libcamera { class V4L2ControlInfo; +class V4L2Controls; class V4L2Device : protected Loggable { @@ -23,6 +24,8 @@ public: bool isOpen() const { return fd_ != -1; } const V4L2ControlInfo *getControlInfo(unsigned int id) const; + int getControls(V4L2Controls *ctrls); + int setControls(V4L2Controls *ctrls); const std::string &deviceNode() const { return deviceNode_; } diff --git a/src/libcamera/v4l2_device.cpp b/src/libcamera/v4l2_device.cpp index b113ff77e3cf..efe30ef856ae 100644 --- a/src/libcamera/v4l2_device.cpp +++ b/src/libcamera/v4l2_device.cpp @@ -125,6 +125,199 @@ const V4L2ControlInfo *V4L2Device::getControlInfo(unsigned int id) const return &it->second; } +/** + * \brief Read controls from the device + * \param[inout] ctrls The list of controls to read + * + * Read the value of each control contained in \a ctrls, and store the + * value in the corresponding \a ctrls entry. + * + * If an integer number less than the requested number of controls is returned, + * only controls up to that index have been successfully read. + * + * Each V4L2Control instance in \a ctrls should be supported by the device and + * accessible (ie the V4L2_CTRL_FLAG_DISABLED flag should not be set), otherwise + * a negative error code (-EINVAL) is returned + * + * \return 0 on success or an error code otherwise + * \retval -EINVAL One of the control is not supported or not accessible + * \retval i The index of the control that failed + */ +int V4L2Device::getControls(V4L2Controls *ctrls) +{ + unsigned int count = ctrls->size(); + if (count == 0) + return 0; + + const V4L2ControlInfo *controlInfo[count] = {}; + struct v4l2_ext_control v4l2Ctrls[count] = {}; + + for (unsigned int i = 0; i < count; ++i) { + V4L2Control *ctrl = (*ctrls)[i]; + + const V4L2ControlInfo *info = getControlInfo(ctrl->id()); + if (!info) { + LOG(V4L2, Error) + << "Control '" << ctrl->id() << "' not found"; + return -EINVAL; + } + + controlInfo[i] = info; + v4l2Ctrls[i].id = info->id(); + } + + struct v4l2_ext_controls v4l2ExtCtrls = {}; + v4l2ExtCtrls.which = V4L2_CTRL_WHICH_CUR_VAL; + v4l2ExtCtrls.controls = v4l2Ctrls; + v4l2ExtCtrls.count = count; + + int ret = ioctl(VIDIOC_G_EXT_CTRLS, &v4l2ExtCtrls); + if (ret) { + unsigned int errorIdx = v4l2ExtCtrls.error_idx; + + /* Generic validation error. */ + if (errorIdx == count) { + LOG(V4L2, Error) << "Unable to read controls: " + << strerror(ret); + return -EINVAL; + } + + /* A specific control failed. */ + LOG(V4L2, Error) << "Unable to read control " << errorIdx + << ": " << strerror(ret); + return errorIdx; + } + + /* + * For each control read from the device, set the value in the + * V4L2Control provided by the caller. + */ + for (unsigned int i = 0; i < count; ++i) { + struct v4l2_ext_control *v4l2Ctrl = &v4l2Ctrls[i]; + const V4L2ControlInfo *info = controlInfo[i]; + V4L2Control *ctrl = (*ctrls)[i]; + + switch (info->type()) { + case V4L2_CTRL_TYPE_INTEGER64: + ctrl->setValue(v4l2Ctrl->value64); + break; + default: + /* + * \todo To be changed when support for string and + * compound controls will be added. + */ + ctrl->setValue(static_cast(v4l2Ctrl->value)); + break; + } + } + + return 0; +} + +/** + * \brief Write controls to the device + * \param[in] ctrls The list of controls to write + * + * Write the value of each V4L2Control contained in \a ctrls. Each + * control should be initialized by the caller with a value, otherwise + * the result of the operation is undefined. + * + * The value of each control is updated to reflect what has been + * actually applied on the device. + * + * If an integer number less than the requested number of controls is returned, + * only controls up to that index have been successfully applied. + * + * Each V4L2Control instance in \a ctrls should be supported by the device and + * accessible (ie the V4L2_CTRL_FLAG_DISABLED flag should not be set), otherwise + * a negative error code (-EINVAL) is returned + * + * \return 0 on success or an error code otherwise + * \retval -EINVAL One of the control is not supported or not accessible + * \retval i The index of the control that failed + */ +int V4L2Device::setControls(V4L2Controls *ctrls) +{ + unsigned int count = ctrls->size(); + if (count == 0) + return 0; + + const V4L2ControlInfo *controlInfo[count] = {}; + struct v4l2_ext_control v4l2Ctrls[count] = {}; + + for (unsigned int i = 0; i < count; ++i) { + V4L2Control *ctrl = (*ctrls)[i]; + + const V4L2ControlInfo *info = getControlInfo(ctrl->id()); + if (!info) { + LOG(V4L2, Error) + << "Control '" << ctrl->id() << "' not found"; + return -EINVAL; + } + + controlInfo[i] = info; + v4l2Ctrls[i].id = info->id(); + + /* Set the v4l2_ext_control value for the write operation. */ + switch (info->type()) { + case V4L2_CTRL_TYPE_INTEGER64: + v4l2Ctrls[i].value64 = ctrl->value(); + break; + default: + /* + * \todo To be changed when support for string and + * compound controls will be added. + */ + v4l2Ctrls[i].value = static_cast(ctrl->value()); + break; + } + } + + struct v4l2_ext_controls v4l2ExtCtrls = {}; + v4l2ExtCtrls.which = V4L2_CTRL_WHICH_CUR_VAL; + v4l2ExtCtrls.controls = v4l2Ctrls; + v4l2ExtCtrls.count = count; + + int ret = ioctl(VIDIOC_S_EXT_CTRLS, &v4l2ExtCtrls); + if (ret) { + unsigned int errorIdx = v4l2ExtCtrls.error_idx; + + /* Generic validation error. */ + if (errorIdx == count) { + LOG(V4L2, Error) << "Unable to read controls: " + << strerror(ret); + return -EINVAL; + } + + /* A specific control failed. */ + LOG(V4L2, Error) << "Unable to read control " << errorIdx + << ": " << strerror(ret); + return errorIdx; + } + + /* Update the control value to what have actually been applied. */ + for (unsigned int i = 0; i < count; ++i) { + struct v4l2_ext_control *v4l2Ctrl = &v4l2Ctrls[i]; + const V4L2ControlInfo *info = controlInfo[i]; + V4L2Control *ctrl = (*ctrls)[i]; + + switch (info->type()) { + case V4L2_CTRL_TYPE_INTEGER64: + ctrl->setValue(v4l2Ctrl->value64); + break; + default: + /* + * \todo To be changed when support for string and + * compound controls will be added. + */ + ctrl->setValue(static_cast(v4l2Ctrl->value)); + break; + } + } + + return 0; +} + /** * \brief Perform an IOCTL system call on the device node * \param[in] request The IOCTL request code From patchwork Thu Jun 20 15:31:42 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1487 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id D7E7060BA1 for ; Thu, 20 Jun 2019 17:30:42 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 5C4A2240005; Thu, 20 Jun 2019 15:30:42 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:42 +0200 Message-Id: <20190620153144.5394-5-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 4/6] libcamera: camera_sensor: Add V4L2 control operations X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:43 -0000 Add operations to get and set control and to retrieve the informations on a V4L2 control. For simple camera sensors, the operations are directly called on the underlying V4L2 subdevice. Signed-off-by: Jacopo Mondi Reviewed-by: Laurent Pinchart --- src/libcamera/camera_sensor.cpp | 65 +++++++++++++++++++++++++++ src/libcamera/include/camera_sensor.h | 6 +++ 2 files changed, 71 insertions(+) diff --git a/src/libcamera/camera_sensor.cpp b/src/libcamera/camera_sensor.cpp index a804a68c9d91..637f86b05701 100644 --- a/src/libcamera/camera_sensor.cpp +++ b/src/libcamera/camera_sensor.cpp @@ -247,6 +247,71 @@ int CameraSensor::setFormat(V4L2SubdeviceFormat *format) return subdev_->setFormat(0, format); } +/** + * \brief Retrieve information about a control + * \param[in] id The control ID + * \return A pointer to the V4L2ControlInfo for control \a id, or nullptr + * if the sensor doesn't have such a control + */ +const V4L2ControlInfo *CameraSensor::getControlInfo(unsigned int id) const +{ + return subdev_->getControlInfo(id); +} + +/** + * \brief Read controls from the sensor + * \param[inout] ctrls The list of controls to read + * + * Read the value of each control contained in \a ctrls, and store the + * value in the corresponding \a ctrls entry. + * + * If an integer number less than the requested number of controls is returned, + * only controls up to that index have been successfully read. + * + * Each V4L2Control instance in \a ctrls should be supported by the sensor and + * accessible (ie the V4L2_CTRL_FLAG_DISABLED flag should not be set), otherwise + * a negative error code (-EINVAL) is returned + * + * \sa V4L2Device::getControls() + * + * \return 0 on success or an error code otherwise + * \retval -EINVAL One of the control is not supported or not accessible + * \retval i The index of the control that failed + */ +int CameraSensor::getControls(V4L2Controls *ctrls) +{ + return subdev_->getControls(ctrls); +} + +/** + * \brief Write controls to the sensor + * \param[in] ctrls The list of controls to write + * + * Write the value of each V4L2Control contained in \a ctrls. Each + * control should be initialized by the caller with a value, otherwise + * the result of the operation is undefined. + * + * The value of each control is updated to reflect what has been + * actually applied on the sensor. + * + * If an integer number less than the requested number of controls is returned, + * only controls up to that index have been successfully applied. + * + * Each V4L2Control instance in \a ctrls should be supported by the sensor and + * accessible (ie the V4L2_CTRL_FLAG_DISABLED flag should not be set), otherwise + * a negative error code (-EINVAL) is returned + * + * \sa V4L2Device::setControls() + * + * \return 0 on success or an error code otherwise + * \retval -EINVAL One of the control is not supported or not accessible + * \retval i The index of the control that failed + */ +int CameraSensor::setControls(V4L2Controls *ctrls) +{ + return subdev_->setControls(ctrls); +} + std::string CameraSensor::logPrefix() const { return "'" + subdev_->entity()->name() + "'"; diff --git a/src/libcamera/include/camera_sensor.h b/src/libcamera/include/camera_sensor.h index b823480241a7..b4f4b66e96c4 100644 --- a/src/libcamera/include/camera_sensor.h +++ b/src/libcamera/include/camera_sensor.h @@ -17,6 +17,8 @@ namespace libcamera { class MediaEntity; +class V4L2ControlInfo; +class V4L2Controls; class V4L2Subdevice; struct V4L2SubdeviceFormat; @@ -41,6 +43,10 @@ public: const Size &size) const; int setFormat(V4L2SubdeviceFormat *format); + const V4L2ControlInfo *getControlInfo(unsigned int id) const; + int getControls(V4L2Controls *ctrls); + int setControls(V4L2Controls *ctrls); + protected: std::string logPrefix() const; From patchwork Thu Jun 20 15:31:43 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1488 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 6C06F611B5 for ; Thu, 20 Jun 2019 17:30:43 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 0E64124000A; Thu, 20 Jun 2019 15:30:42 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:43 +0200 Message-Id: <20190620153144.5394-6-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 5/6] libcamera: ipu3: Set pipe_mode based on stream configuration X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:44 -0000 Set the ImgU pipe_mode control based on the active stream configuration. Use 'Video' pipe mode unless the viewfinder stream is not active. Signed-off-by: Jacopo Mondi Reviewed-by: Laurent Pinchart Reviewed-by: Kieran Bingham --- src/libcamera/pipeline/ipu3/ipu3.cpp | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp index 1c0a9825b4cd..54a6735dfc87 100644 --- a/src/libcamera/pipeline/ipu3/ipu3.cpp +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp @@ -22,6 +22,7 @@ #include "media_device.h" #include "pipeline_handler.h" #include "utils.h" +#include "v4l2_controls.h" #include "v4l2_subdevice.h" #include "v4l2_videodevice.h" @@ -194,6 +195,12 @@ private: class PipelineHandlerIPU3 : public PipelineHandler { public: + static constexpr unsigned int V4L2_CID_IPU3_PIPE_MODE = 0x009819c1; + enum IPU3PipeModes { + IPU3PipeModeVideo = 0, + IPU3PipeModeStillCapture = 1, + }; + PipelineHandlerIPU3(CameraManager *manager); CameraConfiguration *generateConfiguration(Camera *camera, @@ -559,6 +566,17 @@ int PipelineHandlerIPU3::configure(Camera *camera, CameraConfiguration *c) if (ret) return ret; + /* Apply the "pipe_mode" control to the ImgU subdevice. */ + V4L2Controls ctrls; + ctrls.add(V4L2_CID_IPU3_PIPE_MODE, vfStream->active_ ? + IPU3PipeModeVideo : + IPU3PipeModeStillCapture); + ret = imgu->imgu_->setControls(&ctrls); + if (ret) { + LOG(IPU3, Error) << "Unable to set pipe_mode control"; + return ret; + } + return 0; } From patchwork Thu Jun 20 15:31:44 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jacopo Mondi X-Patchwork-Id: 1489 Return-Path: Received: from relay1-d.mail.gandi.net (relay1-d.mail.gandi.net [217.70.183.193]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 1BF3761371 for ; Thu, 20 Jun 2019 17:30:44 +0200 (CEST) X-Originating-IP: 2.224.242.101 Received: from uno.lan (2-224-242-101.ip172.fastwebnet.it [2.224.242.101]) (Authenticated sender: jacopo@jmondi.org) by relay1-d.mail.gandi.net (Postfix) with ESMTPSA id 8EF3D240002; Thu, 20 Jun 2019 15:30:43 +0000 (UTC) From: Jacopo Mondi To: libcamera-devel@lists.libcamera.org Date: Thu, 20 Jun 2019 17:31:44 +0200 Message-Id: <20190620153144.5394-7-jacopo@jmondi.org> X-Mailer: git-send-email 2.21.0 In-Reply-To: <20190620153144.5394-1-jacopo@jmondi.org> References: <20190620153144.5394-1-jacopo@jmondi.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 6/6] [HACK] ipu3: Set and get a few sensor controls X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Jun 2019 15:30:44 -0000 Not to merge patch to demonstrate how to set controls on the image sensor device. Not-Signed-off-by: Jacopo Mondi --- src/libcamera/pipeline/ipu3/ipu3.cpp | 41 ++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) diff --git a/src/libcamera/pipeline/ipu3/ipu3.cpp b/src/libcamera/pipeline/ipu3/ipu3.cpp index 54a6735dfc87..f561b6cd5c53 100644 --- a/src/libcamera/pipeline/ipu3/ipu3.cpp +++ b/src/libcamera/pipeline/ipu3/ipu3.cpp @@ -677,6 +677,47 @@ int PipelineHandlerIPU3::start(Camera *camera) ImgUDevice *imgu = data->imgu_; int ret; + /* --- Get control values --- */ + V4L2Controls controls; + controls.add(V4L2_CID_EXPOSURE); + controls.add(V4L2_CID_ANALOGUE_GAIN); + ret = cio2->sensor_->getControls(&controls); + if (ret) { + LOG(Error) << "Failed to get control values"; + return -EINVAL; + } + + for (const V4L2Control &ctrl : controls) { + unsigned int id = ctrl.id(); + const V4L2ControlInfo *info = cio2->sensor_->getControlInfo(id); + if (!info) + continue; + + LOG(Error) << "Control : " << id << " - name : " << info->name() + << " - value: " << ctrl.value(); + } + + /* --- Set control values --- */ + controls.reset(); + controls.add(V4L2_CID_EXPOSURE, 2046); + controls.add(V4L2_CID_ANALOGUE_GAIN, 1024); + + ret = cio2->sensor_->setControls(&controls); + if (ret) { + LOG(IPU3, Error) << "Failed to set controls"; + return ret; + } + + for (const V4L2Control &ctrl : controls) { + unsigned int id = ctrl.id(); + const V4L2ControlInfo *info = cio2->sensor_->getControlInfo(id); + if (!info) + continue; + + LOG(Error) << "Control : " << id << " - name : " << info->name() + << " - value: " << ctrl.value(); + } + /* * Start the ImgU video devices, buffers will be queued to the * ImgU output and viewfinder when requests will be queued.