Re: [PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-29 Thread David Miller
From: Florian Fainelli 
Date: Fri, 26 May 2017 15:50:51 -0700

> On 05/26/2017 03:07 PM, Vivien Didelot wrote:
>> The port net device passed to b53_fdb_copy is not used. Remove it.
>> 
>> Signed-off-by: Vivien Didelot 
> 
> Reviewed-by: Florian Fainelli 

Applied.


Re: [PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-29 Thread David Miller
From: Florian Fainelli 
Date: Fri, 26 May 2017 15:50:51 -0700

> On 05/26/2017 03:07 PM, Vivien Didelot wrote:
>> The port net device passed to b53_fdb_copy is not used. Remove it.
>> 
>> Signed-off-by: Vivien Didelot 
> 
> Reviewed-by: Florian Fainelli 

Applied.


Re: [PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-26 Thread Florian Fainelli
On 05/26/2017 03:07 PM, Vivien Didelot wrote:
> The port net device passed to b53_fdb_copy is not used. Remove it.
> 
> Signed-off-by: Vivien Didelot 

Reviewed-by: Florian Fainelli 
-- 
Florian


Re: [PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-26 Thread Florian Fainelli
On 05/26/2017 03:07 PM, Vivien Didelot wrote:
> The port net device passed to b53_fdb_copy is not used. Remove it.
> 
> Signed-off-by: Vivien Didelot 

Reviewed-by: Florian Fainelli 
-- 
Florian


[PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-26 Thread Vivien Didelot
The port net device passed to b53_fdb_copy is not used. Remove it.

Signed-off-by: Vivien Didelot 
---
 drivers/net/dsa/b53/b53_common.c | 8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
index fa099ed41652..6a5648a9cb09 100644
--- a/drivers/net/dsa/b53/b53_common.c
+++ b/drivers/net/dsa/b53/b53_common.c
@@ -1281,8 +1281,7 @@ static void b53_arl_search_rd(struct b53_device *dev, u8 
idx,
b53_arl_to_entry(ent, mac_vid, fwd_entry);
 }
 
-static int b53_fdb_copy(struct net_device *dev, int port,
-   const struct b53_arl_entry *ent,
+static int b53_fdb_copy(int port, const struct b53_arl_entry *ent,
struct switchdev_obj_port_fdb *fdb,
switchdev_obj_dump_cb_t *cb)
 {
@@ -1304,7 +1303,6 @@ int b53_fdb_dump(struct dsa_switch *ds, int port,
 switchdev_obj_dump_cb_t *cb)
 {
struct b53_device *priv = ds->priv;
-   struct net_device *dev = ds->ports[port].netdev;
struct b53_arl_entry results[2];
unsigned int count = 0;
int ret;
@@ -1320,13 +1318,13 @@ int b53_fdb_dump(struct dsa_switch *ds, int port,
return ret;
 
b53_arl_search_rd(priv, 0, [0]);
-   ret = b53_fdb_copy(dev, port, [0], fdb, cb);
+   ret = b53_fdb_copy(port, [0], fdb, cb);
if (ret)
return ret;
 
if (priv->num_arl_entries > 2) {
b53_arl_search_rd(priv, 1, [1]);
-   ret = b53_fdb_copy(dev, port, [1], fdb, cb);
+   ret = b53_fdb_copy(port, [1], fdb, cb);
if (ret)
return ret;
 
-- 
2.13.0



[PATCH net-next] net: dsa: b53: remove unused dev argument

2017-05-26 Thread Vivien Didelot
The port net device passed to b53_fdb_copy is not used. Remove it.

Signed-off-by: Vivien Didelot 
---
 drivers/net/dsa/b53/b53_common.c | 8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
index fa099ed41652..6a5648a9cb09 100644
--- a/drivers/net/dsa/b53/b53_common.c
+++ b/drivers/net/dsa/b53/b53_common.c
@@ -1281,8 +1281,7 @@ static void b53_arl_search_rd(struct b53_device *dev, u8 
idx,
b53_arl_to_entry(ent, mac_vid, fwd_entry);
 }
 
-static int b53_fdb_copy(struct net_device *dev, int port,
-   const struct b53_arl_entry *ent,
+static int b53_fdb_copy(int port, const struct b53_arl_entry *ent,
struct switchdev_obj_port_fdb *fdb,
switchdev_obj_dump_cb_t *cb)
 {
@@ -1304,7 +1303,6 @@ int b53_fdb_dump(struct dsa_switch *ds, int port,
 switchdev_obj_dump_cb_t *cb)
 {
struct b53_device *priv = ds->priv;
-   struct net_device *dev = ds->ports[port].netdev;
struct b53_arl_entry results[2];
unsigned int count = 0;
int ret;
@@ -1320,13 +1318,13 @@ int b53_fdb_dump(struct dsa_switch *ds, int port,
return ret;
 
b53_arl_search_rd(priv, 0, [0]);
-   ret = b53_fdb_copy(dev, port, [0], fdb, cb);
+   ret = b53_fdb_copy(port, [0], fdb, cb);
if (ret)
return ret;
 
if (priv->num_arl_entries > 2) {
b53_arl_search_rd(priv, 1, [1]);
-   ret = b53_fdb_copy(dev, port, [1], fdb, cb);
+   ret = b53_fdb_copy(port, [1], fdb, cb);
if (ret)
return ret;
 
-- 
2.13.0