On Wed, Dec 09, 2020 at 08:18:21AM -0800, Jakub Kicinski wrote: > Test robot reports: > > drivers/net/ethernet/netronome/nfp/crypto/tls.c: In function > 'nfp_net_tls_rx_resync_req': > drivers/net/ethernet/netronome/nfp/crypto/tls.c:477:18: warning: variable > 'ipv6h' set but not used [-Wunused-but-set-variable] > 477 | struct ipv6hdr *ipv6h; > | ^~~~~ > In file included from include/linux/compiler_types.h:65, > from <command-line>: > drivers/net/ethernet/netronome/nfp/crypto/tls.c: In function > 'nfp_net_tls_add': > include/linux/compiler_attributes.h:208:41: warning: statement will never be > executed [-Wswitch-unreachable] > 208 | # define fallthrough > __attribute__((__fallthrough__)) > | ^~~~~~~~~~~~~ > drivers/net/ethernet/netronome/nfp/crypto/tls.c:299:3: note: in expansion of > macro 'fallthrough' > 299 | fallthrough; > | ^~~~~~~~~~~ > > Use the IPv6 header in the switch, it doesn't matter which header > we use to read the version field. > > Reported-by: kernel test robot <l...@intel.com> > Signed-off-by: Jakub Kicinski <k...@kernel.org>
Thanks Jakub, this looks good to me. Reviewed-by: Simon Horman <simon.hor...@netronome.com>
