wireguard-linux/drivers/net/ethernet/cavium, branch jd/unified-crypt-queue

net: cavium: Fix build errors due to 'imply CAVIUM_PTP'

2020-04-02T14:01:31Z

If CAVIUM_PTP is m and THUNDER_NIC_VF is y, build fails: drivers/net/ethernet/cavium/thunder/nicvf_main.o: In function 'nicvf_remove': nicvf_main.c:(.text+0x1f0): undefined reference to 'cavium_ptp_put' drivers/net/ethernet/cavium/thunder/nicvf_main.o: In function `nicvf_probe': nicvf_main.c:(.text+0x557c): undefined reference to 'cavium_ptp_get' THUNDER_NIC_VF imply CAVIUM_PTP, which allow the config now, Use IS_REACHABLE() to avoid the vmlinux link error for this case. Reported-by: Hulk Robot Fixes: def2fbffe62c ("kconfig: allow symbols implied by y to become m") Signed-off-by: YueHaibing Signed-off-by: David S. Miller

net: thunderx: remove set but not used variable 'tail'

2020-03-24T04:53:32Z

Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/cavium/thunder/nicvf_queues.c: In function nicvf_sq_free_used_descs: drivers/net/ethernet/cavium/thunder/nicvf_queues.c:1182:12: warning: variable tail set but not used [-Wunused-but-set-variable] It's not used since commit 4863dea3fab01("net: Adding support for Cavium ThunderX network controller"), so remove it. Reported-by: Hulk Robot Signed-off-by: Zheng zengkai Signed-off-by: David S. Miller

liquidio: remove set but not used variable 's'

2020-03-20T04:00:14Z

Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/cavium/liquidio/lio_main.c: In function 'octeon_chip_specific_setup': drivers/net/ethernet/cavium/liquidio/lio_main.c:1378:8: warning: variable 's' set but not used [-Wunused-but-set-variable] It's not used since commit b6334be64d6f ("net/liquidio: Delete driver version assignment") Reported-by: Hulk Robot Signed-off-by: YueHaibing Reviewed-by: Leon Romanovsky Signed-off-by: David S. Miller

net: liquidio: reject unsupported coalescing params

2020-03-10T23:28:53Z

Set ethtool_ops->supported_coalesce_params to let the core reject unsupported coalescing parameters. This driver did not previously reject unsupported parameters. Signed-off-by: Jakub Kicinski Signed-off-by: David S. Miller

net/cavium: Delete N/A assignments for ethtool

2020-03-04T01:54:54Z

There is no need to set N/A for the ethtool fields. Signed-off-by: Leon Romanovsky Signed-off-by: David S. Miller

net/cavium: Clean driver versions

2020-03-04T01:54:54Z

Delete driver and module versions in favor of global linux kernel variant. Signed-off-by: Leon Romanovsky Signed-off-by: David S. Miller

net/liquidio: Delete non-working LIQUIDIO_PACKAGE check

2020-03-04T01:54:54Z

Size of LIQUIDIO_PACKAGE is 0 and it means that checks of package version never worked, delete dead code. Fixes: 3258124534f6 ("liquidio: Consolidate common functionality") Signed-off-by: Leon Romanovsky Signed-off-by: David S. Miller

net/liquidio: Delete driver version assignment

2020-03-04T01:54:54Z

Drop driver version in favor of global to linux kernel version. Signed-off-by: Leon Romanovsky Signed-off-by: David S. Miller

liquidio: Replace zero-length array with flexible-array member

2020-03-04T01:38:56Z

The current codebase makes use of the zero-length array language extension to the C90 standard, but the preferred mechanism to declare variable-length types such as these ones is a flexible array member[1][2], introduced in C99: struct foo { int stuff; struct boo array[]; }; By making use of the mechanism above, we will get a compiler warning in case the flexible array does not occur last in the structure, which will help us prevent some kind of undefined behavior bugs from being inadvertently introduced[3] to the codebase from now on. Also, notice that, dynamic memory allocations won't be affected by this change: "Flexible array members have incomplete type, and so the sizeof operator may not be applied. As a quirk of the original implementation of zero-length arrays, sizeof evaluates to zero."[1] This issue was found with the help of Coccinelle. [1] https://gcc.gnu.org/onlinedocs/gcc/Zero-Length.html [2] https://github.com/KSPP/linux/issues/21 [3] commit 76497732932f ("cxgb3/l2t: Fix undefined behaviour") Signed-off-by: Gustavo A. R. Silva Signed-off-by: David S. Miller

net: cavium: Register driver with PCI subsys IDs

2020-03-02T19:13:58Z

Across Cavium's ThunderX and Marvell's OcteonTx2 silicons the PTP timestamping block's PCI device ID and vendor ID have remained same but the HW architecture has changed. Hence added PCI subsystem IDs to the device table to avoid this driver from being probed on OcteonTx2 silicons. Signed-off-by: Prakash Brahmajyosyula Signed-off-by: Sunil Goutham Signed-off-by: David S. Miller