Patrick Georgi (pgeorgi(a)google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7674
-gerrit
commit 94ca95d9efa13d21f27366f6727b87ac6a44e187
Author: Patrick Georgi <patrick(a)georgi-clan.de>
Date: Sat Dec 6 17:35:25 2014 +0100
edid: fill reserved bits fields in cb_framebuffer
If it's a 4 byte format (as per documentation), there
are some reserved bits, so let's mark them as such...
Change-Id: I50f12cfff2c9bb9d082a5f3c3ac54c0d514d862b
Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de>
---
src/lib/edid.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/lib/edid.c b/src/lib/edid.c
index c8e3203..2028c70 100644
--- a/src/lib/edid.c
+++ b/src/lib/edid.c
@@ -1446,10 +1446,14 @@ void set_vbe_mode_info_valid(struct edid *edid, uintptr_t fb_addr)
* I'm not sure with grahpics you ever can.
*/
edid_fb.bits_per_pixel = edid->framebuffer_bits_per_pixel;
+ edid_fb.reserved_mask_pos = 0;
+ edid_fb.reserved_mask_size = 0;
switch(edid->framebuffer_bits_per_pixel){
case 32:
case 24:
/* packed into 4-byte words */
+ edid_fb.reserved_mask_pos = 24;
+ edid_fb.reserved_mask_size = 8;
edid_fb.red_mask_pos = 16;
edid_fb.red_mask_size = 8;
edid_fb.green_mask_pos = 8;
@@ -1472,8 +1476,6 @@ void set_vbe_mode_info_valid(struct edid *edid, uintptr_t fb_addr)
return;
}
- edid_fb.reserved_mask_pos = 0;
- edid_fb.reserved_mask_size = 0;
vbe_valid = 1;
}
the following patch was just integrated into master:
commit 45b64fbf9dcfb0eba83f4227b11d92905c8d1e03
Author: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
Date: Sat Dec 6 05:33:36 2014 -0600
vendorcode/amd/agesa: Remove unused helper.c file
The contents of these files were guarded by a check for the _MSC_VER
macro, which we don't use.
Change-Id: Ic595c8e6284c54e1449cf21e0cebee8c9ce7c682
Signed-off-by: Alexandru Gagniuc <mr.nuke.me(a)gmail.com>
Reviewed-on: http://review.coreboot.org/7670
Reviewed-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Tested-by: build bot (Jenkins)
See http://review.coreboot.org/7670 for details.
-gerrit
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 4cad7b2a89a2b8ddc0a8544d43f8382db36da7f7
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
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 0d15dde0ea5948a3c8f10da3283ee43d0e26874f
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..78d76b4 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
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/7506
-gerrit
commit ff87c05f8020738c771efcee3cb4e16e5a5ef5ca
Author: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Date: Tue Nov 18 03:17:54 2014 +1100
Revert "src/Kconfig: Don't treat warns as errors on Clang builds yet"
This reverts commit 9b63c9bde2fc3b3a2d42e68618e043cf282bc566.
Signed-off-by: Edward O'Callaghan <eocallaghan(a)alterapraxis.com>
Change-Id: I4f547d20c5096877b2010602a087e41702939f77
---
src/Kconfig | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/Kconfig b/src/Kconfig
index 533bc5c..c48e592 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -1126,13 +1126,9 @@ config ENABLE_APIC_EXT_ID
bool
default n
-# XXX Currently clang/llvm builds are not fully supported,
-# Let us tone down warns treated as errors until we actually build
-# 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