diff options
author | Leonard Crestez <cdleonard@gmail.com> | 2021-11-04 00:17:51 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2021-11-04 11:26:15 +0000 |
commit | 3b65abb8d8a650e50ff5448ac38992ef8a74c584 (patch) | |
tree | a985ffe29cd47ccb1e2433968a488e66adfe6a2c | |
parent | a985442fdecb59504e3a2f1cfdd3c53af017ea5b (diff) | |
download | linux-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.c | 14 |
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) { |