Re: [PATCH v1 5/8] xen/include: include <asm/types.h> in <xen/early_printk.h>

[Julien Grall <julien@xxxxxxx>]

On 06/01/2023 13:14, Oleksii Kurochko wrote:
> <asm/types.h> should be included because second argument of
> early_puts has type 'size_t' which is defined in <asm/types.h>
>
> Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
Acked-by: Julien Grall <jgrall@xxxxxxxxxx>

Cheers,

> ---
>   xen/include/xen/early_printk.h | 2 ++
>   1 file changed, 2 insertions(+)
>
> diff --git a/xen/include/xen/early_printk.h b/xen/include/xen/early_printk.h
> index 0f76c3a74f..abb34687da 100644
> --- a/xen/include/xen/early_printk.h
> +++ b/xen/include/xen/early_printk.h
> @@ -4,6 +4,8 @@
>   #ifndef __XEN_EARLY_PRINTK_H__
>   #define __XEN_EARLY_PRINTK_H__
>   
> +#include <asm/types.h>
> +
>   #ifdef CONFIG_EARLY_PRINTK
>   void early_puts(const char *s, size_t nr);
>   #else
--
Julien Grall