Skip to content

Commit 5451237

Browse files
Subash Abhinov Kasiviswanathandavem330
authored andcommitted
net: qualcomm: rmnet: Remove duplicate setting of rmnet_devices
The rmnet_devices information is already stored in muxed_ep, so storing this in rmnet_devices[] again is redundant. Signed-off-by: Subash Abhinov Kasiviswanathan <subashab@codeaurora.org> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 56470c9 commit 5451237

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

drivers/net/ethernet/qualcomm/rmnet/rmnet_config.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ struct rmnet_port {
3333
struct rmnet_endpoint muxed_ep[RMNET_MAX_LOGICAL_EP];
3434
u32 ingress_data_format;
3535
u32 egress_data_format;
36-
struct net_device *rmnet_devices[RMNET_MAX_LOGICAL_EP];
3736
u8 nr_rmnet_devs;
3837
u8 rmnet_mode;
3938
};

drivers/net/ethernet/qualcomm/rmnet/rmnet_vnd.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,12 @@ int rmnet_vnd_newlink(u8 id, struct net_device *rmnet_dev,
105105
struct rmnet_priv *priv;
106106
int rc;
107107

108-
if (port->rmnet_devices[id])
108+
if (port->muxed_ep[id].egress_dev)
109109
return -EINVAL;
110110

111111
rc = register_netdevice(rmnet_dev);
112112
if (!rc) {
113-
port->rmnet_devices[id] = rmnet_dev;
113+
port->muxed_ep[id].egress_dev = rmnet_dev;
114114
port->nr_rmnet_devs++;
115115

116116
rmnet_dev->rtnl_link_ops = &rmnet_link_ops;
@@ -127,10 +127,10 @@ int rmnet_vnd_newlink(u8 id, struct net_device *rmnet_dev,
127127

128128
int rmnet_vnd_dellink(u8 id, struct rmnet_port *port)
129129
{
130-
if (id >= RMNET_MAX_LOGICAL_EP || !port->rmnet_devices[id])
130+
if (id >= RMNET_MAX_LOGICAL_EP || !port->muxed_ep[id].egress_dev)
131131
return -EINVAL;
132132

133-
port->rmnet_devices[id] = NULL;
133+
port->muxed_ep[id].egress_dev = NULL;
134134
port->nr_rmnet_devs--;
135135
return 0;
136136
}

0 commit comments

Comments
 (0)