net/hinic: fix snprintf length of cable info

Correct length and remove unnecessary brackets.

Fixes: d9ce1917941c ("net/hinic/base: add hardware operation")
Cc: stable@dpdk.org
Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

diff --git a/drivers/net/hinic/base/hinic_pmd_hwdev.c b/drivers/net/hinic/base/hinic_pmd_hwdev.c
index 8b16897adecd11f1d6e72864e2adfb31beba280d..b6c821a2ad1d9a4d6ff15befefa5cee3f4362971 100644 (file)
--- a/drivers/net/hinic/base/hinic_pmd_hwdev.c
+++ b/drivers/net/hinic/base/hinic_pmd_hwdev.c
@@ -1390,7 +1390,7 @@ static void print_cable_info(struct hinic_link_info *info)
        }
 
        memcpy(tmp_vendor, info->vendor_name, sizeof(info->vendor_name));
-       snprintf(tmp_str, (sizeof(tmp_str) - 1),
+       snprintf(tmp_str, sizeof(tmp_str),
                 "Vendor: %s, %s, %s, length: %um, max_speed: %uGbps",
                 tmp_vendor, info->sfp_type ? "SFP" : "QSFP", port_type,
                 info->cable_length, info->cable_max_speed);