On Wed, Feb 26, 2025 at 08:18:35PM +0800, LeoLiu-oc wrote:
> From: LeoLiuoc <LeoLiu-oc@xxxxxxxxxxx>
>
> Remove static from apei_hest_parse() so that it can be called in another
> file.
> +++ b/drivers/acpi/apei/hest.c
> @@ -134,7 +134,7 @@ static bool is_ghes_assist_struct(struct acpi_hest_header *hest_hdr)
>
> typedef int (*apei_hest_func_t)(struct acpi_hest_header *hest_hdr, void *data);
>
> -static int apei_hest_parse(apei_hest_func_t func, void *data)
> +int apei_hest_parse(apei_hest_func_t func, void *data)
> {
> struct acpi_hest_header *hest_hdr;
> int i, rc, len;
> diff --git a/include/acpi/apei.h b/include/acpi/apei.h
> index dc60f7db5524..b79976daa4bb 100644
> --- a/include/acpi/apei.h
> +++ b/include/acpi/apei.h
> @@ -33,6 +33,8 @@ void __init acpi_ghes_init(void);
> static inline void acpi_ghes_init(void) { }
> #endif
>
> +int apei_hest_parse(apei_hest_func_t func, void *data);
Series doesn't build after this patch because we lack the
apei_hest_func_t typedef:
$ make drivers/acpi/apei/
CC drivers/acpi/apei/apei-base.o
In file included from drivers/acpi/apei/apei-base.c:30:
./include/acpi/apei.h:36:21: error: unknown type name ‘apei_hest_func_t’
36 | int apei_hest_parse(apei_hest_func_t func, void *data);
| ^~~~~~~~~~~~~~~~
The kernel must build and function correctly after each and every
patch in the series.
