From 0cc74c492a89f639059292449d90b171876949c7 Mon Sep 17 00:00:00 2001
From: Milan Zamazal <mzamazal@redhat.com>
Date: Fri, 27 Sep 2024 15:46:10 +0200
Subject: [PATCH] libcamera: software_isp: Let IPASoftSimple inherit Module

The Module class is a base class for all IPA modules.
In addition, implement logPrefix() of the module for the softIPA.

Signed-off-by: Milan Zamazal <mzamazal@redhat.com>
Reviewed-by: Umang Jain <umang.jain@ideasonboard.com>
Reviewed-by: Daniel Scally <dan.scally@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/ipa/simple/soft_simple.cpp | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/ipa/simple/soft_simple.cpp b/src/ipa/simple/soft_simple.cpp
index b7746ce09..72321f443 100644
--- a/src/ipa/simple/soft_simple.cpp
+++ b/src/ipa/simple/soft_simple.cpp
@@ -30,6 +30,7 @@
 #include "libipa/camera_sensor_helper.h"
 
 #include "black_level.h"
+#include "module.h"
 
 namespace libcamera {
 LOG_DEFINE_CATEGORY(IPASoft)
@@ -54,7 +55,7 @@ static constexpr float kExposureOptimal = kExposureBinsCount / 2.0;
  */
 static constexpr float kExposureSatisfactory = 0.2;
 
-class IPASoftSimple : public ipa::soft::IPASoftInterface
+class IPASoftSimple : public ipa::soft::IPASoftInterface, public Module
 {
 public:
 	IPASoftSimple()
@@ -76,6 +77,9 @@ class IPASoftSimple : public ipa::soft::IPASoftInterface
 
 	void processStats(const ControlList &sensorControls) override;
 
+protected:
+	std::string logPrefix() const override;
+
 private:
 	void updateExposure(double exposureMSV);
 
@@ -421,6 +425,11 @@ void IPASoftSimple::updateExposure(double exposureMSV)
 	again_ = std::clamp(again_, againMin_, againMax_);
 }
 
+std::string IPASoftSimple::logPrefix() const
+{
+	return "IPASoft";
+}
+
 } /* namespace ipa::soft */
 
 /*