Skip to content

staging: rtl8192u: use min/max macros

Rebecca Mckeever linux-staging lkml outreachy 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest2 patches 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/2staging: rtl8192u: use max macro instead of ternary operatordrivers/staging/rtl8192u/ieee80211/ieee80211_wx.c+1 -11 reply
2/2staging: rtl8192u: use min_t/max_t macros instead of if elsedrivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c+5 -81 reply

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/2] staging: rtl8192u: use min/max macros

These patches replace ternary and if else statements with an equivalent min/max macro. Found with minmax coccinelle script.

---
I noticed that ieee80211.h includes linux/kernel.h and linux/jiffies.h, which both include linux/minmax.h. Would it be a good idea to add #include <linux/minmax.h>
in ieee80211.h in case the content of linux/kernel.h or linux/jiffies.h changes?
---

Rebecca Mckeever (2):
staging: rtl8192u: use max macro instead of ternary operator
staging: rtl8192u: use min_t/max_t macros instead of if else

 drivers/staging/rtl8192u/ieee80211/ieee80211_wx.c   |  2 +-
 drivers/staging/rtl8192u/ieee80211/rtl819x_HTProc.c | 13 +++++--------
 2 files changed, 6 insertions(+), 9 deletions(-)
Signature · 1 line
2.32.0