diff options
author | Barnabás Pőcze <barnabas.pocze@ideasonboard.com> | 2025-02-06 15:59:42 +0100 |
---|---|---|
committer | Barnabás Pőcze <barnabas.pocze@ideasonboard.com> | 2025-03-25 12:54:50 +0100 |
commit | d505bd5360cc1681ae639c19cc48bdd28d24d9f7 (patch) | |
tree | b8940c77394d88aed90b5c97d8b866f8cc6c2022 /src/ipa/rpi/pisp/data/imx708.json | |
parent | b3272f78278ab18cf9467e06d676eb7f1e27b669 (diff) |
Both `CameraManager::Private::{add,remove}Camera()` emit the
`camera{Added,Removed}` signals, respectively, while holding the
lock protecting the list of cameras.
This is problematic because if a callback tries to call `cameras()`,
then the same (non-recursive) lock would be locked again.
Furthermore, there is no real need to hold the lock while user code
is running, so release the lock as soon as possible.
Signed-off-by: Barnabás Pőcze <barnabas.pocze@ideasonboard.com>
Reviewed-by: Harvey Yang <chenghaoyang@chromium.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Diffstat (limited to 'src/ipa/rpi/pisp/data/imx708.json')
0 files changed, 0 insertions, 0 deletions