[coreboot-gerrit] New patch to review for coreboot: 45fe5b4 fsp_baytrail: update printk to use FSP_INFO_LEVEL

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Wed Nov 19 03:56:07 CET 2014


Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/7520

-gerrit

commit 45fe5b43f3dddbaa6e1068b0a3a94710dce1e27e
Author: Martin Roth <martin.roth at se-eng.com>
Date:   Tue Nov 18 19:52:46 2014 -0700

    fsp_baytrail: update printk to use FSP_INFO_LEVEL
    
    Update the printk statements to use FSP_INFO_LEVEL instead of
    BIOS_DEBUG.  These values are currently identical, but by using the
    second #define, it lets them all be changed as a unit.  This can
    be overridden for a particular platform by adding a #define in
    chipset_fsp_util.c.
    
    Change-Id: Idbf7e55090230ec940c7c8cd3ec8632461561428
    Signed-off-by: Martin Roth <martin.roth at se-eng.com>
---
 src/soc/intel/fsp_baytrail/fsp/chipset_fsp_util.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/soc/intel/fsp_baytrail/fsp/chipset_fsp_util.c b/src/soc/intel/fsp_baytrail/fsp/chipset_fsp_util.c
index a3396eb..2b45c73 100644
--- a/src/soc/intel/fsp_baytrail/fsp/chipset_fsp_util.c
+++ b/src/soc/intel/fsp_baytrail/fsp/chipset_fsp_util.c
@@ -87,7 +87,7 @@ static void ConfigureDefaultUpdData(UPD_DATA_REGION *UpdData)
 {
 	ROMSTAGE_CONST struct device *dev;
 	ROMSTAGE_CONST config_t *config;
-	printk(BIOS_DEBUG, "Configure Default UPD Data\n");
+	printk(FSP_INFO_LEVEL, "Configure Default UPD Data\n");
 
 	dev = dev_find_slot(0, SOC_DEV_FUNC);
 	config = dev->chip_info;
@@ -112,13 +112,13 @@ static void ConfigureDefaultUpdData(UPD_DATA_REGION *UpdData)
 
 	UpdData->PcdMrcInitTsegSize = smm_region_size() >> 20;
 
-	printk(BIOS_DEBUG, "GTT Size:\t\t%d MB\n", UpdData->PcdGttSize);
-	printk(BIOS_DEBUG, "Tseg Size:\t\t%d MB\n", UpdData->PcdMrcInitTsegSize);
-	printk(BIOS_DEBUG, "Aperture Size:\t\t%d MB\n",
+	printk(FSP_INFO_LEVEL, "GTT Size:\t\t%d MB\n", UpdData->PcdGttSize);
+	printk(FSP_INFO_LEVEL, "Tseg Size:\t\t%d MB\n", UpdData->PcdMrcInitTsegSize);
+	printk(FSP_INFO_LEVEL, "Aperture Size:\t\t%d MB\n",
 		APERTURE_SIZE_BASE << UpdData->PcdApertureSize);
-	printk(BIOS_DEBUG, "IGD Memory Size:\t%d MB\n",
+	printk(FSP_INFO_LEVEL, "IGD Memory Size:\t%d MB\n",
 		UpdData->PcdIgdDvmt50PreAlloc * IGD_MEMSIZE_MULTIPLIER);
-	printk(BIOS_DEBUG, "MMIO Size:\t\t%d MB\n", UpdData->PcdMrcInitMmioSize);
+	printk(FSP_INFO_LEVEL, "MMIO Size:\t\t%d MB\n", UpdData->PcdMrcInitMmioSize);
 
 	/* Advance dev to PCI device 0.0 */
 	for (dev = &dev_root; dev; dev = dev_find_next_pci_device(dev)){
@@ -190,18 +190,18 @@ static void ConfigureDefaultUpdData(UPD_DATA_REGION *UpdData)
 	}
 
 	if (UpdData->PcdEnableLpe < sizeof(acpi_pci_mode_strings) / sizeof (char *))
-		printk(BIOS_DEBUG, "Lpe:\t\t\t%s\n",
+		printk(FSP_INFO_LEVEL, "Lpe:\t\t\t%s\n",
 			acpi_pci_mode_strings[UpdData->PcdEnableLpe]);
 
 	if (UpdData->PcdeMMCBootMode < sizeof(emmc_mode_strings) / sizeof (char *))
-		printk(BIOS_DEBUG, "eMMC Mode:\t\t%s",
+		printk(FSP_INFO_LEVEL, "eMMC Mode:\t\t%s",
 			emmc_mode_strings[UpdData->PcdeMMCBootMode]);
 
 	if (UpdData->PcdEnableSata)
-		printk(BIOS_DEBUG, "SATA Mode:\t\t%s\n",
+		printk(FSP_INFO_LEVEL, "SATA Mode:\t\t%s\n",
 			UpdData->PcdSataMode?"AHCI":"IDE");
 
-	printk(BIOS_DEBUG, "Xhci:\t\t\t%s\n",
+	printk(FSP_INFO_LEVEL, "Xhci:\t\t\t%s\n",
 		UpdData->PcdEnableXhci?"Enabled":"Disabled");
 
 }



More information about the coreboot-gerrit mailing list