Skip to content

relay: cleanup and const callbacks, take 2

Jani Nikula ath10k ath11k intel-gfx linux-block linux-wireless lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest9 patches 11 repliesIn review 8 of 9 applied

Patches in v1 8 of 9 applied

#Subject and filesSizeRepliesApplied
1/9relay: remove unused buf_mapped and buf_unmapped callbacksinclude/linux/relay.h kernel/relay.c+0 -531 replyApplied
2/9relay: require non-NULL callbacks in relay_open()kernel/relay.c+2 -121 replyApplied
3/9relay: make create_buf_file and remove_buf_file callbacks mandatoryinclude/linux/relay.h kernel/relay.c+7 -251 replyApplied
4/9relay: allow the use of const callback structsinclude/linux/relay.h kernel/relay.c+13 -263 replies
5/9drm/i915: make relay callbacks constdrivers/gpu/drm/i915/gt/uc/intel_guc_log.c+1 -11 replyApplied
6/9ath10k: make relay callbacks constdrivers/net/wireless/ath/ath10k/spectral.c+1 -11 replyApplied
7/9ath11k: make relay callbacks constdrivers/net/wireless/ath/ath11k/spectral.c+1 -11 replyApplied
8/9ath9k: make relay callbacks constdrivers/net/wireless/ath/ath9k/common-spectral.c+1 -11 replyApplied
9/9blktrace: make relay callbacks constkernel/trace/blktrace.c+1 -11 replyApplied

Reviews trailers per patch

All 9 patches have a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchKalle ValoChristoph Hellwig
SeriesA-b
1/9R-b
2/9R-b
3/9R-b
4/9R-b R-b
5/9R-b
6/9A-b R-b
7/9R-b
8/9A-b R-b
9/9R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/9] relay: cleanup and const callbacks, take 2

This is v2 of [1], with a number of cleanups added first based on Christoph's feedback, making the actual constness patch much smaller and cleaner.

I don't know who actually maintains relay, it's not in MAINTAINERS - Cc'd Andrew just in case.

I'd think it would be simplest to queue patches 5-9 via whichever tree the relay patches get merged. They're all one-liners so neglible conflict potential.

BR,
Jani.

[1] http://lore.kernel.org/r/[email protected]

Jani Nikula (9):
relay: remove unused buf_mapped and buf_unmapped callbacks
relay: require non-NULL callbacks in relay_open()
relay: make create_buf_file and remove_buf_file callbacks mandatory
relay: allow the use of const callback structs
drm/i915: make relay callbacks const
ath10k: make relay callbacks const
ath11k: make relay callbacks const
ath9k: make relay callbacks const
blktrace: make relay callbacks const

 drivers/gpu/drm/i915/gt/uc/intel_guc_log.c    |   2 +-
 drivers/net/wireless/ath/ath10k/spectral.c    |   2 +-
 drivers/net/wireless/ath/ath11k/spectral.c    |   2 +-
 .../net/wireless/ath/ath9k/common-spectral.c  |   2 +-
 include/linux/relay.h                         |  29 ++---
 kernel/relay.c                                | 107 +++---------------
 kernel/trace/blktrace.c                       |   2 +-
 7 files changed, 26 insertions(+), 120 deletions(-)
Signature · 7 lines
2.20.1


_______________________________________________
ath10k mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/ath10k