diff options
author | Sachin Gupta <sgupta2m@in.ibm.com> | 2016-10-14 10:16:58 -0500 |
---|---|---|
committer | AMIT J. TENDOLKAR <amit.tendolkar@in.ibm.com> | 2016-10-14 14:22:50 -0400 |
commit | 40051744d8ed7d308cf6a609ed3da43a17dc1332 (patch) | |
tree | 2364fa739dd2c08f9d1a3e0b8f2d3217a69e8acf /src/sbefw/sbeFifoMsgUtils.C | |
parent | e5abfe5f5da515ca51f9ae8524d47822b28e49ea (diff) | |
download | talos-sbe-40051744d8ed7d308cf6a609ed3da43a17dc1332.tar.gz talos-sbe-40051744d8ed7d308cf6a609ed3da43a17dc1332.zip |
Fix for delay, fifo and npllsetup ( placement in memory )
1. FAPI Delay does not use uint64_t when converting
nanoseconds to ppe timebase. Also as delay is working, updated
wait time in test case framework.
2. npll setup procedure is moved to PIBMEM as npll setup
changes sbe frequence and we can not access seeprom before
changing I2C bit divisor rate.
3. In FIFO utils, the empty status of queue is next state. So
we need to use valid flag to see if FIFO was really empty.
Change-Id: I59ccfc70f3013822532110230ec231408675a1a0
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31245
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: SRINIVAS V. POLISETTY <srinivan@in.ibm.com>
Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: AMIT J. TENDOLKAR <amit.tendolkar@in.ibm.com>
Diffstat (limited to 'src/sbefw/sbeFifoMsgUtils.C')
-rw-r--r-- | src/sbefw/sbeFifoMsgUtils.C | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/sbefw/sbeFifoMsgUtils.C b/src/sbefw/sbeFifoMsgUtils.C index 2d18a1d3..75489cae 100644 --- a/src/sbefw/sbeFifoMsgUtils.C +++ b/src/sbefw/sbeFifoMsgUtils.C @@ -150,7 +150,8 @@ uint32_t sbeUpFifoDeq_mult (uint32_t &io_len, } // if Upstream FIFO is empty, - if (l_data.statusOrReserved.fifo_empty) + if ( ( l_data.statusOrReserved.fifo_empty) && + ( !l_data.statusOrReserved.valid_flag)) { pk_sleep(PK_MILLISECONDS(FIFO_WAIT_SLEEP_TIME)); continue; |