From patchwork Fri May 15 16:35:03 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Laurent Pinchart X-Patchwork-Id: 3801 Return-Path: Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [IPv6:2001:4b98:dc2:55:216:3eff:fef7:d647]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 3EA0460E04 for ; Fri, 15 May 2020 18:35:18 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=ideasonboard.com header.i=@ideasonboard.com header.b="gS3f9Zzz"; dkim-atps=neutral Received: from pendragon.bb.dnainternet.fi (81-175-216-236.bb.dnainternet.fi [81.175.216.236]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id BF46E9C5 for ; Fri, 15 May 2020 18:35:17 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1589560517; bh=w+KlM2gYu/rf9GvPMkMw/AFBoClDUXEVmogzCmuvi7M=; h=From:To:Subject:Date:In-Reply-To:References:From; b=gS3f9ZzzUPKyHvJsuGs1HfO13qGt14Agx8hjZx+fdAcaDAt1t+IGD6XzNgQPHnYBu NvBLzjnHzkCUDtJxXhFRajz4LdLzh8aqg4mLiWIz+d9AgN9VHx+gsGFAaOeIoD7MYM A3B229x6pF+EMA5l+UqssY6SmGeSp0abu7qh9Zl8= From: Laurent Pinchart To: libcamera-devel@lists.libcamera.org Date: Fri, 15 May 2020 19:35:03 +0300 Message-Id: <20200515163505.24330-2-laurent.pinchart@ideasonboard.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200515163505.24330-1-laurent.pinchart@ideasonboard.com> References: <20200515163505.24330-1-laurent.pinchart@ideasonboard.com> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v2 2/3] libcamera: Move IPA headers from include/ipa/ to include/libcamera/ipa/ X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 May 2020 16:35:18 -0000 The IPA headers are installed into $prefix/include/libcamera/ipa/, but are located in the source tree in include/ipa/. This requires files within libcamera to include them with #include while a third party IPA would need to use #include Not only is this inconsistent, it can create issues later if IPA headers need to include each other, as the first form of include directive wouldn't be valid once the headers are installed. Fix the problem by moving the IPA headers to include/libcamera/ipa/. Signed-off-by: Laurent Pinchart Acked-by: Jacopo Mondi --- Documentation/Doxyfile.in | 3 +-- include/libcamera/internal/ipa_context_wrapper.h | 2 +- include/libcamera/internal/ipa_manager.h | 4 ++-- include/libcamera/internal/ipa_module.h | 4 ++-- include/libcamera/internal/ipa_proxy.h | 2 +- include/{ => libcamera}/ipa/ipa_controls.h | 0 include/{ => libcamera}/ipa/ipa_interface.h | 0 include/{ => libcamera}/ipa/ipa_module_info.h | 0 include/{ => libcamera}/ipa/ipa_vimc.h | 0 include/{ => libcamera}/ipa/meson.build | 0 include/{ => libcamera}/ipa/raspberrypi.h | 0 include/{ => libcamera}/ipa/rkisp1.h | 0 include/libcamera/meson.build | 1 + include/meson.build | 1 - src/ipa/libipa/ipa_interface_wrapper.cpp | 2 +- src/ipa/libipa/ipa_interface_wrapper.h | 2 +- src/ipa/raspberrypi/raspberrypi.cpp | 7 ++++--- src/ipa/rkisp1/rkisp1.cpp | 7 ++++--- src/ipa/vimc/vimc.cpp | 6 +++--- src/libcamera/control_serializer.cpp | 2 +- src/libcamera/ipa_controls.cpp | 2 +- src/libcamera/ipa_interface.cpp | 2 +- src/libcamera/pipeline/raspberrypi/raspberrypi.cpp | 2 +- src/libcamera/pipeline/rkisp1/rkisp1.cpp | 2 +- src/libcamera/pipeline/vimc/vimc.cpp | 4 ++-- src/libcamera/proxy/ipa_proxy_linux.cpp | 4 ++-- src/libcamera/proxy/ipa_proxy_thread.cpp | 4 ++-- src/libcamera/proxy/worker/ipa_proxy_linux_worker.cpp | 2 +- test/ipa/ipa_interface_test.cpp | 3 +-- 29 files changed, 34 insertions(+), 34 deletions(-) rename include/{ => libcamera}/ipa/ipa_controls.h (100%) rename include/{ => libcamera}/ipa/ipa_interface.h (100%) rename include/{ => libcamera}/ipa/ipa_module_info.h (100%) rename include/{ => libcamera}/ipa/ipa_vimc.h (100%) rename include/{ => libcamera}/ipa/meson.build (100%) rename include/{ => libcamera}/ipa/raspberrypi.h (100%) rename include/{ => libcamera}/ipa/rkisp1.h (100%) diff --git a/Documentation/Doxyfile.in b/Documentation/Doxyfile.in index c3fe5f663f7c..5f4847523c53 100644 --- a/Documentation/Doxyfile.in +++ b/Documentation/Doxyfile.in @@ -794,8 +794,7 @@ WARN_LOGFILE = # spaces. See also FILE_PATTERNS and EXTENSION_MAPPING # Note: If this tag is empty the current directory is searched. -INPUT = "@TOP_SRCDIR@/include/ipa" \ - "@TOP_SRCDIR@/include/libcamera" \ +INPUT = "@TOP_SRCDIR@/include/libcamera" \ "@TOP_SRCDIR@/src/ipa/libipa" \ "@TOP_SRCDIR@/src/libcamera" \ "@TOP_BUILDDIR@/include/libcamera" \ diff --git a/include/libcamera/internal/ipa_context_wrapper.h b/include/libcamera/internal/ipa_context_wrapper.h index a717d4067498..bd770e132c0c 100644 --- a/include/libcamera/internal/ipa_context_wrapper.h +++ b/include/libcamera/internal/ipa_context_wrapper.h @@ -7,7 +7,7 @@ #ifndef __LIBCAMERA_IPA_CONTEXT_WRAPPER_H__ #define __LIBCAMERA_IPA_CONTEXT_WRAPPER_H__ -#include +#include #include "libcamera/internal/control_serializer.h" diff --git a/include/libcamera/internal/ipa_manager.h b/include/libcamera/internal/ipa_manager.h index f07d91fc7bbf..2412d75746ac 100644 --- a/include/libcamera/internal/ipa_manager.h +++ b/include/libcamera/internal/ipa_manager.h @@ -10,8 +10,8 @@ #include #include -#include -#include +#include +#include #include "libcamera/internal/ipa_module.h" #include "libcamera/internal/pipeline_handler.h" diff --git a/include/libcamera/internal/ipa_module.h b/include/libcamera/internal/ipa_module.h index 2f569db0b848..5b54cb31a48a 100644 --- a/include/libcamera/internal/ipa_module.h +++ b/include/libcamera/internal/ipa_module.h @@ -11,8 +11,8 @@ #include #include -#include -#include +#include +#include #include "libcamera/internal/log.h" #include "libcamera/internal/pipeline_handler.h" diff --git a/include/libcamera/internal/ipa_proxy.h b/include/libcamera/internal/ipa_proxy.h index 1111065b36a7..e9a36608bd31 100644 --- a/include/libcamera/internal/ipa_proxy.h +++ b/include/libcamera/internal/ipa_proxy.h @@ -11,7 +11,7 @@ #include #include -#include +#include namespace libcamera { diff --git a/include/ipa/ipa_controls.h b/include/libcamera/ipa/ipa_controls.h similarity index 100% rename from include/ipa/ipa_controls.h rename to include/libcamera/ipa/ipa_controls.h diff --git a/include/ipa/ipa_interface.h b/include/libcamera/ipa/ipa_interface.h similarity index 100% rename from include/ipa/ipa_interface.h rename to include/libcamera/ipa/ipa_interface.h diff --git a/include/ipa/ipa_module_info.h b/include/libcamera/ipa/ipa_module_info.h similarity index 100% rename from include/ipa/ipa_module_info.h rename to include/libcamera/ipa/ipa_module_info.h diff --git a/include/ipa/ipa_vimc.h b/include/libcamera/ipa/ipa_vimc.h similarity index 100% rename from include/ipa/ipa_vimc.h rename to include/libcamera/ipa/ipa_vimc.h diff --git a/include/ipa/meson.build b/include/libcamera/ipa/meson.build similarity index 100% rename from include/ipa/meson.build rename to include/libcamera/ipa/meson.build diff --git a/include/ipa/raspberrypi.h b/include/libcamera/ipa/raspberrypi.h similarity index 100% rename from include/ipa/raspberrypi.h rename to include/libcamera/ipa/raspberrypi.h diff --git a/include/ipa/rkisp1.h b/include/libcamera/ipa/rkisp1.h similarity index 100% rename from include/ipa/rkisp1.h rename to include/libcamera/ipa/rkisp1.h diff --git a/include/libcamera/meson.build b/include/libcamera/meson.build index f9c8e64bd50c..32b8d14211a4 100644 --- a/include/libcamera/meson.build +++ b/include/libcamera/meson.build @@ -24,6 +24,7 @@ libcamera_api = files([ include_dir = join_paths(libcamera_include_dir, 'libcamera') subdir('internal') +subdir('ipa') install_headers(libcamera_api, subdir : include_dir) diff --git a/include/meson.build b/include/meson.build index 4f41c9ceeb4c..2ac9a3a049f2 100644 --- a/include/meson.build +++ b/include/meson.build @@ -3,5 +3,4 @@ libcamera_include_dir = 'libcamera' subdir('android') -subdir('ipa') subdir('libcamera') diff --git a/src/ipa/libipa/ipa_interface_wrapper.cpp b/src/ipa/libipa/ipa_interface_wrapper.cpp index 90f270c021a9..2a2e43abc708 100644 --- a/src/ipa/libipa/ipa_interface_wrapper.cpp +++ b/src/ipa/libipa/ipa_interface_wrapper.cpp @@ -12,7 +12,7 @@ #include #include -#include +#include #include "libcamera/internal/byte_stream_buffer.h" #include "libcamera/internal/camera_sensor.h" diff --git a/src/ipa/libipa/ipa_interface_wrapper.h b/src/ipa/libipa/ipa_interface_wrapper.h index b426034d551d..a1c701599b56 100644 --- a/src/ipa/libipa/ipa_interface_wrapper.h +++ b/src/ipa/libipa/ipa_interface_wrapper.h @@ -9,7 +9,7 @@ #include -#include +#include #include "libcamera/internal/control_serializer.h" diff --git a/src/ipa/raspberrypi/raspberrypi.cpp b/src/ipa/raspberrypi/raspberrypi.cpp index 46b244538e32..9669f2126e6c 100644 --- a/src/ipa/raspberrypi/raspberrypi.cpp +++ b/src/ipa/raspberrypi/raspberrypi.cpp @@ -12,14 +12,15 @@ #include #include -#include -#include -#include #include #include #include +#include +#include +#include #include #include + #include #include "libcamera/internal/camera_sensor.h" diff --git a/src/ipa/rkisp1/rkisp1.cpp b/src/ipa/rkisp1/rkisp1.cpp index 4f6e0342dac1..fbdc908fc816 100644 --- a/src/ipa/rkisp1/rkisp1.cpp +++ b/src/ipa/rkisp1/rkisp1.cpp @@ -14,12 +14,13 @@ #include -#include -#include -#include #include #include +#include +#include +#include #include + #include #include "libcamera/internal/log.h" diff --git a/src/ipa/vimc/vimc.cpp b/src/ipa/vimc/vimc.cpp index 3fa6be6535e2..af278a482b8a 100644 --- a/src/ipa/vimc/vimc.cpp +++ b/src/ipa/vimc/vimc.cpp @@ -5,7 +5,7 @@ * ipa_vimc.cpp - Vimc Image Processing Algorithm module */ -#include +#include #include #include @@ -14,8 +14,8 @@ #include -#include -#include +#include +#include #include diff --git a/src/libcamera/control_serializer.cpp b/src/libcamera/control_serializer.cpp index bf162b6cc471..258db6dfbf5a 100644 --- a/src/libcamera/control_serializer.cpp +++ b/src/libcamera/control_serializer.cpp @@ -11,9 +11,9 @@ #include #include -#include #include #include +#include #include #include "libcamera/internal/byte_stream_buffer.h" diff --git a/src/libcamera/ipa_controls.cpp b/src/libcamera/ipa_controls.cpp index b1d14190e3fe..8fd726513182 100644 --- a/src/libcamera/ipa_controls.cpp +++ b/src/libcamera/ipa_controls.cpp @@ -5,7 +5,7 @@ * ipa_controls.cpp - IPA control handling */ -#include +#include /** * \file ipa_controls.h diff --git a/src/libcamera/ipa_interface.cpp b/src/libcamera/ipa_interface.cpp index c890eadaf6c8..69835e63ab46 100644 --- a/src/libcamera/ipa_interface.cpp +++ b/src/libcamera/ipa_interface.cpp @@ -5,7 +5,7 @@ * ipa_interface.cpp - Image Processing Algorithm interface */ -#include +#include /** * \file ipa_interface.h diff --git a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp index e254e5558d92..07ca9f5d7f53 100644 --- a/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp +++ b/src/libcamera/pipeline/raspberrypi/raspberrypi.cpp @@ -11,9 +11,9 @@ #include #include -#include #include #include +#include #include #include #include diff --git a/src/libcamera/pipeline/rkisp1/rkisp1.cpp b/src/libcamera/pipeline/rkisp1/rkisp1.cpp index 7a0c8771ba54..d807fc2cf535 100644 --- a/src/libcamera/pipeline/rkisp1/rkisp1.cpp +++ b/src/libcamera/pipeline/rkisp1/rkisp1.cpp @@ -13,10 +13,10 @@ #include -#include #include #include #include +#include #include #include diff --git a/src/libcamera/pipeline/vimc/vimc.cpp b/src/libcamera/pipeline/vimc/vimc.cpp index 50a9f2df4c24..ca36348a5eb6 100644 --- a/src/libcamera/pipeline/vimc/vimc.cpp +++ b/src/libcamera/pipeline/vimc/vimc.cpp @@ -14,11 +14,11 @@ #include #include -#include -#include #include #include #include +#include +#include #include #include diff --git a/src/libcamera/proxy/ipa_proxy_linux.cpp b/src/libcamera/proxy/ipa_proxy_linux.cpp index 7d6f8799f8ff..be34f20aa857 100644 --- a/src/libcamera/proxy/ipa_proxy_linux.cpp +++ b/src/libcamera/proxy/ipa_proxy_linux.cpp @@ -7,8 +7,8 @@ #include -#include -#include +#include +#include #include "libcamera/internal/ipa_module.h" #include "libcamera/internal/ipa_proxy.h" diff --git a/src/libcamera/proxy/ipa_proxy_thread.cpp b/src/libcamera/proxy/ipa_proxy_thread.cpp index 03d4c0d927b9..6fbebed2ba72 100644 --- a/src/libcamera/proxy/ipa_proxy_thread.cpp +++ b/src/libcamera/proxy/ipa_proxy_thread.cpp @@ -7,8 +7,8 @@ #include -#include -#include +#include +#include #include "libcamera/internal/ipa_context_wrapper.h" #include "libcamera/internal/ipa_module.h" diff --git a/src/libcamera/proxy/worker/ipa_proxy_linux_worker.cpp b/src/libcamera/proxy/worker/ipa_proxy_linux_worker.cpp index 9ba94d01d0ca..0c4687f737a3 100644 --- a/src/libcamera/proxy/worker/ipa_proxy_linux_worker.cpp +++ b/src/libcamera/proxy/worker/ipa_proxy_linux_worker.cpp @@ -9,8 +9,8 @@ #include #include -#include #include +#include #include #include "libcamera/internal/ipa_module.h" diff --git a/test/ipa/ipa_interface_test.cpp b/test/ipa/ipa_interface_test.cpp index c394377dacd0..2f02af49cce9 100644 --- a/test/ipa/ipa_interface_test.cpp +++ b/test/ipa/ipa_interface_test.cpp @@ -14,10 +14,9 @@ #include #include +#include #include -#include - #include "libcamera/internal/device_enumerator.h" #include "libcamera/internal/ipa_manager.h" #include "libcamera/internal/ipa_module.h"