Furquan Shaikh (furquan@google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17773
-gerrit
commit 4b29f31ecb2bcab8d2b386ccbf0a86e0211b66d3 Author: Furquan Shaikh furquan@chromium.org Date: Wed Dec 7 20:34:32 2016 -0800
drivers/intel/fsp2_0: Include stddef.h in soc_binding.h
soc_binding.h includes FSP headers which define NULL macro. Because of this, including stddef.h after soc_binding.h results in NULL being re-defined. Thus, include stddef.h in soc_binding.h to avoid having users include stddef.h along with soc_binding.h.
Change-Id: I600083c5d8f672518beaa1119f14f67728a433aa Signed-off-by: Furquan Shaikh furquan@chromium.org --- src/drivers/intel/fsp2_0/include/fsp/soc_binding.h | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/src/drivers/intel/fsp2_0/include/fsp/soc_binding.h b/src/drivers/intel/fsp2_0/include/fsp/soc_binding.h index 1ff8435..5c8b441 100644 --- a/src/drivers/intel/fsp2_0/include/fsp/soc_binding.h +++ b/src/drivers/intel/fsp2_0/include/fsp/soc_binding.h @@ -16,6 +16,8 @@ #ifndef _FSP2_0_SOC_BINDING_H_ #define _FSP2_0_SOC_BINDING_H_
+#include <stddef.h> + #pragma pack(push) /* * This file is a implementation specific header. i.e. different