[v5,0/5] ipa: libipa: Introduce V4L2Params
mbox series

Message ID 20251007-v4l2-params-v5-0-8db451a81398@ideasonboard.com
Headers show
Series
  • ipa: libipa: Introduce V4L2Params
Related show

Message

Jacopo Mondi Oct. 7, 2025, 6:17 p.m. UTC 
With the introduction of v4l2-isp.h in the Linux kernel currently in
progress:
https://lore.kernel.org/all/20251007-extensible-parameters-validation-v6-0-5f719d9f39e5@ideasonboard.com/

the MaliC55 ISP driver at version in v11 has been ported to use a
parameters format compatible with the one defined by v4l2 extensible
parameters:
https://lore.kernel.org/all/20250714-c55-v11-0-bc20e460e42a@ideasonboard.com/

This series
1) Imports v4l2-isp in libcamera
2) Update the Mali C55 header and IPA to compile with v11 of the patch
   series that introduces MaliC55 support in Linux
3) Introduces V4L2Params by generalizing RkISP1Params
4) Ports the Mali C55 IPA to use the newly introduced V4L2Params

Tested on rkisp1 and rzv2h

Signed-off-by: Jacopo Mondi <jacopo.mondi@ideasonboard.com>
---
Changes in v5:
- Update to v6 of v4l2 isp
- Import linux/stddef.h instead of re-defining __counted_by()
- Properly memset the whole mali-c55 buffer in the transitory patch
  before MaliC55Params introduction

Changes in v4:
- Mark RkISP1ParamBlock final
- Fix code examples as suggested by Barnabas

Changes in v3:
- Fix Mali-C55 and re-work RkISP1
  The Mali and RkISP1 IPA have a different requirements:

	struct mali_c55_params_awb_gains {
		struct v4l2_params_block_header header;
		__u16 gain00;
		__u16 gain01;
		__u16 gain10;
		__u16 gain11;
	};

	struct rkisp1_ext_params_awb_gain_config {
		struct rkisp1_ext_params_block_header header;
		struct rkisp1_cif_isp_awb_gain_config config;
	} __attribute__((aligned(8)));

  The RkISP1 IPA needs to support legacy and extensible parameter
  formats, and wants to operate on 'struct rkisp1_cif_isp_' basically
  discarding the header.

  The Mali IPA (and all other IPA compatible with v4l2-isp) operates on
  the whole structure and do not need to discard the header.

  Rework V4L2ParamsBlock to return a Span<> matching the whole structure
  (header included) to support Mali correctly.

  This means the RkISP1ParamsBlock derived class has to override all the
  dereference operators to discard the header and allow the IPA to work
  with both the legacy and extensible formats.

  Tested with Mali C55, RkISP1 in extensible and legacy modes.

- Link to v2: https://lore.kernel.org/r/20250916-v4l2-params-v2-0-764471ed6435@ideasonboard.com

Changes in v2:
- Address Barnabas' reviwe that allows to remove the block() overload in
  MaliC55
- Link to v1: https://lore.kernel.org/r/20250829-v4l2-params-v1-0-340773fb69ff@ideasonboard.com

---
Jacopo Mondi (5):
      include: linux: Add stddef.h
      include: linux: Add v4l2-isp.h
      ipa: Update Mali C55 header file and adjust IPA
      ipa: libipa: Introduce V4L2Params
      ipa: mali-c55: Introduce MaliC55Params

 include/linux/mali-c55-config.h         | 161 +++-----------------
 include/linux/media/v4l2-isp.h          |  82 +++++++++++
 include/linux/stddef.h                  |  75 ++++++++++
 src/ipa/libipa/meson.build              |   2 +
 src/ipa/libipa/v4l2_params.cpp          | 254 ++++++++++++++++++++++++++++++++
 src/ipa/libipa/v4l2_params.h            | 142 ++++++++++++++++++
 src/ipa/mali-c55/algorithms/agc.cpp     |  87 +++++------
 src/ipa/mali-c55/algorithms/agc.h       |  14 +-
 src/ipa/mali-c55/algorithms/algorithm.h |   2 +-
 src/ipa/mali-c55/algorithms/awb.cpp     |  64 ++++----
 src/ipa/mali-c55/algorithms/awb.h       |  10 +-
 src/ipa/mali-c55/algorithms/blc.cpp     |  20 +--
 src/ipa/mali-c55/algorithms/blc.h       |   3 +-
 src/ipa/mali-c55/algorithms/lsc.cpp     |  58 +++-----
 src/ipa/mali-c55/algorithms/lsc.h       |   8 +-
 src/ipa/mali-c55/mali-c55.cpp           |  19 +--
 src/ipa/mali-c55/module.h               |   3 +-
 src/ipa/mali-c55/params.h               |  83 +++++++++++
 src/ipa/rkisp1/params.cpp               |  93 +-----------
 src/ipa/rkisp1/params.h                 | 175 ++++++++++++----------
 utils/update-kernel-headers.sh          |   2 +
 21 files changed, 881 insertions(+), 476 deletions(-)
---
base-commit: f0f2aca56611bf9430eb6edc23e9bd55cf96a26d
change-id: 20250829-v4l2-params-d42aa599e78a

Best regards,