<div dir="ltr"><div dir="ltr">Hi Laurent, thank you for the patch,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 24, 2021 at 11:44 AM Umang Jain <<a href="mailto:umang.jain@ideasonboard.com">umang.jain@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Laurent,<br>
<br>
On 5/23/21 8:01 AM, Laurent Pinchart wrote:<br>
> The asynchronous signal delivery mechanism can't be used by application,<br>
> as it requires libcamera to be aware of the application event loop,<br>
> which has been dropped a long time ago. Drop the corresponding<br>
> documentation paragraph.<br>
><br>
> Signed-off-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br>
Reviewed-by: Umang Jain <<a href="mailto:umang.jain@ideasonboard.com" target="_blank">umang.jain@ideasonboard.com</a>><br></blockquote><div><br></div><div>Reviewed-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ---<br>
>   src/libcamera/thread.cpp | 8 --------<br>
>   1 file changed, 8 deletions(-)<br>
><br>
> diff --git a/src/libcamera/thread.cpp b/src/libcamera/thread.cpp<br>
> index bc1dcedea0d1..d59e43966d26 100644<br>
> --- a/src/libcamera/thread.cpp<br>
> +++ b/src/libcamera/thread.cpp<br>
> @@ -64,14 +64,6 @@<br>
>    * be overridden by selecting a different connection type when calling<br>
>    * Signal::connect().<br>
>    *<br>
> - * Asynchronous signal delivery is used internally in libcamera, but is also<br>
> - * available to applications if desired. To use this feature, applications<br>
> - * shall create receiver classes that inherit from the Object class, and<br>
> - * provide an event loop to the CameraManager as explained above. Note that<br>
> - * Object instances created by the application are limited to living in the<br>
> - * application's main thread. Creating Object instances from another thread of<br>
> - * an application causes undefined behaviour.<br>
> - *<br>
>    * \section thread-reentrancy Reentrancy and Thread-Safety<br>
>    *<br>
>    * Through the documentation, several terms are used to define how classes and<br>
<br>
</blockquote></div></div>