[libcamera-devel] [PATCH 7/7] ipa: raspberrypi: Remove atomic variable from Algorithm class

David Plowman david.plowman at raspberrypi.com
Thu Feb 4 10:34:57 CET 2021


Pause() and Resume() are only called synchronously so paused_ does not
need to be atomic.

With this commit, libatomic can finally be removed as a build
dependency.

Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
---
 src/ipa/raspberrypi/controller/algorithm.hpp | 3 +--
 src/ipa/raspberrypi/meson.build              | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/ipa/raspberrypi/controller/algorithm.hpp b/src/ipa/raspberrypi/controller/algorithm.hpp
index e9b040c7..5123c87b 100644
--- a/src/ipa/raspberrypi/controller/algorithm.hpp
+++ b/src/ipa/raspberrypi/controller/algorithm.hpp
@@ -12,7 +12,6 @@
 #include <string>
 #include <memory>
 #include <map>
-#include <atomic>
 
 #include "controller.hpp"
 
@@ -46,7 +45,7 @@ public:
 
 private:
 	Controller *controller_;
-	std::atomic<bool> paused_;
+	bool paused_;
 };
 
 // This code is for automatic registration of Front End algorithms with the
diff --git a/src/ipa/raspberrypi/meson.build b/src/ipa/raspberrypi/meson.build
index 9445cd09..4cdd0434 100644
--- a/src/ipa/raspberrypi/meson.build
+++ b/src/ipa/raspberrypi/meson.build
@@ -5,7 +5,6 @@ ipa_name = 'ipa_rpi'
 rpi_ipa_deps = [
     libcamera_dep,
     dependency('boost'),
-    libatomic,
 ]
 
 rpi_ipa_includes = [
-- 
2.20.1



More information about the libcamera-devel mailing list