Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

net: dsa: kill off dsa_priv.h

The last remnants in dsa_priv.h are a netlink-related definition for
which we create a new header, and DSA_MAX_NUM_OFFLOADING_BRIDGES which
is only used from dsa.c, so move it there.

Some inclusions need to be adjusted now that we no longer have headers
included transitively from dsa_priv.h.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

authored by

Vladimir Oltean and committed by
Jakub Kicinski
5917bfe6 19d05ea7

+14 -25
+3 -1
net/dsa/dsa.c
··· 21 21 22 22 #include "devlink.h" 23 23 #include "dsa.h" 24 - #include "dsa_priv.h" 25 24 #include "master.h" 25 + #include "netlink.h" 26 26 #include "port.h" 27 27 #include "slave.h" 28 28 #include "switch.h" 29 29 #include "tag.h" 30 + 31 + #define DSA_MAX_NUM_OFFLOADING_BRIDGES BITS_PER_LONG 30 32 31 33 static DEFINE_MUTEX(dsa2_mutex); 32 34 LIST_HEAD(dsa_tree_list);
-19
net/dsa/dsa_priv.h
··· 1 - /* SPDX-License-Identifier: GPL-2.0-or-later */ 2 - /* 3 - * net/dsa/dsa_priv.h - Hardware switch handling 4 - * Copyright (c) 2008-2009 Marvell Semiconductor 5 - */ 6 - 7 - #ifndef __DSA_PRIV_H 8 - #define __DSA_PRIV_H 9 - 10 - #include <linux/phy.h> 11 - #include <linux/netdevice.h> 12 - #include <net/dsa.h> 13 - 14 - #define DSA_MAX_NUM_OFFLOADING_BRIDGES BITS_PER_LONG 15 - 16 - /* netlink.c */ 17 - extern struct rtnl_link_ops dsa_link_ops __read_mostly; 18 - 19 - #endif
+1 -1
net/dsa/netlink.c
··· 4 4 #include <linux/netdevice.h> 5 5 #include <net/rtnetlink.h> 6 6 7 - #include "dsa_priv.h" 7 + #include "netlink.h" 8 8 #include "slave.h" 9 9 10 10 static const struct nla_policy dsa_policy[IFLA_DSA_MAX + 1] = {
+8
net/dsa/netlink.h
··· 1 + /* SPDX-License-Identifier: GPL-2.0-or-later */ 2 + 3 + #ifndef __DSA_NETLINK_H 4 + #define __DSA_NETLINK_H 5 + 6 + extern struct rtnl_link_ops dsa_link_ops __read_mostly; 7 + 8 + #endif
+1 -1
net/dsa/slave.c
··· 23 23 #include <linux/netpoll.h> 24 24 25 25 #include "dsa.h" 26 - #include "dsa_priv.h" 27 26 #include "port.h" 28 27 #include "master.h" 28 + #include "netlink.h" 29 29 #include "slave.h" 30 30 #include "tag.h" 31 31
+1 -1
net/dsa/switch.c
··· 13 13 #include <net/switchdev.h> 14 14 15 15 #include "dsa.h" 16 - #include "dsa_priv.h" 16 + #include "netlink.h" 17 17 #include "port.h" 18 18 #include "slave.h" 19 19 #include "switch.h"
-1
net/dsa/tag_8021q.c
··· 7 7 #include <linux/if_vlan.h> 8 8 #include <linux/dsa/8021q.h> 9 9 10 - #include "dsa_priv.h" 11 10 #include "port.h" 12 11 #include "switch.h" 13 12 #include "tag.h"
-1
net/dsa/tag_hellcreek.c
··· 11 11 #include <linux/skbuff.h> 12 12 #include <net/dsa.h> 13 13 14 - #include "dsa_priv.h" 15 14 #include "tag.h" 16 15 17 16 #define HELLCREEK_NAME "hellcreek"