kernel/drivers/net/ethernet/brocade, branch linux-4.3.y

bna: fix error handling

2015-09-29T20:49:53Z

Several functions can return negative value in case of error, so their return type should be fixed as well as type of variables to which this value is assigned. The problem has been detected using proposed semantic patch scripts/coccinelle/tests/assign_signed_to_unsigned.cocci [1]. [1]: http://permalink.gmane.org/gmane.linux.kernel/2046107 Signed-off-by: Andrzej Hajda Signed-off-by: David S. Miller

bna: check for dma mapping errors

2015-09-21T04:36:57Z

Check for DMA mapping errors, recover from them and register them in ethtool stats like other errors. Cc: Rasesh Mody Signed-off-by: Ivan Vecera Acked-by: Rasesh Mody Signed-off-by: David S. Miller

bna: fix interrupts storm caused by erroneous packets

2015-08-10T17:58:37Z

The commit "e29aa33 bna: Enable Multi Buffer RX" moved packets counter increment from the beginning of the NAPI processing loop after the check for erroneous packets so they are never accounted. This counter is used to inform firmware about number of processed completions (packets). As these packets are never acked the firmware fires IRQs for them again and again. Fixes: e29aa33 ("bna: Enable Multi Buffer RX") Signed-off-by: Ivan Vecera Acked-by: Rasesh Mody Signed-off-by: David S. Miller

drivers/net: remove all references to obsolete Ethernet-HOWTO

2015-06-23T13:50:35Z

This howto made sense in the 1990s when users had to manually configure ISA cards with jumpers or vendor utilities, but with the implementation of PCI it became increasingly less and less relevant, to the point where it has been well over a decade since I last updated it. And there is no value in anyone else taking over updating it either. However the references to it continue to spread as boiler plate text from one Kconfig file into the next. We are not doing end users any favours by pointing them at this old document, so lets kill it with fire, once and for all, to hopefully stop any further spread. No code is changed in this commit, just Kconfig help text. Signed-off-by: Paul Gortmaker Signed-off-by: David S. Miller

bna: remove superfluous parentheses

2015-06-21T17:29:02Z

Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller

bna: make pointers to read-only inputs const

2015-06-21T17:29:01Z

Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller

bna: remove unnecessary cast of BIT value

2015-06-21T17:29:00Z

BIT value is already unsigned so casting is not necessary. Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller

bna: use netdev_* and dev_* instead of printk and pr_*

2015-06-11T22:57:18Z

...and remove some of them. It is not necessary to log when .probe() and .remove() are called or when TxQ is started or stopped. Also log level of some of them was changed to more appropriate one (link up/down, firmware loading failure. Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller

bna: fix timeout API argument type

2015-06-11T22:57:17Z

Timeout functions are defined with 'void *' ptr argument. They should be defined directly with 'struct bfa_ioc *' type to avoid type conversions. Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller

bna: use list_for_each_entry where appropriate

2015-06-11T22:57:17Z

Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller