diff options
author | Julius Werner <jwerner@chromium.org> | 2019-11-13 19:50:33 -0800 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-11-18 22:47:13 +0000 |
commit | a2148377b5605e96860476bd7cffbabc6e92542e (patch) | |
tree | 7cd07d20d3283bb141f85ef66b6b86ade1e6ee26 /src/include | |
parent | 85b41445b5c4df5833eceb7e1602408dc6c68662 (diff) | |
download | coreboot-a2148377b5605e96860476bd7cffbabc6e92542e.tar.gz coreboot-a2148377b5605e96860476bd7cffbabc6e92542e.tar.bz2 coreboot-a2148377b5605e96860476bd7cffbabc6e92542e.zip |
include: Make stdbool.h a separate file
This patch moves the traditional POSIX stdbool.h definitions out from
stdint.h into their own file. This helps for using these definitions in
commonlib code which may be compiled in different environments. For
coreboot everything should chain-include this stuff via types.h anyway
so nothing should change.
Change-Id: Ic8d52be80b64d8e9564f3aee8975cb25e4c187f5
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36837
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/include')
-rw-r--r-- | src/include/bootmem.h | 4 | ||||
-rw-r--r-- | src/include/console/usb.h | 2 | ||||
-rw-r--r-- | src/include/device/device.h | 3 | ||||
-rw-r--r-- | src/include/stdbool.h | 16 | ||||
-rw-r--r-- | src/include/stdint.h | 9 | ||||
-rw-r--r-- | src/include/types.h | 3 |
6 files changed, 22 insertions, 15 deletions
diff --git a/src/include/bootmem.h b/src/include/bootmem.h index 2e33fcdf76da..53e2d0f9db0f 100644 --- a/src/include/bootmem.h +++ b/src/include/bootmem.h @@ -16,9 +16,9 @@ #ifndef BOOTMEM_H #define BOOTMEM_H -#include <memrange.h> -#include <stdint.h> #include <boot/coreboot_tables.h> +#include <memrange.h> +#include <types.h> /** * Bootmem types match to LB_MEM tags, except for the following: diff --git a/src/include/console/usb.h b/src/include/console/usb.h index 33edbf6e5fb3..ad57d522dc87 100644 --- a/src/include/console/usb.h +++ b/src/include/console/usb.h @@ -17,7 +17,7 @@ #ifndef _CONSOLE_USB_H_ #define _CONSOLE_USB_H_ -#include <stdint.h> +#include <types.h> void usbdebug_init(void); int usbdebug_hw_init(bool force); diff --git a/src/include/device/device.h b/src/include/device/device.h index 405d816e153f..b1c1651ec9e0 100644 --- a/src/include/device/device.h +++ b/src/include/device/device.h @@ -9,11 +9,10 @@ */ #if !defined(__ROMCC__) -#include <stdint.h> -#include <stddef.h> #include <device/resource.h> #include <device/path.h> #include <device/pci_type.h> +#include <types.h> struct device; struct pci_operations; diff --git a/src/include/stdbool.h b/src/include/stdbool.h new file mode 100644 index 000000000000..2eeb70ef5bbe --- /dev/null +++ b/src/include/stdbool.h @@ -0,0 +1,16 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#ifndef __STDBOOL_H__ +#define __STDBOOL_H__ + +#include <stdint.h> + +#ifdef __ROMCC__ +typedef uint8_t bool; +#else +typedef _Bool bool; +#endif +#define true 1 +#define false 0 + +#endif /* __STDBOOL_H__ */ diff --git a/src/include/stdint.h b/src/include/stdint.h index 0a8e153d6afa..67b0b0be08dd 100644 --- a/src/include/stdint.h +++ b/src/include/stdint.h @@ -101,13 +101,4 @@ typedef uint64_t u64; #define UINTMAX_MAX UINT64_MAX #endif -/* TODO: move into stdbool.h */ -#ifdef __ROMCC__ -typedef uint8_t bool; -#else -typedef _Bool bool; -#endif -#define true 1 -#define false 0 - #endif /* STDINT_H */ diff --git a/src/include/types.h b/src/include/types.h index 5902bc268a7b..30f243ff995c 100644 --- a/src/include/types.h +++ b/src/include/types.h @@ -16,7 +16,8 @@ #ifndef __TYPES_H #define __TYPES_H -/* types.h is supposed to provide stdint and stddef defined in here: */ +/* types.h is supposed to provide the standard headers defined in here: */ +#include <stdbool.h> #include <stdint.h> #include <stddef.h> |