[PATCH v5 00/18] libcamera: introduce Software ISP and Software IPA
Hans de Goede
hdegoede at redhat.com
Mon Mar 11 15:15:04 CET 2024
Here is v5 of the patch-set to add Software ISP support
to libcamera / to the simple pipeline-handler.
Changes in v5 vs v4:
- CI fully passes now!
- Fix some small typos / using unnamed const values pointed
out in Milan's reviews
Since there have been no significant review comments on v4
and since this fully passes CI now I believe that this version
is ready for merging.
git branch for this v5 patch set:
https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/SoftwareISP-v08-for-ci
Older changelogs and links to git branches:
Changes in v4 vs v3:
- Andrey:
[05/18] "libcamera: shared_mem_object: reorganize the code and document the SharedMemObject class"
- documentation fixes
[09/18] "libcamera: ipa: add Soft IPA"
- Use int32_t for again*_ and exposure*_ as this matches
the value the corresponding ControlValue::get() returns
- Check for mmap() returning MAP_FAILED on error
- Drop std::move() called on const SharedFD & argument
- Replace #defines (EXPOSURE_OPTIMAL_VALUE etc) with const expressions
- Use std::clamp() to keep exposure_ and again_ between the *_min_ and
the *_max_
- add comment on non-linear gain value vs gain code
[10/18] "libcamera: introduce SoftwareIsp"
- Fix the check of if sharedParams_ were created OK
new [18/18] "libcamera: Soft IPA: use CameraSensorHelper for analogue gain"
- One can check if his camera sensor has the CameraSensorHelper implemented with something
like 'grep ^REGISTER_CAMERA_SENSOR_HELPER src/ipa/libipa/camera_sensor_helper.cpp'. If
the camera sensor used has no CameraSensorHelper, the Soft IPA works as before this patch.
This fallback option doesn't make the code nicer, and makes it harder to change the AE/AGC
algorithm to work faster, but I decided to keep it for now.
- Milan:
[17/18] "libcamera: software_isp: Apply black level compensation"
- New patch adding black-level compensation autmatically detecting the
sensor blacklevel
- Hans:
- Small typo changes in various comments/docs
- Replace typedef-s with using
Changes in v3 vs v2:
- The Software ISP now is always build when building the simple pipeline
handler, no more -Dpipelines=simple/simple. Instead whether the SoftISP
is used or not is based on the media-controller driver. For now it is
only enabled for the "qcom-camss" driver (Andrey)
- SoftISP factory has been removed, there is a just a single SoftwareISP class now
- Fix the multi-threading issues (Andrey)
- Fully document the SharedMemObject and DmaHeap classes (Andrey)
- Drop SwStats base (integrate into SwStatsCpu class)
- Move headers for classes only used by the SoftISP into src/libcamera/software_isp
- Move SwStats / SwDebayer docs into .cpp and extend it
- Rename many foo_bar symbols to fooBar
- Add constexpr kFooBar values for varies hardcoded sizes like
the yHistoGram having 16 bins and the gammalookuptable having 1024 entries
- Make startFrame() and finishFrame() normal methods instead of
using function pointers for these
- Document how/why an array of src pointers is passed to
the debayer functions
- 12bpp unpacked raw bayer input support (Kieran)
- Add "Software ISP benchmarking" doc
Changes in v2 vs v1:
- Integrated Dennis, Martti and Toon's auto-exposure algorithm
based on the paper which they found which gives us a nice
relatively simple AEC + AGC algorithm
TODO: Add link to paper to source + commit-message
- Integrated Dennis' doxygen comments for all new classes, no more warnings!
- Move AWB gain calculations to the IPA (Andrey)
- Added 8 bpp and 10 bpp unpacked raw bayer input support (Hans)
- Use dma-buf for the output FrameBuffer-s (Andrey)
- Memcpy data from input FrameBuffers to a heap buffer on a line by line
basis to speedup debayering from uncached mem (Hans)
- This addresses all "open issues" from the v1 posting
Changes in v1 vs RFC-v2:
- Add and use SwStats[Cpu] and Debayer[Cpu] classes
- Rename linaro soft-IPA and soft-ISP implementations to simple following
the simple pipeline hander
- Integrate Pavel's swstats and debayer improvements
This has been tested on:
- Qualcomm RB5 board with a mezzanine board equipped with RPi camera v2 (Andrey)
- Lenovo x13s, sc8280xp (Bryan)
- Pinephone (Pavel)
- Debix Model A (Milan)
- Lenovo Thinkpad Yoga X1 yoga gen7/8, Alder Lake/Raptor Lake IPU6EP +
ov2740 10bpp packed + unpacked (Dennis / Hans)
- Dell Latitude 9420, Tiger Lake IPU6 + ov01a1s (RGBI) sensor
(Martti, requires IGIG_GBGR_IGIG_GRGB patches on top)
git branch for v4 of the patch set:
https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/SoftwareISP-v07
git branch for v3 of the patch set:
https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/SoftwareISP-v06
git branch for v2 of the patch set:
https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/SoftwareISP-v05
git branch for v1 of the patch set:
https://gitlab.freedesktop.org/camera/libcamera-softisp/-/commits/SoftwareISP-v04
Regards,
Hans
Andrei Konovalov (2):
libcamera: shared_mem_object: reorganize the code and document the
SharedMemObject class
libcamera: Soft IPA: use CameraSensorHelper for analogue gain
Andrey Konovalov (8):
libcamera: pipeline: simple: fix size adjustment in validate()
libcamera: internal: Move dma_heaps.[h,cpp] to common directories
libcamera: dma_heaps: extend DmaHeap class to support system heap
libcamera: internal: Move SharedMemObject class to a common directory
libcamera: ipa: add Soft IPA
libcamera: introduce SoftwareIsp
libcamera: pipeline: simple: rename converterBuffers_ and related vars
libcamera: pipeline: simple: enable use of Soft ISP and Soft IPA
Hans de Goede (7):
libcamera: software_isp: Add SwStatsCpu class
libcamera: software_isp: Add Debayer base class
libcamera: software_isp: Add DebayerCpu class
libcamera: swstats_cpu: Add support for 8, 10 and 12 bpp unpacked
bayer input
libcamera: debayer_cpu: Add support for 8, 10 and 12 bpp unpacked
bayer input
libcamera: debayer_cpu: Add BGR888 output support
libcamera: Add "Software ISP benchmarking" documentation
Milan Zamazal (1):
libcamera: software_isp: Apply black level compensation
Documentation/Doxyfile.in | 1 +
Documentation/index.rst | 1 +
Documentation/meson.build | 1 +
Documentation/software-isp-benchmarking.rst | 82 ++
.../libcamera/internal}/dma_heaps.h | 14 +-
include/libcamera/internal/meson.build | 3 +
.../libcamera/internal}/shared_mem_object.h | 105 ++-
.../internal/software_isp/debayer_params.h | 52 ++
.../internal/software_isp/meson.build | 7 +
.../internal/software_isp/software_isp.h | 99 +++
.../internal/software_isp/swisp_stats.h | 44 +
include/libcamera/ipa/meson.build | 1 +
include/libcamera/ipa/soft.mojom | 28 +
meson_options.txt | 2 +-
src/ipa/simple/black_level.cpp | 86 ++
src/ipa/simple/black_level.h | 28 +
src/ipa/simple/data/meson.build | 9 +
src/ipa/simple/data/soft.conf | 3 +
src/ipa/simple/meson.build | 30 +
src/ipa/simple/soft_simple.cpp | 359 ++++++++
src/libcamera/dma_heaps.cpp | 164 ++++
src/libcamera/meson.build | 3 +
src/libcamera/pipeline/rpi/vc4/dma_heaps.cpp | 90 --
src/libcamera/pipeline/rpi/vc4/meson.build | 1 -
src/libcamera/pipeline/rpi/vc4/vc4.cpp | 5 +-
src/libcamera/pipeline/simple/simple.cpp | 234 ++++--
src/libcamera/shared_mem_object.cpp | 190 +++++
src/libcamera/software_isp/debayer.cpp | 29 +
src/libcamera/software_isp/debayer.h | 104 +++
src/libcamera/software_isp/debayer_cpu.cpp | 791 ++++++++++++++++++
src/libcamera/software_isp/debayer_cpu.h | 158 ++++
src/libcamera/software_isp/meson.build | 15 +
src/libcamera/software_isp/software_isp.cpp | 349 ++++++++
src/libcamera/software_isp/swstats_cpu.cpp | 336 ++++++++
src/libcamera/software_isp/swstats_cpu.h | 168 ++++
35 files changed, 3390 insertions(+), 202 deletions(-)
create mode 100644 Documentation/software-isp-benchmarking.rst
rename {src/libcamera/pipeline/rpi/vc4 => include/libcamera/internal}/dma_heaps.h (65%)
rename {src/libcamera/pipeline/rpi/common => include/libcamera/internal}/shared_mem_object.h (63%)
create mode 100644 include/libcamera/internal/software_isp/debayer_params.h
create mode 100644 include/libcamera/internal/software_isp/meson.build
create mode 100644 include/libcamera/internal/software_isp/software_isp.h
create mode 100644 include/libcamera/internal/software_isp/swisp_stats.h
create mode 100644 include/libcamera/ipa/soft.mojom
create mode 100644 src/ipa/simple/black_level.cpp
create mode 100644 src/ipa/simple/black_level.h
create mode 100644 src/ipa/simple/data/meson.build
create mode 100644 src/ipa/simple/data/soft.conf
create mode 100644 src/ipa/simple/meson.build
create mode 100644 src/ipa/simple/soft_simple.cpp
create mode 100644 src/libcamera/dma_heaps.cpp
delete mode 100644 src/libcamera/pipeline/rpi/vc4/dma_heaps.cpp
create mode 100644 src/libcamera/shared_mem_object.cpp
create mode 100644 src/libcamera/software_isp/debayer.cpp
create mode 100644 src/libcamera/software_isp/debayer.h
create mode 100644 src/libcamera/software_isp/debayer_cpu.cpp
create mode 100644 src/libcamera/software_isp/debayer_cpu.h
create mode 100644 src/libcamera/software_isp/meson.build
create mode 100644 src/libcamera/software_isp/software_isp.cpp
create mode 100644 src/libcamera/software_isp/swstats_cpu.cpp
create mode 100644 src/libcamera/software_isp/swstats_cpu.h
--
2.44.0
More information about the libcamera-devel
mailing list