[igt-dev] [PATCH v1 04/13] xe_query: Add missing include.
Kamil Konieczny
kamil.konieczny at linux.intel.com
Tue Nov 21 17:00:06 UTC 2023
Hi Francois,
On 2023-11-16 at 14:53:39 +0000, Francois Dugast wrote:
> From: Rodrigo Vivi <rodrigo.vivi at intel.com>
>
LGTM, please remove final dot from subject line at merge.
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> When trying to use xe_for_each_mem_region from a caller
> that is not importing the igt_aux.h, the following build issue
> will occur:
>
> ../lib/xe/xe_query.h:76:38: error: implicit declaration of function ‘igt_fls’ [-Werror=implicit-function-declaration]
> 76 | for (uint64_t __i = 0; __i < igt_fls(__memreg); __i++) \
>
> So, to avoid a dependency chain, let's include from the file
> that is using the helper.
>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> Signed-off-by: Francois Dugast <francois.dugast at intel.com>
> ---
> lib/xe/xe_query.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/lib/xe/xe_query.h b/lib/xe/xe_query.h
> index 38e9aa440..7b3fc3100 100644
> --- a/lib/xe/xe_query.h
> +++ b/lib/xe/xe_query.h
> @@ -11,6 +11,8 @@
>
> #include <stdint.h>
> #include <xe_drm.h>
> +
> +#include "igt_aux.h"
> #include "igt_list.h"
> #include "igt_sizes.h"
>
> --
> 2.34.1
>
More information about the igt-dev
mailing list