[PATCH v4 0/3] staging: octeon: ethernet: logging and struct cleanups

Ayush Mukkanwar posted 3 patches 2 months, 1 week ago
drivers/staging/octeon/ethernet-mem.c | 42 +++++++++--------
drivers/staging/octeon/ethernet-mem.h |  4 +-
drivers/staging/octeon/ethernet-rx.c  |  6 ++-
drivers/staging/octeon/ethernet-rx.h  |  6 +--
drivers/staging/octeon/ethernet.c     | 65 ++++++++++++++++-----------
5 files changed, 73 insertions(+), 50 deletions(-)
[PATCH v4 0/3] staging: octeon: ethernet: logging and struct cleanups
Posted by Ayush Mukkanwar 2 months, 1 week ago
This series replaces pr_*() logging calls with dev_warn() and
netdev_err()/dev_err() in the octeon ethernet staging driver to include
device context in log messages.

Changes since v3:
- Broke the series into much smaller, focused patches to ease review.
- Removed the global cvm_oct_rx_refill_work by introducing a per-device
  struct octeon_ethernet_platform, allocated during probe.
- Retrieved the device pointer via platform_get_drvdata() and passed it
  down the call chain (NAPI poll -> rx refill -> mem free/fill) as the
  first argument to avoid global state.
- Note: Dropped the ethernet-spi.c IRQ handler conversions from this
  series for now, as those errors are SPI4 bus-level.

Ayush Mukkanwar (3):
  staging: octeon: ethernet-mem: replace pr_warn with dev_warn in free
    functions
  staging: octeon: replace pr_warn with dev_warn in fill path
  staging: octeon: ethernet: replace pr_err and pr_info with dev_err and
    netdev_err

 drivers/staging/octeon/ethernet-mem.c | 42 +++++++++--------
 drivers/staging/octeon/ethernet-mem.h |  4 +-
 drivers/staging/octeon/ethernet-rx.c  |  6 ++-
 drivers/staging/octeon/ethernet-rx.h  |  6 +--
 drivers/staging/octeon/ethernet.c     | 65 ++++++++++++++++-----------
 5 files changed, 73 insertions(+), 50 deletions(-)

-- 
2.53.0