[libcamera-devel,v2,0/3] ipa: ipu3: IPAFrameContext queue
mbox series

Message ID 20220506095307.78370-1-umang.jain@ideasonboard.com
Headers show
Series
  • ipa: ipu3: IPAFrameContext queue
Related show

Message

Umang Jain May 6, 2022, 9:53 a.m. UTC 
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(-)