summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLeonard Crestez <cdleonard@gmail.com>2021-11-04 00:17:51 +0200
committerDavid S. Miller <davem@davemloft.net>2021-11-04 11:26:15 +0000
commit3b65abb8d8a650e50ff5448ac38992ef8a74c584 (patch)
treea985ffe29cd47ccb1e2433968a488e66adfe6a2c
parenta985442fdecb59504e3a2f1cfdd3c53af017ea5b (diff)
downloadlinux-3b65abb8d8a650e50ff5448ac38992ef8a74c584.tar.gz
linux-3b65abb8d8a650e50ff5448ac38992ef8a74c584.tar.bz2
linux-3b65abb8d8a650e50ff5448ac38992ef8a74c584.zip
tcp: Use BIT() for OPTION_* constants
Extending these flags using the existing (1 << x) pattern triggers complaints from checkpatch. Instead of ignoring checkpatch modify the existing values to use BIT(x) style in a separate commit. Signed-off-by: Leonard Crestez <cdleonard@gmail.com> Reviewed-by: Eric Dumazet <edumazet@google.com> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/ipv4/tcp_output.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c
index 6f7860e283c6..2e6e5a70168e 100644
--- a/net/ipv4/tcp_output.c
+++ b/net/ipv4/tcp_output.c
@@ -408,13 +408,13 @@ static inline bool tcp_urg_mode(const struct tcp_sock *tp)
return tp->snd_una != tp->snd_up;
}
-#define OPTION_SACK_ADVERTISE (1 << 0)
-#define OPTION_TS (1 << 1)
-#define OPTION_MD5 (1 << 2)
-#define OPTION_WSCALE (1 << 3)
-#define OPTION_FAST_OPEN_COOKIE (1 << 8)
-#define OPTION_SMC (1 << 9)
-#define OPTION_MPTCP (1 << 10)
+#define OPTION_SACK_ADVERTISE BIT(0)
+#define OPTION_TS BIT(1)
+#define OPTION_MD5 BIT(2)
+#define OPTION_WSCALE BIT(3)
+#define OPTION_FAST_OPEN_COOKIE BIT(8)
+#define OPTION_SMC BIT(9)
+#define OPTION_MPTCP BIT(10)
static void smc_options_write(__be32 *ptr, u16 *options)
{