{"id":18388,"url":"https://patchwork.libcamera.org/api/patches/18388/?format=json","web_url":"https://patchwork.libcamera.org/patch/18388/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/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":"<20230313091944.9530-6-jacopo.mondi@ideasonboard.com>","date":"2023-03-13T09:19:43","name":"[libcamera-devel,v2,5/6] libcamera: imx8-isi: Split Bayer/YUV config generation","commit_ref":null,"pull_url":null,"state":"accepted","archived":false,"hash":"b6dd31d1d505487337225d9475a2b5546c93f7ca","submitter":{"id":143,"url":"https://patchwork.libcamera.org/api/people/143/?format=json","name":"Jacopo Mondi","email":"jacopo.mondi@ideasonboard.com"},"delegate":null,"mbox":"https://patchwork.libcamera.org/patch/18388/mbox/","series":[{"id":3803,"url":"https://patchwork.libcamera.org/api/series/3803/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=3803","date":"2023-03-13T09:19:38","name":"libcamera: imx8-isi: Remove pixelformat-2-media-bus map","version":2,"mbox":"https://patchwork.libcamera.org/series/3803/mbox/"}],"comments":"https://patchwork.libcamera.org/api/patches/18388/comments/","check":"pending","checks":"https://patchwork.libcamera.org/api/patches/18388/checks/","tags":{},"headers":{"Return-Path":"","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 74E1EC329E\n\tfor ;\n\tMon, 13 Mar 2023 09:20:10 +0000 (UTC)","from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id E29C462720;\n\tMon, 13 Mar 2023 10:20:09 +0100 (CET)","from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 889B06270D\n\tfor ;\n\tMon, 13 Mar 2023 10:20:05 +0100 (CET)","from uno.homenet.telecomitalia.it\n\t(host-79-33-55-183.retail.telecomitalia.it [79.33.55.183])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 844CB9DE;\n\tMon, 13 Mar 2023 10:20:04 +0100 (CET)"],"DKIM-Signature":["v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org;\n\ts=mail; t=1678699210;\n\tbh=WZjTmmYJZoOSuXxoDtzyE6z5+j7i5oRKdC1kAWdbvIo=;\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=mKsi3re68QJf3tTNuKv9Q+jYzCOFPKkL92F8G+zT2Tg2HNNAUK6OG3opUh287MKAC\n\tNNkJVLS3JOtbXGqkP5YGTBysHE+S69Gi8bRHLhKQ5/gpgnxHiiVAec4poVi9sSTcvR\n\tYcE4j8GgcT5lBYlvJSA7NZaZHc7hdyX0eomG9B5mn+51CPyyri1Nukvtc7A6j0zrfM\n\tzZ8Sf7fAry1vTxcZz+LP3pRX3poDAH8wgt86/Bu59AYUwvWYly4uphKmS/SkkyJ6b1\n\ttxaiKMNwI/0c4Hc3MMPKAa2saU2iGFYwizIZrIsAdJmrrsmnwmHH/NrEMdjs5R+2wv\n\tKjRI8h67hAq/w==","v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1678699205;\n\tbh=WZjTmmYJZoOSuXxoDtzyE6z5+j7i5oRKdC1kAWdbvIo=;\n\th=From:To:Cc:Subject:Date:In-Reply-To:References:From;\n\tb=QgwnC1ZjbW0i0km/Zo5KmzhxRN/otnsmuwRKuJ/3NFOOcWvDg5MaLr2KUUaT4WVmz\n\tn1ot3jQ/Etgrf/ZdAFSutBRFCFgOSrPnRhlOku54Toj4/RpjFjzWD+wzXjCejf8XV3\n\tG0574PI+U2VW0FzmjgtYXwU0ZZVZ7yC0kAS/PPi4="],"Authentication-Results":"lancelot.ideasonboard.com; dkim=pass (1024-bit key; \n\tunprotected) header.d=ideasonboard.com\n\theader.i=@ideasonboard.com\n\theader.b=\"QgwnC1Zj\"; dkim-atps=neutral","To":"libcamera-devel@lists.libcamera.org","Date":"Mon, 13 Mar 2023 10:19:43 +0100","Message-Id":"<20230313091944.9530-6-jacopo.mondi@ideasonboard.com>","X-Mailer":"git-send-email 2.39.0","In-Reply-To":"<20230313091944.9530-1-jacopo.mondi@ideasonboard.com>","References":"<20230313091944.9530-1-jacopo.mondi@ideasonboard.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Subject":"[libcamera-devel] [PATCH v2 5/6] libcamera: imx8-isi: Split\n\tBayer/YUV config generation","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","From":"Jacopo Mondi via libcamera-devel ","Reply-To":"Jacopo Mondi ","Cc":"Jacopo Mondi ","Errors-To":"libcamera-devel-bounces@lists.libcamera.org","Sender":"\"libcamera-devel\" "},"content":"At generateConfiguration() a YUV/RGB pixel format is preferred for the\nStillCapture/VideoRecording/Viewfinder roles, but currently there are no\nguarantees in place that the sensor provides a non-Bayer bus format from\nwhich YUV/RGB can be generated.\n\nThis makes the default configuration generated for those roles not to\nwork if the sensor is a RAW-only one.\n\nTo improve the situation split the configuration generation in two,\none for YUV modes and one for Raw Bayer mode.\n\nStreamRoles assigned to a YUV mode will try to first generate a YUV\nconfiguration and then fallback to RAW if that's what the sensor can\nprovide.\n\nAs an additional requirement, for YUV streams, the generated mode has to\nbe validated with the sensor to confirm the desired sizes can be\ngenerated. In order to test a format use the newly introduced\nCameraSensor::tryFormat().\n\nReported-by: Laurent Pinchart \nSigned-off-by: Jacopo Mondi \nReviewed-by: Paul Elder \nReviewed-by: Daniel Scally \nTested-by: Daniel Scally \n---\n src/libcamera/pipeline/imx8-isi/imx8-isi.cpp | 231 +++++++++++++------\n 1 file changed, 155 insertions(+), 76 deletions(-)","diff":"diff --git a/src/libcamera/pipeline/imx8-isi/imx8-isi.cpp b/src/libcamera/pipeline/imx8-isi/imx8-isi.cpp\nindex 146ba7465012..a4f437f55b26 100644\n--- a/src/libcamera/pipeline/imx8-isi/imx8-isi.cpp\n+++ b/src/libcamera/pipeline/imx8-isi/imx8-isi.cpp\n@@ -145,6 +145,10 @@ private:\n \n \tPipe *pipeFromStream(Camera *camera, const Stream *stream);\n \n+\tStreamConfiguration generateYUVConfiguration(Camera *camera,\n+\t\t\t\t\t\t const Size &size);\n+\tStreamConfiguration generateRawConfiguration(Camera *camera);\n+\n \tvoid bufferReady(FrameBuffer *buffer);\n \n \tMediaDevice *isiDev_;\n@@ -705,6 +709,129 @@ PipelineHandlerISI::PipelineHandlerISI(CameraManager *manager)\n {\n }\n \n+/*\n+ * Generate a StreamConfiguration for YUV/RGB use case.\n+ *\n+ * Verify it the sensor can produce a YUV/RGB media bus format and collect\n+ * all the processed pixel formats the ISI can generate as supported stream\n+ * configurations.\n+ */\n+StreamConfiguration PipelineHandlerISI::generateYUVConfiguration(Camera *camera,\n+\t\t\t\t\t\t\t\t const Size &size)\n+{\n+\tISICameraData *data = cameraData(camera);\n+\tPixelFormat pixelFormat = formats::YUYV;\n+\tunsigned int mbusCode;\n+\n+\tmbusCode = data->getYuvMediaBusFormat(&pixelFormat);\n+\tif (!mbusCode)\n+\t\treturn {};\n+\n+\t/* Adjust the requested size to the sensor's capabilities. */\n+\tconst CameraSensor *sensor = data->sensor_.get();\n+\n+\tV4L2SubdeviceFormat sensorFmt;\n+\tsensorFmt.mbus_code = mbusCode;\n+\tsensorFmt.size = size;\n+\n+\tint ret = sensor->tryFormat(&sensorFmt);\n+\tif (ret) {\n+\t\tLOG(ISI, Error) << \"Failed to try sensor format.\";\n+\t\treturn {};\n+\t}\n+\n+\tSize sensorSize = sensorFmt.size;\n+\n+\t/*\n+\t * Populate the StreamConfiguration.\n+\t *\n+\t * As the sensor supports at least one YUV/RGB media bus format all the\n+\t * processed ones in formatsMap_ can be generated from it.\n+\t */\n+\tstd::map> streamFormats;\n+\n+\tfor (const auto &[pixFmt, pipeFmt] : ISICameraConfiguration::formatsMap_) {\n+\t\tconst PixelFormatInfo &info = PixelFormatInfo::info(pixFmt);\n+\t\tif (info.colourEncoding == PixelFormatInfo::ColourEncodingRAW)\n+\t\t\tcontinue;\n+\n+\t\tstreamFormats[pixFmt] = { { kMinISISize, sensorSize } };\n+\t}\n+\n+\tStreamFormats formats(streamFormats);\n+\n+\tStreamConfiguration cfg(formats);\n+\tcfg.pixelFormat = pixelFormat;\n+\tcfg.size = sensorSize;\n+\tcfg.bufferCount = 4;\n+\n+\treturn cfg;\n+}\n+\n+/*\n+ * Generate a StreamConfiguration for Raw Bayer use case. Verify if the sensor\n+ * can produce the requested RAW bayer format and eventually adjust it to\n+ * the one with the largest bit-depth the sensor can produce.\n+ */\n+StreamConfiguration PipelineHandlerISI::generateRawConfiguration(Camera *camera)\n+{\n+\tstatic const std::map rawFormats = {\n+\t\t{ MEDIA_BUS_FMT_SBGGR8_1X8, formats::SBGGR8 },\n+\t\t{ MEDIA_BUS_FMT_SGBRG8_1X8, formats::SGBRG8 },\n+\t\t{ MEDIA_BUS_FMT_SGRBG8_1X8, formats::SGRBG8 },\n+\t\t{ MEDIA_BUS_FMT_SRGGB8_1X8, formats::SRGGB8 },\n+\t\t{ MEDIA_BUS_FMT_SBGGR10_1X10, formats::SBGGR10 },\n+\t\t{ MEDIA_BUS_FMT_SGBRG10_1X10, formats::SGBRG10 },\n+\t\t{ MEDIA_BUS_FMT_SGRBG10_1X10, formats::SGRBG10 },\n+\t\t{ MEDIA_BUS_FMT_SRGGB10_1X10, formats::SRGGB10 },\n+\t\t{ MEDIA_BUS_FMT_SBGGR12_1X12, formats::SBGGR12 },\n+\t\t{ MEDIA_BUS_FMT_SGBRG12_1X12, formats::SGBRG12 },\n+\t\t{ MEDIA_BUS_FMT_SGRBG12_1X12, formats::SGRBG12 },\n+\t\t{ MEDIA_BUS_FMT_SRGGB12_1X12, formats::SRGGB12 },\n+\t};\n+\n+\tISICameraData *data = cameraData(camera);\n+\tPixelFormat pixelFormat = formats::SBGGR10;\n+\tunsigned int mbusCode;\n+\n+\t/* pixelFormat will be adjusted, if the sensor can produce RAW. */\n+\tmbusCode = data->getRawMediaBusFormat(&pixelFormat);\n+\tif (!mbusCode)\n+\t\treturn {};\n+\n+\t/*\n+\t * Populate the StreamConfiguration with all the supported Bayer\n+\t * formats the sensor can produce.\n+\t */\n+\tstd::map> streamFormats;\n+\tconst CameraSensor *sensor = data->sensor_.get();\n+\n+\tfor (unsigned int code : sensor->mbusCodes()) {\n+\t\t/* Find a Bayer media bus code from the sensor. */\n+\t\tconst BayerFormat &bayerFormat = BayerFormat::fromMbusCode(code);\n+\t\tif (!bayerFormat.isValid())\n+\t\t\tcontinue;\n+\n+\t\tauto it = rawFormats.find(code);\n+\t\tif (it == rawFormats.end()) {\n+\t\t\tLOG(ISI, Warning) << bayerFormat\n+\t\t\t\t\t << \" not supported in ISI formats map.\";\n+\t\t\tcontinue;\n+\t\t}\n+\n+\t\tstreamFormats[it->second] = { { sensor->resolution(), sensor->resolution() } };\n+\t}\n+\n+\tStreamFormats formats(streamFormats);\n+\n+\tStreamConfiguration cfg(formats);\n+\tcfg.size = sensor->resolution();\n+\tcfg.pixelFormat = pixelFormat;\n+\tcfg.bufferCount = 4;\n+\n+\treturn cfg;\n+}\n+\n std::unique_ptr\n PipelineHandlerISI::generateConfiguration(Camera *camera,\n \t\t\t\t\t const StreamRoles &roles)\n@@ -722,79 +849,45 @@ PipelineHandlerISI::generateConfiguration(Camera *camera,\n \t\treturn nullptr;\n \t}\n \n-\tbool isRaw = false;\n \tfor (const auto &role : roles) {\n \t\t/*\n-\t\t * Prefer the following formats\n+\t\t * Prefer the following formats:\n \t\t * - Still Capture: Full resolution YUYV\n \t\t * - ViewFinder/VideoRecording: 1080p YUYV\n-\t\t * - RAW: sensor's native format and resolution\n+\t\t * - RAW: Full resolution Bayer\n \t\t */\n-\t\tstd::map> streamFormats;\n-\t\tPixelFormat pixelFormat;\n-\t\tSize size;\n+\t\tStreamConfiguration cfg;\n \n \t\tswitch (role) {\n \t\tcase StreamRole::StillCapture:\n-\t\t\t/*\n-\t\t\t * \\todo Make sure the sensor can produce non-RAW formats\n-\t\t\t * compatible with the ones supported by the pipeline.\n-\t\t\t */\n-\t\t\tsize = data->sensor_->resolution();\n-\t\t\tpixelFormat = formats::YUYV;\n+\t\t\tcfg = generateYUVConfiguration(camera,\n+\t\t\t\t\t\t data->sensor_->resolution());\n+\t\t\tif (!cfg.pixelFormat.isValid()) {\n+\t\t\t\t/*\n+\t\t\t\t * Fallback to use a Bayer format if that's what\n+\t\t\t\t * the sensor supports.\n+\t\t\t\t */\n+\t\t\t\tcfg = generateRawConfiguration(camera);\n+\t\t\t}\n+\n \t\t\tbreak;\n \n \t\tcase StreamRole::Viewfinder:\n \t\tcase StreamRole::VideoRecording:\n-\t\t\t/*\n-\t\t\t * \\todo Make sure the sensor can produce non-RAW formats\n-\t\t\t * compatible with the ones supported by the pipeline.\n-\t\t\t */\n-\t\t\tsize = PipelineHandlerISI::kPreviewSize;\n-\t\t\tpixelFormat = formats::YUYV;\n-\t\t\tbreak;\n-\n-\t\tcase StreamRole::Raw: {\n-\t\t\t/*\n-\t\t\t * Make sure the sensor can generate a RAW format and\n-\t\t\t * prefer the ones with a larger bitdepth.\n-\t\t\t */\n-\t\t\tconst ISICameraConfiguration::FormatMap::value_type *rawPipeFormat = nullptr;\n-\t\t\tunsigned int maxDepth = 0;\n-\n-\t\t\tfor (unsigned int code : data->sensor_->mbusCodes()) {\n-\t\t\t\tconst BayerFormat &bayerFormat = BayerFormat::fromMbusCode(code);\n-\t\t\t\tif (!bayerFormat.isValid())\n-\t\t\t\t\tcontinue;\n-\n-\t\t\t\t/* Make sure the format is supported by the pipeline handler. */\n-\t\t\t\tauto it = std::find_if(ISICameraConfiguration::formatsMap_.begin(),\n-\t\t\t\t\t\t ISICameraConfiguration::formatsMap_.end(),\n-\t\t\t\t\t\t [code](auto &isiFormat) {\n-\t\t\t\t\t\t\t auto &pipe = isiFormat.second;\n-\t\t\t\t\t\t\t return pipe.sensorCode == code;\n-\t\t\t\t\t\t });\n-\t\t\t\tif (it == ISICameraConfiguration::formatsMap_.end())\n-\t\t\t\t\tcontinue;\n-\n-\t\t\t\tif (bayerFormat.bitDepth > maxDepth) {\n-\t\t\t\t\tmaxDepth = bayerFormat.bitDepth;\n-\t\t\t\t\trawPipeFormat = &(*it);\n-\t\t\t\t}\n-\t\t\t}\n-\n-\t\t\tif (!rawPipeFormat) {\n-\t\t\t\tLOG(ISI, Error)\n-\t\t\t\t\t<< \"Cannot generate a configuration for RAW stream\";\n-\t\t\t\treturn nullptr;\n+\t\t\tcfg = generateYUVConfiguration(camera,\n+\t\t\t\t\t\t PipelineHandlerISI::kPreviewSize);\n+\t\t\tif (!cfg.pixelFormat.isValid()) {\n+\t\t\t\t/*\n+\t\t\t\t * Fallback to use a Bayer format if that's what\n+\t\t\t\t * the sensor supports.\n+\t\t\t\t */\n+\t\t\t\tcfg = generateRawConfiguration(camera);\n \t\t\t}\n \n-\t\t\tsize = data->sensor_->resolution();\n-\t\t\tpixelFormat = rawPipeFormat->first;\n-\n-\t\t\tstreamFormats[pixelFormat] = { { kMinISISize, size } };\n-\t\t\tisRaw = true;\n+\t\t\tbreak;\n \n+\t\tcase StreamRole::Raw: {\n+\t\t\tcfg = generateRawConfiguration(camera);\n \t\t\tbreak;\n \t\t}\n \n@@ -803,26 +896,12 @@ PipelineHandlerISI::generateConfiguration(Camera *camera,\n \t\t\treturn nullptr;\n \t\t}\n \n-\t\t/*\n-\t\t * For non-RAW configurations the ISI can perform colorspace\n-\t\t * conversion. List all the supported output formats here.\n-\t\t */\n-\t\tif (!isRaw) {\n-\t\t\tfor (const auto &[pixFmt, pipeFmt] : ISICameraConfiguration::formatsMap_) {\n-\t\t\t\tconst PixelFormatInfo &info = PixelFormatInfo::info(pixFmt);\n-\t\t\t\tif (info.colourEncoding == PixelFormatInfo::ColourEncodingRAW)\n-\t\t\t\t\tcontinue;\n-\n-\t\t\t\tstreamFormats[pixFmt] = { { kMinISISize, size } };\n-\t\t\t}\n+\t\tif (!cfg.pixelFormat.isValid()) {\n+\t\t\tLOG(ISI, Error)\n+\t\t\t\t<< \"Cannot generate configuration for role: \" << role;\n+\t\t\treturn nullptr;\n \t\t}\n \n-\t\tStreamFormats formats(streamFormats);\n-\n-\t\tStreamConfiguration cfg(formats);\n-\t\tcfg.pixelFormat = pixelFormat;\n-\t\tcfg.size = size;\n-\t\tcfg.bufferCount = 4;\n \t\tconfig->addConfiguration(cfg);\n \t}\n \n","prefixes":["libcamera-devel","v2","5/6"]}