commit 3b87812f004f1b510e36e2fb6a25b54181954190
author Martin Roth <martinroth@google.com> Fri Sep 30 14:43:01 2016 -0600
committer Martin Roth <martinroth@google.com> Sun Oct 02 19:08:15 2016 +0200
tree e7a96357425496d4fdaf8b8414173748e8dc18a1
parent cc3365a0398d02dc2e88c3caf969c0b874d6a3dc

Kconfig: Update default hex values to start with 0x

Kconfig hex values don't need to be in quotes, and should start with
'0x'.  If the default value isn't set this way, Kconfig will add the
0x to the start, and the entry can be added unnecessarily to the
defconfig since it's "different" than what was set by the default.

A check for this has been added to the Kconfig lint tool.

Change-Id: I86f37340682771700011b6285e4b4af41b7e9968
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/16834
Tested-by: build bot (Jenkins)
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>

src/southbridge/amd/cimx/sb800/Kconfig

diff --git a/src/southbridge/amd/cimx/sb800/Kconfig b/src/southbridge/amd/cimx/sb800/Kconfig
index 5748f79..a48b93a 100644
--- a/src/southbridge/amd/cimx/sb800/Kconfig
+++ b/src/southbridge/amd/cimx/sb800/Kconfig
@@ -39,8 +39,8 @@
 
 config IDE_COMBINED_MODE
 	hex
-	default "0x0" if ENABLE_IDE_COMBINED_MODE
-	default "0x1" if !ENABLE_IDE_COMBINED_MODE
+	default 0x0 if ENABLE_IDE_COMBINED_MODE
+	default 0x1 if !ENABLE_IDE_COMBINED_MODE
 
 choice
 	prompt "SATA Mode"
@@ -70,9 +70,9 @@
 config SB800_SATA_MODE
 	hex
 	depends on (SB800_SATA_IDE || SB800_SATA_RAID || SB800_SATA_AHCI)
-	default "0x0" if SB800_SATA_IDE
-	default "0x1" if SB800_SATA_RAID
-	default "0x2" if SB800_SATA_AHCI
+	default 0x0 if SB800_SATA_IDE
+	default 0x1 if SB800_SATA_RAID
+	default 0x2 if SB800_SATA_AHCI
 
 config SB_SUPERIO_HWM
 	bool