[libcamera-devel] [PATCH v3 1/4] libcamera: ipa: raspberrypi: ALSC: Camera mode does not need to be atomic
Naushir Patuck
naush at raspberrypi.com
Tue Aug 4 10:49:09 CEST 2020
Hi David,
Thank you for the patch.
On Sat, 1 Aug 2020 at 09:02, David Plowman
<david.plowman at raspberrypi.com> wrote:
>
> In the libcamera framework, SwitchMode (which overwrites the
> camera_mode) cannot run concurrently with Prepare (which uses it).
>
> Signed-off-by: David Plowman <david.plowman at raspberrypi.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Reviewed-by: Naushir Patuck <naush at raspberrypi.com>
> ---
> src/ipa/raspberrypi/controller/rpi/alsc.hpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/ipa/raspberrypi/controller/rpi/alsc.hpp b/src/ipa/raspberrypi/controller/rpi/alsc.hpp
> index 3806257..e895913 100644
> --- a/src/ipa/raspberrypi/controller/rpi/alsc.hpp
> +++ b/src/ipa/raspberrypi/controller/rpi/alsc.hpp
> @@ -59,7 +59,7 @@ private:
> // configuration is read-only, and available to both threads
> AlscConfig config_;
> bool first_time_;
> - std::atomic<CameraMode> camera_mode_;
> + CameraMode camera_mode_;
> std::thread async_thread_;
> void asyncFunc(); // asynchronous thread function
> std::mutex mutex_;
> --
> 2.20.1
>
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel
More information about the libcamera-devel
mailing list