[coreboot-gerrit] New patch to review for coreboot: qualcomm/ipq40xx: Don't annoy users with ambiguous questions

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue May 10 23:04:51 CEST 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14766

-gerrit

commit 7d4254e070c2a67642d94b397ab5ffa0a6188b2d
Author: Patrick Georgi <pgeorgi at chromium.org>
Date:   Tue May 10 23:03:17 2016 +0200

    qualcomm/ipq40xx: Don't annoy users with ambiguous questions
    
    The code needs to know what kind of part the SoC is, but the question
    was weirdly phrased and also exposed to the user (instead of being a
    silent "select" to do in a board).
    
    Change-Id: I0344c528d86ac047fc49ccff9e149865bbd4b481
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
---
 src/soc/qualcomm/ipq40xx/Kconfig | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/soc/qualcomm/ipq40xx/Kconfig b/src/soc/qualcomm/ipq40xx/Kconfig
index b93f879..605cea2 100644
--- a/src/soc/qualcomm/ipq40xx/Kconfig
+++ b/src/soc/qualcomm/ipq40xx/Kconfig
@@ -19,8 +19,10 @@ config CHROMEOS
 	select RETURN_FROM_VERSTAGE
 
 config IPQ_QFN_PART
-	bool "Is the SoC a BGA part or QFN part"
+	bool
 	default n
+	help
+	  Is the SoC a QFN part (as opposed to a BGA part)
 
 config MBN_ENCAPSULATION
 	depends on USE_BLOBS



More information about the coreboot-gerrit mailing list