summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNicolas Chautru <nicolas.chautru@intel.com>2019-10-22 06:16:17 -0700
committerAkhil Goyal <akhil.goyal@nxp.com>2019-10-23 16:57:06 +0200
commite12aabced4a9db018b3340a084568a76a217872b (patch)
tree60cbce747e09b713434fbfaae8d370ab771a8bd8
parent6d608a6ccdf18f13f26b01f9889bda85caa25323 (diff)
downloaddpdk-e12aabced4a9db018b3340a084568a76a217872b.zip
dpdk-e12aabced4a9db018b3340a084568a76a217872b.tar.gz
dpdk-e12aabced4a9db018b3340a084568a76a217872b.tar.xz
baseband/fpga_lte_fec: fix polling of MMIO register
Polling of MMIO register could misreport the actual value set dynamically in hardware as the variable was not set explicitly to volatile integer. Fixes: efd453698c49 ("baseband/fpga_lte_fec: add driver for FEC on FPGA") Cc: stable@dpdk.org Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
-rw-r--r--drivers/baseband/fpga_lte_fec/fpga_lte_fec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
index 2fc7f11..8bd10b4 100644
--- a/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
+++ b/drivers/baseband/fpga_lte_fec/fpga_lte_fec.c
@@ -889,7 +889,7 @@ fpga_queue_stop(struct rte_bbdev *dev, uint16_t queue_id)
* completed. If completion flag is not updated within 1ms it is
* considered as a failure.
*/
- while (!(*((uint8_t *)d->flush_queue_status + q->q_idx) & payload)) {
+ while (!(*((volatile uint8_t *)d->flush_queue_status + q->q_idx) & payload)) {
if (counter > timeout) {
rte_bbdev_log(ERR, "FPGA Queue Flush failed for queue %d",
queue_id);