Re: [PATCH 1/5] ASN.1: Add missing include <linux/types.h>

[Jonathan Cameron <Jonathan.Cameron@Huawei.com>]

On Mon, 29 Jul 2024 15:47:00 +0200
Lukas Wunner <lukas@wunner.de> wrote:

> If <linux/asn1_decoder.h> is the first header included from a .c file
> (due to headers being sorted alphabetically), the compiler complains:
> 
> include/linux/asn1_decoder.h:18:29: error: unknown type name 'size_t'
> 
> Fix it.
> 
> Signed-off-by: Lukas Wunner <lukas@wunner.de>

FWIW
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

I see asn1_encoder.h already has this include so this
is nice and consistent.

However that header includes bug.h which should probably be
pushed down into the relevant c files.  Not relevant for this
series though so one for another day (or never :)

> ---
>  include/linux/asn1_decoder.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/linux/asn1_decoder.h b/include/linux/asn1_decoder.h
> index 83f9c6e1e5e9..b41bce82a191 100644
> --- a/include/linux/asn1_decoder.h
> +++ b/include/linux/asn1_decoder.h
> @@ -9,6 +9,7 @@
>  #define _LINUX_ASN1_DECODER_H
>  
>  #include <linux/asn1.h>
> +#include <linux/types.h>
>  
>  struct asn1_decoder;
>