<div dir="ltr"><div dir="ltr">Hi Laurent, thank you for the patch.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 9, 2021 at 11:26 AM <<a href="mailto:paul.elder@ideasonboard.com">paul.elder@ideasonboard.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Laurent,<br>
<br>
On Wed, Jun 09, 2021 at 12:27:14AM +0300, Laurent Pinchart wrote:<br>
> The ipu3_agc.h forward-declares the IPACameraSensorInfo structure, but<br>
> incorrectly declares it as a class. This causes a compilation error with<br>
> clang:<br>
> <br>
> include/libcamera/ipa/core_ipa_interface.h:24:1: error: 'IPACameraSensorInfo' defined as a struct here but previously declared as a class; this is valid, but may result in linker errors under the Microsoft C++ ABI [-Werror,-Wmismatched-tags]<br>
> struct IPACameraSensorInfo<br>
> ^<br>
> ../../src/ipa/ipu3/ipu3_agc.h:21:1: note: did you mean struct here?<br>
> class IPACameraSensorInfo;<br>
> ^~~~~<br>
> struct<br>
> <br>
> Fix it.<br>
> <br>
> Fixes: 384a53d3cdf7 ("ipa: ipu3: Calculate line duration from IPACameraSensorInfo")<br>
> Signed-off-by: Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>><br>
<br>
Reviewed-by: Paul Elder <<a href="mailto:paul.elder@ideasonboard.com" target="_blank">paul.elder@ideasonboard.com</a>><br><br></blockquote><div><br></div><div>Reviewed-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ---<br>
>  src/ipa/ipu3/ipu3_agc.h | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> <br>
> diff --git a/src/ipa/ipu3/ipu3_agc.h b/src/ipa/ipu3/ipu3_agc.h<br>
> index 99a582a9fdcf..f3d4055761e6 100644<br>
> --- a/src/ipa/ipu3/ipu3_agc.h<br>
> +++ b/src/ipa/ipu3/ipu3_agc.h<br>
> @@ -18,7 +18,7 @@<br>
>  <br>
>  namespace libcamera {<br>
>  <br>
> -class IPACameraSensorInfo;<br>
> +struct IPACameraSensorInfo;<br>
>  <br>
>  namespace ipa::ipu3 {<br>
>  <br>
> -- <br>
> Regards,<br>
> <br>
> Laurent Pinchart<br>
> <br>
</blockquote></div></div>