summaryrefslogtreecommitdiffstats
path: root/drivers/net/ethernet/netronome
diff options
context:
space:
mode:
authorNiklas Söderlund <niklas.soderlund@corigine.com>2021-07-22 13:25:02 +0200
committerDavid S. Miller <davem@davemloft.net>2021-07-22 14:46:03 +0200
commit4431531c482a2c05126caaa9fcc5053a4a5c495b (patch)
tree6765ccd40b87d93e36c903581c30590afef3b3d2 /drivers/net/ethernet/netronome
parentipv6: fix "'ioam6_if_id_max' defined but not used" warn (diff)
downloadlinux-4431531c482a2c05126caaa9fcc5053a4a5c495b.tar.xz
linux-4431531c482a2c05126caaa9fcc5053a4a5c495b.zip
nfp: fix return statement in nfp_net_parse_meta()
The return type of the function is bool and while NULL do evaluate to false it's not very nice, fix this by explicitly returning false. There is no functional change. Signed-off-by: Niklas Söderlund <niklas.soderlund@corigine.com> Signed-off-by: Louis Peens <louis.peens@corigine.com> Signed-off-by: Simon Horman <simon.horman@corigine.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ethernet/netronome')
-rw-r--r--drivers/net/ethernet/netronome/nfp/nfp_net_common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
index 5dfa4799c34f..ed2ade2a4f04 100644
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
@@ -1697,7 +1697,7 @@ nfp_net_parse_meta(struct net_device *netdev, struct nfp_meta_parsed *meta,
case NFP_NET_META_RESYNC_INFO:
if (nfp_net_tls_rx_resync_req(netdev, data, pkt,
pkt_len))
- return NULL;
+ return false;
data += sizeof(struct nfp_net_tls_resync_req);
break;
default: