mbox series

[v3,net-next,0/7] smsc W=1 warning fixes

Message ID 20201110030248.1480413-1-andrew@lunn.ch (mailing list archive)
Headers show
Series smsc W=1 warning fixes | expand

Message

Andrew Lunn Nov. 10, 2020, 3:02 a.m. UTC
Fixup various W=1 warnings, and then add COMPILE_TEST support, which
explains why these where missed on the previous pass.

v2:
Use while (0)
Rework buffer alignment to make it clearer

v3:
Access the length from the hardware and Use __always_unused to tell the
compiler we want to discard the value.

Andrew Lunn (7):
  drivers: net: smc91x: Fix set but unused W=1 warning
  drivers: net: smc91x: Fix missing kerneldoc reported by W=1
  drivers: net: smc911x: Work around set but unused status
  drivers: net: smc911x: Fix set but unused status because of DBG macro
  drivers: net: smc911x: Fix passing wrong number of parameters to DBG()
    macro
  drivers: net: smc911x: Fix cast from pointer to integer of different
    size
  drivers: net: smsc: Add COMPILE_TEST support

 drivers/net/ethernet/smsc/Kconfig   |  6 +++---
 drivers/net/ethernet/smsc/smc911x.c | 17 +++++++++--------
 drivers/net/ethernet/smsc/smc91x.c  |  9 ++++++++-
 3 files changed, 20 insertions(+), 12 deletions(-)

Comments

Jakub Kicinski Nov. 12, 2020, 10:51 p.m. UTC | #1
On Tue, 10 Nov 2020 04:02:41 +0100 Andrew Lunn wrote:
> Fixup various W=1 warnings, and then add COMPILE_TEST support, which
> explains why these where missed on the previous pass.
> 
> v2:
> Use while (0)
> Rework buffer alignment to make it clearer
> 
> v3:
> Access the length from the hardware and Use __always_unused to tell the
> compiler we want to discard the value.

Applied, thank you!