[coreboot] New patch to review for coreboot: a071c33 Fix AMD S3 block generator on Cygwin

Patrick Georgi (patrick@georgi-clan.de) gerrit at coreboot.org
Tue Jul 3 18:09:45 CEST 2012


Patrick Georgi (patrick at georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1164

-gerrit

commit a071c337b4129b84db56f162dbc12f86f82389f9
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Tue Jul 3 18:07:04 2012 +0200

    Fix AMD S3 block generator on Cygwin
    
    awk on Cygwin created the UTF-8 value for the 0xff code point,
    which makes it two bytes wide. This broke the build.
    
    Change-Id: I4937ae7ce1136ba7a76d05b42f9dd2771203175d
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 src/southbridge/amd/Makefile.inc |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/southbridge/amd/Makefile.inc b/src/southbridge/amd/Makefile.inc
index d7b5645..9bc58cd 100644
--- a/src/southbridge/amd/Makefile.inc
+++ b/src/southbridge/amd/Makefile.inc
@@ -20,7 +20,8 @@ ifeq ($(CONFIG_CPU_AMD_AGESA), y)
 
 $(obj)/s3.rom:
 	echo "    S3 NVRAM   0xffff0000 (S3 storage area)"
-	awk 'BEGIN {for (i=0; i<20480; i++) {printf "%c", 255}}' > $@.tmp
+	# force C locale, so cygwin awk doesn't try to interpret the 0xff below as UTF-8 (or worse)
+	LC_ALL=C awk 'BEGIN {for (i=0; i<20480; i++) {printf "%c", 255}}' > $@.tmp
 	mv $@.tmp $@
 
 cbfs-files-y += s3nv




More information about the coreboot mailing list