[DPDK/ethdev Bug 1807] net/bnxt: misuse of comma operator

[DPDK/ethdev Bug 1807] net/bnxt: misuse of comma operator

[bugzilla]

http://bugs.dpdk.org/show_bug.cgi?id=1807

            Bug ID: 1807
           Summary: net/bnxt: misuse of comma operator
           Product: DPDK
           Version: 20.11
          Hardware: All
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: ethdev
          Assignee: dev@dpdk.org
          Reporter: stephen@networkplumber.org
  Target Milestone: ---

If DPDK is built with clang and -Wcomma in drivers, then bnxt has some open
issues:

[846/3622] Compiling C object drivers/...xt.a.p/net_bnxt_tf_ulp_bnxt_ulp_tf.c.o
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1302:17: warning: possible misuse of
comma operator here [-Wcomma]
 1302 |         parms.dir = dir,
      |                        ^
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1302:2: note: cast expression to void
to silence warning
 1302 |         parms.dir = dir,
      |         ^~~~~~~~~~~~~~~
      |         (void)(        )
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1303:19: warning: possible misuse of
comma operator here [-Wcomma]
 1303 |         parms.type = type,
      |                          ^
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1303:2: note: cast expression to void
to silence warning
 1303 |         parms.type = type,
      |         ^~~~~~~~~~~~~~~~~
      |         (void)(          )
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1304:23: warning: possible misuse of
comma operator here [-Wcomma]
 1304 |         parms.offset = offset,
      |                              ^
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1304:2: note: cast expression to void
to silence warning
 1304 |         parms.offset = offset,
      |         ^~~~~~~~~~~~~~~~~~~~~
      |         (void)(              )
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1305:39: warning: possible misuse of
comma operator here [-Wcomma]
 1305 |         parms.config = (uint8_t *)&global_cfg,
      |                                              ^
../drivers/net/bnxt/tf_ulp/bnxt_ulp_tf.c:1305:2: note: cast expression to void
to silence warning
 1305 |         parms.config = (uint8_t *)&global_cfg,
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |         (void)(                              )

-- 
You are receiving this mail because:
You are the assignee for the bug.