Patch Detail
Show a patch.
GET /api/1.1/patches/25068/?format=api
{ "id": 25068, "url": "https://patchwork.libcamera.org/api/1.1/patches/25068/?format=api", "web_url": "https://patchwork.libcamera.org/patch/25068/", "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": "<20251114-exposure-limits-v3-16-b7c07feba026@ideasonboard.com>", "date": "2025-11-14T14:17:11", "name": "[v3,16/19] ipa: libipa: Remove maxExposureTime from sensor configuration", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "130936775c028577bf6eba971b1eea7990d1fc56", "submitter": { "id": 143, "url": "https://patchwork.libcamera.org/api/1.1/people/143/?format=api", "name": "Jacopo Mondi", "email": "jacopo.mondi@ideasonboard.com" }, "delegate": null, "mbox": "https://patchwork.libcamera.org/patch/25068/mbox/", "series": [ { "id": 5590, "url": "https://patchwork.libcamera.org/api/1.1/series/5590/?format=api", "web_url": "https://patchwork.libcamera.org/project/libcamera/list/?series=5590", "date": "2025-11-14T14:16:55", "name": "libipa: agc: Calculate exposure limits", "version": 3, "mbox": "https://patchwork.libcamera.org/series/5590/mbox/" } ], "comments": "https://patchwork.libcamera.org/api/patches/25068/comments/", "check": "pending", "checks": "https://patchwork.libcamera.org/api/patches/25068/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 01B3EC3241\n\tfor <parsemail@patchwork.libcamera.org>;\n\tFri, 14 Nov 2025 14:17:42 +0000 (UTC)", "from lancelot.ideasonboard.com (localhost [IPv6:::1])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTP id 9BF6660AA7;\n\tFri, 14 Nov 2025 15:17:42 +0100 (CET)", "from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id 0571860AAB\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tFri, 14 Nov 2025 15:17:24 +0100 (CET)", "from [192.168.1.101] (93-61-96-190.ip145.fastwebnet.it\n\t[93.61.96.190])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 067041E5A;\n\tFri, 14 Nov 2025 15:15:22 +0100 (CET)" ], "Authentication-Results": "lancelot.ideasonboard.com; dkim=pass (1024-bit key;\n\tunprotected) header.d=ideasonboard.com header.i=@ideasonboard.com\n\theader.b=\"GmVqZ7R0\"; dkim-atps=neutral", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1763129723;\n\tbh=AIdwxNDRiTVtX5iE/ejoMpkXEM33NwchcLz9mIXnhXs=;\n\th=From:Date:Subject:References:In-Reply-To:To:Cc:From;\n\tb=GmVqZ7R0gYvKzBBYZUS+YJIL53/bBCjU3SH+bzMfd8As3ICVfTxH+1TIEe/7th60Z\n\tTDUKYMP3cc86sZjoUvOnRuUT10f69r9fAcDmbxvNgWMN3KRnypxEFJ+++gdDTEf7z+\n\tn8/qW19kUXb4xMLGttLxdS8OYS3xjtkVs1u76JiQ=", "From": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "Date": "Fri, 14 Nov 2025 15:17:11 +0100", "Subject": "[PATCH v3 16/19] ipa: libipa: Remove maxExposureTime from sensor\n\tconfiguration", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"utf-8\"", "Content-Transfer-Encoding": "7bit", "Message-Id": "<20251114-exposure-limits-v3-16-b7c07feba026@ideasonboard.com>", "References": "<20251114-exposure-limits-v3-0-b7c07feba026@ideasonboard.com>", "In-Reply-To": "<20251114-exposure-limits-v3-0-b7c07feba026@ideasonboard.com>", "To": "=?utf-8?q?Niklas_S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>,\n\tRobert Mader <robert.mader@collabora.com>, \n\tlibcamera-devel@lists.libcamera.org", "Cc": "Jacopo Mondi <jacopo.mondi@ideasonboard.com>", "X-Mailer": "b4 0.14.2", "X-Developer-Signature": "v=1; a=openpgp-sha256; l=13821;\n\ti=jacopo.mondi@ideasonboard.com; h=from:subject:message-id;\n\tbh=AIdwxNDRiTVtX5iE/ejoMpkXEM33NwchcLz9mIXnhXs=;\n\tb=owEBbQKS/ZANAwAKAXI0Bo8WoVY8AcsmYgBpFznsqlFgcBGcHZlihm+Ni4oglPlZjY0BtbH4o\n\tpPEQGW1Wn6JAjMEAAEKAB0WIQS1xD1IgJogio9YOMByNAaPFqFWPAUCaRc57AAKCRByNAaPFqFW\n\tPKZID/0Wy74iL7GKwpBdJAKqDuuDOpj9bFQz8g4meNo/eorIIz+qmynrrBsGz7IvX1YDzmS9qRo\n\tUnMtqbUkbPn8BrwgpoKtadKuEOrtjuUgKTyHTf88gV48lrRAWl3+tAIBdBI3VV4HGarPs5AW/Bq\n\tylMNJrn6uK/pV/AKEuDPWRbL1JS711fyOBKM7wZ83lsSsf6LY7+yq/4q2UnYF2GCUJ3EQkmdwZG\n\ttJ28Y3pLCnE/do0fzCCcvOI2rKx1VFuUl2JXleYbL/GII7zztv0nB6qZ/HwGme/j4N0cJoC1neJ\n\tWzAlm4A5YIA5DCd08yNBPZkVRytFZeQqyls6a7mnFKqYNNFasbAur7ldbd2Iqw+MX8FBS/WVn0R\n\tAJO4j3EgCLTqScjK5ciCwf07WUOevvGGgfpwOQsqCifGZKlJ7fw3pcYklVvKDfljRzoxDvfw3ia\n\tTOJ2hzyb9fR4DDTrni6EcbjpflVhdH+RIAPygssS6d1SRC9tNQcYZxo9TRgi9g5d7CQ5pClORuh\n\tQ2swjVO/AYWoq029glmh2Aq14cTRsWvjr9opqm+Ruo85p+hDfQ+zqVEKkGxwZfsN4gOu7MJf3I+\n\tDiwK60agEQUfJDibR8usV4kelsQQnVTFlzr5te+Pn5eqVgmQVNyZ/vdweUCKGiclJjxtcYWM5um\n\tqvGvA+ootCouvyw==", "X-Developer-Key": "i=jacopo.mondi@ideasonboard.com; a=openpgp;\n\tfpr=72392EDC88144A65C701EA9BA5826A2587AD026B", "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>", "Errors-To": "libcamera-devel-bounces@lists.libcamera.org", "Sender": "\"libcamera-devel\" <libcamera-devel-bounces@lists.libcamera.org>" }, "content": "Now that we have moved the AgcMeanLuminance class and the associated\nhelpers to calculate the maximum achievable shutter time using the\nframe duration, remove the maxExposureTime parameter from all the IPAs\nusing AgcMeanLuminance.\n\nRemove maxExposureTime from all IPA sensor configuration data, as it is\nnow unused and its value depends on the frame duration.\n\nDo not remove minExposureTime as this instead effectively represents\na sensor parameters which remains stable during a streaming session.\n\nSigned-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>\n---\n src/ipa/ipu3/algorithms/agc.cpp | 15 +++------------\n src/ipa/ipu3/algorithms/agc.h | 4 ----\n src/ipa/ipu3/ipa_context.cpp | 3 ---\n src/ipa/ipu3/ipa_context.h | 1 -\n src/ipa/ipu3/ipu3.cpp | 2 --\n src/ipa/libipa/agc_mean_luminance.cpp | 4 ----\n src/ipa/libipa/agc_mean_luminance.h | 1 -\n src/ipa/mali-c55/algorithms/agc.cpp | 1 -\n src/ipa/mali-c55/ipa_context.h | 1 -\n src/ipa/mali-c55/mali-c55.cpp | 2 --\n src/ipa/rkisp1/algorithms/agc.cpp | 2 --\n src/ipa/rkisp1/ipa_context.cpp | 3 ---\n src/ipa/rkisp1/ipa_context.h | 1 -\n src/ipa/rkisp1/rkisp1.cpp | 1 -\n 14 files changed, 3 insertions(+), 38 deletions(-)", "diff": "diff --git a/src/ipa/ipu3/algorithms/agc.cpp b/src/ipa/ipu3/algorithms/agc.cpp\nindex c9d41f93cff5b81710b76592303f1e0d10697326..8fc2d7310c5c6e08b3eff6cc575c221d21c1131f 100644\n--- a/src/ipa/ipu3/algorithms/agc.cpp\n+++ b/src/ipa/ipu3/algorithms/agc.cpp\n@@ -57,11 +57,6 @@ static constexpr utils::Duration kMaxExposureTime = 60ms;\n /* Histogram constants */\n static constexpr uint32_t knumHistogramBins = 256;\n \n-Agc::Agc()\n-\t: minExposureTime_(0s), maxExposureTime_(0s)\n-{\n-}\n-\n /**\n * \\brief Initialise the AGC algorithm from tuning files\n * \\param[in] context The shared IPA context\n@@ -101,10 +96,6 @@ int Agc::configure(IPAContext &context,\n \tstride_ = configuration.grid.stride;\n \tbdsGrid_ = configuration.grid.bdsGrid;\n \n-\tminExposureTime_ = configuration.sensor.minExposureTime;\n-\tmaxExposureTime_ = std::min(configuration.sensor.maxExposureTime,\n-\t\t\t\t kMaxExposureTime);\n-\n \tminAnalogueGain_ = std::max(configuration.sensor.minAnalogueGain, kMinAnalogueGain);\n \tmaxAnalogueGain_ = configuration.sensor.maxAnalogueGain;\n \n@@ -117,10 +108,10 @@ int Agc::configure(IPAContext &context,\n \n \tAgcMeanLuminance::SensorConfiguration sensorConfig;\n \tsensorConfig.lineDuration = context.configuration.sensor.lineDuration;\n-\tsensorConfig.minExposureTime = minExposureTime_;\n+\tsensorConfig.minExposureTime = configuration.sensor.minExposureTime;\n \tsensorConfig.minFrameDuration = context.configuration.sensor.minFrameDuration;\n-\tsensorConfig.maxFrameDuration = context.configuration.sensor.maxFrameDuration;\n-\tsensorConfig.maxExposureTime = maxExposureTime_;\n+\tsensorConfig.maxFrameDuration = std::min(context.configuration.sensor.maxFrameDuration,\n+\t\t\t\t\t\t kMaxExposureTime);\n \tsensorConfig.minAnalogueGain = minAnalogueGain_;\n \tsensorConfig.maxAnalogueGain = maxAnalogueGain_;\n \ndiff --git a/src/ipa/ipu3/algorithms/agc.h b/src/ipa/ipu3/algorithms/agc.h\nindex 890c271b44627f337c0d8a85853e1ecc3bae1318..4c2e4e3e5913d87a5af420b17f6c487aeb6cc37a 100644\n--- a/src/ipa/ipu3/algorithms/agc.h\n+++ b/src/ipa/ipu3/algorithms/agc.h\n@@ -27,7 +27,6 @@ namespace ipa::ipu3::algorithms {\n class Agc : public Algorithm, public AgcMeanLuminance\n {\n public:\n-\tAgc();\n \t~Agc() = default;\n \n \tint init(IPAContext &context, const YamlObject &tuningData) override;\n@@ -42,9 +41,6 @@ private:\n \tHistogram parseStatistics(const ipu3_uapi_stats_3a *stats,\n \t\t\t\t const ipu3_uapi_grid_config &grid);\n \n-\tutils::Duration minExposureTime_;\n-\tutils::Duration maxExposureTime_;\n-\n \tdouble minAnalogueGain_;\n \tdouble maxAnalogueGain_;\n \ndiff --git a/src/ipa/ipu3/ipa_context.cpp b/src/ipa/ipu3/ipa_context.cpp\nindex 47ae750f0ddb859e1a08e75e255b8decacd730a1..4281ffba5f7fc841e47132986631c0ee44c898ff 100644\n--- a/src/ipa/ipu3/ipa_context.cpp\n+++ b/src/ipa/ipu3/ipa_context.cpp\n@@ -111,9 +111,6 @@ namespace libcamera::ipa::ipu3 {\n * \\var IPASessionConfiguration::agc.minExposureTime\n * \\brief Minimum exposure time supported with the configured sensor\n *\n- * \\var IPASessionConfiguration::agc.maxExposureTime\n- * \\brief Maximum exposure time supported with the configured sensor\n- *\n * \\var IPASessionConfiguration::agc.minAnalogueGain\n * \\brief Minimum analogue gain supported with the configured sensor\n *\ndiff --git a/src/ipa/ipu3/ipa_context.h b/src/ipa/ipu3/ipa_context.h\nindex 33f3fe3ae1b503d47887873046def0835cd53894..11b9c286d56ac6f30d446e3721f7e5db4364a6fb 100644\n--- a/src/ipa/ipu3/ipa_context.h\n+++ b/src/ipa/ipu3/ipa_context.h\n@@ -38,7 +38,6 @@ struct IPASessionConfiguration {\n \t\tutils::Duration lineDuration;\n \t\tSize size;\n \t\tutils::Duration minExposureTime;\n-\t\tutils::Duration maxExposureTime;\n \t\tutils::Duration minFrameDuration;\n \t\tutils::Duration maxFrameDuration;\n \t\tdouble minAnalogueGain;\ndiff --git a/src/ipa/ipu3/ipu3.cpp b/src/ipa/ipu3/ipu3.cpp\nindex 0c8651c5235f9e4e9944eb88595aeef41f016310..61e8030d722f430e702a8b8f7e2880caa961fa8e 100644\n--- a/src/ipa/ipu3/ipu3.cpp\n+++ b/src/ipa/ipu3/ipu3.cpp\n@@ -206,7 +206,6 @@ void IPAIPU3::updateSessionConfiguration(const IPACameraSensorInfo &sensorInfo,\n \n \tconst ControlInfo &v4l2Exposure = sensorControls.find(V4L2_CID_EXPOSURE)->second;\n \tint32_t minExposure = v4l2Exposure.min().get<int32_t>();\n-\tint32_t maxExposure = v4l2Exposure.max().get<int32_t>();\n \n \tconst ControlInfo &v4l2Gain = sensorControls.find(V4L2_CID_ANALOGUE_GAIN)->second;\n \tint32_t minGain = v4l2Gain.min().get<int32_t>();\n@@ -226,7 +225,6 @@ void IPAIPU3::updateSessionConfiguration(const IPACameraSensorInfo &sensorInfo,\n \t * \\todo take VBLANK into account for maximum exposure time\n \t */\n \tcontext_.configuration.sensor.minExposureTime = minExposure * context_.configuration.sensor.lineDuration;\n-\tcontext_.configuration.sensor.maxExposureTime = maxExposure * context_.configuration.sensor.lineDuration;\n \tcontext_.configuration.sensor.minFrameDuration = frameHeights[0] *\n \t\t\t\t\t\t\t context_.configuration.sensor.lineDuration;\n \tcontext_.configuration.sensor.maxFrameDuration = frameHeights[1] *\ndiff --git a/src/ipa/libipa/agc_mean_luminance.cpp b/src/ipa/libipa/agc_mean_luminance.cpp\nindex 725a23ef2f6f612c6d3408701246db7415fd8327..b930e1f7240d4936aa8dc850657bbbf9c2f3a11f 100644\n--- a/src/ipa/libipa/agc_mean_luminance.cpp\n+++ b/src/ipa/libipa/agc_mean_luminance.cpp\n@@ -115,9 +115,6 @@ static constexpr double kMaxRelativeLuminanceTarget = 0.95;\n * \\var AgcMeanLuminance::SensorConfiguration::minExposureTime\n * \\brief The sensor minimum exposure time in microseconds\n *\n- * \\var AgcMeanLuminance::SensorConfiguration::maxExposureTime\n- * \\brief The sensor maximum exposure time in microseconds\n- *\n * \\var AgcMeanLuminance::SensorConfiguration::minFrameDuration\n * \\brief The sensor minimum frame duration in microseconds\n *\n@@ -372,7 +369,6 @@ void AgcMeanLuminance::configure(const SensorConfiguration &config,\n \t\tExposureModeHelper::SensorConfiguration sensorConfig;\n \t\tsensorConfig.lineDuration_ = config.lineDuration;\n \t\tsensorConfig.minExposureTime_ = config.minExposureTime;\n-\t\tsensorConfig.maxExposureTime_ = config.maxExposureTime;\n \t\tsensorConfig.minFrameDuration_ = config.minFrameDuration;\n \t\tsensorConfig.maxFrameDuration_ = config.maxFrameDuration;\n \t\tsensorConfig.minGain_ = config.minAnalogueGain;\ndiff --git a/src/ipa/libipa/agc_mean_luminance.h b/src/ipa/libipa/agc_mean_luminance.h\nindex acbefc4e5765413bc803417eae1dbd0a943bc95e..93a0959bbd9e0d6ec42248f2d3b19253ad389ae6 100644\n--- a/src/ipa/libipa/agc_mean_luminance.h\n+++ b/src/ipa/libipa/agc_mean_luminance.h\n@@ -45,7 +45,6 @@ public:\n \tstruct SensorConfiguration {\n \t\tutils::Duration lineDuration;\n \t\tutils::Duration minExposureTime;\n-\t\tutils::Duration maxExposureTime;\n \t\tutils::Duration minFrameDuration;\n \t\tutils::Duration maxFrameDuration;\n \t\tdouble minAnalogueGain;\ndiff --git a/src/ipa/mali-c55/algorithms/agc.cpp b/src/ipa/mali-c55/algorithms/agc.cpp\nindex 91b1438f7e5ca0498373c86fd75b91f9c5a81c3f..a0b55694aad292f8a080d8266470797ac0cc2c25 100644\n--- a/src/ipa/mali-c55/algorithms/agc.cpp\n+++ b/src/ipa/mali-c55/algorithms/agc.cpp\n@@ -176,7 +176,6 @@ int Agc::configure(IPAContext &context,\n \tAgcMeanLuminance::SensorConfiguration sensorConfig;\n \tsensorConfig.lineDuration = context.configuration.sensor.lineDuration;\n \tsensorConfig.minExposureTime = context.configuration.sensor.minShutterSpeed;\n-\tsensorConfig.maxExposureTime = context.configuration.sensor.maxShutterSpeed;\n \tsensorConfig.minFrameDuration = context.configuration.sensor.minFrameDuration;\n \tsensorConfig.maxFrameDuration = context.configuration.sensor.maxFrameDuration;\n \tsensorConfig.minAnalogueGain = context.configuration.sensor.minAnalogueGain;\ndiff --git a/src/ipa/mali-c55/ipa_context.h b/src/ipa/mali-c55/ipa_context.h\nindex 3b64cb7571729d4af162def5b2316331b3561af1..828103f21451d9f7f4998c3faedc8fb6a1e7a2ec 100644\n--- a/src/ipa/mali-c55/ipa_context.h\n+++ b/src/ipa/mali-c55/ipa_context.h\n@@ -29,7 +29,6 @@ struct IPASessionConfiguration {\n \t\tutils::Duration lineDuration;\n \t\tuint32_t blackLevel;\n \t\tutils::Duration minShutterSpeed;\n-\t\tutils::Duration maxShutterSpeed;\n \t\tutils::Duration minFrameDuration;\n \t\tutils::Duration maxFrameDuration;\n \t\tdouble minAnalogueGain;\ndiff --git a/src/ipa/mali-c55/mali-c55.cpp b/src/ipa/mali-c55/mali-c55.cpp\nindex 491ae71a06dbede967bfbe1bcdcab25d177ad691..02f5dfb76eae073858ec688746b7e12ec072e567 100644\n--- a/src/ipa/mali-c55/mali-c55.cpp\n+++ b/src/ipa/mali-c55/mali-c55.cpp\n@@ -170,7 +170,6 @@ void IPAMaliC55::updateSessionConfiguration(const IPACameraSensorInfo &info,\n \n \tconst ControlInfo &v4l2Exposure = sensorControls.find(V4L2_CID_EXPOSURE)->second;\n \tint32_t minExposure = v4l2Exposure.min().get<int32_t>();\n-\tint32_t maxExposure = v4l2Exposure.max().get<int32_t>();\n \tint32_t defExposure = v4l2Exposure.def().get<int32_t>();\n \n \tconst ControlInfo &v4l2Gain = sensorControls.find(V4L2_CID_ANALOGUE_GAIN)->second;\n@@ -191,7 +190,6 @@ void IPAMaliC55::updateSessionConfiguration(const IPACameraSensorInfo &info,\n \tutils::Duration lineDuration = info.minLineLength * 1.0s / info.pixelRate;\n \tcontext_.configuration.sensor.lineDuration = lineDuration;\n \tcontext_.configuration.sensor.minShutterSpeed = minExposure * lineDuration;\n-\tcontext_.configuration.sensor.maxShutterSpeed = maxExposure * lineDuration;\n \tcontext_.configuration.sensor.minFrameDuration = frameHeights[0] * lineDuration;\n \tcontext_.configuration.sensor.maxFrameDuration = frameHeights[1] * lineDuration;\n \tcontext_.configuration.sensor.minAnalogueGain = context_.camHelper->gain(minGain);\ndiff --git a/src/ipa/rkisp1/algorithms/agc.cpp b/src/ipa/rkisp1/algorithms/agc.cpp\nindex b9a94ba03c910f73420579dd6737d8d46b26e576..a2ecd5c46a1fbcb728e23ed83b37b89fcdb80d84 100644\n--- a/src/ipa/rkisp1/algorithms/agc.cpp\n+++ b/src/ipa/rkisp1/algorithms/agc.cpp\n@@ -203,8 +203,6 @@ int Agc::configure(IPAContext &context, const IPACameraSensorInfo &configInfo)\n \tAgcMeanLuminance::SensorConfiguration sensorConfig;\n \tsensorConfig.lineDuration = context.configuration.sensor.lineDuration;\n \tsensorConfig.minExposureTime = context.configuration.sensor.minExposureTime;\n-\tsensorConfig.maxExposureTime = context.configuration.sensor.maxExposureTime;\n-\tsensorConfig.minFrameDuration = context.configuration.sensor.minFrameDuration;\n \tsensorConfig.maxFrameDuration = context.configuration.sensor.maxFrameDuration;\n \tsensorConfig.minAnalogueGain = context.configuration.sensor.minAnalogueGain;\n \tsensorConfig.maxAnalogueGain = context.configuration.sensor.maxAnalogueGain;\ndiff --git a/src/ipa/rkisp1/ipa_context.cpp b/src/ipa/rkisp1/ipa_context.cpp\nindex 15cb0afe9fe8d266d645a27cc3a3e440a0dd2413..40c5d244cf524e9ca3455183ee02237892e70312 100644\n--- a/src/ipa/rkisp1/ipa_context.cpp\n+++ b/src/ipa/rkisp1/ipa_context.cpp\n@@ -89,9 +89,6 @@ namespace libcamera::ipa::rkisp1 {\n * \\var IPASessionConfiguration::sensor.minExposureTime\n * \\brief Minimum exposure time supported with the sensor\n *\n- * \\var IPASessionConfiguration::sensor.maxExposureTime\n- * \\brief Maximum exposure time supported with the sensor\n- *\n * \\var IPASessionConfiguration::sensor.minAnalogueGain\n * \\brief Minimum analogue gain supported with the sensor\n *\ndiff --git a/src/ipa/rkisp1/ipa_context.h b/src/ipa/rkisp1/ipa_context.h\nindex 5fe727bd0b508617d993d226ae785056a3771ce0..52764dbc0a8f579332f0785baf86ba7a1f6db9e2 100644\n--- a/src/ipa/rkisp1/ipa_context.h\n+++ b/src/ipa/rkisp1/ipa_context.h\n@@ -61,7 +61,6 @@ struct IPASessionConfiguration {\n \n \tstruct {\n \t\tutils::Duration minExposureTime;\n-\t\tutils::Duration maxExposureTime;\n \t\tutils::Duration minFrameDuration;\n \t\tutils::Duration maxFrameDuration;\n \t\tdouble minAnalogueGain;\ndiff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp\nindex f25e477f0fb77241bd1ccddb7778205e58bdc8a9..4da7cf36400df897ef4392fb264b1f6401391ad0 100644\n--- a/src/ipa/rkisp1/rkisp1.cpp\n+++ b/src/ipa/rkisp1/rkisp1.cpp\n@@ -269,7 +269,6 @@ int IPARkISP1::configure(const IPAConfigInfo &ipaConfig,\n \t * on the sensor, update it with the controls.\n \t */\n \tcontext_.configuration.sensor.minExposureTime = minExposure * lineDuration;\n-\tcontext_.configuration.sensor.maxExposureTime = maxExposure * lineDuration;\n \tcontext_.configuration.sensor.minFrameDuration = frameHeights[0] * lineDuration;\n \tcontext_.configuration.sensor.maxFrameDuration = frameHeights[1] * lineDuration;\n \tcontext_.configuration.sensor.minAnalogueGain = context_.camHelper->gain(minGain);\n", "prefixes": [ "v3", "16/19" ] }