[PATCH 0/7] staging: octeon: remove typedefs for enums and structs

Eric Wu posted 7 patches 1 month, 2 weeks ago
.../executive/cvmx-helper-util.c              |  2 +-
.../cavium-octeon/executive/cvmx-helper.c     |  8 ++--
arch/mips/cavium-octeon/executive/cvmx-pko.c  |  6 +--
arch/mips/cavium-octeon/executive/cvmx-spi.c  | 16 +++----
.../include/asm/octeon/cvmx-helper-util.h     |  2 +-
arch/mips/include/asm/octeon/cvmx-helper.h    |  6 +--
arch/mips/include/asm/octeon/cvmx-pip.h       |  6 +--
arch/mips/include/asm/octeon/cvmx-pko.h       | 26 +++++-----
arch/mips/include/asm/octeon/cvmx-pow.h       | 12 ++---
arch/mips/include/asm/octeon/cvmx-spi.h       | 38 +++++++--------
drivers/staging/octeon/ethernet.c             |  6 +--
drivers/staging/octeon/octeon-stubs.h         | 48 +++++++++----------
12 files changed, 88 insertions(+), 88 deletions(-)
[PATCH 0/7] staging: octeon: remove typedefs for enums and structs
Posted by Eric Wu 1 month, 2 weeks ago
The staging octeon driver's TODO list includes "general code review and
clean up". This series addresses that by removing unnecessary typedefs
for enums and structs, converting them to plain types as required by the
Linux kernel coding style (Documentation/process/coding-style.rst,
section 5).

All typedefs ending in _t have been removed from the staging driver
stubs (drivers/staging/octeon/octeon-stubs.h) and correspondingly updated
in the arch/mips/ implementation headers and source files.

The series converts:
- 5 enum typedefs to plain enums:
  cvmx_spi_mode_t, cvmx_helper_interface_mode_t, cvmx_pow_wait_t,
  cvmx_pko_lock_t, cvmx_pko_status_t
- 2 struct typedefs to plain structs:
  cvmx_pko_port_status_t, cvmx_pip_port_status_t

No functional change intended. All patches have been compile-tested
for both MIPS (cavium_octeon_defconfig) and x86_64 (with
CONFIG_STAGING=y, CONFIG_OCTEON_ETHERNET=y, CONFIG_COMPILE_TEST=y).

Eric Wu (7):
  staging: octeon: convert cvmx_spi_mode_t from typedef to plain enum
  staging: octeon: convert cvmx_helper_interface_mode_t from typedef to
    plain enum
  staging: octeon: convert cvmx_pow_wait_t from typedef to plain enum
  staging: octeon: convert cvmx_pko_lock_t from typedef to plain enum
  staging: octeon: convert cvmx_pko_status_t from typedef to plain enum
  staging: octeon: convert cvmx_pko_port_status_t from typedef to plain
    struct
  staging: octeon: convert cvmx_pip_port_status_t from typedef to plain
    struct

 .../executive/cvmx-helper-util.c              |  2 +-
 .../cavium-octeon/executive/cvmx-helper.c     |  8 ++--
 arch/mips/cavium-octeon/executive/cvmx-pko.c  |  6 +--
 arch/mips/cavium-octeon/executive/cvmx-spi.c  | 16 +++----
 .../include/asm/octeon/cvmx-helper-util.h     |  2 +-
 arch/mips/include/asm/octeon/cvmx-helper.h    |  6 +--
 arch/mips/include/asm/octeon/cvmx-pip.h       |  6 +--
 arch/mips/include/asm/octeon/cvmx-pko.h       | 26 +++++-----
 arch/mips/include/asm/octeon/cvmx-pow.h       | 12 ++---
 arch/mips/include/asm/octeon/cvmx-spi.h       | 38 +++++++--------
 drivers/staging/octeon/ethernet.c             |  6 +--
 drivers/staging/octeon/octeon-stubs.h         | 48 +++++++++----------
 12 files changed, 88 insertions(+), 88 deletions(-)

-- 
2.43.0