[libcamera-devel] [PATCH v2 0/3] ipa: ipu3: IPAFrameContext queue

Umang Jain umang.jain at ideasonboard.com
Fri May 6 11:53:04 CEST 2022


Changes in v2:
- Majorly change the interpretation of Frame context
  Currently, everything is stored in a single struct i.e.
  IPAFrameContext. It reflects the current state of the algorithms
  and also stores values required to prepare frame metadata.
  Split these two things in separate containters IPAActiveState
  and IPAFrameContext.
- The above design helps to rid of prevFrameContext and nextFrameContext
  concepts as they can get error=prone easily.
- Have a queue of IPAFrameContexts which is holding members per-frame
  related /only/. All algorithm current (or active) state values go in
  IPAActiveState which is independently updated by the algorithms.

Umang Jain (3):
  ipa: ipu3: Add a frameCompleted() helper
  ipa: ipu3: Rework IPAFrameContext
  ipa: ipu3: Introduce a IPAFrameContext queue

 src/ipa/ipu3/algorithms/af.cpp           |  42 +++++-----
 src/ipa/ipu3/algorithms/agc.cpp          |  23 +++---
 src/ipa/ipu3/algorithms/agc.h            |   2 +-
 src/ipa/ipu3/algorithms/awb.cpp          |  16 ++--
 src/ipa/ipu3/algorithms/tone_mapping.cpp |  10 +--
 src/ipa/ipu3/ipa_context.cpp             | 100 ++++++++++++++---------
 src/ipa/ipu3/ipa_context.h               |  22 +++--
 src/ipa/ipu3/ipu3.cpp                    |  50 +++++++++---
 8 files changed, 165 insertions(+), 100 deletions(-)

-- 
2.31.1



More information about the libcamera-devel mailing list