From 17e8f6f71c0a03bd011675f83e93912317527885 Mon Sep 17 00:00:00 2001 From: Paul Elder Date: Thu, 22 Apr 2021 16:47:03 +0900 Subject: utils: ipc: Include instead of forward-declare CameraSensorInfo For structs defined in core.mojom that have the skipHeader tag, if they're only used in function parameters (in a mojom file) then a forward-declaration is sufficient. However, if the struct is used in another struct in a mojom file, then the forward-declaration is insufficient, and the definition needs to be included. Do so for CameraSensorInfo, which is the only forward-declared struct in ipa_interface.h, and update the documentation comment. Signed-off-by: Paul Elder Tested-by: Umang Jain Reviewed-by: Laurent Pinchart --- include/libcamera/ipa/core.mojom | 3 +-- include/libcamera/ipa/ipa_interface.h | 6 +++--- 2 files changed, 4 insertions(+), 5 deletions(-) (limited to 'include') diff --git a/include/libcamera/ipa/core.mojom b/include/libcamera/ipa/core.mojom index 5363f1c5..70de71ea 100644 --- a/include/libcamera/ipa/core.mojom +++ b/include/libcamera/ipa/core.mojom @@ -38,8 +38,7 @@ * implemented in ipa_data_serializer.h, as it cannot be defined in mojom * - [skipHeader] and [skipSerdes] only work here in core.mojom. * - If a struct definition has [skipHeader], then the header where the - * struct is defined must be #included (or the struct forward-declared) in - * ipa_interface.h + * struct is defined must be #included in ipa_interface.h * - If a field in a struct has a FileDescriptor, but is not explicitly * defined so in mojom, then the field must be marked with the [hasFd] * attribute. diff --git a/include/libcamera/ipa/ipa_interface.h b/include/libcamera/ipa/ipa_interface.h index 5d99e2cf..dfe1b40a 100644 --- a/include/libcamera/ipa/ipa_interface.h +++ b/include/libcamera/ipa/ipa_interface.h @@ -18,15 +18,15 @@ #include #include +#include "libcamera/internal/camera_sensor.h" + namespace libcamera { /* * Structs that are defined in core.mojom and have the skipHeader tag must be - * forward-declared or #included here. + * #included here. */ -struct CameraSensorInfo; - class IPAInterface { public: -- cgit v1.2.1