| Message ID | 20210208094036.6846-2-david.plowman@raspberrypi.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series |
|
| Related | show |
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@raspberrypi.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@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
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
Pause() and Resume() are only called synchronously so paused_ does not need to be atomic. Signed-off-by: David Plowman <david.plowman@raspberrypi.com> --- src/ipa/raspberrypi/controller/algorithm.hpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)