aboutsummaryrefslogtreecommitdiffstats
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 05:46:03 -0700
commit4431531c482a2c05126caaa9fcc5053a4a5c495b (patch)
tree6765ccd40b87d93e36c903581c30590afef3b3d2
parentipv6: fix "'ioam6_if_id_max' defined but not used" warn (diff)
downloadlinux-dev-4431531c482a2c05126caaa9fcc5053a4a5c495b.tar.xz
linux-dev-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>
-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: