kernel/drivers/net/ethernet/brocade/bna, 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

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

bna: get rid of private macros for manipulation with lists

2015-06-11T22:57:17Z

Remove macros for manipulation with struct list_head and replace them with standard ones. Signed-off-by: Ivan Vecera Signed-off-by: David S. Miller