{"id":3574,"url":"https://patchwork.libcamera.org/api/1.1/patches/3574/?format=json","web_url":"https://patchwork.libcamera.org/patch/3574/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20200427213236.333777-6-jacopo@jmondi.org>","date":"2020-04-27T21:32:34","name":"[libcamera-devel,v4,5/7] libcamera: v4l2_subdevice: Add format information","commit_ref":null,"pull_url":null,"state":"superseded","archived":false,"hash":"c6fca1ef8ec856ae40814253941738425002c9ef","submitter":{"id":3,"url":"https://patchwork.libcamera.org/api/1.1/people/3/?format=json","name":"Jacopo Mondi","email":"jacopo@jmondi.org"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/3574/mbox/","series":[{"id":837,"url":"https://patchwork.libcamera.org/api/1.1/series/837/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=837","date":"2020-04-27T21:32:29","name":"libcamera: Add CameraSensorInfo","version":4,"mbox":"https://patchwork.libcamera.org/series/837/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/3574/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/3574/checks/","tags":{},"headers":{"Return-Path":"","Received":["from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net\n\t[217.70.183.194])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 4823860B0A\n\tfor ;\n\tMon, 27 Apr 2020 23:29:36 +0200 (CEST)","from uno.homenet.telecomitalia.it (a-ur1-85.tin.it\n\t[212.216.150.148]) (Authenticated sender: jacopo@jmondi.org)\n\tby relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 9955440002;\n\tMon, 27 Apr 2020 21:29:35 +0000 (UTC)"],"X-Originating-IP":"212.216.150.148","From":"Jacopo Mondi ","To":"libcamera-devel@lists.libcamera.org","Date":"Mon, 27 Apr 2020 23:32:34 +0200","Message-Id":"<20200427213236.333777-6-jacopo@jmondi.org>","X-Mailer":"git-send-email 2.26.1","In-Reply-To":"<20200427213236.333777-1-jacopo@jmondi.org>","References":"<20200427213236.333777-1-jacopo@jmondi.org>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v4 5/7] libcamera: v4l2_subdevice: Add\n\tformat information","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","X-List-Received-Date":"Mon, 27 Apr 2020 21:29:36 -0000"},"content":"Define a map of static information associated with a media bus code.\nStart by reporting the bits-per-pixel for each media bus code defined by\nthe V4L2 kernel API, to later expand it when necessary.\n\nAdd to the V4L2SubdeviceFormat class a method to return the bits per\npixel, retrieved by inspecting the static map of format information.\n\nWhile at there, remove a rouge map inclusion from header file.\n\nSigned-off-by: Jacopo Mondi \n\n---\n src/libcamera/include/v4l2_subdevice.h | 2 +-\n src/libcamera/v4l2_subdevice.cpp | 104 +++++++++++++++++++++++++\n 2 files changed, 105 insertions(+), 1 deletion(-)\n\n--\n2.26.1","diff":"diff --git a/src/libcamera/include/v4l2_subdevice.h b/src/libcamera/include/v4l2_subdevice.h\nindex 27ba5b17f61e..d0e565dbdaab 100644\n--- a/src/libcamera/include/v4l2_subdevice.h\n+++ b/src/libcamera/include/v4l2_subdevice.h\n@@ -7,7 +7,6 @@\n #ifndef __LIBCAMERA_V4L2_SUBDEVICE_H__\n #define __LIBCAMERA_V4L2_SUBDEVICE_H__\n\n-#include \n #include \n #include \n\n@@ -27,6 +26,7 @@ struct V4L2SubdeviceFormat {\n \tSize size;\n\n \tconst std::string toString() const;\n+\tuint8_t bitsPerPixel() const;\n };\n\n class V4L2Subdevice : public V4L2Device\ndiff --git a/src/libcamera/v4l2_subdevice.cpp b/src/libcamera/v4l2_subdevice.cpp\nindex 74788ce7cf4f..93fe4b8c3d32 100644\n--- a/src/libcamera/v4l2_subdevice.cpp\n+++ b/src/libcamera/v4l2_subdevice.cpp\n@@ -14,6 +14,7 @@\n #include \n #include \n\n+#include \n #include \n\n #include \n@@ -32,6 +33,92 @@ namespace libcamera {\n\n LOG_DECLARE_CATEGORY(V4L2)\n\n+namespace {\n+\n+/*\n+ * \\var formatInfoMap\n+ * \\brief A map that associates bits per pixel to V4L2 media bus codes\n+ */\n+const std::map formatInfoMap = {\n+\t{ V4L2_MBUS_FMT_RGB444_2X8_PADHI_BE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB444_2X8_PADHI_LE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB555_2X8_PADHI_BE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB555_2X8_PADHI_LE, 16 },\n+\t{ V4L2_MBUS_FMT_BGR565_2X8_BE, 16 },\n+\t{ V4L2_MBUS_FMT_BGR565_2X8_LE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB565_2X8_BE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB565_2X8_LE, 16 },\n+\t{ V4L2_MBUS_FMT_RGB666_1X18, 18 },\n+\t{ V4L2_MBUS_FMT_RGB888_1X24, 24 },\n+\t{ V4L2_MBUS_FMT_RGB888_2X12_BE, 24 },\n+\t{ V4L2_MBUS_FMT_RGB888_2X12_LE, 24 },\n+\t{ V4L2_MBUS_FMT_ARGB8888_1X32, 32 },\n+\t{ V4L2_MBUS_FMT_Y8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_UV8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_UYVY8_1_5X8, 40 },\n+\t{ V4L2_MBUS_FMT_VYUY8_1_5X8, 40 },\n+\t{ V4L2_MBUS_FMT_YUYV8_1_5X8, 40 },\n+\t{ V4L2_MBUS_FMT_YVYU8_1_5X8, 40 },\n+\t{ V4L2_MBUS_FMT_UYVY8_2X8, 16 },\n+\t{ V4L2_MBUS_FMT_VYUY8_2X8, 16 },\n+\t{ V4L2_MBUS_FMT_YUYV8_2X8, 16 },\n+\t{ V4L2_MBUS_FMT_YVYU8_2X8, 16 },\n+\t{ V4L2_MBUS_FMT_Y10_1X10, 10 },\n+\t{ V4L2_MBUS_FMT_UYVY10_2X10, 20 },\n+\t{ V4L2_MBUS_FMT_VYUY10_2X10, 20 },\n+\t{ V4L2_MBUS_FMT_YUYV10_2X10, 20 },\n+\t{ V4L2_MBUS_FMT_YVYU10_2X10, 20 },\n+\t{ V4L2_MBUS_FMT_Y12_1X12, 12 },\n+\t{ V4L2_MBUS_FMT_UYVY8_1X16, 16 },\n+\t{ V4L2_MBUS_FMT_VYUY8_1X16, 16 },\n+\t{ V4L2_MBUS_FMT_YUYV8_1X16, 16 },\n+\t{ V4L2_MBUS_FMT_YVYU8_1X16, 16 },\n+\t{ V4L2_MBUS_FMT_YDYUYDYV8_1X16, 16 },\n+\t{ V4L2_MBUS_FMT_UYVY10_1X20, 20 },\n+\t{ V4L2_MBUS_FMT_VYUY10_1X20, 20 },\n+\t{ V4L2_MBUS_FMT_YUYV10_1X20, 20 },\n+\t{ V4L2_MBUS_FMT_YVYU10_1X20, 20 },\n+\t{ V4L2_MBUS_FMT_YUV10_1X30, 30 },\n+\t{ V4L2_MBUS_FMT_AYUV8_1X32, 32 },\n+\t{ V4L2_MBUS_FMT_UYVY12_2X12, 24 },\n+\t{ V4L2_MBUS_FMT_VYUY12_2X12, 24 },\n+\t{ V4L2_MBUS_FMT_YUYV12_2X12, 24 },\n+\t{ V4L2_MBUS_FMT_YVYU12_2X12, 24 },\n+\t{ V4L2_MBUS_FMT_UYVY12_1X24, 24 },\n+\t{ V4L2_MBUS_FMT_VYUY12_1X24, 24 },\n+\t{ V4L2_MBUS_FMT_YUYV12_1X24, 24 },\n+\t{ V4L2_MBUS_FMT_YVYU12_1X24, 24 },\n+\t{ V4L2_MBUS_FMT_SBGGR8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGBRG8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGRBG8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SRGGB8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_ALAW8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGBRG10_ALAW8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGRBG10_ALAW8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SRGGB10_ALAW8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_DPCM8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGBRG10_DPCM8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SGRBG10_DPCM8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SRGGB10_DPCM8_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_BE, 16 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE, 16 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_2X8_PADLO_BE, 16 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_2X8_PADLO_LE, 16 },\n+\t{ V4L2_MBUS_FMT_SBGGR10_1X10, 10 },\n+\t{ V4L2_MBUS_FMT_SGBRG10_1X10, 10 },\n+\t{ V4L2_MBUS_FMT_SGRBG10_1X10, 10 },\n+\t{ V4L2_MBUS_FMT_SRGGB10_1X10, 10 },\n+\t{ V4L2_MBUS_FMT_SBGGR12_1X12, 24 },\n+\t{ V4L2_MBUS_FMT_SGBRG12_1X12, 24 },\n+\t{ V4L2_MBUS_FMT_SGRBG12_1X12, 24 },\n+\t{ V4L2_MBUS_FMT_SRGGB12_1X12, 24 },\n+\t{ V4L2_MBUS_FMT_JPEG_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_S5C_UYVY_JPEG_1X8, 8 },\n+\t{ V4L2_MBUS_FMT_AHSV8888_1X32, 32 },\n+};\n+\n+}; /* namespace */\n+\n /**\n * \\struct V4L2SubdeviceFormat\n * \\brief The V4L2 sub-device image format and sizes\n@@ -81,6 +168,23 @@ const std::string V4L2SubdeviceFormat::toString() const\n \treturn ss.str();\n }\n\n+/**\n+ * \\brief Retrieve the number of bits per pixel for the V4L2 subdevice format\n+ * \\return The number of bits per pixel for the format, or default it to 8 if\n+ * the format's media bus code is not supported\n+ */\n+uint8_t V4L2SubdeviceFormat::bitsPerPixel() const\n+{\n+\tconst auto it = formatInfoMap.find(mbus_code);\n+\tif (it == formatInfoMap.end()) {\n+\t\tLOG(V4L2, Error) << \"No information available for format '\"\n+\t\t\t\t << toString() << \"'\";\n+\t\treturn 0;\n+\t}\n+\n+\treturn it->second;\n+}\n+\n /**\n * \\class V4L2Subdevice\n * \\brief A V4L2 subdevice as exposed by the Linux kernel\n","prefixes":["libcamera-devel","v4","5/7"]}