[PATCH v5 02/18] libcamera: software_isp: Move BlackLevel to libcamera::ipa::soft
Milan Zamazal
mzamazal at redhat.com
Fri Aug 30 09:25:38 CEST 2024
IPA modules use custom namespaces for all their internal components to
avoid namespace clashes. The simple IPA module for the software ISP uses
libcamera::ipa::soft for this purpose. It however defines an internal
class named BlackLevel in the root of the libcamera namespace, making it
prone to clashes. Move it to the ipa::soft namespace along with the rest
of the code.
Signed-off-by: Milan Zamazal <mzamazal at redhat.com>
Reviewed-by: Umang Jain <umang.jain at ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Daniel Scally <dan.scally at ideasonboard.com>
---
src/ipa/simple/black_level.cpp | 5 +++++
src/ipa/simple/black_level.h | 4 ++++
2 files changed, 9 insertions(+)
diff --git a/src/ipa/simple/black_level.cpp b/src/ipa/simple/black_level.cpp
index cc490eb5..37e0109c 100644
--- a/src/ipa/simple/black_level.cpp
+++ b/src/ipa/simple/black_level.cpp
@@ -15,6 +15,8 @@ namespace libcamera {
LOG_DEFINE_CATEGORY(IPASoftBL)
+namespace ipa::soft {
+
/**
* \class BlackLevel
* \brief Object providing black point level for software ISP
@@ -85,4 +87,7 @@ void BlackLevel::update(SwIspStats::Histogram &yHistogram)
}
};
}
+
+} /* namespace ipa::soft */
+
} /* namespace libcamera */
diff --git a/src/ipa/simple/black_level.h b/src/ipa/simple/black_level.h
index 5e032f9f..a04230c9 100644
--- a/src/ipa/simple/black_level.h
+++ b/src/ipa/simple/black_level.h
@@ -14,6 +14,8 @@
namespace libcamera {
+namespace ipa::soft {
+
class BlackLevel
{
public:
@@ -26,4 +28,6 @@ private:
bool blackLevelSet_;
};
+} /* namespace ipa::soft */
+
} /* namespace libcamera */
--
2.44.1
More information about the libcamera-devel
mailing list