aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVasant Hegde <hegdevasant@linux.vnet.ibm.com>2016-07-02 21:07:41 +0530
committerStewart Smith <stewart@linux.vnet.ibm.com>2016-08-26 18:39:19 +1000
commit7185dbc41abc43d42f38117cfde2cb1fb1400a1d (patch)
treea93117f5a3e1abbf84bdd7d720b7879b33abe580
parent4a74a1ea1c27d7f0e46d2df0e87a9f9958da6a6c (diff)
downloadskiboot-7185dbc41abc43d42f38117cfde2cb1fb1400a1d.zip
skiboot-7185dbc41abc43d42f38117cfde2cb1fb1400a1d.tar.gz
skiboot-7185dbc41abc43d42f38117cfde2cb1fb1400a1d.tar.bz2
FSP/ELOG: Remove redundant elog state
OPAL gets elog notification from service processor which contains log information. Once we get notification we start reading log data and change elog state to ELOG_STATE_FETCHING. Hence we don't need ELOG_STATE_FETCHED_INFO state. Lets remove this variable. Also in some places we have used this state after sending event information to host. Replace such usage with better state (ELOG_STATE_HOST_INFO). Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
-rw-r--r--hw/fsp/fsp-elog-write.c4
-rw-r--r--include/fsp-elog.h1
2 files changed, 2 insertions, 3 deletions
diff --git a/hw/fsp/fsp-elog-write.c b/hw/fsp/fsp-elog-write.c
index f38c236..d4edc50 100644
--- a/hw/fsp/fsp-elog-write.c
+++ b/hw/fsp/fsp-elog-write.c
@@ -156,7 +156,7 @@ bool opal_elog_info(uint64_t *opal_elog_id, uint64_t *opal_elog_size)
} else {
*opal_elog_id = head->plid;
*opal_elog_size = head->log_size;
- fsp_elog_write_set_head_state(ELOG_STATE_FETCHED_INFO);
+ fsp_elog_write_set_head_state(ELOG_STATE_HOST_INFO);
rc = true;
}
}
@@ -190,7 +190,7 @@ bool opal_elog_read(uint64_t *buffer, uint64_t opal_elog_size,
bool rc = false;
lock(&elog_write_to_host_lock);
- if (elog_write_to_host_head_state == ELOG_STATE_FETCHED_INFO) {
+ if (elog_write_to_host_head_state == ELOG_STATE_HOST_INFO) {
log_data = list_top(&elog_write_to_host_pending,
struct errorlog, link);
if (!log_data) {
diff --git a/include/fsp-elog.h b/include/fsp-elog.h
index a796f5e..eb9c28f 100644
--- a/include/fsp-elog.h
+++ b/include/fsp-elog.h
@@ -29,7 +29,6 @@
*/
enum elog_head_state {
ELOG_STATE_FETCHING, /*In the process of reading log from FSP. */
- ELOG_STATE_FETCHED_INFO,/* Indicates reading log info is completed */
ELOG_STATE_FETCHED_DATA,/* Indicates reading log is completed */
ELOG_STATE_HOST_INFO, /* Host read log info */
ELOG_STATE_NONE, /* Indicates to fetch next log */