Skip to content

Commit 0fb52a2

Browse files
shemmingerdavem330
authored andcommitted
team: cleanup netpoll clode
This started out with fixing a sparse warning, then I realized that the wrapper function team_netpoll_info could just be collapsed away by rolling it into the enable code. Signed-off-by: Stephen Hemminger <[email protected]> Acked-by: Jiri Pirko <[email protected]> Signed-off-by: David S. Miller <[email protected]>
1 parent 93d8bf9 commit 0fb52a2

File tree

1 file changed

+8
-17
lines changed

1 file changed

+8
-17
lines changed

drivers/net/team/team.c

Lines changed: 8 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1037,6 +1037,9 @@ static int team_port_enable_netpoll(struct team *team, struct team_port *port,
10371037
struct netpoll *np;
10381038
int err;
10391039

1040+
if (!team->dev->npinfo)
1041+
return 0;
1042+
10401043
np = kzalloc(sizeof(*np), gfp);
10411044
if (!np)
10421045
return -ENOMEM;
@@ -1063,12 +1066,6 @@ static void team_port_disable_netpoll(struct team_port *port)
10631066
__netpoll_cleanup(np);
10641067
kfree(np);
10651068
}
1066-
1067-
static struct netpoll_info *team_netpoll_info(struct team *team)
1068-
{
1069-
return team->dev->npinfo;
1070-
}
1071-
10721069
#else
10731070
static int team_port_enable_netpoll(struct team *team, struct team_port *port,
10741071
gfp_t gfp)
@@ -1078,10 +1075,6 @@ static int team_port_enable_netpoll(struct team *team, struct team_port *port,
10781075
static void team_port_disable_netpoll(struct team_port *port)
10791076
{
10801077
}
1081-
static struct netpoll_info *team_netpoll_info(struct team *team)
1082-
{
1083-
return NULL;
1084-
}
10851078
#endif
10861079

10871080
static void __team_port_change_port_added(struct team_port *port, bool linkup);
@@ -1163,13 +1156,11 @@ static int team_port_add(struct team *team, struct net_device *port_dev)
11631156
goto err_vids_add;
11641157
}
11651158

1166-
if (team_netpoll_info(team)) {
1167-
err = team_port_enable_netpoll(team, port, GFP_KERNEL);
1168-
if (err) {
1169-
netdev_err(dev, "Failed to enable netpoll on device %s\n",
1170-
portname);
1171-
goto err_enable_netpoll;
1172-
}
1159+
err = team_port_enable_netpoll(team, port, GFP_KERNEL);
1160+
if (err) {
1161+
netdev_err(dev, "Failed to enable netpoll on device %s\n",
1162+
portname);
1163+
goto err_enable_netpoll;
11731164
}
11741165

11751166
err = netdev_master_upper_dev_link(port_dev, dev);

0 commit comments

Comments
 (0)