mainboard/intel: Fix checkpatch errors in minnowmax

This fixes the following issues, with no functional changes:

ERROR:POINTER_LOCATION: "foo * bar" should be "foo *bar"
ERROR:SPACING: space required after that ',' (ctx:VxV)
WARNING:LONG_LINE_COMMENT: line over 80 characters
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
ERROR:FUNCTION_WITHOUT_ARGS: Bad function definition
ERROR:SPACING: space prohibited before that close parenthesis ')'
WARNING:RETURN_VOID: void function return statements are not generally
useful

2 unfixed issues:
ERROR:COMPLEX_MACRO: Macros with complex values should be enclosed in
parentheses

Verified that the binary was the same before and after the changes.

Change-Id: Ie9afb50e268f4140872e39fe8bede231a43d5cc6
Signed-off-by: Martin Roth <gaumless@gmail.com>
Reviewed-on: https://review.coreboot.org/19078
Tested-by: build bot (Jenkins)
Reviewed-by: Werner Zeh <werner.zeh@siemens.com>
Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar@intel.com>
Reviewed-by: Philippe Mathieu-Daudé <philippe.mathieu.daude@gmail.com>
Reviewed-by: York Yang <york.yang@intel.com>

4 files changed, 197 insertions, 198 deletions

diff --git a/src/mainboard/intel/minnowmax/fadt.c b/src/mainboard/intel/minnowmax/fadt.c
index 36e4655a32..cd8eefdd67 100644
--- a/src/mainboard/intel/minnowmax/fadt.c
+++ b/src/mainboard/intel/minnowmax/fadt.c
@@ -17,11 +17,11 @@
 #include <arch/acpi.h>
 #include <soc/acpi.h>
 
-void acpi_create_fadt(acpi_fadt_t * fadt, acpi_facs_t * facs, void *dsdt)
+void acpi_create_fadt(acpi_fadt_t *fadt, acpi_facs_t *facs, void *dsdt)
 {
 	acpi_header_t *header = &(fadt->header);
 
-	acpi_fill_in_fadt(fadt,facs,dsdt);
+	acpi_fill_in_fadt(fadt, facs, dsdt);
 
 	/* Platform specific customizations go here */
 
diff --git a/src/mainboard/intel/minnowmax/gpio.c b/src/mainboard/intel/minnowmax/gpio.c
index d0f1b1fb55..b0f78c8b57 100644
--- a/src/mainboard/intel/minnowmax/gpio.c
+++ b/src/mainboard/intel/minnowmax/gpio.c
@@ -31,189 +31,190 @@
 
 /* NCORE GPIOs */
 static const struct soc_gpio_map gpncore_gpio_map[] = {
-	GPIO_FUNC2,				/* GPIO_S0_NC[00] - HDMI_HPD */
-	GPIO_FUNC2,				/* GPIO_S0_NC[01] - HDMI_DDCDAT */
-	GPIO_FUNC2,				/* GPIO_S0_NC[02] - HDMI_DDCCLK */
-	GPIO_NC,				/* GPIO_S0_NC[03] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[04] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[05] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[06] - No Connect */
-	GPIO_FUNC2,				/* GPIO_S0_NC[07] - DDI1_DDCDAT */
-	GPIO_NC,				/* GPIO_S0_NC[08] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[09] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[10] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[11] - No Connect */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S0_NC[12] - TP15 */
-	GPIO_NC,				/* GPIO_S0_NC[13] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[14] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[15] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[16] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[17] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[18] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[19] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[20] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[21] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[22] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[23] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[24] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[25] - No Connect */
-	GPIO_NC,				/* GPIO_S0_NC[26] - No Connect */
+	GPIO_FUNC2,			// GPIO_S0_NC[00] - HDMI_HPD
+	GPIO_FUNC2,			// GPIO_S0_NC[01] - HDMI_DDCDAT
+	GPIO_FUNC2,			// GPIO_S0_NC[02] - HDMI_DDCCLK
+	GPIO_NC,			// GPIO_S0_NC[03] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[04] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[05] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[06] - No Connect
+	GPIO_FUNC2,			// GPIO_S0_NC[07] - DDI1_DDCDAT
+	GPIO_NC,			// GPIO_S0_NC[08] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[09] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[10] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[11] - No Connect
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S0_NC[12] - TP15
+	GPIO_NC,			// GPIO_S0_NC[13] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[14] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[15] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[16] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[17] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[18] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[19] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[20] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[21] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[22] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[23] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[24] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[25] - No Connect
+	GPIO_NC,			// GPIO_S0_NC[26] - No Connect
 	GPIO_END
 };
 
-/* SCORE GPIOs (GPIO_S0_SC_XX)*/
+/* SCORE GPIOs (GPIO_S0_SC_XX) */
 static const struct soc_gpio_map gpscore_gpio_map[] = {
-	GPIO_FUNC1,				/* GPIO_S0_SC[000] - SATA_GP0 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[001] - SATA_GP1 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[002] - SATA_LED_B */
-	GPIO_FUNC1,				/* GPIO_S0_SC[003] - PCIE_CLKREQ_0 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[004] - PCIE_CLKREQ_1 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[005] - PCIE_CLKREQ_2 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[006] - PCIE_CLKREQ_3 */
-	GPIO_FUNC2,				/* GPIO_S0_SC[007] - SD3_WP */
-	GPIO_NC,				/* GPIO_S0_SC[008] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[009] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[010] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[011] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[012] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[013] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[014] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[015] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[016] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[017] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[018] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[019] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[020] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[021] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[022] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[023] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[024] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[025] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[026] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[027] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[028] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[029] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[030] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[031] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[032] - No Connect */
-	GPIO_FUNC1,				/* GPIO_S0_SC[033] - SD3_CLK */
-	GPIO_FUNC1,				/* GPIO_S0_SC[034] - SD3_D0 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[035] - SD3_D1 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[036] - SD3_D2 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[037] - SD3_D3 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[038] - SD3_CD# */
-	GPIO_FUNC1,				/* GPIO_S0_SC[039] - SD3_CMD */
-	GPIO_FUNC1,				/* GPIO_S0_SC[040] - TP12 (SD3_1P8EN) */
-	GPIO_FUNC1,				/* GPIO_S0_SC[041] - TP11 (/SD3_PWREN) */
-	GPIO_NC,				/* GPIO_S0_SC[042] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[043] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[044] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[045] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[046] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[047] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[048] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[049] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[050] - No Connect */
-	GPIO_FUNC1,				/* GPIO_S0_SC[051] - PCU_SMB_DATA */
-	GPIO_FUNC1,				/* GPIO_S0_SC[052] - PCU_SMB_CLK */
-	GPIO_FUNC1,				/* GPIO_S0_SC[053] - PCU_SMB_ALERT */
-	GPIO_FUNC1,				/* GPIO_S0_SC[054] - ILB_8254_SPKR */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S0_SC[055] - TP8 (GPIO_S0_SC_55) */
-	GPIO_FUNC0,				/* GPIO_S0_SC[056] - GPIO_S0_SC_56 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[057] - PCU_UART3_TXD */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S0_SC[058] - TP9 (GPIO_S0_SC_58) */
-	GPIO_FUNC0,				/* GPIO_S0_SC[059] - HDMI_DCDC_ENB */
-	GPIO_FUNC0,				/* GPIO_S0_SC[060] - HDMI_LDSW_ENB */
-	GPIO_FUNC1,				/* GPIO_S0_SC[061] - PCU_UART3_RXD */
-	GPIO_FUNC1,				/* GPIO_S0_SC[062] - LPE_I2S_CLK */
-	GPIO_FUNC1,				/* GPIO_S0_SC[063] - LPE_I2S_FRM */
-	GPIO_FUNC1,				/* GPIO_S0_SC[064] - LPE_I2S_DATIN */
-	GPIO_FUNC1,				/* GPIO_S0_SC[065] - LPE_I2S_DATOUT */
-	GPIO_FUNC1,				/* GPIO_S0_SC[066] - SOC_SIO_SPI_CS1 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[067] - SOC_SIO_SPI_MISO */
-	GPIO_FUNC1,				/* GPIO_S0_SC[068] - SOC_SIO_SPI_MOSI */
-	GPIO_FUNC1,				/* GPIO_S0_SC[069] - SOC_SIO_SPI_CLK */
-	GPIO_FUNC1,				/* GPIO_S0_SC[070] - SIO_UART1_RXD */
-	GPIO_FUNC1,				/* GPIO_S0_SC[071] - SIO_UART1_TXD */
-	GPIO_FUNC1,				/* GPIO_S0_SC[072] - SIO_UART1_RTSB */
-	GPIO_FUNC1,				/* GPIO_S0_SC[073] - SIO_UART1_CTSB */
-	GPIO_FUNC1,				/* GPIO_S0_SC[074] - SIO_UART2_RXD */
-	GPIO_FUNC1,				/* GPIO_S0_SC[075] - SIO_UART2_TXD */
-	GPIO_NC,				/* GPIO_S0_SC[076] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[077] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[078] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[079] - No Connect */
-	GPIO_FUNC1,				/* GPIO_S0_SC[080] - TP6 (SIO_I2C1_SDA) */
-	GPIO_FUNC1,				/* GPIO_S0_SC[081] - TP5 (SIO_I2C1_SCL) */
-	GPIO_NC,				/* GPIO_S0_SC[082] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[083] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[084] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[085] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[086] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[087] - No Connect */
-	GPIO_FUNC1,				/* GPIO_S0_SC[088] - LSS_I2C_SDA */
-	GPIO_FUNC1,				/* GPIO_S0_SC[089] - LSS_I2C_SCL */
-	GPIO_FUNC1,				/* GPIO_S0_SC[090] - EXP_I2C_SDA */
-	GPIO_FUNC1,				/* GPIO_S0_SC[091] - EXP_I2C_SCL */
-	GPIO_FUNC(1, PULL_UP, 20K),		/* GPIO_S0_SC[092] - TP13 */
-	GPIO_FUNC(1, PULL_UP, 20K),		/* GPIO_S0_SC[093] - TP16 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[094] - SOC_PWM0 */
-	GPIO_FUNC1,				/* GPIO_S0_SC[095] - SOC_PWM1 */
-	GPIO_NC,				/* GPIO_S0_SC[096] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[097] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[098] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[099] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[100] - No Connect */
-	GPIO_NC,				/* GPIO_S0_SC[101] - No Connect */
+	GPIO_FUNC1,			// GPIO_S0_SC[000] - SATA_GP0
+	GPIO_FUNC1,			// GPIO_S0_SC[001] - SATA_GP1
+	GPIO_FUNC1,			// GPIO_S0_SC[002] - SATA_LED_B
+	GPIO_FUNC1,			// GPIO_S0_SC[003] - PCIE_CLKREQ_0
+	GPIO_FUNC1,			// GPIO_S0_SC[004] - PCIE_CLKREQ_1
+	GPIO_FUNC1,			// GPIO_S0_SC[005] - PCIE_CLKREQ_2
+	GPIO_FUNC1,			// GPIO_S0_SC[006] - PCIE_CLKREQ_3
+	GPIO_FUNC2,			// GPIO_S0_SC[007] - SD3_WP
+	GPIO_NC,			// GPIO_S0_SC[008] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[009] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[010] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[011] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[012] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[013] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[014] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[015] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[016] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[017] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[018] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[019] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[020] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[021] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[022] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[023] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[024] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[025] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[026] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[027] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[028] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[029] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[030] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[031] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[032] - No Connect
+	GPIO_FUNC1,			// GPIO_S0_SC[033] - SD3_CLK
+	GPIO_FUNC1,			// GPIO_S0_SC[034] - SD3_D0
+	GPIO_FUNC1,			// GPIO_S0_SC[035] - SD3_D1
+	GPIO_FUNC1,			// GPIO_S0_SC[036] - SD3_D2
+	GPIO_FUNC1,			// GPIO_S0_SC[037] - SD3_D3
+	GPIO_FUNC1,			// GPIO_S0_SC[038] - SD3_CD#
+	GPIO_FUNC1,			// GPIO_S0_SC[039] - SD3_CMD
+	GPIO_FUNC1,			// GPIO_S0_SC[040] - TP12 (SD3_1P8EN)
+	GPIO_FUNC1,			// GPIO_S0_SC[041] - TP11 (/SD3_PWREN)
+	GPIO_NC,			// GPIO_S0_SC[042] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[043] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[044] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[045] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[046] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[047] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[048] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[049] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[050] - No Connect
+	GPIO_FUNC1,			// GPIO_S0_SC[051] - PCU_SMB_DATA
+	GPIO_FUNC1,			// GPIO_S0_SC[052] - PCU_SMB_CLK
+	GPIO_FUNC1,			// GPIO_S0_SC[053] - PCU_SMB_ALERT
+	GPIO_FUNC1,			// GPIO_S0_SC[054] - ILB_8254_SPKR
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S0_SC[055] - TP8 (GPIO_S0_SC_55)
+	GPIO_FUNC0,			// GPIO_S0_SC[056] - GPIO_S0_SC_56
+	GPIO_FUNC1,			// GPIO_S0_SC[057] - PCU_UART3_TXD
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S0_SC[058] - TP9 (GPIO_S0_SC_58)
+	GPIO_FUNC0,			// GPIO_S0_SC[059] - HDMI_DCDC_ENB
+	GPIO_FUNC0,			// GPIO_S0_SC[060] - HDMI_LDSW_ENB
+	GPIO_FUNC1,			// GPIO_S0_SC[061] - PCU_UART3_RXD
+	GPIO_FUNC1,			// GPIO_S0_SC[062] - LPE_I2S_CLK
+	GPIO_FUNC1,			// GPIO_S0_SC[063] - LPE_I2S_FRM
+	GPIO_FUNC1,			// GPIO_S0_SC[064] - LPE_I2S_DATIN
+	GPIO_FUNC1,			// GPIO_S0_SC[065] - LPE_I2S_DATOUT
+	GPIO_FUNC1,			// GPIO_S0_SC[066] - SOC_SIO_SPI_CS1
+	GPIO_FUNC1,			// GPIO_S0_SC[067] - SOC_SIO_SPI_MISO
+	GPIO_FUNC1,			// GPIO_S0_SC[068] - SOC_SIO_SPI_MOSI
+	GPIO_FUNC1,			// GPIO_S0_SC[069] - SOC_SIO_SPI_CLK
+	GPIO_FUNC1,			// GPIO_S0_SC[070] - SIO_UART1_RXD
+	GPIO_FUNC1,			// GPIO_S0_SC[071] - SIO_UART1_TXD
+	GPIO_FUNC1,			// GPIO_S0_SC[072] - SIO_UART1_RTSB
+	GPIO_FUNC1,			// GPIO_S0_SC[073] - SIO_UART1_CTSB
+	GPIO_FUNC1,			// GPIO_S0_SC[074] - SIO_UART2_RXD
+	GPIO_FUNC1,			// GPIO_S0_SC[075] - SIO_UART2_TXD
+	GPIO_NC,			// GPIO_S0_SC[076] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[077] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[078] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[079] - No Connect
+	GPIO_FUNC1,			// GPIO_S0_SC[080] - TP6 (SIO_I2C1_SDA)
+	GPIO_FUNC1,			// GPIO_S0_SC[081] - TP5 (SIO_I2C1_SCL)
+	GPIO_NC,			// GPIO_S0_SC[082] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[083] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[084] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[085] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[086] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[087] - No Connect
+	GPIO_FUNC1,			// GPIO_S0_SC[088] - LSS_I2C_SDA
+	GPIO_FUNC1,			// GPIO_S0_SC[089] - LSS_I2C_SCL
+	GPIO_FUNC1,			// GPIO_S0_SC[090] - EXP_I2C_SDA
+	GPIO_FUNC1,			// GPIO_S0_SC[091] - EXP_I2C_SCL
+	GPIO_FUNC(1, PULL_UP, 20K),	// GPIO_S0_SC[092] - TP13
+	GPIO_FUNC(1, PULL_UP, 20K),	// GPIO_S0_SC[093] - TP16
+	GPIO_FUNC1,			// GPIO_S0_SC[094] - SOC_PWM0
+	GPIO_FUNC1,			// GPIO_S0_SC[095] - SOC_PWM1
+	GPIO_NC,			// GPIO_S0_SC[096] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[097] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[098] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[099] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[100] - No Connect
+	GPIO_NC,			// GPIO_S0_SC[101] - No Connect
 	GPIO_END
 };
 
 /* SSUS GPIOs (GPIO_S5) */
 static const struct soc_gpio_map gpssus_gpio_map[] = {
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[00] - SOC_GPIO_S5_0 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[01] - SOC_GPIO_S5_1 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[02] - SOC_GPIO_S5_2 */
-	GPIO_FUNC6,				/* GPIO_S5[03] - mPCIE_WAKEB */
-	GPIO_NC,				/* GPIO_S5[04] - No Connect */
-	GPIO_INPUT,				/* GPIO_S5[05] - BOM_OP1 - Memory: 0=1GB 1=2GB or 4GB*/
-	GPIO_INPUT,				/* GPIO_S5[06] - BOM_OP2 */
-	GPIO_INPUT,				/* GPIO_S5[07] - BOM_OP3 */
-	GPIO_OUT_HIGH_LEGACY,			/* GPIO_S5[08] - SOC_USB_HOST_EN0 */
-	GPIO_OUT_HIGH_LEGACY,			/* GPIO_S5[09] - SOC_USB_HOST_EN1 */
-	GPIO_OUT_HIGH_LEGACY,			/* GPIO_S5[10] - GPIO_S5_10_UNLOCK */
-	GPIO_FUNC0,				/* GPIO_S5[11] - SUSPWRDNACK (TP14) */
-	GPIO_FUNC0,				/* GPIO_S5[12] - PMC_SUSCLK0 */
-	GPIO_FUNC1,				/* GPIO_S5[13] - PMC_SLP_S0IX (TP10) */
-	GPIO_FUNC1,				/* GPIO_S5[14] - GPIO_S514_J20 */
-	GPIO_FUNC0,				/* GPIO_S5[15] - PMC_PCIE_WAKE_R */
-	GPIO_FUNC0,				/* GPIO_S5[16] - PMC_PWRBTN */
-	GPIO_NC,				/* GPIO_S5[17] - No Connect */
-	GPIO_FUNC1,				/* GPIO_S5[18] - LPCPD_L (TP7) */
-	GPIO_FUNC0,				/* GPIO_S5[19] - SOC_USB_HOST_OC0 */
-	GPIO_FUNC0,				/* GPIO_S5[20] - SOC_USB_HOST_OC1 */
-	GPIO_FUNC0,				/* GPIO_S5[21] - SOC_SPI_CS1B */
-	GPIO_INPUT_PD,				/* GPIO_S5[22] - NC or LED D2 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[23] - XDP_H_OBSDATA_A0 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[24] - XDP_H_OBSDATA_A1 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[25] - XDP_H_OBSDATA_A2 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[26] - XDP_H_OBSDATA_A3 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[27] - EXP_GPIO1 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[28] - EXP_GPIO2 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[29] - EXP_GPIO3 */
-	GPIO_FUNC(0, PULL_UP, 20K),		/* GPIO_S5[30] - EXP_GPIO4 */
-	GPIO_NC,				/* GPIO_S5[31] - No Connect */
-	GPIO_NC,				/* GPIO_S5[32] - No Connect */
-	GPIO_NC,				/* GPIO_S5[33] - No Connect */
-	GPIO_NC,				/* GPIO_S5[34] - No Connect */
-	GPIO_NC,				/* GPIO_S5[35] - No Connect */
-	GPIO_NC,				/* GPIO_S5[36] - No Connect */
-	GPIO_NC,				/* GPIO_S5[37] - No Connect */
-	GPIO_NC,				/* GPIO_S5[38] - No Connect */
-	GPIO_NC,				/* GPIO_S5[39] - No Connect */
-	GPIO_NC,				/* GPIO_S5[40] - No Connect */
-	GPIO_NC,				/* GPIO_S5[41] - No Connect */
-	GPIO_NC,				/* GPIO_S5[42] - No Connect */
-	GPIO_NC,				/* GPIO_S5[43] - No Connect */
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[00] - SOC_GPIO_S5_0
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[01] - SOC_GPIO_S5_1
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[02] - SOC_GPIO_S5_2
+	GPIO_FUNC6,			// GPIO_S5[03] - mPCIE_WAKEB
+	GPIO_NC,			// GPIO_S5[04] - No Connect
+	GPIO_INPUT,			// GPIO_S5[05] - BOM_OP1
+					// Memory: 0=1GB 1=2GB or 4GB
+	GPIO_INPUT,			// GPIO_S5[06] - BOM_OP2
+	GPIO_INPUT,			// GPIO_S5[07] - BOM_OP3
+	GPIO_OUT_HIGH_LEGACY,		// GPIO_S5[08] - SOC_USB_HOST_EN0
+	GPIO_OUT_HIGH_LEGACY,		// GPIO_S5[09] - SOC_USB_HOST_EN1
+	GPIO_OUT_HIGH_LEGACY,		// GPIO_S5[10] - GPIO_S5_10_UNLOCK
+	GPIO_FUNC0,			// GPIO_S5[11] - SUSPWRDNACK (TP14)
+	GPIO_FUNC0,			// GPIO_S5[12] - PMC_SUSCLK0
+	GPIO_FUNC1,			// GPIO_S5[13] - PMC_SLP_S0IX (TP10)
+	GPIO_FUNC1,			// GPIO_S5[14] - GPIO_S514_J20
+	GPIO_FUNC0,			// GPIO_S5[15] - PMC_PCIE_WAKE_R
+	GPIO_FUNC0,			// GPIO_S5[16] - PMC_PWRBTN
+	GPIO_NC,			// GPIO_S5[17] - No Connect
+	GPIO_FUNC1,			// GPIO_S5[18] - LPCPD_L (TP7)
+	GPIO_FUNC0,			// GPIO_S5[19] - SOC_USB_HOST_OC0
+	GPIO_FUNC0,			// GPIO_S5[20] - SOC_USB_HOST_OC1
+	GPIO_FUNC0,			// GPIO_S5[21] - SOC_SPI_CS1B
+	GPIO_INPUT_PD,			// GPIO_S5[22] - NC or LED D2
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[23] - XDP_H_OBSDATA_A0
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[24] - XDP_H_OBSDATA_A1
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[25] - XDP_H_OBSDATA_A2
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[26] - XDP_H_OBSDATA_A3
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[27] - EXP_GPIO1
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[28] - EXP_GPIO2
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[29] - EXP_GPIO3
+	GPIO_FUNC(0, PULL_UP, 20K),	// GPIO_S5[30] - EXP_GPIO4
+	GPIO_NC,			// GPIO_S5[31] - No Connect
+	GPIO_NC,			// GPIO_S5[32] - No Connect
+	GPIO_NC,			// GPIO_S5[33] - No Connect
+	GPIO_NC,			// GPIO_S5[34] - No Connect
+	GPIO_NC,			// GPIO_S5[35] - No Connect
+	GPIO_NC,			// GPIO_S5[36] - No Connect
+	GPIO_NC,			// GPIO_S5[37] - No Connect
+	GPIO_NC,			// GPIO_S5[38] - No Connect
+	GPIO_NC,			// GPIO_S5[39] - No Connect
+	GPIO_NC,			// GPIO_S5[40] - No Connect
+	GPIO_NC,			// GPIO_S5[41] - No Connect
+	GPIO_NC,			// GPIO_S5[42] - No Connect
+	GPIO_NC,			// GPIO_S5[43] - No Connect
 	GPIO_END
 };
 
@@ -225,7 +226,7 @@ static struct soc_gpio_config gpio_config = {
 	.sus_dirq = NULL,
 };
 
-struct soc_gpio_config* mainboard_get_gpios(void)
+struct soc_gpio_config *mainboard_get_gpios(void)
 {
 	return &gpio_config;
 }
diff --git a/src/mainboard/intel/minnowmax/irqroute.h b/src/mainboard/intel/minnowmax/irqroute.h
index 0b194aa10d..20281b7e3f 100644
--- a/src/mainboard/intel/minnowmax/irqroute.h
+++ b/src/mainboard/intel/minnowmax/irqroute.h
@@ -21,25 +21,25 @@
 #include <soc/intel/fsp_baytrail/include/soc/pci_devs.h>
 
 /*
- *IR02h GFX      INT(A) 	- PIRQ A
- *IR10h EMMC	 INT(ABCD)	- PIRQ DEFG
- *IR11h SDIO     INT(A) 	- PIRQ B
- *IR12h SD       INT(A) 	- PIRQ C
- *IR13h SATA     INT(A) 	- PIRQ D
- *IR14h XHCI     INT(A) 	- PIRQ E
- *IR15h LP Audio INT(A) 	- PIRQ F
- *IR17h MMC      INT(A) 	- PIRQ F
- *IR18h SIO      INT(ABCD)	- PIRQ BADC
- *IR1Ah TXE      INT(A)		- PIRQ F
- *IR1Bh HD Audio INT(A)		- PIRQ G
- *IR1Ch PCIe     INT(ABCD)	- PIRQ EFGH
- *IR1Dh EHCI     INT(A)		- PIRQ D
- *IR1Eh SIO      INT(ABCD)	- PIRQ BDEF
- *IR1Fh LPC      INT(ABCD)	- PIRQ HGBC
+ *IR02h GFX      INT(A)     - PIRQ A
+ *IR10h EMMC     INT(ABCD)  - PIRQ DEFG
+ *IR11h SDIO     INT(A)     - PIRQ B
+ *IR12h SD       INT(A)     - PIRQ C
+ *IR13h SATA     INT(A)     - PIRQ D
+ *IR14h XHCI     INT(A)     - PIRQ E
+ *IR15h LP Audio INT(A)     - PIRQ F
+ *IR17h MMC      INT(A)     - PIRQ F
+ *IR18h SIO      INT(ABCD)  - PIRQ BADC
+ *IR1Ah TXE      INT(A)     - PIRQ F
+ *IR1Bh HD Audio INT(A)     - PIRQ G
+ *IR1Ch PCIe     INT(ABCD)  - PIRQ EFGH
+ *IR1Dh EHCI     INT(A)     - PIRQ D
+ *IR1Eh SIO      INT(ABCD)  - PIRQ BDEF
+ *IR1Fh LPC      INT(ABCD)  - PIRQ HGBC
  */
 
 /* PCIe bridge routing */
-#define BRIDGE1_DEV	PCIE_DEV
+#define BRIDGE1_DEV PCIE_DEV
 
 /* PCI bridge IRQs need to be updated in both tables and need to match */
 #define PCIE_BRIDGE_IRQ_ROUTES \
diff --git a/src/mainboard/intel/minnowmax/romstage.c b/src/mainboard/intel/minnowmax/romstage.c
index 7ee4f216e0..6f40207ffc 100644
--- a/src/mainboard/intel/minnowmax/romstage.c
+++ b/src/mainboard/intel/minnowmax/romstage.c
@@ -26,7 +26,7 @@
  * /brief mainboard call for setup that needs to be done before fsp init
  *
  */
-void early_mainboard_romstage_entry()
+void early_mainboard_romstage_entry(void)
 {
 
 }
@@ -45,7 +45,7 @@ void get_func_disables(uint32_t *fd_mask, uint32_t *fd2_mask)
  * /brief mainboard call for setup that needs to be done after fsp init
  *
  */
-void late_mainboard_romstage_entry()
+void late_mainboard_romstage_entry(void)
 {
 
 }
@@ -70,7 +70,7 @@ void romstage_fsp_rt_buffer_callback(FSP_INIT_RT_BUFFER *FspRtBuffer)
 		UpdData->PcdMemoryParameters.DIMMDensity
 		+= (DIMM_DENSITY_4G_BIT - DIMM_DENSITY_2G_BIT);
 	printk(BIOS_NOTICE, "%s GB Minnowboard Max detected.\n",
-			gpio5 ? "2 / 4" : "1" );
+			gpio5 ? "2 / 4" : "1");
 	/* Update XHCI UPD value if required */
 	get_option(&use_xhci, "use_xhci_over_ehci");
 	if ((use_xhci < 2) && (use_xhci != UpdData->PcdEnableXhci)) {
@@ -78,6 +78,4 @@ void romstage_fsp_rt_buffer_callback(FSP_INIT_RT_BUFFER *FspRtBuffer)
 		printk(FSP_INFO_LEVEL, "Xhci updated from CMOS:\t\t\t%s\n",
 			UpdData->PcdEnableXhci?"Enabled":"Disabled");
 	}
-
-	return;
 }