Patrick Georgi ([email protected]) just uploaded a new patch set to 
gerrit, which you can find at http://review.coreboot.org/1164

-gerrit

commit a071c337b4129b84db56f162dbc12f86f82389f9
Author: Patrick Georgi <[email protected]>
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 <[email protected]>
---
 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}}' > [email protected]
+       # 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}}' > 
[email protected]
        mv [email protected] $@
 
 cbfs-files-y += s3nv

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to