netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125
@ 2017-07-20 19:25 Florian Fainelli
  2017-07-20 21:41 ` Vivien Didelot
  2017-07-20 22:22 ` David Miller
  0 siblings, 2 replies; 3+ messages in thread
From: Florian Fainelli @ 2017-07-20 19:25 UTC (permalink / raw)
  To: netdev; +Cc: davem, Florian Fainelli, Andrew Lunn, Vivien Didelot, open list

The BCM53125 entry was missing an arl_entries member which would
basically prevent the ARL search from terminating properly. This switch
has 4 ARL entries, so add that.

Fixes: 1da6df85c6fb ("net: dsa: b53: Implement ARL add/del/dump operations")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 drivers/net/dsa/b53/b53_common.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/net/dsa/b53/b53_common.c b/drivers/net/dsa/b53/b53_common.c
index e68d368e20ac..7f36d3e3c98b 100644
--- a/drivers/net/dsa/b53/b53_common.c
+++ b/drivers/net/dsa/b53/b53_common.c
@@ -1665,6 +1665,7 @@ static const struct b53_chip_data b53_switch_chips[] = {
 		.dev_name = "BCM53125",
 		.vlans = 4096,
 		.enabled_ports = 0xff,
+		.arl_entries = 4,
 		.cpu_port = B53_CPU_PORT,
 		.vta_regs = B53_VTA_REGS,
 		.duplex_reg = B53_DUPLEX_STAT_GE,
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125
  2017-07-20 19:25 [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 Florian Fainelli
@ 2017-07-20 21:41 ` Vivien Didelot
  2017-07-20 22:22 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: Vivien Didelot @ 2017-07-20 21:41 UTC (permalink / raw)
  To: Florian Fainelli, netdev; +Cc: davem, Florian Fainelli, Andrew Lunn, open list

Florian Fainelli <f.fainelli@gmail.com> writes:

> The BCM53125 entry was missing an arl_entries member which would
> basically prevent the ARL search from terminating properly. This switch
> has 4 ARL entries, so add that.
>
> Fixes: 1da6df85c6fb ("net: dsa: b53: Implement ARL add/del/dump operations")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Reviewed-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125
  2017-07-20 19:25 [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 Florian Fainelli
  2017-07-20 21:41 ` Vivien Didelot
@ 2017-07-20 22:22 ` David Miller
  1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2017-07-20 22:22 UTC (permalink / raw)
  To: f.fainelli; +Cc: netdev, andrew, vivien.didelot, linux-kernel

From: Florian Fainelli <f.fainelli@gmail.com>
Date: Thu, 20 Jul 2017 12:25:22 -0700

> The BCM53125 entry was missing an arl_entries member which would
> basically prevent the ARL search from terminating properly. This switch
> has 4 ARL entries, so add that.
> 
> Fixes: 1da6df85c6fb ("net: dsa: b53: Implement ARL add/del/dump operations")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Applied and queued up for -stable, thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-07-20 22:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-20 19:25 [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 Florian Fainelli
2017-07-20 21:41 ` Vivien Didelot
2017-07-20 22:22 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).