Patch Detail
Show a patch.
GET /api/1.1/patches/17040/?format=api
{ "id": 17040, "url": "https://patchwork.libcamera.org/api/1.1/patches/17040/?format=api", "web_url": "https://patchwork.libcamera.org/patch/17040/", "project": { "id": 1, "url": "https://patchwork.libcamera.org/api/1.1/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": "<20220809143531.7473-2-rishikeshdonadkar@gmail.com>", "date": "2022-08-09T14:35:30", "name": "[libcamera-devel,v2,1/2] gstreamer: Provide ColorSpace => colorimetry mappings", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "f3e2a50d37d7ad7db1ca80fa981b9184b1b2fd4f", "submitter": { "id": 118, "url": "https://patchwork.libcamera.org/api/1.1/people/118/?format=api", "name": "Rishikesh Donadkar", "email": "rishikeshdonadkar@gmail.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/17040/mbox/", "series": [ { "id": 3396, "url": "https://patchwork.libcamera.org/api/1.1/series/3396/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=3396", "date": "2022-08-09T14:35:29", "name": "Multiple colorimetry support for libcamerasrc.", "version": 2, "mbox": "https://patchwork.libcamera.org/series/3396/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/17040/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/17040/checks/", "tags": {}, "headers": { "Return-Path": "<libcamera-devel-bounces@lists.libcamera.org>", "X-Original-To": "parsemail@patchwork.libcamera.org", "Delivered-To": "parsemail@patchwork.libcamera.org", "Received": [ "from lancelot.ideasonboard.com (lancelot.ideasonboard.com\n\t[92.243.16.209])\n\tby patchwork.libcamera.org (Postfix) with ESMTPS id 73611C3272\n\tfor <parsemail@patchwork.libcamera.org>;\n\tTue, 9 Aug 2022 14:36:02 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 17A176332F;\n\tTue, 9 Aug 2022 16:36:02 +0200 (CEST)", "from mail-pj1-x102e.google.com (mail-pj1-x102e.google.com\n\t[IPv6:2607:f8b0:4864:20::102e])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id BB71A61FAA\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 9 Aug 2022 16:36:00 +0200 (CEST)", "by mail-pj1-x102e.google.com with SMTP id\n\tp14-20020a17090a74ce00b001f4d04492faso12359394pjl.4\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tTue, 09 Aug 2022 07:36:00 -0700 (PDT)", "from localhost.localdomain ([49.36.99.220])\n\tby smtp.googlemail.com with ESMTPSA id\n\th12-20020a17090ac38c00b001f4fb21c11asm10269264pjt.21.2022.08.09.07.35.56\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 09 Aug 2022 07:35:58 -0700 (PDT)" ], "DKIM-Signature": [ "v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1660055762;\n\tbh=awl4xklTtuUU5Mb59OEOY17zUCdX1Th1MBNR3eCSro8=;\n\th=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe:\n\tList-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:\n\tFrom;\n\tb=GmAdukRs3IqVWyG0RcWHiv0Ji1sTKPtJLoPa3/1dfJdNJeKvb+xhH9dnDbjg6DEls\n\tVPP9AaSNlfsLBNlvxP5ZI0Ks8mrRg3aVI0LtkWGJAM45+/ric0TA2N5h4IMg4GoHRB\n\t8AtWKm2M179GajwQuY4DAzpY0eprGPnXBLj2mrUlq3+Z3Dr1nrgeM20uy3/lpPBaK8\n\tMOIQUV3qd8qUNynXJQE8kVs9ueaprReIak1vvBWB0Z0dNW++oDR2XdGNOSRVIEKDGN\n\tscUl2G0puIbXBA2XkDg/Tqt1USCW3kmXRyOiZuUyLMBkwOGvl3ZTYSd65ajzPeMafa\n\tYO5h0zqonbTow==", "v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112;\n\th=content-transfer-encoding:mime-version:references:in-reply-to\n\t:message-id:date:subject:cc:to:from:from:to:cc;\n\tbh=pPvN2mIa4PubgtmGzgfN7j7/BdvXVBboFMCswkgpSAQ=;\n\tb=WlL/bmKF0yH4CzBCmqR+mKG52M43X2O9nLLOVvkZkPVvQ+sFh8nyoU4wl5Xjljn2tH\n\tXqkFMAaDe9N2rk9+le2KiYBRgpKuIGQ6uA0/1usS2+KFI0BVH7yFQtAEBixilN8reExo\n\t5eVTQm9wqSk3bFi5zjuHBPXlEjndYgkgotNHPnOF9UjwQDL8tndIuA6MedV11oOp7Jj8\n\tZybNtuYmWNhVrf3t/lXpkOiF3xEPwOl/iNKRkDMUV25WZreKx8dKKyHju5sH0klUs/FR\n\t+vnDaXOBvWTkAPG9bFqjyOVZXCsioqWOFUIjDT0KB4d1VHuqlWrB2Ylf3Xn2QZuyxxlt\n\tRidg==" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (2048-bit key; \n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"WlL/bmKF\"; dkim-atps=neutral", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20210112;\n\th=content-transfer-encoding:mime-version:references:in-reply-to\n\t:message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc;\n\tbh=pPvN2mIa4PubgtmGzgfN7j7/BdvXVBboFMCswkgpSAQ=;\n\tb=eRMDLxvsQQ+oLMdi2dfPt977ZRwlxx7vJwCj8KQbNQ16DsPdiXPw4h5uA9omC9tV7n\n\tOmnaM7XS4hebV7hKBh0wkU3oEAcAcou1uZm7s6Tx8ZWZ+bcwxXjV4Ho3F+Wbt7sHcIyQ\n\t4t09HDPGHdG5xwlwzCSjzVFW5wlgxQClsbsRa0fMpQp8c/3UbWKYR3a8P2F3O+qPjx2h\n\tuK//aDlvQNv+iyCnkweSsGF3kaejoqLuy/06VCx3F9lRQEBJNC+Y3KfoLasLOLg4Z2Z3\n\tvBk2rY+Yk12qwBBd71MeZ+uB2g5YtZvO8uahajSIzYq2x8FrBJAwsB+Z9f5ZDPnkZ2X1\n\tV2aQ==", "X-Gm-Message-State": "ACgBeo1DOoz/qY57LbI6C0cWiGAafJXAFH/cwUjTq1rDF2Py5xyy6R5o\n\tB9qf397gvmSCbdZDrVE8/eUCHzKgVRF8NChV", "X-Google-Smtp-Source": "AA6agR4sNxXlJhl+4iE/anhRbe9OrSOMS1Se/+OfeMFMlIbpuEWyWjjAv1y7fFtq3EpCDRysf2VB8A==", "X-Received": "by 2002:a17:903:24e:b0:16b:a02d:41fc with SMTP id\n\tj14-20020a170903024e00b0016ba02d41fcmr23933162plh.121.1660055759104; \n\tTue, 09 Aug 2022 07:35:59 -0700 (PDT)", "To": "libcamera-devel@lists.libcamera.org", "Date": "Tue, 9 Aug 2022 20:05:30 +0530", "Message-Id": "<20220809143531.7473-2-rishikeshdonadkar@gmail.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20220809143531.7473-1-rishikeshdonadkar@gmail.com>", "References": "<20220809143531.7473-1-rishikeshdonadkar@gmail.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[libcamera-devel] [PATCH v2 1/2] gstreamer: Provide ColorSpace =>\n\tcolorimetry mappings", "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>", "From": "Rishikesh Donadkar via libcamera-devel\n\t<libcamera-devel@lists.libcamera.org>", "Reply-To": "Rishikesh Donadkar <rishikeshdonadkar@gmail.com>", "Cc": "Rishikesh Donadkar <rishikeshdonadkar@gmail.com>,\n\tnicolas.dufresne@collabora.com, vedantparanjape160201@gmail.com", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Provide libcamera::ColorSpace => colorimetry mappings via:\n- GstVideoColorimetry colorimetry_from_colorspace(colorspace);\n\nRead the ColorSpace from the StreamConfiguration, convert it to\ncolorimetry and update the colorimetry into the caps.\n\nThis patch handles colorimetry in libcamera->GStreamer direction.\nColorimetry support form GStreamer->libcamera direction is provided\nthe next patch.\n\nSigned-off-by: Rishikesh Donadkar <rishikeshdonadkar@gmail.com>\nSigned-off-by: Umang Jain <umang.jain@ideasonboard.com>\n---\n src/gstreamer/gstlibcamera-utils.cpp | 71 ++++++++++++++++++++++++++++\n 1 file changed, 71 insertions(+)", "diff": "diff --git a/src/gstreamer/gstlibcamera-utils.cpp b/src/gstreamer/gstlibcamera-utils.cpp\nindex c97c0d43..dbb47c5a 100644\n--- a/src/gstreamer/gstlibcamera-utils.cpp\n+++ b/src/gstreamer/gstlibcamera-utils.cpp\n@@ -45,6 +45,65 @@ static struct {\n \t/* \\todo NV42 is used in libcamera but is not mapped in GStreamer yet. */\n };\n \n+static GstVideoColorimetry\n+colorimetry_from_colorspace(const ColorSpace &colorSpace)\n+{\n+\tGstVideoColorimetry colorimetry;\n+\n+\tswitch (colorSpace.primaries) {\n+\tcase ColorSpace::Primaries::Raw:\n+\t\tcolorimetry.primaries = GST_VIDEO_COLOR_PRIMARIES_UNKNOWN;\n+\t\tbreak;\n+\tcase ColorSpace::Primaries::Smpte170m:\n+\t\tcolorimetry.primaries = GST_VIDEO_COLOR_PRIMARIES_SMPTE170M;\n+\t\tbreak;\n+\tcase ColorSpace::Primaries::Rec709:\n+\t\tcolorimetry.primaries = GST_VIDEO_COLOR_PRIMARIES_BT709;\n+\t\tbreak;\n+\tcase ColorSpace::Primaries::Rec2020:\n+\t\tcolorimetry.primaries = GST_VIDEO_COLOR_PRIMARIES_BT2020;\n+\t\tbreak;\n+\t}\n+\n+\tswitch (colorSpace.transferFunction) {\n+\tcase ColorSpace::TransferFunction::Linear:\n+\t\tcolorimetry.transfer = GST_VIDEO_TRANSFER_GAMMA10;\n+\t\tbreak;\n+\tcase ColorSpace::TransferFunction::Srgb:\n+\t\tcolorimetry.transfer = GST_VIDEO_TRANSFER_SRGB;\n+\t\tbreak;\n+\tcase ColorSpace::TransferFunction::Rec709:\n+\t\tcolorimetry.transfer = GST_VIDEO_TRANSFER_BT709;\n+\t\tbreak;\n+\t}\n+\n+\tswitch (colorSpace.ycbcrEncoding) {\n+\tcase ColorSpace::YcbcrEncoding::None:\n+\t\tcolorimetry.matrix = GST_VIDEO_COLOR_MATRIX_RGB;\n+\t\tbreak;\n+\tcase ColorSpace::YcbcrEncoding::Rec601:\n+\t\tcolorimetry.matrix = GST_VIDEO_COLOR_MATRIX_BT601;\n+\t\tbreak;\n+\tcase ColorSpace::YcbcrEncoding::Rec709:\n+\t\tcolorimetry.matrix = GST_VIDEO_COLOR_MATRIX_BT709;\n+\t\tbreak;\n+\tcase ColorSpace::YcbcrEncoding::Rec2020:\n+\t\tcolorimetry.matrix = GST_VIDEO_COLOR_MATRIX_BT2020;\n+\t\tbreak;\n+\t}\n+\n+\tswitch (colorSpace.range) {\n+\tcase ColorSpace::Range::Full:\n+\t\tcolorimetry.range = GST_VIDEO_COLOR_RANGE_0_255;\n+\t\tbreak;\n+\tcase ColorSpace::Range::Limited:\n+\t\tcolorimetry.range = GST_VIDEO_COLOR_RANGE_16_235;\n+\t\tbreak;\n+\t}\n+\n+\treturn colorimetry;\n+}\n+\n static GstVideoFormat\n pixel_format_to_gst_format(const PixelFormat &format)\n {\n@@ -139,6 +198,18 @@ gst_libcamera_stream_configuration_to_caps(const StreamConfiguration &stream_cfg\n \t\t\t \"width\", G_TYPE_INT, stream_cfg.size.width,\n \t\t\t \"height\", G_TYPE_INT, stream_cfg.size.height,\n \t\t\t nullptr);\n+\n+\tif (stream_cfg.colorSpace) {\n+\t\tGstVideoColorimetry colorimetry = colorimetry_from_colorspace(stream_cfg.colorSpace.value());\n+\t\tgchar *colorimetry_str = gst_video_colorimetry_to_string(&colorimetry);\n+\n+\t\tif (colorimetry_str)\n+\t\t\tgst_structure_set(s, \"colorimetry\", G_TYPE_STRING, colorimetry_str, nullptr);\n+\t\telse\n+\t\t\tg_error(\"Got invalid colorimetry from ColorSpace: %s\",\n+\t\t\t\tColorSpace::toString(stream_cfg.colorSpace).c_str());\n+\t}\n+\n \tgst_caps_append_structure(caps, s);\n \n \treturn caps;\n", "prefixes": [ "libcamera-devel", "v2", "1/2" ] }