[PATCH 10/16] libcamera: software_isp: Remove per-frame params buffers TODO item

Milan Zamazal mzamazal at redhat.com
Mon Aug 12 13:49:59 CEST 2024


Parameters are stored in per-frame buffers and shared properly now.

Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
---
 src/libcamera/software_isp/TODO | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/src/libcamera/software_isp/TODO b/src/libcamera/software_isp/TODO
index a50db668..db1115dd 100644
--- a/src/libcamera/software_isp/TODO
+++ b/src/libcamera/software_isp/TODO
@@ -53,24 +53,6 @@ stats in hardware, such as the i.MX7), but please keep it on your radar.
 
 ---
 
-5. Store ISP parameters in per-frame buffers
-
-> /**
->  * \fn void Debayer::process(FrameBuffer *input, FrameBuffer *output, DebayerParams params)
->  * \brief Process the bayer data into the requested format.
->  * \param[in] input The input buffer.
->  * \param[in] output The output buffer.
->  * \param[in] params The parameters to be used in debayering.
->  *
->  * \note DebayerParams is passed by value deliberately so that a copy is passed
->  * when this is run in another thread by invokeMethod().
->  */
-
-Possibly something to address later, by storing ISP parameters in
-per-frame buffers like we do for hardware ISPs.
-
----
-
 6. Input buffer copying configuration
 
 > DebayerCpu::DebayerCpu(std::unique_ptr<SwStatsCpu> stats)
-- 
2.44.1



More information about the libcamera-devel mailing list