Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/36556 )
Change subject: sb/intel/i82801jx/nvs.h: include required header ......................................................................
sb/intel/i82801jx/nvs.h: include required header
u8, u16, ... are defined in stdint.h.
Change-Id: I045438a7e754d4da936cf2ded411d055f2f69b45 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/southbridge/intel/i82801jx/nvs.h 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/56/36556/1
diff --git a/src/southbridge/intel/i82801jx/nvs.h b/src/southbridge/intel/i82801jx/nvs.h index ca66985..88944c0 100644 --- a/src/southbridge/intel/i82801jx/nvs.h +++ b/src/southbridge/intel/i82801jx/nvs.h @@ -13,6 +13,8 @@ * GNU General Public License for more details. */
+#include <stdint.h> + typedef struct { /* Miscellaneous */ u16 osys; /* 0x00 - Operating System */
HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36556 )
Change subject: sb/intel/i82801jx/nvs.h: include required header ......................................................................
Patch Set 1: Code-Review+2
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36556 )
Change subject: sb/intel/i82801jx/nvs.h: include required header ......................................................................
Patch Set 2: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/36556 )
Change subject: sb/intel/i82801jx/nvs.h: include required header ......................................................................
sb/intel/i82801jx/nvs.h: include required header
u8, u16, ... are defined in stdint.h.
Change-Id: I045438a7e754d4da936cf2ded411d055f2f69b45 Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/36556 Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: HAOUAS Elyes ehaouas@noos.fr Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/southbridge/intel/i82801jx/nvs.h 1 file changed, 2 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified HAOUAS Elyes: Looks good to me, approved Angel Pons: Looks good to me, approved
diff --git a/src/southbridge/intel/i82801jx/nvs.h b/src/southbridge/intel/i82801jx/nvs.h index ca66985..88944c0 100644 --- a/src/southbridge/intel/i82801jx/nvs.h +++ b/src/southbridge/intel/i82801jx/nvs.h @@ -13,6 +13,8 @@ * GNU General Public License for more details. */
+#include <stdint.h> + typedef struct { /* Miscellaneous */ u16 osys; /* 0x00 - Operating System */
-
Angel Pons (Code Review) -
Arthur Heymans (Code Review)
-
HAOUAS Elyes (Code Review)
-
Patrick Georgi (Code Review)