New patch to review for coreboot: northbridge/amd/amdht: Reduce excessive romstage array size
Timothy Pearson (tpearson@raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12573
-gerrit
commit d136e2ce5d8361b4ae444b935484286ddd753b90 Author: Timothy Pearson tpearson@raptorengineeringinc.com Date: Mon Nov 16 11:48:38 2015 -0600
northbridge/amd/amdht: Reduce excessive romstage array size
Change-Id: Ibcdf5d3927375da5cb72987ae83eaaa789ab9a70 Signed-off-by: Timothy Pearson tpearson@raptorengineeringinc.com --- src/northbridge/amd/amdht/ht_wrapper.c | 48 +++++++++++++++++----------------- 1 file changed, 24 insertions(+), 24 deletions(-)
diff --git a/src/northbridge/amd/amdht/ht_wrapper.c b/src/northbridge/amd/amdht/ht_wrapper.c index 18c0ac4..129b02d 100644 --- a/src/northbridge/amd/amdht/ht_wrapper.c +++ b/src/northbridge/amd/amdht/ht_wrapper.c @@ -91,25 +91,25 @@ static const char * event_class_string_decodes[] = { };
static const char * event_string_decodes[] = { - [HT_EVENT_COH_EVENTS] = "HT_EVENT_COH_EVENTS", - [HT_EVENT_COH_NO_TOPOLOGY] = "HT_EVENT_COH_NO_TOPOLOGY", - [HT_EVENT_COH_LINK_EXCEED] = "HT_EVENT_COH_LINK_EXCEED", - [HT_EVENT_COH_FAMILY_FEUD] = "HT_EVENT_COH_FAMILY_FEUD", - [HT_EVENT_COH_NODE_DISCOVERED] = "HT_EVENT_COH_NODE_DISCOVERED", - [HT_EVENT_COH_MPCAP_MISMATCH] = "HT_EVENT_COH_MPCAP_MISMATCH", - [HT_EVENT_NCOH_EVENTS] = "HT_EVENT_NCOH_EVENTS", - [HT_EVENT_NCOH_BUID_EXCEED] = "HT_EVENT_NCOH_BUID_EXCEED", - [HT_EVENT_NCOH_LINK_EXCEED] = "HT_EVENT_NCOH_LINK_EXCEED", - [HT_EVENT_NCOH_BUS_MAX_EXCEED] = "HT_EVENT_NCOH_BUS_MAX_EXCEED", - [HT_EVENT_NCOH_CFG_MAP_EXCEED] = "HT_EVENT_NCOH_CFG_MAP_EXCEED", - [HT_EVENT_NCOH_DEVICE_FAILED] = "HT_EVENT_NCOH_DEVICE_FAILED", - [HT_EVENT_NCOH_AUTO_DEPTH] = "HT_EVENT_NCOH_AUTO_DEPTH", - [HT_EVENT_OPT_EVENTS] = "HT_EVENT_OPT_EVENTS", - [HT_EVENT_OPT_REQUIRED_CAP_RETRY] = "HT_EVENT_OPT_REQUIRED_CAP_RETRY", - [HT_EVENT_OPT_REQUIRED_CAP_GEN3] = "HT_EVENT_OPT_REQUIRED_CAP_GEN3", - [HT_EVENT_HW_EVENTS] = "HT_EVENT_HW_EVENTS", - [HT_EVENT_HW_SYNCHFLOOD] = "HT_EVENT_HW_SYNCHFLOOD", - [HT_EVENT_HW_HTCRC] = "HT_EVENT_HW_HTCRC" + [(HT_EVENT_COH_EVENTS & 0xfff)] = "HT_EVENT_COH_EVENTS", + [(HT_EVENT_COH_NO_TOPOLOGY & 0xfff)] = "HT_EVENT_COH_NO_TOPOLOGY", + [(HT_EVENT_COH_LINK_EXCEED & 0xfff)] = "HT_EVENT_COH_LINK_EXCEED", + [(HT_EVENT_COH_FAMILY_FEUD & 0xfff)] = "HT_EVENT_COH_FAMILY_FEUD", + [(HT_EVENT_COH_NODE_DISCOVERED & 0xfff)] = "HT_EVENT_COH_NODE_DISCOVERED", + [(HT_EVENT_COH_MPCAP_MISMATCH & 0xfff)] = "HT_EVENT_COH_MPCAP_MISMATCH", + [(HT_EVENT_NCOH_EVENTS & 0xfff)] = "HT_EVENT_NCOH_EVENTS", + [(HT_EVENT_NCOH_BUID_EXCEED & 0xfff)] = "HT_EVENT_NCOH_BUID_EXCEED", + [(HT_EVENT_NCOH_LINK_EXCEED & 0xfff)] = "HT_EVENT_NCOH_LINK_EXCEED", + [(HT_EVENT_NCOH_BUS_MAX_EXCEED & 0xfff)] = "HT_EVENT_NCOH_BUS_MAX_EXCEED", + [(HT_EVENT_NCOH_CFG_MAP_EXCEED & 0xfff)] = "HT_EVENT_NCOH_CFG_MAP_EXCEED", + [(HT_EVENT_NCOH_DEVICE_FAILED & 0xfff)] = "HT_EVENT_NCOH_DEVICE_FAILED", + [(HT_EVENT_NCOH_AUTO_DEPTH & 0xfff)] = "HT_EVENT_NCOH_AUTO_DEPTH", + [(HT_EVENT_OPT_EVENTS & 0xfff)] = "HT_EVENT_OPT_EVENTS", + [(HT_EVENT_OPT_REQUIRED_CAP_RETRY & 0xfff)] = "HT_EVENT_OPT_REQUIRED_CAP_RETRY", + [(HT_EVENT_OPT_REQUIRED_CAP_GEN3 & 0xfff)] = "HT_EVENT_OPT_REQUIRED_CAP_GEN3", + [(HT_EVENT_HW_EVENTS & 0xfff)] = "HT_EVENT_HW_EVENTS", + [(HT_EVENT_HW_SYNCHFLOOD & 0xfff)] = "HT_EVENT_HW_SYNCHFLOOD", + [(HT_EVENT_HW_HTCRC & 0xfff)] = "HT_EVENT_HW_HTCRC" };
/** @@ -146,7 +146,7 @@ static void AMD_CB_EventNotify (u8 evtClass, u16 event, const u8 *pEventData0) case HT_EVENT_COH_NO_TOPOLOGY: case HT_EVENT_COH_LINK_EXCEED: case HT_EVENT_COH_FAMILY_FEUD: - printk(log_level, event_string_decodes[event]); + printk(log_level, event_string_decodes[event & 0xfff]); break; case HT_EVENT_COH_NODE_DISCOVERED: { @@ -163,11 +163,11 @@ static void AMD_CB_EventNotify (u8 evtClass, u16 event, const u8 *pEventData0) case HT_EVENT_NCOH_LINK_EXCEED: case HT_EVENT_NCOH_BUS_MAX_EXCEED: case HT_EVENT_NCOH_CFG_MAP_EXCEED: - printk(log_level, event_string_decodes[event]); + printk(log_level, event_string_decodes[event & 0xfff]); break; case HT_EVENT_NCOH_DEVICE_FAILED: { - printk(log_level, event_string_decodes[event]); + printk(log_level, event_string_decodes[event & 0xfff]); sHtEventNcohDeviceFailed *evt = (sHtEventNcohDeviceFailed*)pEventData0; printk(log_level, ": node %d link %d depth: %d attemptedBUID: %d", evt->node, evt->link, evt->depth, evt->attemptedBUID); @@ -176,7 +176,7 @@ static void AMD_CB_EventNotify (u8 evtClass, u16 event, const u8 *pEventData0) } case HT_EVENT_NCOH_AUTO_DEPTH: { - printk(log_level, event_string_decodes[event]); + printk(log_level, event_string_decodes[event & 0xfff]); sHtEventNcohAutoDepth *evt = (sHtEventNcohAutoDepth*)pEventData0; printk(log_level, ": node %d link %d depth: %d", evt->node, evt->link, evt->depth); @@ -189,7 +189,7 @@ static void AMD_CB_EventNotify (u8 evtClass, u16 event, const u8 *pEventData0) case HT_EVENT_HW_EVENTS: case HT_EVENT_HW_SYNCHFLOOD: case HT_EVENT_HW_HTCRC: - printk(log_level, event_string_decodes[event]); + printk(log_level, event_string_decodes[event & 0xfff]); break; default: printk(log_level, "HT_EVENT_UNKNOWN");
-
Timothy Pearson