liquidio_set_rxcsum_command is a local function only, no need to expose it outside of lio_main.c so declare it static and make sparse happy.
Signed-off-by: Nicholas Mc Guire <hof...@osadl.org> --- Detected by sparse: CHECK drivers/net/ethernet/cavium/liquidio/lio_main.c drivers/net/ethernet/cavium/liquidio/lio_main.c:3193:5: warning: symbol 'liquidio_set_rxcsum_command' was not declared. Should it be static? Patch was compile tested with: x86_64_defconfig + CONFIG_LIQUIDIO=m Patch is against 4.8.0-rc2 (localversion-next is -next-20160822) drivers/net/ethernet/cavium/liquidio/lio_main.c | 2 +- 1 file changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/ethernet/cavium/liquidio/lio_main.c b/drivers/net/ethernet/cavium/liquidio/lio_main.c index bdd6c82..184f1ac 100644 --- a/drivers/net/ethernet/cavium/liquidio/lio_main.c +++ b/drivers/net/ethernet/cavium/liquidio/lio_main.c @@ -3190,7 +3190,7 @@ static int liquidio_vlan_rx_kill_vid(struct net_device *netdev, * OCTNET_CMD_RXCSUM_DISABLE * @returns SUCCESS or FAILURE */ -int liquidio_set_rxcsum_command(struct net_device *netdev, int command, +static int liquidio_set_rxcsum_command(struct net_device *netdev, int command, u8 rx_cmd) { struct lio *lio = GET_LIO(netdev); -- 2.1.4