Software ISP TODOs
Milan Zamazal
mzamazal at redhat.com
Tue Jul 30 16:13:52 CEST 2024
Hi,
I have been working on some of the software ISP TODOs, here is an
overview of the current state, FYI. See
https://git.libcamera.org/libcamera/libcamera.git/tree/src/libcamera/software_isp/TODO
for descriptions of the TODOs.
*
1. Setting F_SEAL_SHRINK and F_SEAL_GROW after ftruncate()
Better to be addressed by someone who fully understands what's the right
thing to do.
*
2. Reconsider stats sharing
3. Remove statsReady signal
5. Store ISP parameters in per-frame buffers
#2 + #5 part of the patches I work on (not yet posted but working for
me), maybe I'll handle #3 too, it's all WIP.
*
6. Input buffer copying configuration
7. Performance measurement configuration
Part of the posted global configuration file patches.
*
8. DebayerCpu cleanups
This is partially addressed by my patches, which compute the lookup
tables elsewhere and get rid of getStatsFd method at least. But
DebayerCpu still keeps gathering stats, I don't think we can do much
about it. I'd keep this TODO restricted to what can be reasonably done.
Let's also see what Bryan invents while working on DebayerGpu.
*
8. Decouple pipeline and IPA naming
Better to be addressed by someone who knows what's the right thing to
do.
*
9. Doxyfile cleanup
Not yet handled.
*
10. Switch to libipa/algorithm.h API in processStats
11. Improve handling the sensor controls which take effect with a delay
12. Use DelayedControls class in ispStatsReady()
Part of the posted software ISP refactoring patches.
*
13. Improve black level and colour gains application
I don't think we know how to resolve this (it's similar to #8 Debayer
cleanups) and I don't see this as urgent (e.g. the refactoring related
to #10-#12 is way more important for further work on software ISP).
Regards,
Milan
More information about the libcamera-devel
mailing list