[coreboot-gerrit] Patch set updated for coreboot: e2d1bb3 mainboards: fix spd generation

Aaron Durbin (adurbin@google.com) gerrit at coreboot.org
Wed Apr 1 21:43:54 CEST 2015


Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9196

-gerrit

commit e2d1bb3a78c54dd07dbe02758ed857d29a9d2964
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Wed Apr 1 08:49:41 2015 -0500

    mainboards: fix spd generation
    
    echo is evaluated by a shell builtin producing non-binary
    spd data of the form '-e -n \<byte>'. Correct this by
    using printf builtin which does the equivalent and is
    more cross platform friendly.
    
    Boards changed:
    gizmosphere/gizmo
    gizmosphere/gizmo2
    google/bolt
    google/falco
    google/link
    google/peppy
    google/rambi
    google/samus
    google/slippy
    pcengines/apu1
    
    Change-Id: Iefdaf59903b9682cc88c94fd991883b560616492
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/gizmosphere/gizmo/Makefile.inc  | 2 +-
 src/mainboard/gizmosphere/gizmo2/Makefile.inc | 2 +-
 src/mainboard/google/bolt/Makefile.inc        | 2 +-
 src/mainboard/google/falco/Makefile.inc       | 2 +-
 src/mainboard/google/link/Makefile.inc        | 2 +-
 src/mainboard/google/peppy/Makefile.inc       | 2 +-
 src/mainboard/google/rambi/spd/Makefile.inc   | 2 +-
 src/mainboard/google/samus/spd/Makefile.inc   | 2 +-
 src/mainboard/google/slippy/Makefile.inc      | 2 +-
 src/mainboard/pcengines/apu1/Makefile.inc     | 2 +-
 10 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/mainboard/gizmosphere/gizmo/Makefile.inc b/src/mainboard/gizmosphere/gizmo/Makefile.inc
index 10a15e3..8592d7e 100644
--- a/src/mainboard/gizmosphere/gizmo/Makefile.inc
+++ b/src/mainboard/gizmosphere/gizmo/Makefile.inc
@@ -45,7 +45,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/gizmosphere/gizmo2/Makefile.inc b/src/mainboard/gizmosphere/gizmo2/Makefile.inc
index 8a1c388..215b6b5 100644
--- a/src/mainboard/gizmosphere/gizmo2/Makefile.inc
+++ b/src/mainboard/gizmosphere/gizmo2/Makefile.inc
@@ -38,7 +38,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/bolt/Makefile.inc b/src/mainboard/google/bolt/Makefile.inc
index e6c6eb3..9471e33 100644
--- a/src/mainboard/google/bolt/Makefile.inc
+++ b/src/mainboard/google/bolt/Makefile.inc
@@ -36,7 +36,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/falco/Makefile.inc b/src/mainboard/google/falco/Makefile.inc
index fbc2462..2866de9 100644
--- a/src/mainboard/google/falco/Makefile.inc
+++ b/src/mainboard/google/falco/Makefile.inc
@@ -44,7 +44,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/link/Makefile.inc b/src/mainboard/google/link/Makefile.inc
index 62746ad..b0920e7f 100644
--- a/src/mainboard/google/link/Makefile.inc
+++ b/src/mainboard/google/link/Makefile.inc
@@ -37,7 +37,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 
 # Include spd rom data
 $(SPD_BIN): $(SPD_DEPS)
-	for f in $^; do for c in $$(cat $$f); do echo -e -n "\\x$$c"; done; done > $@
+	for f in $^; do for c in $$(cat $$f); do printf "\\x$$c"; done; done > $@
 
 cbfs-files-y += spd.bin
 spd.bin-file := $(SPD_BIN)
diff --git a/src/mainboard/google/peppy/Makefile.inc b/src/mainboard/google/peppy/Makefile.inc
index 96c63f0..4079bc9 100644
--- a/src/mainboard/google/peppy/Makefile.inc
+++ b/src/mainboard/google/peppy/Makefile.inc
@@ -43,7 +43,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/rambi/spd/Makefile.inc b/src/mainboard/google/rambi/spd/Makefile.inc
index 2e7d750..38a1224 100644
--- a/src/mainboard/google/rambi/spd/Makefile.inc
+++ b/src/mainboard/google/rambi/spd/Makefile.inc
@@ -40,7 +40,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/spd/$(f).
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/samus/spd/Makefile.inc b/src/mainboard/google/samus/spd/Makefile.inc
index df63118..c279338 100644
--- a/src/mainboard/google/samus/spd/Makefile.inc
+++ b/src/mainboard/google/samus/spd/Makefile.inc
@@ -45,7 +45,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/spd/$(f).
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/google/slippy/Makefile.inc b/src/mainboard/google/slippy/Makefile.inc
index ec75717..cf5097a 100644
--- a/src/mainboard/google/slippy/Makefile.inc
+++ b/src/mainboard/google/slippy/Makefile.inc
@@ -37,7 +37,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 
diff --git a/src/mainboard/pcengines/apu1/Makefile.inc b/src/mainboard/pcengines/apu1/Makefile.inc
index 904dfbe..b4b1b9f 100644
--- a/src/mainboard/pcengines/apu1/Makefile.inc
+++ b/src/mainboard/pcengines/apu1/Makefile.inc
@@ -47,7 +47,7 @@ SPD_DEPS := $(foreach f, $(SPD_SOURCES), src/mainboard/$(MAINBOARDDIR)/$(f).spd.
 $(SPD_BIN): $(SPD_DEPS)
 	for f in $+; \
 	  do for c in $$(cat $$f | grep -v ^#); \
-	    do echo -e -n "\\x$$c"; \
+	    do printf "\\x$$c"; \
 	  done; \
 	done > $@
 



More information about the coreboot-gerrit mailing list