These test programs are includedd as binary files in U-Boot to avoid
having to build them (and associated toolchain differences). Instructions
on building are in the files themselves, but it seems better to provide
a Makefile which can be manually run when desired.

Add a Makefile, separate from the normal build system, to handle this.

Signed-off-by: Simon Glass <s...@chromium.org>
---

 tools/binman/test/Makefile              | 36 +++++++++++++++++++++++++++++++++
 tools/binman/test/u_boot_no_ucode_ptr.c |  4 ----
 tools/binman/test/u_boot_ucode_ptr.c    |  4 ----
 3 files changed, 36 insertions(+), 8 deletions(-)
 create mode 100644 tools/binman/test/Makefile

diff --git a/tools/binman/test/Makefile b/tools/binman/test/Makefile
new file mode 100644
index 00000000000..786d1b05778
--- /dev/null
+++ b/tools/binman/test/Makefile
@@ -0,0 +1,36 @@
+#
+# Builds test programs
+#
+# Copyright (C) 2017 Google, Inc
+# Written by Simon Glass <s...@chromium.org>
+#
+# SPDX-License-Identifier:      GPL-2.0+
+#
+
+CFLAGS := -march=i386 -m32 -nostdlib -I ../../../include
+
+LDS_UCODE := -T u_boot_ucode_ptr.lds
+
+TARGETS = u_boot_ucode_ptr u_boot_no_ucode_ptr
+
+all: $(TARGETS)
+
+u_boot_no_ucode_ptr: CFLAGS += $(LDS_UCODE)
+u_boot_no_ucode_ptr: u_boot_no_ucode_ptr.c
+
+u_boot_ucode_ptr: CFLAGS += $(LDS_UCODE)
+u_boot_ucode_ptr: u_boot_ucode_ptr.c
+
+clean:
+       rm -f $(TARGETS)
+
+help:
+       @echo "Makefile for binman test programs"
+       @echo
+       @echo "Intended for use on x86 hosts"
+       @echo
+       @echo "Targets:"
+       @echo
+       @echo -e "\thelp        - Print help (this is it!)"
+       @echo -e "\tall - Builds test programs (default targget)"
+       @echo -e "\tclean       - Delete output files"
diff --git a/tools/binman/test/u_boot_no_ucode_ptr.c 
b/tools/binman/test/u_boot_no_ucode_ptr.c
index a17bb4c6c20..c4a2b85fc95 100644
--- a/tools/binman/test/u_boot_no_ucode_ptr.c
+++ b/tools/binman/test/u_boot_no_ucode_ptr.c
@@ -5,10 +5,6 @@
  *
  * Simple program to create a bad _dt_ucode_base_size symbol to create an
  * error when it is used. This is used by binman tests.
- *
- * Build with:
- * cc -march=i386 -m32 -o u_boot_no_ucode_ptr -T u_boot_ucode_ptr.lds \
-       -nostdlib u_boot_no_ucode_ptr.c
  */
 
 static unsigned long not__dt_ucode_base_size[2]
diff --git a/tools/binman/test/u_boot_ucode_ptr.c 
b/tools/binman/test/u_boot_ucode_ptr.c
index 434c9f44003..24f349fb9e4 100644
--- a/tools/binman/test/u_boot_ucode_ptr.c
+++ b/tools/binman/test/u_boot_ucode_ptr.c
@@ -5,10 +5,6 @@
  *
  * Simple program to create a _dt_ucode_base_size symbol which can be read
  * by 'nm'. This is used by binman tests.
- *
- * Build with:
- * cc -march=i386 -m32 -o u_boot_ucode_ptr -T u_boot_ucode_ptr.lds -nostdlib \
-       u_boot_ucode_ptr.c
  */
 
 static unsigned long _dt_ucode_base_size[2]
-- 
2.15.0.448.gf294e3d99a-goog

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to