[libcamera-devel] [PATCH v7.1 13/14] ipa: raspberrypi: Use YamlParser to replace dependency on boost

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Jul 27 23:21:58 CEST 2022


The subject line should have read

"[PATCH v7.1 08/14] ipa: raspberrypi: Use YamlParser to replace dependency on boost"

Sorry about that.

On Wed, Jul 27, 2022 at 08:43:52PM +0300, Laurent Pinchart via libcamera-devel wrote:
> The Raspberry Pi IPA module depends on boost only to parse the JSON
> tuning data files. As libcamera depends on libyaml, use the YamlParser
> class to parse those files and drop the dependency on boost.
> 
> Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Tested-by: Naushir Patuck <naush at raspberrypi.com>
> ---
> Changes since v7:
> 
> - Guard against invalid number of entries or invalid entries in
>   Pwl::read()
> - Don't use default value for reference_Y and reference_lux in lux.cpp
> 
> Changes since v6:
> 
> - Propagate tuning data read errors
> ---
>  README.rst                                    |   6 -
>  src/ipa/raspberrypi/controller/algorithm.cpp  |   2 +-
>  src/ipa/raspberrypi/controller/algorithm.h    |   6 +-
>  src/ipa/raspberrypi/controller/controller.cpp |  27 ++--
>  src/ipa/raspberrypi/controller/pwl.cpp        |  27 ++--
>  src/ipa/raspberrypi/controller/pwl.h          |   4 +-
>  src/ipa/raspberrypi/controller/rpi/agc.cpp    | 136 ++++++++++--------
>  src/ipa/raspberrypi/controller/rpi/agc.h      |  10 +-
>  src/ipa/raspberrypi/controller/rpi/alsc.cpp   | 105 +++++++-------
>  src/ipa/raspberrypi/controller/rpi/alsc.h     |   2 +-
>  src/ipa/raspberrypi/controller/rpi/awb.cpp    | 134 ++++++++++-------
>  src/ipa/raspberrypi/controller/rpi/awb.h      |   8 +-
>  .../controller/rpi/black_level.cpp            |  12 +-
>  .../raspberrypi/controller/rpi/black_level.h  |   2 +-
>  src/ipa/raspberrypi/controller/rpi/ccm.cpp    |  47 +++---
>  src/ipa/raspberrypi/controller/rpi/ccm.h      |   4 +-
>  .../raspberrypi/controller/rpi/contrast.cpp   |  28 ++--
>  src/ipa/raspberrypi/controller/rpi/contrast.h |   2 +-
>  src/ipa/raspberrypi/controller/rpi/dpc.cpp    |   7 +-
>  src/ipa/raspberrypi/controller/rpi/dpc.h      |   2 +-
>  src/ipa/raspberrypi/controller/rpi/geq.cpp    |  10 +-
>  src/ipa/raspberrypi/controller/rpi/geq.h      |   2 +-
>  src/ipa/raspberrypi/controller/rpi/lux.cpp    |  30 +++-
>  src/ipa/raspberrypi/controller/rpi/lux.h      |   2 +-
>  src/ipa/raspberrypi/controller/rpi/noise.cpp  |  14 +-
>  src/ipa/raspberrypi/controller/rpi/noise.h    |   2 +-
>  src/ipa/raspberrypi/controller/rpi/sdn.cpp    |   6 +-
>  src/ipa/raspberrypi/controller/rpi/sdn.h      |   2 +-
>  .../raspberrypi/controller/rpi/sharpen.cpp    |   8 +-
>  src/ipa/raspberrypi/controller/rpi/sharpen.h  |   2 +-
>  src/ipa/raspberrypi/meson.build               |   1 -
>  src/ipa/raspberrypi/raspberrypi.cpp           |   1 +
>  32 files changed, 374 insertions(+), 277 deletions(-)

[snip]

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list