From 3994cf0c21c496b979be340bab68c6718ebf0bcc Mon Sep 17 00:00:00 2001 From: KiritiGowda Date: Wed, 15 Jul 2026 13:55:17 -0700 Subject: [PATCH 1/3] Install NNEF-Tools parser headers alongside the sample impl Add an install(DIRECTORY) rule for kernels/NNEF-Tools/parser/cpp/include/ so that cnnef.h and the nnef/ headers are installed into the include directory alongside the OpenVX headers. This allows downstream consumers (e.g. the CTS) to find them via OPENVX_INCLUDES without needing a separate -I pointing into the sample impl source tree. Co-Authored-By: Claude --- kernels/CMakeLists.txt | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/kernels/CMakeLists.txt b/kernels/CMakeLists.txt index 1548d1c..5892267 100644 --- a/kernels/CMakeLists.txt +++ b/kernels/CMakeLists.txt @@ -47,10 +47,13 @@ add_library (${TARGET_NAME} STATIC ${OVX_SRC_NNEF_LIB} ) # no longer pulls it in transitively, so force-include it for this target. target_compile_options(${TARGET_NAME} PRIVATE -include limits) -install ( TARGETS ${TARGET_NAME} +install ( TARGETS ${TARGET_NAME} RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR} LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} ) + +install ( DIRECTORY ${CMAKE_SOURCE_DIR}/kernels/NNEF-Tools/parser/cpp/include/ + DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} ) set_target_properties( ${TARGET_NAME} PROPERTIES FOLDER ${HELPER_FOLDER} ) endif (OPENVX_CONFORMANCE_NNEF_IMPORT) From 92202ec5a8023fb10186aea2c88334093a822885 Mon Sep 17 00:00:00 2001 From: KiritiGowda Date: Wed, 15 Jul 2026 13:57:05 -0700 Subject: [PATCH 2/3] Fix NNEF header install destination (use literal 'include') CMAKE_INSTALL_INCLUDEDIR is empty because GNUInstallDirs is not included in this project; use the literal 'include' to match the pattern used by the rest of the sample impl. Co-Authored-By: Claude --- kernels/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernels/CMakeLists.txt b/kernels/CMakeLists.txt index 5892267..381ff29 100644 --- a/kernels/CMakeLists.txt +++ b/kernels/CMakeLists.txt @@ -53,7 +53,7 @@ install ( TARGETS ${TARGET_NAME} LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} ) install ( DIRECTORY ${CMAKE_SOURCE_DIR}/kernels/NNEF-Tools/parser/cpp/include/ - DESTINATION ${CMAKE_INSTALL_INCLUDEDIR} ) + DESTINATION include ) set_target_properties( ${TARGET_NAME} PROPERTIES FOLDER ${HELPER_FOLDER} ) endif (OPENVX_CONFORMANCE_NNEF_IMPORT) From 8a76a9d999bf23a3d2e4ab9be3af9b9bd7ee3b37 Mon Sep 17 00:00:00 2001 From: KiritiGowda Date: Wed, 15 Jul 2026 14:04:36 -0700 Subject: [PATCH 3/3] Fix -Wdiscarded-qualifiers in vxSendUserEvent The spec declares vx_event_t.user_event_parameter as void* but vxSendUserEvent takes const void*. Cast to void* on assignment to match the spec-defined field type. Co-Authored-By: Claude --- sample/framework/vx_event_queue.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sample/framework/vx_event_queue.c b/sample/framework/vx_event_queue.c index e03fdea..792ae45 100755 --- a/sample/framework/vx_event_queue.c +++ b/sample/framework/vx_event_queue.c @@ -160,7 +160,7 @@ VX_API_ENTRY vx_status VX_API_CALL vxSendUserEvent(vx_context context, vx_uint32 event.type = VX_EVENT_USER; event.timestamp = ownCaptureTime(); event.app_value = id; - event.event_info.user_event.user_event_parameter = parameter; + event.event_info.user_event.user_event_parameter = (void *)parameter; return ownPipelinePostEvent(context, &event); }