sb,soc/intel: Reduce preprocessor use with ME debugging

Change-Id: Iedd54730f140b6a7a40834f00d558ed99a345077
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36639
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
This commit is contained in:
Kyösti Mälkki 2019-11-06 06:32:27 +02:00 committed by Patrick Georgi
parent 82c0e7e3d5
commit c86fc8e63d
9 changed files with 70 additions and 93 deletions

View File

@ -493,11 +493,7 @@ struct me_fwcaps {
void intel_me_hsio_version(uint16_t *version, uint16_t *checksum); void intel_me_hsio_version(uint16_t *version, uint16_t *checksum);
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
/* Defined in me_status.c for both romstage and ramstage */ /* Defined in me_status.c for both romstage and ramstage */
void intel_me_status(void); void intel_me_status(void);
#else
static inline void intel_me_status(void) { }
#endif
#endif #endif

View File

@ -58,11 +58,13 @@ static const char *me_bios_path_values[] = {
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u8 *mei_base_address; static u8 *mei_base_address;
#if CONFIG(DEBUG_INTEL_ME)
static void mei_dump(void *ptr, int dword, int offset, const char *type) static void mei_dump(void *ptr, int dword, int offset, const char *type)
{ {
struct mei_csr *csr; struct mei_csr *csr;
if (!CONFIG(DEBUG_INTEL_ME))
return;
printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset); printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset);
switch (offset) { switch (offset) {
@ -88,9 +90,6 @@ static void mei_dump(void *ptr, int dword, int offset, const char *type)
break; break;
} }
} }
#else
# define mei_dump(ptr, dword, offset, type) do {} while (0)
#endif
/* /*
* ME/MEI access helpers using memcpy to avoid aliasing. * ME/MEI access helpers using memcpy to avoid aliasing.
@ -483,7 +482,6 @@ 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)
static inline void print_cap(const char *name, int state) static inline void print_cap(const char *name, int state)
{ {
printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n", printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n",
@ -536,7 +534,6 @@ static void me_print_fwcaps(mbp_mefwcaps *cap)
print_cap("TLS", cap->tls); print_cap("TLS", cap->tls);
print_cap("Wireless LAN (WLAN)", cap->wlan); print_cap("Wireless LAN (WLAN)", cap->wlan);
} }
#endif
/* Send END OF POST message to the ME */ /* Send END OF POST message to the ME */
static int mkhi_end_of_post(void) static int mkhi_end_of_post(void)
@ -804,9 +801,8 @@ static void intel_me_print_mbp(me_bios_payload *mbp_data)
{ {
me_print_fw_version(mbp_data->fw_version_name); me_print_fw_version(mbp_data->fw_version_name);
#if CONFIG(DEBUG_INTEL_ME) 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",
@ -912,12 +908,12 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev)
} }
/* Dump out the MBP contents. */ /* Dump out the MBP contents. */
#if CONFIG(DEBUG_INTEL_ME) if (CONFIG(DEBUG_INTEL_ME)) {
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);
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 }
#define ASSIGN_FIELD_PTR(field_, val_) \ #define ASSIGN_FIELD_PTR(field_, val_) \
{ \ { \

View File

@ -34,8 +34,6 @@ static inline void me_read_dword_ptr(void *ptr, int offset)
memcpy(ptr, &dword, sizeof(dword)); memcpy(ptr, &dword, sizeof(dword));
} }
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
/* HFS1[3:0] Current Working State Values */ /* HFS1[3:0] Current Working State Values */
static const char *me_cws_values[] = { static const char *me_cws_values[] = {
[ME_HFS_CWS_RESET] = "Reset", [ME_HFS_CWS_RESET] = "Reset",
@ -210,6 +208,9 @@ static const char *me_progress_policy_values[] = {
void intel_me_status(void) void intel_me_status(void)
{ {
if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL < BIOS_DEBUG)
return;
struct me_hfs _hfs, *hfs = &_hfs; struct me_hfs _hfs, *hfs = &_hfs;
struct me_hfs2 _hfs2, *hfs2 = &_hfs2; struct me_hfs2 _hfs2, *hfs2 = &_hfs2;
@ -302,7 +303,6 @@ void intel_me_status(void)
} }
printk(BIOS_DEBUG, "\n"); printk(BIOS_DEBUG, "\n");
} }
#endif
void intel_me_hsio_version(uint16_t *version, uint16_t *checksum) void intel_me_hsio_version(uint16_t *version, uint16_t *checksum)
{ {

View File

@ -60,11 +60,13 @@ static const char *me_bios_path_values[] = {
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
#if CONFIG(DEBUG_INTEL_ME)
static void mei_dump(void *ptr, int dword, int offset, const char *type) static void mei_dump(void *ptr, int dword, int offset, const char *type)
{ {
struct mei_csr *csr; struct mei_csr *csr;
if (!CONFIG(DEBUG_INTEL_ME))
return;
printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset); printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset);
switch (offset) { switch (offset) {
@ -90,9 +92,6 @@ static void mei_dump(void *ptr, int dword, int offset, const char *type)
break; break;
} }
} }
#else
# define mei_dump(ptr,dword,offset,type) do {} while (0)
#endif
/* /*
* ME/MEI access helpers using memcpy to avoid aliasing. * ME/MEI access helpers using memcpy to avoid aliasing.
@ -373,9 +372,8 @@ static int mkhi_end_of_post(void)
} }
#endif #endif
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) && !defined(__SMM__)
/* Get ME firmware version */ /* Get ME firmware version */
static int mkhi_get_fw_version(void) static int __unused mkhi_get_fw_version(void)
{ {
struct me_fw_version version; struct me_fw_version version;
struct mkhi_header mkhi = { struct mkhi_header mkhi = {
@ -412,7 +410,7 @@ static inline void print_cap(const char *name, int state)
} }
/* Get ME Firmware Capabilities */ /* Get ME Firmware Capabilities */
static int mkhi_get_fwcaps(void) static int __unused mkhi_get_fwcaps(void)
{ {
u32 rule_id = 0; u32 rule_id = 0;
struct me_fwcaps cap; struct me_fwcaps cap;
@ -454,7 +452,6 @@ static int mkhi_get_fwcaps(void)
return 0; return 0;
} }
#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 */
@ -714,12 +711,12 @@ static void intel_me_init(struct device *dev)
if (intel_mei_setup(dev) < 0) if (intel_mei_setup(dev) < 0)
break; break;
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) {
/* Print ME firmware version */ /* Print ME firmware version */
mkhi_get_fw_version(); mkhi_get_fw_version();
/* Print ME firmware capabilities */ /* Print ME firmware capabilities */
mkhi_get_fwcaps(); mkhi_get_fwcaps();
#endif }
/* /*
* Leave the ME unlocked in this path. * Leave the ME unlocked in this path.

View File

@ -62,11 +62,14 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data);
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
#if CONFIG(DEBUG_INTEL_ME)
static void mei_dump(void *ptr, int dword, int offset, const char *type) static void mei_dump(void *ptr, int dword, int offset, const char *type)
{ {
struct mei_csr *csr; struct mei_csr *csr;
if (!CONFIG(DEBUG_INTEL_ME))
return;
printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset); printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset);
switch (offset) { switch (offset) {
@ -92,9 +95,6 @@ static void mei_dump(void *ptr, int dword, int offset, const char *type)
break; break;
} }
} }
#else
# define mei_dump(ptr,dword,offset,type) do {} while (0)
#endif
/* /*
* ME/MEI access helpers using memcpy to avoid aliasing. * ME/MEI access helpers using memcpy to avoid aliasing.
@ -350,14 +350,13 @@ static inline int mei_sendrecv(struct mei_header *mei, struct mkhi_header *mkhi,
return 0; return 0;
} }
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) && !defined(__SMM__)
static inline void print_cap(const char *name, int state) static inline void print_cap(const char *name, int state)
{ {
printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n", printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n",
name, state ? " en" : "dis"); name, state ? " en" : "dis");
} }
static void me_print_fw_version(mbp_fw_version_name *vers_name) static void __unused me_print_fw_version(mbp_fw_version_name *vers_name)
{ {
if (!vers_name->major_version) { if (!vers_name->major_version) {
printk(BIOS_ERR, "ME: mbp missing version report\n"); printk(BIOS_ERR, "ME: mbp missing version report\n");
@ -395,7 +394,7 @@ static int mkhi_get_fwcaps(mefwcaps_sku *cap)
} }
/* Get ME Firmware Capabilities */ /* Get ME Firmware Capabilities */
static void me_print_fwcaps(mbp_fw_caps *caps_section) static void __unused me_print_fwcaps(mbp_fw_caps *caps_section)
{ {
mefwcaps_sku *cap = &caps_section->fw_capabilities; mefwcaps_sku *cap = &caps_section->fw_capabilities;
if (!caps_section->available) { if (!caps_section->available) {
@ -421,7 +420,6 @@ static void me_print_fwcaps(mbp_fw_caps *caps_section)
print_cap("TLS", cap->tls); print_cap("TLS", cap->tls);
print_cap("Wireless LAN (WLAN)", cap->wlan); print_cap("Wireless LAN (WLAN)", cap->wlan);
} }
#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 */
@ -719,10 +717,10 @@ static void intel_me_init(struct device *dev)
} }
#endif #endif
#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);
me_print_fwcaps(&mbp_data.fw_caps_sku); me_print_fwcaps(&mbp_data.fw_caps_sku);
#endif }
/* /*
* Leave the ME unlocked in this path. * Leave the ME unlocked in this path.

View File

@ -18,7 +18,6 @@
#include <console/console.h> #include <console/console.h>
#include "me.h" #include "me.h"
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
/* HFS1[3:0] Current Working State Values */ /* HFS1[3:0] Current Working State Values */
static const char *me_cws_values[] = { static const char *me_cws_values[] = {
[ME_HFS_CWS_RESET] = "Reset", [ME_HFS_CWS_RESET] = "Reset",
@ -137,11 +136,12 @@ static const char *me_progress_policy_values[] = {
[0x0f] = "ME cannot access the chipset descriptor region", [0x0f] = "ME cannot access the chipset descriptor region",
[0x10] = "Required VSCC values for flash parts do not match", [0x10] = "Required VSCC values for flash parts do not match",
}; };
#endif
void intel_me_status(struct me_hfs *hfs, struct me_gmes *gmes) void intel_me_status(struct me_hfs *hfs, struct me_gmes *gmes)
{ {
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL < BIOS_DEBUG)
return;
/* Check Current States */ /* Check Current States */
printk(BIOS_DEBUG, "ME: FW Partition Table : %s\n", printk(BIOS_DEBUG, "ME: FW Partition Table : %s\n",
hfs->fpt_bad ? "BAD" : "OK"); hfs->fpt_bad ? "BAD" : "OK");
@ -204,5 +204,4 @@ void intel_me_status(struct me_hfs *hfs, struct me_gmes *gmes)
printk(BIOS_DEBUG, "Unknown 0x%02x", gmes->current_state); printk(BIOS_DEBUG, "Unknown 0x%02x", gmes->current_state);
} }
printk(BIOS_DEBUG, "\n"); printk(BIOS_DEBUG, "\n");
#endif
} }

View File

@ -59,11 +59,13 @@ static const char *me_bios_path_values[] = {
/* MMIO base address for MEI interface */ /* MMIO base address for MEI interface */
static u32 *mei_base_address; static u32 *mei_base_address;
#if CONFIG(DEBUG_INTEL_ME)
static void mei_dump(void *ptr, int dword, int offset, const char *type) static void mei_dump(void *ptr, int dword, int offset, const char *type)
{ {
struct mei_csr *csr; struct mei_csr *csr;
if (!CONFIG(DEBUG_INTEL_ME))
return;
printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset); printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset);
switch (offset) { switch (offset) {
@ -89,9 +91,6 @@ static void mei_dump(void *ptr, int dword, int offset, const char *type)
break; break;
} }
} }
#else
# define mei_dump(ptr,dword,offset,type) do {} while (0)
#endif
/* /*
* ME/MEI access helpers using memcpy to avoid aliasing. * ME/MEI access helpers using memcpy to avoid aliasing.

View File

@ -66,11 +66,13 @@ void intel_me_mbp_clear(pci_devfn_t dev);
void intel_me_mbp_clear(struct device *dev); void intel_me_mbp_clear(struct device *dev);
#endif #endif
#if CONFIG(DEBUG_INTEL_ME)
static void mei_dump(void *ptr, int dword, int offset, const char *type) static void mei_dump(void *ptr, int dword, int offset, const char *type)
{ {
struct mei_csr *csr; struct mei_csr *csr;
if (!CONFIG(DEBUG_INTEL_ME))
return;
printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset); printk(BIOS_SPEW, "%-9s[%02x] : ", type, offset);
switch (offset) { switch (offset) {
@ -96,9 +98,6 @@ static void mei_dump(void *ptr, int dword, int offset, const char *type)
break; break;
} }
} }
#else
# define mei_dump(ptr,dword,offset,type) do {} while (0)
#endif
/* /*
* ME/MEI access helpers using memcpy to avoid aliasing. * ME/MEI access helpers using memcpy to avoid aliasing.
@ -380,7 +379,6 @@ static int mei_recv_msg(void *header, int header_bytes,
return mei_wait_for_me_ready(); return mei_wait_for_me_ready();
} }
#if CONFIG(DEBUG_INTEL_ME) || defined(__SMM__)
static inline int mei_sendrecv_mkhi(struct mkhi_header *mkhi, static inline int mei_sendrecv_mkhi(struct mkhi_header *mkhi,
void *req_data, int req_bytes, void *req_data, int req_bytes,
void *rsp_data, int rsp_bytes) void *rsp_data, int rsp_bytes)
@ -418,7 +416,6 @@ static inline int mei_sendrecv_mkhi(struct mkhi_header *mkhi,
return 0; return 0;
} }
#endif /* CONFIG_DEBUG_INTEL_ME || __SMM__ */
/* /*
* mbp give up routine. This path is taken if hfs.mpb_rdy is 0 or the read * mbp give up routine. This path is taken if hfs.mpb_rdy is 0 or the read
@ -469,8 +466,7 @@ void intel_me_mbp_clear(struct device *dev)
} }
} }
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) && !defined(__SMM__) static void __unused me_print_fw_version(mbp_fw_version_name *vers_name)
static void me_print_fw_version(mbp_fw_version_name *vers_name)
{ {
if (!vers_name) { if (!vers_name) {
printk(BIOS_ERR, "ME: mbp missing version report\n"); printk(BIOS_ERR, "ME: mbp missing version report\n");
@ -482,7 +478,6 @@ 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)
static inline void print_cap(const char *name, int state) static inline void print_cap(const char *name, int state)
{ {
printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n", printk(BIOS_DEBUG, "ME Capability: %-41s : %sabled\n",
@ -510,7 +505,7 @@ static int mkhi_get_fwcaps(mbp_mefwcaps *cap)
} }
/* Get ME Firmware Capabilities */ /* Get ME Firmware Capabilities */
static void me_print_fwcaps(mbp_mefwcaps *cap) static void __unused me_print_fwcaps(mbp_mefwcaps *cap)
{ {
mbp_mefwcaps local_caps; mbp_mefwcaps local_caps;
if (!cap) { if (!cap) {
@ -535,8 +530,6 @@ static void me_print_fwcaps(mbp_mefwcaps *cap)
print_cap("TLS", cap->tls); print_cap("TLS", cap->tls);
print_cap("Wireless LAN (WLAN)", cap->wlan); print_cap("Wireless LAN (WLAN)", cap->wlan);
} }
#endif /* CONFIG_DEBUG_INTEL_ME */
#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 */
@ -851,11 +844,11 @@ static void intel_me_init(struct device *dev)
if (intel_me_read_mbp(&mbp_data, dev)) if (intel_me_read_mbp(&mbp_data, dev))
return; return;
#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)
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",
@ -865,7 +858,7 @@ static void intel_me_init(struct device *dev)
printk(BIOS_DEBUG, "ME: Platform Reset to CPU Reset: %u ms\n", printk(BIOS_DEBUG, "ME: Platform Reset to CPU Reset: %u ms\n",
mbp_data.plat_time->pltrst_cpurst_time_ms); mbp_data.plat_time->pltrst_cpurst_time_ms);
} }
#endif }
/* Set clock enables according to devicetree */ /* Set clock enables according to devicetree */
if (config && config->icc_clock_disable) if (config && config->icc_clock_disable)
@ -1004,15 +997,15 @@ static int intel_me_read_mbp(me_bios_payload *mbp_data, struct device *dev)
#endif #endif
/* Dump out the MBP contents. */ /* Dump out the MBP contents. */
#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) 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 }
#define ASSIGN_FIELD_PTR(field_,val_) \ #define ASSIGN_FIELD_PTR(field_,val_) \
{ \ { \

View File

@ -18,7 +18,6 @@
#include <console/console.h> #include <console/console.h>
#include "me.h" #include "me.h"
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG)
/* HFS1[3:0] Current Working State Values */ /* HFS1[3:0] Current Working State Values */
static const char *me_cws_values[] = { static const char *me_cws_values[] = {
[ME_HFS_CWS_RESET] = "Reset", [ME_HFS_CWS_RESET] = "Reset",
@ -138,11 +137,12 @@ static const char *me_progress_policy_values[] = {
[ME_HFS2_STATE_POLICY_DESCRIPTOR_ERR] = "ME cannot access the chipset descriptor region", [ME_HFS2_STATE_POLICY_DESCRIPTOR_ERR] = "ME cannot access the chipset descriptor region",
[ME_HFS2_STATE_POLICY_VSCC_NO_MATCH] = "Required VSCC values for flash parts do not match", [ME_HFS2_STATE_POLICY_VSCC_NO_MATCH] = "Required VSCC values for flash parts do not match",
}; };
#endif
void intel_me_status(struct me_hfs *hfs, struct me_hfs2 *hfs2) void intel_me_status(struct me_hfs *hfs, struct me_hfs2 *hfs2)
{ {
#if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= BIOS_DEBUG) if (CONFIG_DEFAULT_CONSOLE_LOGLEVEL < BIOS_DEBUG)
return;
/* Check Current States */ /* Check Current States */
printk(BIOS_DEBUG, "ME: FW Partition Table : %s\n", printk(BIOS_DEBUG, "ME: FW Partition Table : %s\n",
hfs->fpt_bad ? "BAD" : "OK"); hfs->fpt_bad ? "BAD" : "OK");
@ -206,5 +206,4 @@ void intel_me_status(struct me_hfs *hfs, struct me_hfs2 *hfs2)
hfs2->progress_code, hfs2->current_state); hfs2->progress_code, hfs2->current_state);
} }
printk(BIOS_DEBUG, "\n"); printk(BIOS_DEBUG, "\n");
#endif
} }