Re: [PATCH 1/2] net: systemport: Remove unused txchk accessors

From: Florian Fainelli
Date: Tue Oct 15 2024 - 13:32:45 EST


On 10/15/24 10:29, Vladimir Oltean wrote:
On Tue, Oct 15, 2024 at 10:24:57AM -0700, Florian Fainelli wrote:
Vladimir reported the following warning with clang-16 and W=1:

warning: unused function 'txchk_readl' [-Wunused-function]
BCM_SYSPORT_IO_MACRO(txchk, SYS_PORT_TXCHK_OFFSET);
note: expanded from macro 'BCM_SYSPORT_IO_MACRO'

warning: unused function 'txchk_writel' [-Wunused-function]
note: expanded from macro 'BCM_SYSPORT_IO_MACRO'

warning: unused function 'tbuf_readl' [-Wunused-function]
BCM_SYSPORT_IO_MACRO(tbuf, SYS_PORT_TBUF_OFFSET);
note: expanded from macro 'BCM_SYSPORT_IO_MACRO'

warning: unused function 'tbuf_writel' [-Wunused-function]
note: expanded from macro 'BCM_SYSPORT_IO_MACRO'

I mean the warnings do say that the tbuf macros are unused too, yet you
only remove txchk? I did ask yesterday if that should also be deleted,
and with it the other unused transmit buffer macros?
https://lore.kernel.org/netdev/20241014174056.sxcgikf4hhw2olp7@skbuf/

See my response to my own posting.
--
Florian