[AMD Official Use Only - AMD Internal Distribution Only] Acked-by: Selwin Sebastian<[email protected]>
-----Original Message----- From: Ande, Venkat Kumar <[email protected]> Sent: Thursday, November 13, 2025 10:11 PM To: [email protected] Cc: Sebastian, Selwin <[email protected]>; Ande, Venkat Kumar <[email protected]>; [email protected] Subject: [PATCH v2] net/axgbe: fix the build warning for GCC-16 Build warning reported for local variable set and not used on GCC-16. Fix the issue by using the variable for missed register write operation. ../drivers/net/axgbe/axgbe_ethdev.c: In function ‘axgbe_timesync_disable’: ../drivers/net/axgbe/axgbe_ethdev.c:1744:22: warning: variable ‘mac_tscr’ set but not used [-Wunused-but-set-variable=] 1744 | unsigned int mac_tscr = 0; | ^~~~~~~~ Bugzilla ID: 1819 Fixes: e04449488fdb ("net/axgbe: support IEEE 1588 PTP") Cc: [email protected] Signed-off-by: Venkat Kumar Ande <[email protected]> --- drivers/net/axgbe/axgbe_ethdev.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/net/axgbe/axgbe_ethdev.c b/drivers/net/axgbe/axgbe_ethdev.c index 31d35ff182..cf3b0d9ef5 100644 --- a/drivers/net/axgbe/axgbe_ethdev.c +++ b/drivers/net/axgbe/axgbe_ethdev.c @@ -1742,6 +1742,7 @@ axgbe_timesync_disable(struct rte_eth_dev *dev) { struct axgbe_port *pdata = dev->data->dev_private; unsigned int mac_tscr = 0; + unsigned int value = 0; /*disable timestamp for all pkts*/ AXGMAC_SET_BITS(mac_tscr, MAC_TSCR, TSENALL, 0); @@ -1751,6 +1752,11 @@ axgbe_timesync_disable(struct rte_eth_dev *dev) AXGMAC_SET_BITS(mac_tscr, MAC_TSCR, TSCFUPDT, 0); /*disable time stamp*/ AXGMAC_SET_BITS(mac_tscr, MAC_TSCR, TSENA, 0); + + value = AXGMAC_IOREAD(pdata, MAC_TSCR); + value |= mac_tscr; + AXGMAC_IOWRITE(pdata, MAC_TSCR, value); + return 0; } -- 2.40.1.windows.1
