checkpatch.pl had a few complaints in the last set of MPTCP patches:
ERROR: code indent should use tabs where possible
+^I         subflow, sk->sk_family, icsk->icsk_af_ops, target, mapped);$
CHECK: Comparison to NULL could be written "!new_ctx"
+	if (new_ctx == NULL) {
ERROR: "foo * bar" should be "foo *bar"
+static const struct proto_ops * tcp_proto_ops(struct sock *sk)
Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
 
 static void __mptcp_close(struct sock *sk, long timeout);
 
-static const struct proto_ops * tcp_proto_ops(struct sock *sk)
+static const struct proto_ops *tcp_proto_ops(struct sock *sk)
 {
 #if IS_ENABLED(CONFIG_IPV6)
        if (sk->sk_family == AF_INET6)
 
        target = mapped ? &subflow_v6m_specific : subflow_default_af_ops(sk);
 
        pr_debug("subflow=%p family=%d ops=%p target=%p mapped=%d",
-                subflow, sk->sk_family, icsk->icsk_af_ops, target, mapped);
+                subflow, sk->sk_family, icsk->icsk_af_ops, target, mapped);
 
        if (likely(icsk->icsk_af_ops == target))
                return;
        }
 
        new_ctx = subflow_create_ctx(newsk, priority);
-       if (new_ctx == NULL) {
+       if (!new_ctx) {
                subflow_ulp_fallback(newsk, old_ctx);
                return;
        }