Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8157
-gerrit
commit 2f20015dfb0551fd63f4e2bc062ee1ce2bedd846
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Wed Jan 7 15:53:00 2015 +1100
southbridge/intel/lynxpoint/me_9.x.c: Avoid unused func warn
Put a guard around this function to avoid compiler warnings
pretaining to unused functions when a false branch is hit on
a guard further down the file.
Change-Id: Ie4955ee9df6904c38848f46226b53be37d9fa239
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
src/southbridge/intel/lynxpoint/me_9.x.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/southbridge/intel/lynxpoint/me_9.x.c b/src/southbridge/intel/lynxpoint/me_9.x.c
index dfed6de..8b0049a 100644
--- a/src/southbridge/intel/lynxpoint/me_9.x.c
+++ b/src/southbridge/intel/lynxpoint/me_9.x.c
@@ -374,6 +374,7 @@ static int mei_recv_msg(void *header, int header_bytes,
return mei_wait_for_me_ready();
}
+#if IS_ENABLED (CONFIG_DEBUG_INTEL_ME) || __SMM__
static inline int mei_sendrecv_mkhi(struct mkhi_header *mkhi,
void *req_data, int req_bytes,
void *rsp_data, int rsp_bytes)
@@ -411,6 +412,7 @@ static inline int mei_sendrecv_mkhi(struct mkhi_header *mkhi,
return 0;
}
+#endif
static inline int mei_sendrecv_icc(struct icc_header *icc,
void *req_data, int req_bytes,
Edward O'Callaghan (eocallaghan(a)alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8136
-gerrit
commit fe73a674dba51afe46ea28a9a18ad8dc0562f3c8
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Jan 6 14:58:31 2015 +1100
Makefile: clang - ramp up some more warnings
Change-Id: I5262cba22d262f5d74d1edd958b59c2695271293
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
---
Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Makefile b/Makefile
index 6ef9838..ce2b727 100644
--- a/Makefile
+++ b/Makefile
@@ -119,7 +119,7 @@ ifeq ($(CONFIG_COMPILER_LLVM_CLANG),y)
# 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
# Tone down some clang warnings
-CFLAGS_x86_32 += -Wno-unused-variable -Wno-unused-function -Wno-tautological-compare -Wno-shift-overflow
+CFLAGS_x86_32 += -Wno-unused-variable -Wno-tautological-compare -Wno-shift-overflow
CC_x86_32:=clang
HOSTCC := clang
the following patch was just integrated into master:
commit 97398464695e17f76add7a5dcbf4622cec17826a
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Thu Jan 1 04:09:04 2015 +1100
northbridge/amd/agesa/family1{4,5}: Remove cruft from dimmSpd.c
Remove useless comment pretaining to abusing pragma's for old
GCC/GDB interaction issues.
Change-Id: Ic83a0285ac947a23699a81a82b89de08a47ab052
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Reviewed-on: http://review.coreboot.org/8017
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich(a)gmail.com>
See http://review.coreboot.org/8017 for details.
-gerrit
Felix Held (felix-coreboot(a)felixheld.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8090
-gerrit
commit df3ba88b532152ab2aab620cbe77e2cf910720a9
Author: Felix Held <felix-coreboot(a)felixheld.de>
Date: Sun Jan 4 17:33:41 2015 +0100
southbridge/hudson: only enable XHCI when USE_BLOBS is set
only enable XHCI when the firmware is present
Change-Id: I1de706d753da76a9e7c4fec2c5a5cae9e639c3da
Signed-off-by: Felix Held <felix-coreboot(a)felixheld.de>
---
src/southbridge/amd/agesa/hudson/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/southbridge/amd/agesa/hudson/Kconfig b/src/southbridge/amd/agesa/hudson/Kconfig
index f957c48..510db70 100644
--- a/src/southbridge/amd/agesa/hudson/Kconfig
+++ b/src/southbridge/amd/agesa/hudson/Kconfig
@@ -45,7 +45,7 @@ config EHCI_BAR
config HUDSON_XHCI_ENABLE
bool "Enable Hudson XHCI Controller"
- default y
+ default y if USE_BLOBS
help
The XHCI controller must be enabled and the XHCI firmware
must be added in order to have USB 3.0 support configured
Felix Held (felix-coreboot(a)felixheld.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8089
-gerrit
commit 6f52fd7ad8520ce9fde74a6e54aec40ebb8369dc
Author: Felix Held <felix-coreboot(a)felixheld.de>
Date: Sun Jan 4 17:19:59 2015 +0100
hudson: reintroduce HUDSON_DISABLE_IMC option
HUDSON_DISABLE_IMC was introduced in commit 1a59039c
option gets used in commit 99799d66 for asus/f2a85-m
option gets killed in commit 1b1b795f, but is still used in source agesa based
boards (asus/f2a85-m and asus/f2a85-m_le)
The HUDSON_DISABLE_IMC option is reintroduced to exclude the IMC firmware on
boards where the fan control is done by the SIO/EC and not by the FCH.
Defaulting HUDSON_IMC_FWM to n, making it depend on USE_BLOBS and explicitly
sourcing it in the board Kconfig file does not work as expected; when we
explicitly select HUDSON_IMC_FWM in the mainboard Kconfig and USE_BLOBS is n,
we get a warning due to unmet dependencies, but the IMC firmware still gets
included in CBFS.
Change-Id: I448a0dc7c9fef504f89cfd192ca74172425fb3e0
Signed-off-by: Felix Held <felix-coreboot(a)felixheld.de>
---
src/southbridge/amd/agesa/hudson/Kconfig | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/src/southbridge/amd/agesa/hudson/Kconfig b/src/southbridge/amd/agesa/hudson/Kconfig
index acc2d72..f957c48 100644
--- a/src/southbridge/amd/agesa/hudson/Kconfig
+++ b/src/southbridge/amd/agesa/hudson/Kconfig
@@ -59,8 +59,13 @@ config HUDSON_XHCI_FWM
help
Add Hudson 2/3/4 XHCI Firmware to support the onboard USB 3.0
+config HUDSON_DISABLE_IMC
+ bool
+ default n
+
config HUDSON_IMC_FWM
bool "Add imc firmware"
+ depends on !HUDSON_DISABLE_IMC
default y if USE_BLOBS
help
Add Hudson 2/3/4 IMC Firmware to support the onboard fan control