[v3,0/2] imx8-isi: Move isi routing into acquireDevice
mbox series

Message ID 20251028160453.3802642-1-antoine.bouyer@nxp.com
Headers show
Series
  • imx8-isi: Move isi routing into acquireDevice
Related show

Message

Antoine Bouyer Oct. 28, 2025, 4:04 p.m. UTC 
Submit imx8-isi fix on behalf of Andrei. As stated in the commit message,
this patch fixes an issue introduced by the imx8-isi multicamera support
patch, merged in v0.5.2.

Issue is about running multiple processes trying to list cameras while
one process is already using one (or more) camera(s).

Tested on i.MX8MP evk.

---
Changes in v3:
- Move PipelineHandler::useCount_ variable to protected instead of private.
- Use PipelineHandler::useCount_ variable instead of local acquireCount_.
- Remove useless PipelineHandlerISI::releaseDevice().

link to v2: https://patchwork.libcamera.org/patch/24720/ 

Changes in v2:
- Return boolean instead of int if an error is returned by setRouting in
  acquireDevice.
- Check acquireCount_ is higher than 0 instead of 1, and increase counter
  only when setRouting is successful, or if counter is already non null.

link to v1: https://patchwork.libcamera.org/patch/24706/

---
Andrei Gansari (1):
  pipeline: imx8-isi: Delay ISI routes config to acquire() time

Antoine Bouyer (1):
  libcamera: pipeline_handler: Move useCount_ to protected

 include/libcamera/internal/pipeline_handler.h |  2 +-
 src/libcamera/pipeline/imx8-isi/imx8-isi.cpp  | 30 +++++++++++++++----
 2 files changed, 25 insertions(+), 7 deletions(-)