Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7697
-gerrit
commit 9aaa10850750bc43a42ace601bbf82a8fb64fd44
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Mon Dec 8 05:45:05 2014 +1100
southbridge/intel/fsp_rangeley: Fix missing entry-point prototype
In '-ffreestanding' main() is just as any other function and so
it needs a type-signature. Fixes a clang warning.
Change-Id: I59eff28d77bd2a7f0c598c5846240344b7cec4d4
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/southbridge/intel/fsp_rangeley/romstage.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/southbridge/intel/fsp_rangeley/romstage.c b/src/southbridge/intel/fsp_rangeley/romstage.c
index fd8b3b2..2174a60 100644
--- a/src/southbridge/intel/fsp_rangeley/romstage.c
+++ b/src/southbridge/intel/fsp_rangeley/romstage.c
@@ -39,6 +39,7 @@
#include <cpu/x86/msr.h>
#include "gpio.h"
+void main(FSP_INFO_HEADER *fsp_info_header); // XXX: find a better place
void main(FSP_INFO_HEADER *fsp_info_header)
{
uint32_t fd_mask = 0;
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7672
-gerrit
commit bd2defad7a4924c31e0315fd1cfc6b4c3c7f0cde
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Sun Dec 7 02:46:07 2014 +1100
TEST clang warn as error
Change-Id: Iac4efa59f63437fd6c7678368f4c62ba005d2103
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
src/Kconfig | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index eaba3e4..e0246ff 100644
--- a/Makefile
+++ b/Makefile
@@ -117,7 +117,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# FIXME: armv7/aarch64 won't build right now
# NOTE: clang puts compiler-rt under lib/linux/libclang_rt.builtins-i386.a
# this means the triple is i386-linux-elf instead of i386-none-elf
-CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
+CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32 -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare
CC_x86_32:=clang
HOSTCC := clang
diff --git a/src/Kconfig b/src/Kconfig
index 533bc5c..7f033a2 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1131,8 +1131,7 @@ config ENABLE_APIC_EXT_ID
# the whole treee.
config WARNINGS_ARE_ERRORS
bool
- default y if !COMPILER_LLVM_CLANG
- default n if COMPILER_LLVM_CLANG
+ default y
# The four POWER_BUTTON_DEFAULT_ENABLE, POWER_BUTTON_DEFAULT_DISABLE,
# POWER_BUTTON_FORCE_ENABLE and POWER_BUTTON_FORCE_DISABLE options are
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7672
-gerrit
commit f70b1266395d45927ce03ef770a9e6ec627f3e7c
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Sun Dec 7 02:46:07 2014 +1100
TEST clang warn as error
Change-Id: Iac4efa59f63437fd6c7678368f4c62ba005d2103
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
src/Kconfig | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index eaba3e4..e0246ff 100644
--- a/Makefile
+++ b/Makefile
@@ -117,7 +117,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# FIXME: armv7/aarch64 won't build right now
# NOTE: clang puts compiler-rt under lib/linux/libclang_rt.builtins-i386.a
# this means the triple is i386-linux-elf instead of i386-none-elf
-CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
+CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32 -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare
CC_x86_32:=clang
HOSTCC := clang
diff --git a/src/Kconfig b/src/Kconfig
index 533bc5c..7f033a2 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1131,8 +1131,7 @@ config ENABLE_APIC_EXT_ID
# the whole treee.
config WARNINGS_ARE_ERRORS
bool
- default y if !COMPILER_LLVM_CLANG
- default n if COMPILER_LLVM_CLANG
+ default y
# The four POWER_BUTTON_DEFAULT_ENABLE, POWER_BUTTON_DEFAULT_DISABLE,
# POWER_BUTTON_FORCE_ENABLE and POWER_BUTTON_FORCE_DISABLE options are
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7672
-gerrit
commit 9b63ff81c89ad77d88084dc3b3d1df4d131ae904
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Sun Dec 7 02:46:07 2014 +1100
TEST clang warn as error
Change-Id: Iac4efa59f63437fd6c7678368f4c62ba005d2103
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
src/Kconfig | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index eaba3e4..e0246ff 100644
--- a/Makefile
+++ b/Makefile
@@ -117,7 +117,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# FIXME: armv7/aarch64 won't build right now
# NOTE: clang puts compiler-rt under lib/linux/libclang_rt.builtins-i386.a
# this means the triple is i386-linux-elf instead of i386-none-elf
-CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
+CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32 -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare
CC_x86_32:=clang
HOSTCC := clang
diff --git a/src/Kconfig b/src/Kconfig
index 533bc5c..7f033a2 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1131,8 +1131,7 @@ config ENABLE_APIC_EXT_ID
# the whole treee.
config WARNINGS_ARE_ERRORS
bool
- default y if !COMPILER_LLVM_CLANG
- default n if COMPILER_LLVM_CLANG
+ default y
# The four POWER_BUTTON_DEFAULT_ENABLE, POWER_BUTTON_DEFAULT_DISABLE,
# POWER_BUTTON_FORCE_ENABLE and POWER_BUTTON_FORCE_DISABLE options are
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7672
-gerrit
commit c665023bcf63033940b4c26e76be6e090da61de5
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Sun Dec 7 02:46:07 2014 +1100
TEST clang warn as error
Change-Id: Iac4efa59f63437fd6c7678368f4c62ba005d2103
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
src/Kconfig | 3 +--
2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/Makefile b/Makefile
index eaba3e4..5a7e687 100644
--- a/Makefile
+++ b/Makefile
@@ -117,7 +117,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# FIXME: armv7/aarch64 won't build right now
# NOTE: clang puts compiler-rt under lib/linux/libclang_rt.builtins-i386.a
# this means the triple is i386-linux-elf instead of i386-none-elf
-CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32
+CFLAGS_x86_32 = -no-integrated-as -Qunused-arguments -target i386-linux-elf -m32 -Wno-unused-variable -Wno-unused-function
CC_x86_32:=clang
HOSTCC := clang
diff --git a/src/Kconfig b/src/Kconfig
index 533bc5c..7f033a2 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1131,8 +1131,7 @@ config ENABLE_APIC_EXT_ID
# the whole treee.
config WARNINGS_ARE_ERRORS
bool
- default y if !COMPILER_LLVM_CLANG
- default n if COMPILER_LLVM_CLANG
+ default y
# The four POWER_BUTTON_DEFAULT_ENABLE, POWER_BUTTON_DEFAULT_DISABLE,
# POWER_BUTTON_FORCE_ENABLE and POWER_BUTTON_FORCE_DISABLE options are