<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<div dir="auto">Thank you! I used v0.4.0 and didnt check carefully at master.</div>
<div dir="auto">Please ignore my patch</div>
<div dir="auto"><br>
</div>
<div dir="auto">Best regards</div>
<div dir="auto">Tho</div>
<div id="ms-outlook-mobile-body-separator-line" dir="auto"><br>
</div>
<div id="ms-outlook-mobile-signature" dir="auto">Tải xuống <a href="https://aka.ms/AAb9ysg">
Outlook dành cho Android</a></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Laurent Pinchart <laurent.pinchart@ideasonboard.com><br>
<b>Sent:</b> Friday, March 28, 2025 6:03:04 PM<br>
<b>To:</b> THO DAT NGUYEN/LGEDV ADVANCED PLATFORM TEAM <tho3.nguyen@lge.com><br>
<b>Cc:</b> libcamera-devel@lists.libcamera.org <libcamera-devel@lists.libcamera.org><br>
<b>Subject:</b> Re: [PATCH] dma_buf_allocator.h: Fix build with gcc-15</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hello,<br>
<br>
Thank you for the patch.<br>
<br>
On Fri, Mar 28, 2025 at 01:03:38PM +0900, tho3.nguyen wrote:<br>
> To fix:<br>
> ...<br>
> In file included from ../git/src/libcamera/dma_buf_allocator.cpp:9:<br>
> ../git/include/libcamera/internal/dma_buf_allocator.h:66:19: error: 'uint64_t' has not been declared<br>
> 66 | void sync(uint64_t step);<br>
> | ^~~~~~~~<br>
> ../git/include/libcamera/internal/dma_buf_allocator.h:17:1: note: 'uint64_t' is defined in header '<cstdint>'; this is probably fixable by adding '#include <cstdint>'<br>
> ...<br>
> <br>
> Signed-off-by: tho3.nguyen <tho3.nguyen@lge.com><br>
> ---<br>
> include/libcamera/internal/dma_buf_allocator.h | 1 +<br>
> 1 file changed, 1 insertion(+)<br>
> <br>
> diff --git a/include/libcamera/internal/dma_buf_allocator.h b/include/libcamera/internal/dma_buf_allocator.h<br>
> index 13600915..c22b9bbd 100644<br>
> --- a/include/libcamera/internal/dma_buf_allocator.h<br>
> +++ b/include/libcamera/internal/dma_buf_allocator.h<br>
> @@ -11,6 +11,7 @@<br>
> #include <stdint.h><br>
> #include <string><br>
> #include <vector><br>
> +#include <cstdint><br>
<br>
This file already includes stdint.h, which should provide a definition<br>
for uint64_t. Does it really fail to compile ?<br>
<br>
> <br>
> #include <libcamera/base/flags.h><br>
> #include <libcamera/base/shared_fd.h><br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</div>
</span></font></div>
</body>
</html>