From 527c1e615b4c6616abb5c7b14c2ff5b04a029175 Mon Sep 17 00:00:00 2001 From: Edward Cree Date: Tue, 12 May 2020 14:24:34 +0100 Subject: [PATCH 1/2] sfc: actually wire up siena_check_caps() Assign it to siena_a0_nic_type.check_caps function pointer. Fixes: be904b855200 ("sfc: make capability checking a nic_type function") Signed-off-by: Edward Cree Signed-off-by: David S. Miller --- drivers/net/ethernet/sfc/siena.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/sfc/siena.c b/drivers/net/ethernet/sfc/siena.c index ed1cb6caa69d9..d8b052979b1bc 100644 --- a/drivers/net/ethernet/sfc/siena.c +++ b/drivers/net/ethernet/sfc/siena.c @@ -1093,4 +1093,5 @@ const struct efx_nic_type siena_a0_nic_type = { 1 << HWTSTAMP_FILTER_PTP_V1_L4_EVENT | 1 << HWTSTAMP_FILTER_PTP_V2_L4_EVENT), .rx_hash_key_size = 16, + .check_caps = siena_check_caps, }; From 1b0cde4091877cd7fe4b29f67645cc391b86c9ca Mon Sep 17 00:00:00 2001 From: Edward Cree Date: Tue, 12 May 2020 14:24:58 +0100 Subject: [PATCH 2/2] sfc: siena_check_caps() can be static Reported-by: Jakub Kicinski Reported-by: kbuild test robot Signed-off-by: Edward Cree Signed-off-by: David S. Miller --- drivers/net/ethernet/sfc/siena.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/sfc/siena.c b/drivers/net/ethernet/sfc/siena.c index d8b052979b1bc..891e9fb6abeca 100644 --- a/drivers/net/ethernet/sfc/siena.c +++ b/drivers/net/ethernet/sfc/siena.c @@ -948,8 +948,8 @@ static int siena_mtd_probe(struct efx_nic *efx) #endif /* CONFIG_SFC_MTD */ -unsigned int siena_check_caps(const struct efx_nic *efx, - u8 flag, u32 offset) +static unsigned int siena_check_caps(const struct efx_nic *efx, + u8 flag, u32 offset) { /* Siena did not support MC_CMD_GET_CAPABILITIES */ return 0;