Nico Huber has posted comments on this change. ( https://review.coreboot.org/29312 )
Change subject: src: Add missing include <stdint.h> ......................................................................
Patch Set 10:
(4 comments)
https://review.coreboot.org/#/c/29312/10/src/soc/intel/baytrail/include/soc/... File src/soc/intel/baytrail/include/soc/iomap.h:
https://review.coreboot.org/#/c/29312/10/src/soc/intel/baytrail/include/soc/... PS10, Line 19: #include <stdint.h> needs __ACPI__ guard
https://review.coreboot.org/#/c/29312/10/src/soc/intel/baytrail/include/soc/... PS10, Line 82: #ifndef __ACPI__ This guards the C code below from being included in ASL. The #include needs the same guard.
https://review.coreboot.org/#/c/29312/10/src/soc/intel/braswell/include/soc/... File src/soc/intel/braswell/include/soc/iomap.h:
https://review.coreboot.org/#/c/29312/10/src/soc/intel/braswell/include/soc/... PS10, Line 20: #include <stdint.h> same here
https://review.coreboot.org/#/c/29312/10/src/soc/intel/fsp_baytrail/include/... File src/soc/intel/fsp_baytrail/include/soc/iomap.h:
https://review.coreboot.org/#/c/29312/10/src/soc/intel/fsp_baytrail/include/... PS10, Line 19: #include <stdint.h> and here