[PATCH v4 0/4] libcamera: converter: Replace usage of stream index by Stream pointer

[Umang Jain]

The v4l2_converter_m2m and simple pipeline handler are adapt to
use the new interface. This work roped in software ISP as well,
which also seems to use indexes (although it doesn't implement converter
interface) because of a common conversionQueue_ queue used for
converter_ and swIsp_.

Patch 1/4 and 2/4 drop redundant  validation of outputs std::map<>.

Patch 3/4 renames private class V4L2M2MConverter::Stream to
V4L2M2Mconverter::V4L2M2MStream

Patch 4/4 replaces the stream index usage from converter interface and
soft isp.

---
Tested with mxc-isi running software isp on i.MX8MP platform
---

changes in v4:
- Emplace V4L2M2MStream stream only after configure() succeeds.
  (trivial shuffle of ordering of the call).
- Collect R-b tags

Changes in v3:
- Fix issue while running soft-isp
- Fix missing Doxygen changes

changes in v2:
- Split out patches 1/4 and 2/4 to tackle validation/sanity check
- Rename V4L2M2MConverter::Stream to V4L2M2Mconverter::V4L2M2MStream

Umang Jain (4):
  converter: converter_v4l2_m2m: Rectify streams sanity check
  libcamera: software_isp: Drop unnecessary sanity check
  converter: converter_v4l2_m2m: Rename private Stream class
  libcamera: converter: Replace usage of stream index by Stream pointer

 include/libcamera/internal/converter.h        |  5 +-
 .../internal/converter/converter_v4l2_m2m.h   | 13 +--
 .../internal/software_isp/software_isp.h      |  5 +-
 src/libcamera/converter.cpp                   |  6 +-
 .../converter/converter_v4l2_m2m.cpp          | 79 ++++++++++---------
 src/libcamera/pipeline/simple/simple.cpp      | 14 ++--
 src/libcamera/software_isp/software_isp.cpp   | 26 +++---
 7 files changed, 73 insertions(+), 75 deletions(-)

-- 
2.44.0