Skip to contentSkip to conversation

8390: core cleanups

PATCH Superseded Armin Wolf 2 participants 3 messages Aug 29, 2020 – Aug 31, 2020

Subsystems (2)

v3 of 5 Superseded Latest is v5 →

lore
Share & tools
lore

Armin Wolf

author
#0 lore raw

v3 cover 8390: core cleanups

The purpose of this patchset is to do some
cleanups in lib8390.c and 8390.c.

While most changes are coding-style related, pr_cont() usage in lib8390.c was replaced by
a more SMP-safe construct.

Other functional changes include the removal of version-printing in lib8390.c so modules using lib8390.c do not need a global version-string in order to compile successfully.

Patches do compile and run flawless on 5.9.0-rc1 with a RTL8029AS nic using ne2k-pci.

v3 changes:
- swap commits to not break buildability (sorry)
- move MODULE_LICENCE at the bottom and remove MODULE_VERSION in 8390.c

v2 changes:
- change "librarys" to "libraries" in 8390.c commit
- improve 8390.c commit
- prevent uneven whitespaces in error message (lib8390.c)
- do not destroy kernel doc comments in lib8390.c
- fix some typos in lib8390.c

Armin Wolf (2):
lib8390: Fix coding-style issues and remove verion printing 8390: Miscellaneous cleanups

 drivers/net/ethernet/8390/8390.c    |  21 +-
 drivers/net/ethernet/8390/lib8390.c | 590 ++++++++++++++--------------
 2 files changed, 308 insertions(+), 303 deletions(-)

--
2.20.1

maintainer
#1 lore raw

From: Armin Wolf <[email protected]>
Date: Sat, 29 Aug 2020 13:17:32 +0200

Quoted 7 lines

The purpose of this patchset is to do some
cleanups in lib8390.c and 8390.c.

While most changes are coding-style related, pr_cont() usage in lib8390.c was replaced by
a more SMP-safe construct.

Quoted 7 lines

Other functional changes include the removal of version-printing in lib8390.c so modules using lib8390.c do not need a global version-string in order to compile successfully.

Patches do compile and run flawless on 5.9.0-rc1 with a RTL8029AS nic using ne2k-pci.

...

Series applied to net-next.

maintainer
#2 lore raw

From: David Miller <[email protected]>
Date: Mon, 31 Aug 2020 12:36:21 -0700 (PDT)

Quoted 15 lines

From: Armin Wolf <[email protected]>
Date: Sat, 29 Aug 2020 13:17:32 +0200

Quoted 12 lines

The purpose of this patchset is to do some
cleanups in lib8390.c and 8390.c.

While most changes are coding-style related, pr_cont() usage in lib8390.c was replaced by
a more SMP-safe construct.

Other functional changes include the removal of version-printing in lib8390.c so modules using lib8390.c do not need a global version-string in order to compile successfully.

Quoted 5 lines
Quoted 2 lines

Patches do compile and run flawless on 5.9.0-rc1 with a RTL8029AS nic using ne2k-pci.

...

Series applied to net-next.

Actually reverted, this adds a new build warning:

drivers/net/ethernet/8390/ax88796.c:58:22: warning: ‘version’ defined but not used [-Wunused-variable]

   58 | static unsigned char version[] = "ax88796.c: Copyright 2005,2007 Simtec Electronics\n";
      |                      ^~~~~~~

C-c C-cmake[2]: *** Deleting file 'net/sunrpc/auth.o'