[libcamera-devel] [PATCH v4 03/12] libcamera: controls: Construct from valid values

[Jacopo Mondi]

Add two constructors to the ControlInfo class that allows creating
a class instance from the list of the control valid values with
an optional default one.

Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
---
 include/libcamera/controls.h |  5 ++++
 src/libcamera/controls.cpp   | 47 ++++++++++++++++++++++++++++++++++++
 2 files changed, 52 insertions(+)

diff --git a/include/libcamera/controls.h b/include/libcamera/controls.h
index 80944efc133a..0ae4fd002726 100644
--- a/include/libcamera/controls.h
+++ b/include/libcamera/controls.h
@@ -268,10 +268,14 @@ public:
 	explicit ControlInfo(const ControlValue &min = 0,
 			     const ControlValue &max = 0,
 			     const ControlValue &def = 0);
+	explicit ControlInfo(const ControlValue *values, unsigned int numValues,
+			     const ControlValue &def);
+	explicit ControlInfo(const ControlValue *values, unsigned int numValues);
 
 	const ControlValue &min() const { return min_; }
 	const ControlValue &max() const { return max_; }
 	const ControlValue &def() const { return def_; }
+	const std::vector<ControlValue> &values() const { return values_; }
 
 	std::string toString() const;
 
@@ -289,6 +293,7 @@ private:
 	ControlValue min_;
 	ControlValue max_;
 	ControlValue def_;
+	std::vector<ControlValue> values_;
 };
 
 using ControlIdMap = std::unordered_map<unsigned int, const ControlId *>;
diff --git a/src/libcamera/controls.cpp b/src/libcamera/controls.cpp
index dca782667d88..2a2f04a598b6 100644
--- a/src/libcamera/controls.cpp
+++ b/src/libcamera/controls.cpp
@@ -491,6 +491,42 @@ ControlInfo::ControlInfo(const ControlValue &min,
 {
 }
 
+/**
+ * \brief Construct a ControlInfo from the list of values values and a default
+ * \param[in] values The control valid values
+ * \param[in] numValues The number or valid values
+ * \param[in] def The control default value
+ *
+ * Construct a ControlInfo from a list of valid values. The ControlInfo
+ * minimum and maximum values are assigned to the first and last members of
+ * the values list respectively.
+ */
+ControlInfo::ControlInfo(const ControlValue *values, unsigned int numValues,
+			 const ControlValue &def)
+	: def_(def)
+{
+	min_ = values[0];
+	max_ = values[numValues - 1];
+
+	for (unsigned int i = 0; i < numValues; ++i)
+		values_.push_back(values[i]);
+}
+
+/**
+ * \brief Construct a ControlInfo from the list of valid values
+ * \param[in] values The control valid values
+ * \param[in] numValues The number or valid values
+ *
+ * Construct a ControlInfo from a list of valid values. The ControlInfo
+ * minimum and maximum values are assigned to the first and last members of
+ * the values list respectively. The ControlInfo default value is set to be
+ * equal to the minimum one.
+ */
+ControlInfo::ControlInfo(const ControlValue *values, unsigned int numValues)
+	: ControlInfo(values, numValues, values[0])
+{
+}
+
 /**
  * \fn ControlInfo::min()
  * \brief Retrieve the minimum value of the control
@@ -519,6 +555,17 @@ ControlInfo::ControlInfo(const ControlValue &min,
  * \return A ControlValue with the default value for the control
  */
 
+/**
+ * \fn ControlInfo::values()
+ * \brief Retrieve the list of valid values
+ *
+ * For controls that support a pre-defined number of values, the enumeration of
+ * those is reported through a vector of ControlValue instances accessible with
+ * this method.
+ *
+ * \return A vector of ControlValue representing the control valid values
+ */
+
 /**
  * \brief Provide a string representation of the ControlInfo
  */
-- 
2.28.0