GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

lynxpoint: me: Disable some verbose messages 

The default ME output is quite verbose and not all that useful
unless you are actively debugging the ME and then you can enable
the CONFIG_DEBUG_INTEL_ME option.

This commit silences the firmware capabilities and the MBP output.

Change-Id: I2b8abcb34ae0d00d9a38d029979e84ee0d0ca287
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/65252
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: http://review.coreboot.org/4452
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
This commit is contained in:
Duncan Laurie 2013-08-08 15:40:01 -07:00 committed by Patrick Georgi
parent c45dc1c081
commit 0b3cd36061
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/southbridge/intel/lynxpoint/me_9.x.c
View File

@ -496,6 +496,7 @@ static void me_print_fw_version(mbp_fw_version_name *vers_name)
vers_name->hotfix_version, vers_name->build_version); vers_name->hotfix_version, vers_name->build_version);
} }
#if CONFIG_DEBUG_INTEL_ME
/* Get ME Firmware Capabilities */ /* Get ME Firmware Capabilities */
static int mkhi_get_fwcaps(mbp_mefwcaps *cap) static int mkhi_get_fwcaps(mbp_mefwcaps *cap)
{ {
@ -543,6 +544,7 @@ static void me_print_fwcaps(mbp_mefwcaps *cap)
print_cap("Wireless LAN (WLAN)", cap->wlan); print_cap("Wireless LAN (WLAN)", cap->wlan);
} }
#endif #endif
#endif
#if CONFIG_CHROMEOS && 0 /* DISABLED */ #if CONFIG_CHROMEOS && 0 /* DISABLED */
/* Tell ME to issue a global reset */ /* Tell ME to issue a global reset */
@ -836,7 +838,9 @@ static void intel_me_init(device_t dev)
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) #if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
me_print_fw_version(mbp_data.fw_version_name); me_print_fw_version(mbp_data.fw_version_name);
#if CONFIG_DEBUG_INTEL_ME
me_print_fwcaps(mbp_data.fw_capabilities); me_print_fwcaps(mbp_data.fw_capabilities);
#endif
if (mbp_data.plat_time) { if (mbp_data.plat_time) {
printk(BIOS_DEBUG, "ME: Wake Event to ME Reset: %u ms\n", printk(BIOS_DEBUG, "ME: Wake Event to ME Reset: %u ms\n",
@ -1001,9 +1005,11 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, device_t dev)
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) #if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
printk(BIOS_INFO, "ME MBP: Header: items: %d, size dw: %d\n", printk(BIOS_INFO, "ME MBP: Header: items: %d, size dw: %d\n",
mbp->header.num_entries, mbp->header.mbp_size); mbp->header.num_entries, mbp->header.mbp_size);
#if CONFIG_DEBUG_INTEL_ME
for (i = 0; i < mbp->header.mbp_size - 1; i++) { for (i = 0; i < mbp->header.mbp_size - 1; i++) {
printk(BIOS_INFO, "ME MBP: %04x: 0x%08x\n", i, mbp->data[i]); printk(BIOS_INFO, "ME MBP: %04x: 0x%08x\n", i, mbp->data[i]);
} }
#endif
#endif #endif
#define ASSIGN_FIELD_PTR(field_,val_) \ #define ASSIGN_FIELD_PTR(field_,val_) \
@ -1047,8 +1053,8 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, device_t dev)
ASSIGN_FIELD_PTR(nfc_data, &mbp->data[i+1]); ASSIGN_FIELD_PTR(nfc_data, &mbp->data[i+1]);
default: default:
printk(BIOS_ERR, "ME MBP: unknown item 0x%x @ dw offset 0x%x\n", printk(BIOS_ERR, "ME MBP: unknown item 0x%x @ "
mbp->data[i], i); "dw offset 0x%x\n", mbp->data[i], i);
break; break;
} }
i += item->length; i += item->length;