]> git.baikalelectronics.ru Git - kernel.git/commit
sfc: ethtool: Refactor to remove fallthrough comments in case blocks
authorJoe Perches <joe@perches.com>
Wed, 11 Mar 2020 02:41:41 +0000 (19:41 -0700)
committerDavid S. Miller <davem@davemloft.net>
Thu, 12 Mar 2020 18:29:28 +0000 (11:29 -0700)
commitbd1eaf5cbd04f5ab9bd422ce0d2189bf58959164
tree2ae8d47710fa60ac77abd6a1e336f18bcaf12050
parenta64e341a1cd5d345a1e56b29c92682ada09ab2dc
sfc: ethtool: Refactor to remove fallthrough comments in case blocks

Converting fallthrough comments to fallthrough; creates warnings
in this code when compiled with gcc.

This code is overly complicated and reads rather better with a
little refactoring and no fallthrough uses at all.

Remove the fallthrough comments and simplify the written source
code while reducing the object code size.

Consolidate duplicated switch/case blocks for IPV4 and IPV6.

defconfig x86-64 with sfc:

$ size drivers/net/ethernet/sfc/ethtool.o*
   text    data     bss     dec     hex filename
  10055      12       0   10067    2753 drivers/net/ethernet/sfc/ethtool.o.new
  10135      12       0   10147    27a3 drivers/net/ethernet/sfc/ethtool.o.old

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Martin Habets <mhabets@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/sfc/ethtool.c