Patch Detail
Show a patch.
GET /api/patches/2327/?format=api
{ "id": 2327, "url": "https://patchwork.libcamera.org/api/patches/2327/?format=api", "web_url": "https://patchwork.libcamera.org/patch/2327/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/projects/1/?format=api", "name": "libcamera", "link_name": "libcamera", "list_id": "libcamera_core", "list_email": "libcamera-devel@lists.libcamera.org", "web_url": "", "scm_url": "", "webscm_url": "" }, "msgid": "<20191119164421.68983-4-jacopo@jmondi.org>", "date": "2019-11-19T16:44:18", "name": "[libcamera-devel,RFC,3/6] libcamera: properties: Generate Properties", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "209d63d72e21239a61bcfab75a82e680a89945ed", "submitter": { "id": 3, "url": "https://patchwork.libcamera.org/api/people/3/?format=api", "name": "Jacopo Mondi", "email": "jacopo@jmondi.org" }, "delegate": { "id": 15, "url": "https://patchwork.libcamera.org/api/users/15/?format=api", "username": "jmondi", "first_name": "Jacopo", "last_name": "Mondi", "email": "jacopo@jmondi.org" }, "mbox": "https://patchwork.libcamera.org/patch/2327/mbox/", "series": [ { "id": 571, "url": "https://patchwork.libcamera.org/api/series/571/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=571", "date": "2019-11-19T16:44:15", "name": "Collect Camera properties", "version": 1, "mbox": "https://patchwork.libcamera.org/series/571/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/2327/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/2327/checks/", "tags": {}, "headers": { "Return-Path": "<jacopo@jmondi.org>", "Received": [ "from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net\n\t[217.70.183.200])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 59A526138B\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 19 Nov 2019 17:42:27 +0100 (CET)", "from uno.lan (93-34-114-233.ip49.fastwebnet.it [93.34.114.233])\n\t(Authenticated sender: jacopo@jmondi.org)\n\tby relay7-d.mail.gandi.net (Postfix) with ESMTPSA id E2FE520010;\n\tTue, 19 Nov 2019 16:42:26 +0000 (UTC)" ], "X-Originating-IP": "93.34.114.233", "From": "Jacopo Mondi <jacopo@jmondi.org>", "To": "libcamera-devel@lists.libcamera.org", "Date": "Tue, 19 Nov 2019 17:44:18 +0100", "Message-Id": "<20191119164421.68983-4-jacopo@jmondi.org>", "X-Mailer": "git-send-email 2.23.0", "In-Reply-To": "<20191119164421.68983-1-jacopo@jmondi.org>", "References": "<20191119164421.68983-1-jacopo@jmondi.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [RFC 3/6] libcamera: properties: Generate\n\tProperties", "X-BeenThere": "libcamera-devel@lists.libcamera.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "<libcamera-devel.lists.libcamera.org>", "List-Unsubscribe": "<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>", "List-Archive": "<https://lists.libcamera.org/pipermail/libcamera-devel/>", "List-Post": "<mailto:libcamera-devel@lists.libcamera.org>", "List-Help": "<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>", "List-Subscribe": "<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>", "X-List-Received-Date": "Tue, 19 Nov 2019 16:42:27 -0000" }, "content": "Re-use the ControlIDs generation infrastructure to define ids and values\nfor libcamera properties.\n\nThis generates two additional files, on that enumerates the properties ids\n(include/libcamera/property_ids.h) and one the defines Control instances\nfor each property (src/libcamera/property_ids.cpp)\n\nSigned-off-by: Jacopo Mondi <jacopo@jmondi.org>\n---\n include/libcamera/meson.build | 9 ++++++\n include/libcamera/property_ids.h.in | 33 ++++++++++++++++++++++\n src/libcamera/meson.build | 6 ++++\n src/libcamera/property_ids.cpp.in | 43 +++++++++++++++++++++++++++++\n src/libcamera/property_ids.yaml | 32 +++++++++++++++++++++\n 5 files changed, 123 insertions(+)\n create mode 100644 include/libcamera/property_ids.h.in\n create mode 100644 src/libcamera/property_ids.cpp.in\n create mode 100644 src/libcamera/property_ids.yaml", "diff": "diff --git a/include/libcamera/meson.build b/include/libcamera/meson.build\nindex 99abf0609940..795cc00f97c6 100644\n--- a/include/libcamera/meson.build\n+++ b/include/libcamera/meson.build\n@@ -31,7 +31,16 @@ control_ids_h = custom_target('control_ids_h',\n install : true,\n install_dir : join_paths('include', include_dir))\n \n+property_ids_h = custom_target('property_ids_h',\n+ input : files('../../src/libcamera/property_ids.yaml', 'property_ids.h.in'),\n+ output : 'property_ids.h',\n+ depend_files : gen_controls,\n+ command : [gen_controls, '-o', '@OUTPUT@', '@INPUT@'],\n+ install : true,\n+ install_dir : join_paths('include', include_dir))\n+\n libcamera_api += control_ids_h\n+libcamera_api += property_ids_h\n \n gen_header = files('gen-header.sh')\n \ndiff --git a/include/libcamera/property_ids.h.in b/include/libcamera/property_ids.h.in\nnew file mode 100644\nindex 000000000000..25fa52cdfd17\n--- /dev/null\n+++ b/include/libcamera/property_ids.h.in\n@@ -0,0 +1,33 @@\n+/* SPDX-License-Identifier: LGPL-2.1-or-later */\n+/*\n+ * Copyright (C) 2019, Google Inc.\n+ *\n+ * property_ids.h : Property ID list\n+ *\n+ * This file is auto-generated. Do not edit.\n+ */\n+\n+#ifndef __LIBCAMERA_PROPERTY_IDS_H__\n+#define __LIBCAMERA_PROPERTY_IDS_H__\n+\n+#include <stdint.h>\n+\n+#include <libcamera/controls.h>\n+\n+namespace libcamera {\n+\n+namespace properties {\n+\n+enum {\n+${ids}\n+};\n+\n+${controls}\n+\n+extern const ControlIdMap properties;\n+\n+} /* namespace propertiess */\n+\n+} /* namespace libcamera */\n+\n+#endif // __LIBCAMERA_property_IDS_H__\ndiff --git a/src/libcamera/meson.build b/src/libcamera/meson.build\nindex c4f965bd7413..0f85414e672f 100644\n--- a/src/libcamera/meson.build\n+++ b/src/libcamera/meson.build\n@@ -73,7 +73,13 @@ control_ids_cpp = custom_target('control_ids_cpp',\n depend_files : gen_controls,\n command : [gen_controls, '-o', '@OUTPUT@', '@INPUT@'])\n \n+property_ids_cpp = custom_target('property_ids_cpp',\n+ input : files('property_ids.yaml', 'property_ids.cpp.in'),\n+ output : 'property_ids.cpp',\n+ depend_files : gen_controls,\n+ command : [gen_controls, '-o', '@OUTPUT@', '@INPUT@'])\n libcamera_sources += control_ids_cpp\n+libcamera_sources += property_ids_cpp\n libcamera_sources += control_ids_h\n \n gen_version = join_paths(meson.source_root(), 'utils', 'gen-version.sh')\ndiff --git a/src/libcamera/property_ids.cpp.in b/src/libcamera/property_ids.cpp.in\nnew file mode 100644\nindex 000000000000..1e9ff33f1ec7\n--- /dev/null\n+++ b/src/libcamera/property_ids.cpp.in\n@@ -0,0 +1,43 @@\n+/* SPDX-License-Identifier: LGPL-2.1-or-later */\n+/*\n+ * Copyright (C) 2019, Google Inc.\n+ *\n+ * property_ids.cpp : Property ID list\n+ *\n+ * This file is auto-generated. Do not edit.\n+ */\n+\n+#include <libcamera/property_ids.h>\n+\n+/**\n+ * \\file property_ids.h\n+ * \\brief Camera property identifiers\n+ */\n+\n+namespace libcamera {\n+\n+/**\n+ * \\brief Namespace for libcamera controls\n+ */\n+namespace properties {\n+\n+${controls_doc}\n+\n+#ifndef __DOXYGEN__\n+/*\n+ * Keep the controls definitions hidden from doxygen as it incorrectly parses\n+ * them as functions.\n+ */\n+${controls_def}\n+#endif\n+\n+/**\n+ * \\brief List of all supported libcamera properties\n+ */\n+extern const ControlIdMap properties {\n+${controls_map}\n+};\n+\n+} /* namespace properties */\n+\n+} /* namespace libcamera */\ndiff --git a/src/libcamera/property_ids.yaml b/src/libcamera/property_ids.yaml\nnew file mode 100644\nindex 000000000000..829f7886d94d\n--- /dev/null\n+++ b/src/libcamera/property_ids.yaml\n@@ -0,0 +1,32 @@\n+# SPDX-License-Identifier: LGPL-2.1-or-later\n+#\n+# Copyright (C) 2019, Google Inc.\n+#\n+%YAML 1.2\n+---\n+controls:\n+ - Location:\n+ type: int32_t\n+ description: |\n+ Camera mounting location\n+ values:\n+ - name: \"CAMERA_LOCATION_FRONT\"\n+ value: 0\n+ description: |\n+ The camera is mounted on the front side of the device, facing the\n+ user\n+ - name: \"CAMERA_LOCATION_BACK\"\n+ value: 1\n+ description: |\n+ The camera is mounted on the back facing side of the device\n+ - name: \"CAMERA_LOCATION_EXTERNAL\"\n+ value: 2\n+ description: |\n+ The camera attached to the device in a way that allows it to\n+ move freely\n+\n+ - Rotation:\n+ type: int32_t\n+ description: |\n+ Camera mounting rotation\n+...\n", "prefixes": [ "libcamera-devel", "RFC", "3/6" ] }