net/bnxt: misuse of comma operator

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: [email protected]
          Reporter: [email protected]

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.