[PATCH] dma_buf_allocator.h: Fix build with gcc-15

Laurent Pinchart laurent.pinchart at ideasonboard.com
Fri Mar 28 12:03:04 CET 2025


Hello,

Thank you for the patch.

On Fri, Mar 28, 2025 at 01:03:38PM +0900, tho3.nguyen wrote:
> To fix:
> ...
> In file included from ../git/src/libcamera/dma_buf_allocator.cpp:9:
> ../git/include/libcamera/internal/dma_buf_allocator.h:66:19: error: 'uint64_t' has not been declared
>    66 |         void sync(uint64_t step);
>       |                   ^~~~~~~~
> ../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>'
> ...
> 
> Signed-off-by: tho3.nguyen <tho3.nguyen at lge.com>
> ---
>  include/libcamera/internal/dma_buf_allocator.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/libcamera/internal/dma_buf_allocator.h b/include/libcamera/internal/dma_buf_allocator.h
> index 13600915..c22b9bbd 100644
> --- a/include/libcamera/internal/dma_buf_allocator.h
> +++ b/include/libcamera/internal/dma_buf_allocator.h
> @@ -11,6 +11,7 @@
>  #include <stdint.h>
>  #include <string>
>  #include <vector>
> +#include <cstdint>

This file already includes stdint.h, which should provide a definition
for uint64_t. Does it really fail to compile ?

>  
>  #include <libcamera/base/flags.h>
>  #include <libcamera/base/shared_fd.h>

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list