[coreboot-gerrit] New patch to review for coreboot: Kconfig: add comments for toolchain choices

[Martin Roth (martinroth@google.com)]

Martin Roth (martinroth at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13053

-gerrit

commit c5a62fb479f405953102aa9c223813e291042c99
Author: Martin Roth <martinroth at google.com>
Date:   Tue Jan 19 12:01:09 2016 -0700

    Kconfig: add comments for toolchain choices
    
    - While we're working on fixing clang for coreboot, mark it as not
    currently working so that it doesn't look like a reasonable choice.
    
    -Add help on how to make the toolchains
    
    Change-Id: Ib37093ca98d0328fad40dd7886c98d00f78bd58e
    Signed-off-by: Martin Roth <martinroth at google.com>
---
 src/Kconfig | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/src/Kconfig b/src/Kconfig
index 6a0209d..2a196e1 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -58,6 +58,14 @@ choice
 	help
 	  This option allows you to select the compiler used for building
 	  coreboot.
+	  You must build the coreboot crosscompiler for the board that you
+	  have selected.
+
+	  To build all the GCC crosscompilers (takes a LONG time), run:
+	    make crossgcc
+
+	  For help on individual architectures, run the command:
+	    make help_toolchain 
 
 config COMPILER_GCC
 	bool "GCC"
@@ -67,9 +75,14 @@ config COMPILER_GCC
 	  For details see http://gcc.gnu.org.
 
 config COMPILER_LLVM_CLANG
-	bool "LLVM/clang"
-	help
-	  Use LLVM/clang to build coreboot.
+	bool "LLVM/clang (TESTING ONLY - Not currently working)"
+	help
+	  Use LLVM/clang to build coreboot.  To use this, you must build the
+	  coreboot version of the clang compiler.  Run the command
+	    make clang
+	  Note that this option is not currently working correctly and should
+	  really only be selected if you're trying to work on getting clang
+	  operational. 
 
 	  For details see http://clang.llvm.org.