mb/lenovo/t420(s): Do minor cosmetic changes

Align the whitespace and do some cosmetic changes. This makes it easier
to fold these two boards into a variant setup.

Change-Id: I53bdd90ae47b52dfdfec27229c6b904487fa2081
Signed-off-by: Peter Lemenkov <lemenkov@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/40380
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

5 files changed, 11 insertions, 8 deletions

diff --git a/src/mainboard/lenovo/t420s/Kconfig b/src/mainboard/lenovo/t420s/Kconfig
index f29d50fe96..e2fd824f33 100644
--- a/src/mainboard/lenovo/t420s/Kconfig
+++ b/src/mainboard/lenovo/t420s/Kconfig
@@ -16,10 +16,10 @@ config BOARD_SPECIFIC_OPTIONS
 	select HAVE_CMOS_DEFAULT
 	select HAVE_ACPI_RESUME
 	select INTEL_INT15
-	select MAINBOARD_HAS_LIBGFXINIT
-	select GFX_GMA_PANEL_1_ON_LVDS
 	select MAINBOARD_HAS_LPC_TPM
 	select MAINBOARD_HAS_TPM1
+	select MAINBOARD_HAS_LIBGFXINIT
+	select GFX_GMA_PANEL_1_ON_LVDS
 	select DRIVERS_LENOVO_HYBRID_GRAPHICS
 	select INTEL_GMA_HAVE_VBT
 	select MAINBOARD_USES_IFD_GBE_REGION
diff --git a/src/mainboard/lenovo/t420s/devicetree.cb b/src/mainboard/lenovo/t420s/devicetree.cb
index 1f794c1b24..b3399c32a3 100644
--- a/src/mainboard/lenovo/t420s/devicetree.cb
+++ b/src/mainboard/lenovo/t420s/devicetree.cb
@@ -1,4 +1,5 @@
 chip northbridge/intel/sandybridge
+	# IGD Displays
 	register "gfx" = "GMA_STATIC_DISPLAYS(1)"
 
 	# Enable DisplayPort Hotplug with 6ms pulse
@@ -66,6 +67,7 @@ chip northbridge/intel/sandybridge
 
 			register "c2_latency" = "101"  # c2 not supported
 
+			# device specific SPI configuration
 			register "spi_uvscc" = "0x2005"
 			register "spi_lvscc" = "0x2005"
 
diff --git a/src/mainboard/lenovo/t420s/early_init.c b/src/mainboard/lenovo/t420s/early_init.c
index 212be38fec..4f5f69d4dc 100644
--- a/src/mainboard/lenovo/t420s/early_init.c
+++ b/src/mainboard/lenovo/t420s/early_init.c
@@ -35,8 +35,8 @@ static void hybrid_graphics_init(void)
 }
 
 const struct southbridge_usb_port mainboard_usb_ports[] = {
-	{ 0, 1, -1 }, /* P0 empty */
-	{ 1, 1,  1 }, /* P1 system port 2 (To system port) (EHCI debug), OC 1 */
+	{ 0, 1, -1 }, /* P0: empty */
+	{ 1, 1,  1 }, /* P1: system port 2 (To system port) (EHCI debug), OC 1 */
 	{ 1, 1, -1 }, /* P2: HALF MINICARD (WLAN) no oc */
 	{ 1, 0, -1 }, /* P3: WWAN, no OC */
 	{ 1, 1, -1 }, /* P4: smartcard, no OC */
diff --git a/src/mainboard/lenovo/t420s/gpio.c b/src/mainboard/lenovo/t420s/gpio.c
index 0b64efe268..daca020da0 100644
--- a/src/mainboard/lenovo/t420s/gpio.c
+++ b/src/mainboard/lenovo/t420s/gpio.c
@@ -2,6 +2,7 @@
 /* This file is part of the coreboot project. */
 
 #include <southbridge/intel/common/gpio.h>
+
 static const struct pch_gpio_set1 pch_gpio_set1_mode = {
 	.gpio0  = GPIO_MODE_GPIO,   // -USB30_SMIB - input
 	.gpio1  = GPIO_MODE_GPIO,   // -EC_SCI - input
@@ -69,7 +70,7 @@ static const struct pch_gpio_set1 pch_gpio_set1_direction = {
 	.gpio28 = GPIO_DIR_OUTPUT,
 	.gpio29 = GPIO_DIR_OUTPUT,
 	.gpio30 = GPIO_DIR_OUTPUT,
-	.gpio31 = GPIO_DIR_INPUT
+	.gpio31 = GPIO_DIR_INPUT,
 };
 
 static const struct pch_gpio_set1 pch_gpio_set1_level = {
@@ -108,8 +109,8 @@ static const struct pch_gpio_set1 pch_gpio_set1_level = {
 };
 
 static const struct pch_gpio_set1 pch_gpio_set1_invert = {
-	.gpio0 = GPIO_INVERT,
-	.gpio1 = GPIO_INVERT,
+	.gpio0  = GPIO_INVERT,
+	.gpio1  = GPIO_INVERT,
 	.gpio13 = GPIO_INVERT,
 };
 
diff --git a/src/mainboard/lenovo/t420s/smihandler.c b/src/mainboard/lenovo/t420s/smihandler.c
index 84b9c3a732..443299573a 100644
--- a/src/mainboard/lenovo/t420s/smihandler.c
+++ b/src/mainboard/lenovo/t420s/smihandler.c
@@ -6,8 +6,8 @@
 #include <cpu/x86/smm.h>
 #include <ec/acpi/ec.h>
 #include <ec/lenovo/h8/h8.h>
-#include <southbridge/intel/common/pmutil.h>
 #include <southbridge/intel/bd82x6x/pch.h>
+#include <southbridge/intel/common/pmutil.h>
 
 #define GPE_EC_SCI	1
 #define GPE_EC_WAKE	13