Error observed with gcc 11 under development
gcc (GCC) 11.0.0 20200920 (experimental)

build error:
../drivers/raw/dpaa2_qdma/dpaa2_qdma.c: In function ‘rte_qdma_reset’:
../drivers/raw/dpaa2_qdma/dpaa2_qdma.c:454:17:
        warning: this ‘if’ clause does not guard...
        [-Wmisleading-indentation]
  454 |         if (qdma_vqs[i].in_use && (qdma_vqs[i].num_enqueues !=
      |         ^~
../drivers/raw/dpaa2_qdma/dpaa2_qdma.c:457:25:
        note: ...this statement, but the latter is misleadingly indented
        as if it were guarded by the ‘if’
  457 |                         return -EBUSY;
      |                         ^~~~~~

Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com>
---
 drivers/raw/dpaa2_qdma/dpaa2_qdma.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/raw/dpaa2_qdma/dpaa2_qdma.c 
b/drivers/raw/dpaa2_qdma/dpaa2_qdma.c
index 0b9c4e3d7..2cba71c63 100644
--- a/drivers/raw/dpaa2_qdma/dpaa2_qdma.c
+++ b/drivers/raw/dpaa2_qdma/dpaa2_qdma.c
@@ -452,9 +452,10 @@ rte_qdma_reset(void)
        /* In case there are pending jobs on any VQ, return -EBUSY */
        for (i = 0; i < qdma_dev.max_vqs; i++) {
                if (qdma_vqs[i].in_use && (qdma_vqs[i].num_enqueues !=
-                   qdma_vqs[i].num_dequeues))
+                           qdma_vqs[i].num_dequeues)) {
                        DPAA2_QDMA_ERR("Jobs are still pending on VQ: %d", i);
                        return -EBUSY;
+               }
        }
 
        /* Reset HW queues */
-- 
2.26.2

Reply via email to