[libcamera-devel] [PATCH 1/2] ipa: raspberrypi: Remove atomic variable from Algorithm class
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Mon Feb 8 16:48:06 CET 2021
Hi David,
Thank you for the patch.
On Mon, Feb 08, 2021 at 09:40:35AM +0000, David Plowman wrote:
> Pause() and Resume() are only called synchronously so paused_ does not
> need to be atomic.
>
> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> ---
> src/ipa/raspberrypi/controller/algorithm.hpp | 3 +--
> 1 file changed, 1 insertion(+), 2 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
--
Regards,
Laurent Pinchart
More information about the libcamera-devel
mailing list