@@ -266,7 +266,7 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
266266 const int rx_stat_cnt = ARRAY_SIZE (aq_ethtool_queue_rx_stat_names );
267267 const int tx_stat_cnt = ARRAY_SIZE (aq_ethtool_queue_tx_stat_names );
268268 char tc_string [8 ];
269- int tc ;
269+ unsigned int tc ;
270270
271271 memset (tc_string , 0 , sizeof (tc_string ));
272272 memcpy (p , aq_ethtool_stat_names ,
@@ -275,22 +275,20 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
275275
276276 for (tc = 0 ; tc < cfg -> tcs ; tc ++ ) {
277277 if (cfg -> is_qos )
278- snprintf (tc_string , 8 , "TC%d " , tc );
278+ snprintf (tc_string , 8 , "TC%u " , tc );
279279
280280 for (i = 0 ; i < cfg -> vecs ; i ++ ) {
281281 for (si = 0 ; si < rx_stat_cnt ; si ++ ) {
282- snprintf ( p , ETH_GSTRING_LEN ,
282+ ethtool_sprintf ( & p ,
283283 aq_ethtool_queue_rx_stat_names [si ],
284284 tc_string ,
285285 AQ_NIC_CFG_TCVEC2RING (cfg , tc , i ));
286- p += ETH_GSTRING_LEN ;
287286 }
288287 for (si = 0 ; si < tx_stat_cnt ; si ++ ) {
289- snprintf ( p , ETH_GSTRING_LEN ,
288+ ethtool_sprintf ( & p ,
290289 aq_ethtool_queue_tx_stat_names [si ],
291290 tc_string ,
292291 AQ_NIC_CFG_TCVEC2RING (cfg , tc , i ));
293- p += ETH_GSTRING_LEN ;
294292 }
295293 }
296294 }
@@ -305,20 +303,18 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
305303
306304 for (i = 0 ; i < max (rx_ring_cnt , tx_ring_cnt ); i ++ ) {
307305 for (si = 0 ; si < rx_stat_cnt ; si ++ ) {
308- snprintf ( p , ETH_GSTRING_LEN ,
306+ ethtool_sprintf ( & p ,
309307 aq_ethtool_queue_rx_stat_names [si ],
310308 tc_string ,
311309 i ? PTP_HWST_RING_IDX : ptp_ring_idx );
312- p += ETH_GSTRING_LEN ;
313310 }
314311 if (i >= tx_ring_cnt )
315312 continue ;
316313 for (si = 0 ; si < tx_stat_cnt ; si ++ ) {
317- snprintf ( p , ETH_GSTRING_LEN ,
314+ ethtool_sprintf ( & p ,
318315 aq_ethtool_queue_tx_stat_names [si ],
319316 tc_string ,
320317 i ? PTP_HWST_RING_IDX : ptp_ring_idx );
321- p += ETH_GSTRING_LEN ;
322318 }
323319 }
324320 }
@@ -338,9 +334,8 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
338334 for (si = 0 ;
339335 si < ARRAY_SIZE (aq_macsec_txsc_stat_names );
340336 si ++ ) {
341- snprintf ( p , ETH_GSTRING_LEN ,
337+ ethtool_sprintf ( & p ,
342338 aq_macsec_txsc_stat_names [si ], i );
343- p += ETH_GSTRING_LEN ;
344339 }
345340 aq_txsc = & nic -> macsec_cfg -> aq_txsc [i ];
346341 for (sa = 0 ; sa < MACSEC_NUM_AN ; sa ++ ) {
@@ -349,10 +344,9 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
349344 for (si = 0 ;
350345 si < ARRAY_SIZE (aq_macsec_txsa_stat_names );
351346 si ++ ) {
352- snprintf ( p , ETH_GSTRING_LEN ,
347+ ethtool_sprintf ( & p ,
353348 aq_macsec_txsa_stat_names [si ],
354349 i , sa );
355- p += ETH_GSTRING_LEN ;
356350 }
357351 }
358352 }
@@ -369,10 +363,9 @@ static void aq_ethtool_get_strings(struct net_device *ndev,
369363 for (si = 0 ;
370364 si < ARRAY_SIZE (aq_macsec_rxsa_stat_names );
371365 si ++ ) {
372- snprintf ( p , ETH_GSTRING_LEN ,
366+ ethtool_sprintf ( & p ,
373367 aq_macsec_rxsa_stat_names [si ],
374368 i , sa );
375- p += ETH_GSTRING_LEN ;
376369 }
377370 }
378371 }
0 commit comments