neigh_update_ext_learned has one caller in neighbour.c so does not need
to be defined in the header. Move it and in the process remove the
intialization of ndm_flags and just set it based on the flags check.
Signed-off-by: David Ahern <dsahern@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
        } while (read_seqretry(&n->ha_lock, seq));
 }
 
-static inline void neigh_update_ext_learned(struct neighbour *neigh, u32 flags,
-                                           int *notify)
-{
-       u8 ndm_flags = 0;
-
-       if (!(flags & NEIGH_UPDATE_F_ADMIN))
-               return;
-
-       ndm_flags |= (flags & NEIGH_UPDATE_F_EXT_LEARNED) ? NTF_EXT_LEARNED : 0;
-       if ((neigh->flags ^ ndm_flags) & NTF_EXT_LEARNED) {
-               if (ndm_flags & NTF_EXT_LEARNED)
-                       neigh->flags |= NTF_EXT_LEARNED;
-               else
-                       neigh->flags &= ~NTF_EXT_LEARNED;
-               *notify = 1;
-       }
-}
-
 static inline void neigh_update_is_router(struct neighbour *neigh, u32 flags,
                                          int *notify)
 {
 
        write_unlock_bh(&n->tbl->lock);
 }
 
+static void neigh_update_ext_learned(struct neighbour *neigh, u32 flags,
+                                    int *notify)
+{
+       u8 ndm_flags;
+
+       if (!(flags & NEIGH_UPDATE_F_ADMIN))
+               return;
+
+       ndm_flags = (flags & NEIGH_UPDATE_F_EXT_LEARNED) ? NTF_EXT_LEARNED : 0;
+       if ((neigh->flags ^ ndm_flags) & NTF_EXT_LEARNED) {
+               if (ndm_flags & NTF_EXT_LEARNED)
+                       neigh->flags |= NTF_EXT_LEARNED;
+               else
+                       neigh->flags &= ~NTF_EXT_LEARNED;
+               *notify = 1;
+       }
+}
+
 static bool neigh_del(struct neighbour *n, struct neighbour __rcu **np,
                      struct neigh_table *tbl)
 {