diff options
| author | Elyes Haouas <ehaouas@noos.fr> | 2024-05-27 10:31:19 +0200 |
|---|---|---|
| committer | Elyes Haouas <ehaouas@noos.fr> | 2024-05-28 03:22:34 +0000 |
| commit | 94bfdd12821d0473e8bac1920342b0271a3771d5 (patch) | |
| tree | 738d196da2b335441df2e2aa419d06694f8ce61f | |
| parent | 0554fc10c0dea70366992e56f4d344b528364f3a (diff) | |
tree: Remove unused <stdarg.h>
<stdarg.h> header is used to define macros for handling variable
argument lists in functions like printf. It does not depend on the string
or memory manipulation functions provided by <string.h>.
So let follow conventions and include only the necessary headers in each
header file.
Change-Id: I07ffc65b7feefb8ec4ab8dd268113f9ed8d24685
Signed-off-by: Elyes Haouas <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/82664
Reviewed-by: Maximilian Brune <maximilian.brune@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/include/stdarg.h | 3 | ||||
| -rw-r--r-- | src/include/string.h | 1 | ||||
| -rw-r--r-- | src/soc/amd/common/psp_verstage/psp_verstage.c | 1 |
3 files changed, 1 insertions, 4 deletions
diff --git a/src/include/stdarg.h b/src/include/stdarg.h index c5a8cd8dbe..f55fcc3019 100644 --- a/src/include/stdarg.h +++ b/src/include/stdarg.h @@ -1,8 +1,7 @@ /* SPDX-License-Identifier: GPL-2.0-only */ /** - * Note: This file is only for POSIX compatibility, and is meant to be - * chain-included via string.h. + * Note: This file is only for POSIX compatibility. */ #ifndef STDARG_H diff --git a/src/include/string.h b/src/include/string.h index 92ea5e5f7f..4a06556d6d 100644 --- a/src/include/string.h +++ b/src/include/string.h @@ -3,7 +3,6 @@ #ifndef STRING_H #define STRING_H -#include <stdarg.h> /* IWYU pragma: export */ #include <stddef.h> #include <stdio.h> /* IWYU pragma: export */ diff --git a/src/soc/amd/common/psp_verstage/psp_verstage.c b/src/soc/amd/common/psp_verstage/psp_verstage.c index e71d75ac32..54945b9236 100644 --- a/src/soc/amd/common/psp_verstage/psp_verstage.c +++ b/src/soc/amd/common/psp_verstage/psp_verstage.c @@ -21,7 +21,6 @@ #include <security/vboot/symbols.h> #include <security/vboot/vboot_common.h> #include <arch/stages.h> -#include <stdarg.h> #include <timestamp.h> extern char _bss_start, _bss_end; |