Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1693187 > unrolled thread

[PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125

Started byFlorian Fainelli <f.fainelli@gmail.com>
First post2017-07-20 21:30 +0200
Last post2017-07-21 00:30 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 Florian Fainelli <f.fainelli@gmail.com> - 2017-07-20 21:30 +0200
    Re: [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-07-20 23:50 +0200
    Re: [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125 David Miller <davem@davemloft.net> - 2017-07-21 00:30 +0200

#1693187 — [PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125

FromFlorian Fainelli <f.fainelli@gmail.com>
Date2017-07-20 21:30 +0200
Subject[PATCH net] net: dsa: b53: Add missing ARL entries for BCM53125
Message-ID<u5pj3-3bi-13@gated-at.bofh.it>
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

[toc] | [next] | [standalone]


#1693247

FromVivien Didelot <vivien.didelot@savoirfairelinux.com>
Date2017-07-20 23:50 +0200
Message-ID<u5ruy-4sb-7@gated-at.bofh.it>
In reply to#1693187
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>

[toc] | [prev] | [next] | [standalone]


#1693263

FromDavid Miller <davem@davemloft.net>
Date2017-07-21 00:30 +0200
Message-ID<u5s7h-4UV-21@gated-at.bofh.it>
In reply to#1693187
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.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web