[v2,0/5] ipa: simple: Introduce metadata reporting
mbox series

Message ID 20241219211010.103310-1-mzamazal@redhat.com
Headers show
Series
  • ipa: simple: Introduce metadata reporting
Related show

Message

Milan Zamazal Dec. 19, 2024, 9:10 p.m. UTC 
Provide metadata reporting for the software ISP. Initially the Black
Levels, ColourGains and Contrast are reported, but also serve as a
direction to add further metadata or internal reporting of the software
ISP.

Running cam -c1 -C10 --metadata now produces the following:

1668.359684 (14.95 fps) cam0-stream0 seq: 000004 bytesused: 15069888
	SensorBlackLevels = [ 4096, 4096, 4096, 4096 ]
	ColourGains = [ 0.316406, 0.299805 ]
	SensorTimestamp = 1668359684000

Changes in v2:
- Taken over from Kieran.
- Updated for master.
- Black level is reported as uint16_t, according to the control
  documentation.
- Contrast value is reported in metadata.
- The metadata used to proces the image is reported rather than the
  metadata determined from the image.
- Metadata is reported using a new signal rather than (ab)using
  setIspParams; this change is less or more needed due to changed simple
  IPA structure since v1.
- Frame -> request mapping is introduced to make sure the metadata is
  assigned to the right request.

Kieran Bingham (3):
  ipa: simple: softisp: Extend to pass metadata
  ipa: simple: Report the ColourGains in metadata
  ipa: simple: Report black levels in metadata

Milan Zamazal (2):
  libcamera: software_isp: Track frames and requests
  ipa: simple: Report contrast in metadata

 .../internal/software_isp/software_isp.h      |   2 +
 include/libcamera/ipa/soft.mojom              |   1 +
 src/ipa/simple/algorithms/awb.cpp             |  11 +-
 src/ipa/simple/algorithms/blc.cpp             |   9 +-
 src/ipa/simple/algorithms/lut.cpp             |  11 ++
 src/ipa/simple/algorithms/lut.h               |   5 +
 src/ipa/simple/soft_simple.cpp                |   7 +-
 src/libcamera/pipeline/simple/simple.cpp      | 108 +++++++++++++++++-
 src/libcamera/software_isp/software_isp.cpp   |  11 ++
 9 files changed, 151 insertions(+), 14 deletions(-)