diff options
author | Jon Paul Maloy <jon.maloy@ericsson.com> | 2015-05-14 10:46:13 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-05-14 12:24:45 -0400 |
commit | a6bf70f792963b32e410e5c3d2f96903265b090a (patch) | |
tree | 11ca9a3895e3976b9f3ab34b3ce79d077076ae03 /net/tipc/node.c | |
parent | 75b44b018eb086fb461aa0351d2ecb1eba684302 (diff) | |
download | linux-a6bf70f792963b32e410e5c3d2f96903265b090a.tar.gz linux-a6bf70f792963b32e410e5c3d2f96903265b090a.tar.bz2 linux-a6bf70f792963b32e410e5c3d2f96903265b090a.zip |
tipc: simplify include dependencies
When we try to add new inline functions in the code, we sometimes
run into circular include dependencies.
The main problem is that the file core.h, which really should be at
the root of the dependency chain, instead is a leaf. I.e., core.h
includes a number of header files that themselves should be allowed
to include core.h. In reality this is unnecessary, because core.h does
not need to know the full signature of any of the structs it refers to,
only their type declaration.
In this commit, we remove all dependencies from core.h towards any
other tipc header file.
As a consequence of this change, we can now move the function
tipc_own_addr(net) from addr.c to addr.h, and make it inline.
There are no functional changes in this commit.
Reviewed-by: Erik Hugne <erik.hugne@ericsson.com>
Reviewed-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc/node.c')
-rw-r--r-- | net/tipc/node.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/tipc/node.c b/net/tipc/node.c index 22c059ad2999..eb3856bb8c5a 100644 --- a/net/tipc/node.c +++ b/net/tipc/node.c @@ -39,6 +39,7 @@ #include "node.h" #include "name_distr.h" #include "socket.h" +#include "bcast.h" static void node_lost_contact(struct tipc_node *n_ptr); static void node_established_contact(struct tipc_node *n_ptr); |