Skip to content

Commit c2fbcc9

Browse files
ksornekanguy11
authored andcommitted
iavf: Refactor string format to avoid static analysis warnings
Change format to match variable type that is used in string. Use %u format for unsigned variable and %d format for signed variable to remove static analysis warnings. Signed-off-by: Michal Swiatkowski <[email protected]> Signed-off-by: Karen Sornek <[email protected]> Tested-by: George Kuruvinakunnel <[email protected]> Signed-off-by: Tony Nguyen <[email protected]>
1 parent fbe66f5 commit c2fbcc9

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

drivers/net/ethernet/intel/iavf/iavf_main.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -463,14 +463,14 @@ iavf_request_traffic_irqs(struct iavf_adapter *adapter, char *basename)
463463

464464
if (q_vector->tx.ring && q_vector->rx.ring) {
465465
snprintf(q_vector->name, sizeof(q_vector->name),
466-
"iavf-%s-TxRx-%d", basename, rx_int_idx++);
466+
"iavf-%s-TxRx-%u", basename, rx_int_idx++);
467467
tx_int_idx++;
468468
} else if (q_vector->rx.ring) {
469469
snprintf(q_vector->name, sizeof(q_vector->name),
470-
"iavf-%s-rx-%d", basename, rx_int_idx++);
470+
"iavf-%s-rx-%u", basename, rx_int_idx++);
471471
} else if (q_vector->tx.ring) {
472472
snprintf(q_vector->name, sizeof(q_vector->name),
473-
"iavf-%s-tx-%d", basename, tx_int_idx++);
473+
"iavf-%s-tx-%u", basename, tx_int_idx++);
474474
} else {
475475
/* skip this unused q_vector */
476476
continue;

drivers/net/ethernet/intel/iavf/iavf_virtchnl.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1025,7 +1025,7 @@ static void iavf_print_link_message(struct iavf_adapter *adapter)
10251025
} else if (link_speed_mbps == SPEED_UNKNOWN) {
10261026
snprintf(speed, IAVF_MAX_SPEED_STRLEN, "%s", "Unknown Mbps");
10271027
} else {
1028-
snprintf(speed, IAVF_MAX_SPEED_STRLEN, "%u %s",
1028+
snprintf(speed, IAVF_MAX_SPEED_STRLEN, "%d %s",
10291029
link_speed_mbps, "Mbps");
10301030
}
10311031

0 commit comments

Comments
 (0)