[libcamera-devel] [PATCH v2] libcamera: raspberrypi: Document vc_sm_cma pointer sharing with IPA

[Niklas Söderlund]

Sharing a pointer with the IPA is fragile as it will not work with all
IPC mechanisms. Document that it's a temporary workaround that should be
reworked.

Signed-off-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
---
* Changes since v1
- Make comment less alarmist.
---
 src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
index 9d887b706c3f9670..f4966f8628eedbdd 100644
--- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
+++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp
@@ -1014,6 +1014,11 @@ int PipelineHandlerRPi::configureIPA(Camera *camera)
 		/*
 		 * The vcsm allocation will always be in the memory region
 		 * < 32-bits to allow Videocore to access the memory.
+		 *
+		 * \todo Sending a pointer to the IPA is a workaround for
+		 * vc_sm_cma not yet supporting dmabuf. This will not work with
+		 * IPA module isolation and should be reworked when vc_sma_cma
+		 * will permit.
 		 */
 		IPAOperationData op;
 		op.operation = RPI_IPA_EVENT_LS_TABLE_ALLOCATION;
-- 
2.27.0