Re: [PATCH net v1] lan743x: fix ethernet frame cutoff issue

2021-04-09 Thread George McCollister
4 bytes of the FCS; not just 2") > Link: https://lore.kernel.org/lkml/20210408172353.21143-1-thesve...@gmail.com/ > Signed-off-by: Sven Van Asbroeck I'm glad everyone was able to work together to get this fixed properly without any figure pointing or mud slinging! Kudos everyone.

Re: [PATCH net v1] Revert "lan743x: trim all 4 bytes of the FCS; not just 2"

2021-04-08 Thread George McCollister
On Thu, Apr 8, 2021 at 1:35 PM Sven Van Asbroeck wrote: > > Hi George, > > On Thu, Apr 8, 2021 at 2:26 PM Sven Van Asbroeck wrote: > > > > George, I will send a patch for you to try shortly. Except if you're > > already ahead :) > > Would this work for you? It does for me. Works for me too. > >

Re: [PATCH net v1] Revert "lan743x: trim all 4 bytes of the FCS; not just 2"

2021-04-08 Thread George McCollister
On Thu, Apr 8, 2021 at 12:46 PM Sven Van Asbroeck wrote: > > Hi George, > > On Thu, Apr 8, 2021 at 1:36 PM George McCollister > wrote: > > > > Can you explain the difference in behavior with what I was observing > > on the LAN7431? > > I'm not usin

Re: [PATCH net v1] Revert "lan743x: trim all 4 bytes of the FCS; not just 2"

2021-04-08 Thread George McCollister
Sven Van Asbroeck > --- > > To: Bryan Whitehead > To: "David S. Miller" > To: Jakub Kicinski > To: George McCollister > Cc: unglinuxdri...@microchip.com > Cc: netdev@vger.kernel.org > Cc: linux-ker...@vger.kernel.org > > drivers/net/ethernet/microchip/la

Re: net: phylink: phylink_helper_basex_speed issues with 1000base-x

2021-04-02 Thread George McCollister
On Thu, Apr 1, 2021 at 5:33 PM Russell King - ARM Linux admin wrote: > > Hi, > > I hadn't responded earlier because I wanted to think about it more, > but then forgot about this email. > > On Thu, Mar 25, 2021 at 11:36:26AM -0500, George McCollister wrote: > > When

Re: DSA

2021-03-31 Thread George McCollister
t. If any of you have interest in getting these patches upstream it would be great to work with someone on it. You can find what I have here: https://github.com/gmccollister/linux/tree/net-i210-mv88e6390 Regards, George McCollister

net: phylink: phylink_helper_basex_speed issues with 1000base-x

2021-03-25 Thread George McCollister
for MLO_AN_INBAND or also for ML_AN_FIXED? Do we want phylink_validate called in phylink_create even though it gets called anyway for MLO_AN_INBAND and ML_AN_FIXED later? Regards, George McCollister

Re: [PATCH net] net: dsa: don't assign an error value to tag_ops

2021-03-22 Thread George McCollister
On Mon, Mar 22, 2021 at 3:46 PM Vladimir Oltean wrote: > > On Mon, Mar 22, 2021 at 03:26:50PM -0500, George McCollister wrote: > > Use a temporary variable to hold the return value from > > dsa_tag_driver_get() instead of assigning it to dst->tag_ops. Leaving > > an

[PATCH net] net: dsa: don't assign an error value to tag_ops

2021-03-22 Thread George McCollister
keep a copy of the tagging protocol in the DSA switch tree") Signed-off-by: George McCollister --- net/dsa/dsa2.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/net/dsa/dsa2.c b/net/dsa/dsa2.c index eb709d988c54..8f9e35e1aa89 100644 --- a/net/dsa/dsa2.c +++

[PATCH net] net: dsa: xrs700x: check if partner is same as port in hsr join

2021-03-08 Thread George McCollister
idn't work depending on the order the ports were added to the hsr device. Fixes: bd62e6f5e6a9 ("net: dsa: xrs700x: add HSR offloading support") Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-)

[PATCH net] lan743x: trim all 4 bytes of the FCS; not just 2

2021-03-05 Thread George McCollister
Trim all 4 bytes of the received FCS; not just 2 of them. Leaving 2 bytes of the FCS on the frame breaks DSA tailing tag drivers. Fixes: a8db76d40e4d ("lan743x: boost performance on cpu archs w/o dma cache snooping") Signed-off-by: George McCollister --- drivers/net/ethernet

Re: [RFC PATCH net-next 10/12] Documentation: networking: dsa: add paragraph for the HSR/PRP offload

2021-02-25 Thread George McCollister
On Wed, Feb 24, 2021 at 7:42 PM Andrew Lunn wrote: > > > +IEC 62439-3 (HSR/PRP) > > +- > > + > > +The Parallel Redundancy Protocol (PRP) is a network redundancy protocol > > which > > +works by duplicating and sequence numbering packets through two > > independent L2 > > +net

Re: [PATCH] net: hsr: add support for EntryForgetTime

2021-02-22 Thread George McCollister
On Mon, Feb 22, 2021 at 7:38 AM Wenzel, Marco wrote: > > On Fri, Feb 19, 2021 at 2:14 PM : George McCollister > wrote: > > > > On Fri, Feb 19, 2021 at 2:27 AM Wenzel, Marco > eberle.de> wrote: > > > > > > On Thu, Feb 18, 2021 at 6:06 PM : George M

Re: [RFC PATCH net-next 10/12] Documentation: networking: dsa: add paragraph for the HSR/PRP offload

2021-02-22 Thread George McCollister
On Sun, Feb 21, 2021 at 3:34 PM Vladimir Oltean wrote: > > From: Vladimir Oltean > > Add a short summary of the methods that a driver writer must implement > for offloading a HSR/PRP network interface. > > Cc: George McCollister > Signed-off-by: Vladimir Oltean

Re: [PATCH net-next] net: dsa: Fix dependencies with HSR

2021-02-20 Thread George McCollister
still fail to > link because it calls hsr_get_version() and for that one there is no > easy solution. Thanks for looking into this. It's not something I've run into before. It didn't occur to me what would happen if HSR was a module. I'll look out for this in the future. Sorry for the inconvenience. Reviewed-by: George McCollister -George > -- > Florian

Re: [PATCH] net: hsr: add support for EntryForgetTime

2021-02-19 Thread George McCollister
On Fri, Feb 19, 2021 at 2:27 AM Wenzel, Marco wrote: > > On Thu, Feb 18, 2021 at 6:06 PM : George McCollister > wrote: > > > > On Thu, Feb 18, 2021 at 9:01 AM Marco Wenzel > eberle.de> wrote: > > > > > > In IEC 62439-3 EntryForgetTime is defined w

Re: [PATCH] net: hsr: add support for EntryForgetTime

2021-02-18 Thread George McCollister
mmunication problem? > -- > 2.30.0 > scripts/checkpatch.pl gives errors about DOS line endings but once that is resolved this looks good. I tested it on an HSR network with the software implementation and the xrs700x which uses offloading and everything still works. I don't have a way to force anything on the HSR network to reuse sequence numbers after 400ms. Reviewed-by: George McCollister

Re: HSR/PRP sequence counter issue with Cisco Redbox

2021-02-17 Thread George McCollister
On Wed, Feb 17, 2021 at 7:14 AM Wenzel, Marco wrote: > > On Mon, Feb 15, 2021 at 5:49 PM George McCollister > wrote: > > > > On Mon, Feb 15, 2021 at 6:30 AM Wenzel, Marco > eberle.de> wrote: [snip] > > I was not so familiar with kernel patching until now and h

Re: HSR/PRP sequence counter issue with Cisco Redbox

2021-02-15 Thread George McCollister
n.h b/net/hsr/hsr_main.h > index 7dc92ce5a134..f79ca55d6986 100644 > --- a/net/hsr/hsr_main.h > +++ b/net/hsr/hsr_main.h > @@ -21,6 +21,7 @@ > #define HSR_LIFE_CHECK_INTERVAL 2000 /* ms */ > #define HSR_NODE_FORGET_TIME 6 /* ms */ > #def

Re: [PATCH net-next 4/4] net: dsa: don't set skb->offload_fwd_mark when not offloading the bridge

2021-02-15 Thread George McCollister
On Sun, Feb 14, 2021 at 9:54 AM Vladimir Oltean wrote: [snip] > diff --git a/net/dsa/tag_xrs700x.c b/net/dsa/tag_xrs700x.c > index 858cdf9d2913..215ecceea89e 100644 > --- a/net/dsa/tag_xrs700x.c > +++ b/net/dsa/tag_xrs700x.c > @@ -45,8 +45,7 @@ static struct sk_buff *xrs700x_rcv(struct sk_buff *sk

[PATCH net-next v3 4/4] net: dsa: xrs700x: add HSR offloading support

2021-02-09 Thread George McCollister
Add offloading for HSR/PRP (IEC 62439-3) tag insertion, tag removal forwarding and duplication supported by the xrs7000 series switches. Only HSR v1 and PRP v1 are supported by the xrs7000 series switches (HSR v0 is not). Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x.c

[PATCH net-next v3 3/4] net: dsa: add support for offloading HSR

2021-02-09 Thread George McCollister
driver should then set netdev feature flags for the HSR/PRP operation that it offloads. NETIF_F_HW_HSR_TAG_INS NETIF_F_HW_HSR_TAG_RM NETIF_F_HW_HSR_FWD NETIF_F_HW_HSR_DUP Signed-off-by: George McCollister --- include/net/dsa.h | 13 + net/dsa/dsa_priv.h | 11

[PATCH net-next v3 2/4] net: hsr: add offloading support

2021-02-09 Thread George McCollister
ports. Export is_hsr_master so DSA can tell them apart from other devices in dsa_slave_changeupper. Signed-off-by: George McCollister --- Documentation/networking/netdev-features.rst | 21 + include/linux/if_hsr.h | 27 +++ include

[PATCH net-next v3 1/4] net: hsr: generate supervision frame without HSR/PRP tag

2021-02-09 Thread George McCollister
allow offloading. Signed-off-by: George McCollister --- net/hsr/hsr_device.c | 39 +++ net/hsr/hsr_forward.c | 8 +++- 2 files changed, 14 insertions(+), 33 deletions(-) diff --git a/net/hsr/hsr_device.c b/net/hsr/hsr_device.c index ab953a1a0d6c..ec6a68b403d5 1

[PATCH net-next v3 0/4] add HSR offloading support for DSA switches

2021-02-09 Thread George McCollister
void. * Remove hsr_init_skb() proto argument. (Vladimir Oltean) George McCollister (4): net: hsr: generate supervision frame without HSR/PRP tag net: hsr: add offloading support net: dsa: add support for offloading HSR net: dsa: xrs700x: add HSR offloading support Documentation/networking/n

Re: [PATCH net-next v2 3/4] net: dsa: add support for offloading HSR

2021-02-09 Thread George McCollister
On Tue, Feb 9, 2021 at 12:51 PM Vladimir Oltean wrote: > > On Tue, Feb 09, 2021 at 12:37:38PM -0600, George McCollister wrote: > > On Tue, Feb 9, 2021 at 11:20 AM Vladimir Oltean wrote: > > > > > > On Mon, Feb 08, 2021 at 11:21:26AM -0600, George McCollister wrote

[PATCH net-next 2/2] net: dsa: xrs700x: use of_match_ptr() on xrs700x_mdio_dt_ids

2021-02-09 Thread George McCollister
Use of_match_ptr() on xrs700x_mdio_dt_ids so that NULL is substituted when CONFIG_OF isn't defined. This will prevent unnecessary use of xrs700x_mdio_dt_ids when CONFIG_OF isn't defined. Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x_mdio.c | 3 ++- 1 file

[PATCH net-next 1/2] net: dsa: xrs700x: fix unused warning for of_device_id

2021-02-09 Thread George McCollister
Reported-by: kernel test robot Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x_i2c.c | 2 +- drivers/net/dsa/xrs700x/xrs700x_mdio.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/dsa/xrs700x/xrs700x_i2c.c b/drivers/net/dsa/xrs700

Re: [PATCH net-next v2 3/4] net: dsa: add support for offloading HSR

2021-02-09 Thread George McCollister
On Tue, Feb 9, 2021 at 11:20 AM Vladimir Oltean wrote: > > On Mon, Feb 08, 2021 at 11:21:26AM -0600, George McCollister wrote: > > > If you return zero, the software fallback is never going to kick in. > > > > For join and leave? How is this not a problem for the b

Re: [PATCH net-next v2 0/4] add HSR offloading support for DSA switches

2021-02-09 Thread George McCollister
On Tue, Feb 9, 2021 at 8:38 AM Tobias Waldekranz wrote: > > On Mon, Feb 08, 2021 at 15:09, George McCollister > wrote: > > On Mon, Feb 8, 2021 at 2:16 PM Tobias Waldekranz > > wrote: > >> > >> On Thu, Feb 04, 2021 at 15:59, George McCollister > &g

Re: [PATCH net-next v2 0/4] add HSR offloading support for DSA switches

2021-02-08 Thread George McCollister
On Mon, Feb 8, 2021 at 2:16 PM Tobias Waldekranz wrote: > > On Thu, Feb 04, 2021 at 15:59, George McCollister > wrote: > > Add support for offloading HSR/PRP (IEC 62439-3) tag insertion, tag > > removal, forwarding and duplication on DSA switches. > > This series add

Re: [PATCH net-next v2 1/4] net: hsr: generate supervision frame without HSR/PRP tag

2021-02-08 Thread George McCollister
On Sat, Feb 6, 2021 at 7:26 PM Vladimir Oltean wrote: > > On Thu, Feb 04, 2021 at 03:59:23PM -0600, George McCollister wrote: > > @@ -289,23 +286,12 @@ static void send_hsr_supervision_frame(struct > > hsr_port *master, > > hsr->announce_count++; > &g

Re: [PATCH net-next v2 3/4] net: dsa: add support for offloading HSR

2021-02-08 Thread George McCollister
On Sat, Feb 6, 2021 at 5:29 PM Vladimir Oltean wrote: > > On Thu, Feb 04, 2021 at 03:59:25PM -0600, George McCollister wrote: > > @@ -1935,6 +1936,19 @@ static int dsa_slave_changeupper(struct net_device > > *dev, > > dsa_port_lag_l

Re: [RESEND PATCH net-next 1/4] net: hsr: generate supervision frame without HSR tag

2021-02-08 Thread George McCollister
On Sat, Feb 6, 2021 at 5:43 PM Vladimir Oltean wrote: > > On Tue, Feb 02, 2021 at 08:49:25AM -0600, George McCollister wrote: > > > > > Why is it such a big deal if supervision frames have HSR/PRP tag or > > > > > not? > > > > > > > > B

Re: [PATCH net-next v2 4/4] net: dsa: xrs700x: add HSR offloading support

2021-02-08 Thread George McCollister
On Sat, Feb 6, 2021 at 5:53 PM Vladimir Oltean wrote: > > On Thu, Feb 04, 2021 at 03:59:26PM -0600, George McCollister wrote: > > +static int xrs700x_hsr_join(struct dsa_switch *ds, int port, > > + struct net_device *hsr) > > +{ &g

[PATCH net-next v2 1/4] net: hsr: generate supervision frame without HSR/PRP tag

2021-02-04 Thread George McCollister
allow offloading. Signed-off-by: George McCollister --- net/hsr/hsr_device.c | 32 net/hsr/hsr_forward.c | 6 +- 2 files changed, 9 insertions(+), 29 deletions(-) diff --git a/net/hsr/hsr_device.c b/net/hsr/hsr_device.c index ab953a1a0d6c..161b8da6a21d 100644 ---

[PATCH net-next v2 0/4] add HSR offloading support for DSA switches

2021-02-04 Thread George McCollister
. (Vladimir Oltean) * eliminate IFF_HSR and use is_hsr_master instead. (Vladimir Oltean) * Make hsr_handle_sup_frame handle skb_std as well (required when offloading) * Don't add hsr tag for HSR v0 supervisory frames. * Fixed tag insertion offloading for PRP. George McCollister (4): net

[PATCH net-next v2 3/4] net: dsa: add support for offloading HSR

2021-02-04 Thread George McCollister
driver should then set netdev feature flags for the HSR/PRP operation that it offloads. NETIF_F_HW_HSR_TAG_INS NETIF_F_HW_HSR_TAG_RM NETIF_F_HW_HSR_FWD NETIF_F_HW_HSR_DUP Signed-off-by: George McCollister --- include/net/dsa.h | 13 + net/dsa/dsa_priv.h | 11

[PATCH net-next v2 2/4] net: hsr: add offloading support

2021-02-04 Thread George McCollister
ports. Export is_hsr_master so DSA can tell them apart from other devices in dsa_slave_changeupper. Signed-off-by: George McCollister --- Documentation/networking/netdev-features.rst | 21 + include/linux/if_hsr.h | 27 +++ include

[PATCH net-next v2 4/4] net: dsa: xrs700x: add HSR offloading support

2021-02-04 Thread George McCollister
Add offloading for HSR/PRP (IEC 62439-3) tag insertion, tag removal forwarding and duplication supported by the xrs7000 series switches. Only HSR v1 and PRP v1 are supported by the xrs7000 series switches (HSR v0 is not). Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x.c

Re: [PATCH net-next 1/2] net: add EXPORT_INDIRECT_CALLABLE wrapper

2021-02-04 Thread George McCollister
I don't see the second patch. Regards, George McCollister

Re: [RESEND PATCH net-next 2/4] net: hsr: add offloading support

2021-02-03 Thread George McCollister
On Mon, Feb 1, 2021 at 9:23 AM Vladimir Oltean wrote: [snip] > > @@ -357,6 +367,7 @@ static void hsr_forward_do(struct hsr_frame_info *frame) > > { > > struct hsr_port *port; > > struct sk_buff *skb; > > + bool sent = false; > > > > hsr_for_each_port(frame->port_rcv->hsr, po

Re: [PATCH net-next] net: dsa: xrs700x: Correctly address device over I2C

2021-02-02 Thread George McCollister
[2] = reg >> 7 & 0xff; > - buf[3] = (reg & 0x7f) << 1 | 1; > - buf[4] = val >> 8 & 0xff; > - buf[5] = val & 0xff; > + cmd.reg = cpu_to_be32(reg); > + cmd.val = cpu_to_be16(val); > > - ret = i2c_master_send(i2c, buf, sizeof(buf)); > + ret = i2c_master_send(i2c, (char *)&cmd, sizeof(cmd)); > if (ret < 0) { > dev_err(dev, "xrs i2c_master_send returned %d\n", ret); > return ret; > -- > 2.17.1 > Reviewed-by: George McCollister

Re: [RESEND PATCH net-next 1/4] net: hsr: generate supervision frame without HSR tag

2021-02-02 Thread George McCollister
On Mon, Feb 1, 2021 at 6:37 PM Vladimir Oltean wrote: > > On Mon, Feb 01, 2021 at 01:43:43PM -0600, George McCollister wrote: > > > > diff --git a/net/hsr/hsr_device.c b/net/hsr/hsr_device.c > > > > index ab953a1a0d6c..161b8da6a21d 100644 > > > > --- a/

Re: [RESEND PATCH net-next 4/4] net: dsa: xrs700x: add HSR offloading support

2021-02-01 Thread George McCollister
On Mon, Feb 1, 2021 at 9:29 AM Vladimir Oltean wrote: > > On Mon, Feb 01, 2021 at 08:05:03AM -0600, George McCollister wrote: > > Add offloading for HSR/PRP (IEC 62439-3) tag insertion, tag removal > > forwarding and duplication supported by the xrs7000 series switches. >

Re: [RESEND PATCH net-next 2/4] net: hsr: add offloading support

2021-02-01 Thread George McCollister
On Mon, Feb 1, 2021 at 9:23 AM Vladimir Oltean wrote: > > On Mon, Feb 01, 2021 at 08:05:01AM -0600, George McCollister wrote: > > Add support for offloading of HSR/PRP (IEC 62439-3) tag insertion > > tag removal, duplicate generation and forwarding. > > > > Use a n

Re: [RESEND PATCH net-next 1/4] net: hsr: generate supervision frame without HSR tag

2021-02-01 Thread George McCollister
On Mon, Feb 1, 2021 at 8:59 AM Vladimir Oltean wrote: > > On Mon, Feb 01, 2021 at 08:05:00AM -0600, George McCollister wrote: > > Generate supervision frame without HSR/PRP tag and rely on existing > > code which inserts it later. > > This will allow HSR/PRP tag insertion

[RESEND PATCH net-next 0/4] add HSR offloading support for DSA switches

2021-02-01 Thread George McCollister
lease repost" Changes since RFC: * Split hsr and dsa patches. (Florian Fainelli) George McCollister (4): net: hsr: generate supervision frame without HSR tag net: hsr: add offloading support net: dsa: add support for offloading HSR net: dsa: xrs700x: add HSR offloading support Doc

[RESEND PATCH net-next 4/4] net: dsa: xrs700x: add HSR offloading support

2021-02-01 Thread George McCollister
Add offloading for HSR/PRP (IEC 62439-3) tag insertion, tag removal forwarding and duplication supported by the xrs7000 series switches. Only HSR v1 and PRP v1 are supported by the xrs7000 series switches (HSR v0 is not). Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x.c

[RESEND PATCH net-next 3/4] net: dsa: add support for offloading HSR

2021-02-01 Thread George McCollister
driver should then set netdev feature flags for the HSR/PRP operation that it offloads. NETIF_F_HW_HSR_TAG_INS NETIF_F_HW_HSR_TAG_RM NETIF_F_HW_HSR_FWD NETIF_F_HW_HSR_DUP Signed-off-by: George McCollister --- include/net/dsa.h | 13 + net/dsa/dsa_priv.h | 11

[RESEND PATCH net-next 1/4] net: hsr: generate supervision frame without HSR tag

2021-02-01 Thread George McCollister
Generate supervision frame without HSR/PRP tag and rely on existing code which inserts it later. This will allow HSR/PRP tag insertions to be offloaded in the future. Signed-off-by: George McCollister --- net/hsr/hsr_device.c | 32 net/hsr/hsr_forward.c | 10

[RESEND PATCH net-next 2/4] net: hsr: add offloading support

2021-02-01 Thread George McCollister
redundant ports. This is required because the inserted HSR/PRP header/trailer must contain the same sequence number on the frames sent out both redundant ports. Signed-off-by: George McCollister --- Documentation/networking/netdev-features.rst | 20 include/linux/if_hsr.h

Re: HSR/PRP sequence counter issue with Cisco Redbox

2021-01-28 Thread George McCollister
ot;Time after which an entry is removed from the duplicate table" with a value of 400ms and states devices should usually be configured to keep entries in the table for a much shorter time. hsr_framereg.c needs to be reworked to handle this according to the specification. > > Thanks > Marco Wenzel Regards, George McCollister

Re: [RFC PATCH net-next 2/3] net: hsr: add DSA offloading support

2021-01-26 Thread George McCollister
On Sun, Jan 24, 2021 at 2:43 AM Vladimir Oltean wrote: > [snip] > > I would like to give HSR a spin to get a better idea of what you're doing, but > it's kinda hard when this happens out of the box, with none of your changes > already: > > [ 1385.000453] hsr0: hw csum failure > [ 1385.004105] skb

Re: [RFC PATCH net-next 2/3] net: hsr: add DSA offloading support

2021-01-25 Thread George McCollister
On Sun, Jan 24, 2021 at 11:29 AM Vladimir Oltean wrote: > > On Fri, Jan 22, 2021 at 09:59:47AM -0600, George McCollister wrote: > > diff --git a/net/dsa/slave.c b/net/dsa/slave.c > > index f2fb433f3828..fc7e3ff11c5c 100644 > > --- a/net/dsa/slave.c > > +++ b/

Re: [RFC PATCH net-next 2/3] net: hsr: add DSA offloading support

2021-01-22 Thread George McCollister
On Fri, Jan 22, 2021 at 11:56 AM Florian Fainelli wrote: > > > > On 1/22/2021 7:59 AM, George McCollister wrote: > > Add support for offloading of HSR/PRP (IEC 62439-3) tag insertion > > tag removal, duplicate generation and forwarding on DSA switches. > > > >

[RFC PATCH net-next 2/3] net: hsr: add DSA offloading support

2021-01-22 Thread George McCollister
automatically sending a single frame from the CPU port to both redundant ports. This is required because the inserted HSR/PRP header/trailer must contain the same sequence number on the frames sent out both redundant ports. Signed-off-by: George McCollister --- Documentation/networking/netdev

[RFC PATCH net-next 0/3] add HSR offloading support for DSA switches

2021-01-22 Thread George McCollister
series. Since many may not be familiar with HSR/PRP let me know if I omitted anything in the commit messages which might have seemed obvious to me. George McCollister (3): net: hsr: generate supervision frame without HSR tag net: hsr: add DSA offloading support net: dsa: xrs700x: add HSR

[RFC PATCH net-next 3/3] net: dsa: xrs700x: add HSR offloading support

2021-01-22 Thread George McCollister
Add offloading for HSR/PRP (IEC 62439-3) tag insertion, tag removal forwarding and duplication supported by the xrs7000 series switches. Only HSR v1 and PRP v1 are supported by the xrs7000 series switches (HSR v0 is not). Signed-off-by: George McCollister --- drivers/net/dsa/xrs700x/xrs700x.c

[RFC PATCH net-next 1/3] net: hsr: generate supervision frame without HSR tag

2021-01-22 Thread George McCollister
Generate supervision frame without HSR/PRP tag and rely on existing code which inserts it later. This will allow HSR/PRP tag insertions to be offloaded in the future. Signed-off-by: George McCollister --- net/hsr/hsr_device.c | 32 net/hsr/hsr_forward.c | 10

[PATCH] MAINTAINERS: add entry for Arrow SpeedChips XRS7000 driver

2021-01-20 Thread George McCollister
Add myself as maintainer of the Arrow SpeedChips XRS7000 series Ethernet switch driver. Suggested-by: Jakub Kicinski Signed-off-by: George McCollister --- MAINTAINERS | 8 1 file changed, 8 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 096b584e7fed..992530a481a0 100644

Re: [PATCH net-next v5 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-15 Thread George McCollister
On Thu, Jan 14, 2021 at 4:48 PM Vladimir Oltean wrote: > > On Thu, Jan 14, 2021 at 01:57:33PM -0600, George McCollister wrote: [snip] > > Reviewed-by: Vladimir Oltean > > This driver is good to go, just one small nitpick below, you can fix it > up afterwards if you wan

[PATCH net-next v5 3/3] dt-bindings: net: dsa: add bindings for xrs700x switches

2021-01-14 Thread George McCollister
Add documentation and an example for Arrow SpeedChips XRS7000 Series single chip Ethernet switches. Signed-off-by: George McCollister Reviewed-by: Florian Fainelli --- .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 73 ++ 1 file changed, 73 insertions(+) create mode

[PATCH net-next v5 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-14 Thread George McCollister
. Support for advanced features such as PTP and HSR/PRP (IEC 62439-3 Clause 5 & 4) is not included in this patch and may be added at a later date. Signed-off-by: George McCollister --- drivers/net/dsa/Kconfig| 2 + drivers/net/dsa/Makefile | 1 + drivers/net

[PATCH net-next v5 1/3] dsa: add support for Arrow XRS700x tag trailer

2021-01-14 Thread George McCollister
Add support for Arrow SpeedChips XRS700x single byte tag trailer. This is modeled on tag_trailer.c which works in a similar way. Signed-off-by: George McCollister Reviewed-by: Andrew Lunn Reviewed-by: Florian Fainelli --- include/net/dsa.h | 2 ++ net/dsa/Kconfig | 6 + net

[PATCH net-next v5 0/3] Arrow SpeedChips XRS700x DSA Driver

2021-01-14 Thread George McCollister
i2c/mdio data before registering. (Florian Fainelli) George McCollister (3): dsa: add support for Arrow XRS700x tag trailer net: dsa: add Arrow SpeedChips XRS700x driver dt-bindings: net: dsa: add bindings for xrs700x switches .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 73 +++ d

Re: [PATCH net-next v4 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-14 Thread George McCollister
On Thu, Jan 14, 2021 at 12:32 PM Vladimir Oltean wrote: > > > May boil down to preference too, but I don't believe "dev" is a happy > > > name to give to a driver private data structure. > > > > There are other drivers in the subsystem that do this. If there was a > > consistent pattern followed i

Re: [PATCH net-next v4 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-14 Thread George McCollister
On Thu, Jan 14, 2021 at 11:28 AM Florian Fainelli wrote: > > On 1/13/21 6:59 AM, George McCollister wrote: > > Add a driver with initial support for the Arrow SpeedChips XRS7000 > > series of gigabit Ethernet switch chips which are typically used in > > critical

Re: [PATCH net-next v4 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-14 Thread George McCollister
On Wed, Jan 13, 2021 at 7:57 PM Vladimir Oltean wrote: > > What PHY interface types does the switch support as of this patch? > No RGMII delay configuration needed? > Port 0: RMII Port 1-3: RGMII For RGMII the documentation states: "PCB is required to add 1.5 ns to 2.0 ns more delay to the clock

Re: [PATCH net-next v4 1/3] dsa: add support for Arrow XRS700x tag trailer

2021-01-14 Thread George McCollister
On Wed, Jan 13, 2021 at 7:05 PM Vladimir Oltean wrote: > > +++ b/net/dsa/tag_xrs700x.c > > @@ -0,0 +1,67 @@ > > +// SPDX-License-Identifier: GPL-2.0+ > > +/* > > + * XRS700x tag format handling > > + * Copyright (c) 2008-2009 Marvell Semiconductor > > Why does Marvell get copyright? What Andrew s

[PATCH net-next v4 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2021-01-13 Thread George McCollister
. Support for advanced features such as PTP and HSR/PRP (IEC 62439-3 Clause 5 & 4) is not included in this patch and may be added at a later date. Signed-off-by: George McCollister --- drivers/net/dsa/Kconfig| 2 + drivers/net/dsa/Makefile | 1 + drivers/net

[PATCH net-next v4 0/3] Arrow SpeedChips XRS700x DSA Driver

2021-01-13 Thread George McCollister
in devicetree examples. (Rob Herring) * Use ethernet-port(s) in devicetree examples. (Rob Herring) * Use strscpy() instead of strlcpy(). George McCollister (3): dsa: add support for Arrow XRS700x tag trailer net: dsa: add Arrow SpeedChips XRS700x driver dt-bindings: net: dsa: add bindings for xr

[PATCH net-next v4 3/3] dt-bindings: net: dsa: add bindings for xrs700x switches

2021-01-13 Thread George McCollister
Add documentation and an example for Arrow SpeedChips XRS7000 Series single chip Ethernet switches. Signed-off-by: George McCollister Reviewed-by: Florian Fainelli --- .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 73 ++ 1 file changed, 73 insertions(+) create mode

[PATCH net-next v4 1/3] dsa: add support for Arrow XRS700x tag trailer

2021-01-13 Thread George McCollister
Add support for Arrow SpeedChips XRS700x single byte tag trailer. This is modeled on tag_trailer.c which works in a similar way. Signed-off-by: George McCollister Reviewed-by: Andrew Lunn Reviewed-by: Florian Fainelli --- include/net/dsa.h | 2 ++ net/dsa/Kconfig | 6 + net

Re: [PATCH net-next v3 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-12-07 Thread George McCollister
On Mon, Dec 7, 2020 at 6:40 PM kernel test robot wrote: >drivers/net/dsa/xrs700x/xrs700x.c:511:3: error: 'const struct > dsa_switch_ops' has no member named 'get_stats64' This patch depends on "net: dsa: add optional stats64 support" being merged first: https://lore.kernel.org/netdev/2020120

[PATCH net-next v3 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-12-07 Thread George McCollister
. Support for advanced features such as PTP and HSR/PRP (IEC 62439-3 Clause 5 & 4) is not included in this patch and may be added at a later date. Signed-off-by: George McCollister --- This patch depends on "net: dsa: add optional stats64 support" being merged first. https://lore.ker

[PATCH net-next v3 3/3] dt-bindings: net: dsa: add bindings for xrs700x switches

2020-12-07 Thread George McCollister
Add documentation and an example for Arrow SpeedChips XRS7000 Series single chip Ethernet switches. Signed-off-by: George McCollister Reviewed-by: Florian Fainelli --- .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 74 ++ 1 file changed, 74 insertions(+) create mode

[PATCH net-next v3 1/3] dsa: add support for Arrow XRS700x tag trailer

2020-12-07 Thread George McCollister
Add support for Arrow SpeedChips XRS700x single byte tag trailer. This is modeled on tag_trailer.c which works in a similar way. Signed-off-by: George McCollister Reviewed-by: Andrew Lunn Reviewed-by: Florian Fainelli --- include/net/dsa.h | 2 ++ net/dsa/Kconfig | 6 + net

[PATCH net-next v3 0/3] Arrow SpeedChips XRS700x DSA Driver

2020-12-07 Thread George McCollister
tats via .get_stats64. (Jakub Kicinski, Vladimir Oltean) * Use a 3 second polling rate for counters. George McCollister (3): dsa: add support for Arrow XRS700x tag trailer net: dsa: add Arrow SpeedChips XRS700x driver dt-bindings: net: dsa: add bindings for xrs700x switches .../devi

Re: [PATCH v4 net-next 1/2] net: dsa: add optional stats64 support

2020-12-07 Thread George McCollister
On Fri, Dec 4, 2020 at 8:59 AM Oleksij Rempel wrote: > > Allow DSA drivers to export stats64 > > Signed-off-by: Oleksij Rempel > Reviewed-by: Vladimir Oltean Reviewed-by: George McCollister I've already updated my xrs700x dsa driver for v3 to use this. I'm blocked on

Re: [PATCH v4 net-next 2/2] net: dsa: qca: ar9331: export stats64

2020-12-04 Thread George McCollister
On Fri, Dec 4, 2020 at 8:59 AM Oleksij Rempel wrote: > > Add stats support for the ar9331 switch. > > Signed-off-by: Oleksij Rempel > --- > drivers/net/dsa/qca/ar9331.c | 247 ++- > 1 file changed, 246 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/dsa/

Re: [PATCH v4 net-next 2/2] net: dsa: qca: ar9331: export stats64

2020-12-04 Thread George McCollister
On Fri, Dec 4, 2020 at 8:59 AM Oleksij Rempel wrote: > > Add stats support for the ar9331 switch. > > Signed-off-by: Oleksij Rempel > --- > drivers/net/dsa/qca/ar9331.c | 247 ++- > 1 file changed, 246 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/dsa/

Re: [PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-30 Thread George McCollister
On Mon, Nov 30, 2020 at 5:50 PM Vladimir Oltean wrote: > > On Mon, Nov 30, 2020 at 10:52:35AM -0600, George McCollister wrote: > > Another possible option could be replacing for_each_netdev_rcu with > > for_each_netdev_srcu and using list_for_each_entry_srcu (though it&#x

Re: [PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-30 Thread George McCollister
On Fri, Nov 27, 2020 at 8:15 PM Jakub Kicinski wrote: > > On Sat, 28 Nov 2020 03:41:06 +0200 Vladimir Oltean wrote: > > Jakub, I would like to hear more from you. I would still like to try > > this patch out. You clearly have a lot more background with the code. > > Well, I've seen people run into

Re: [PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-27 Thread George McCollister
On Fri, Nov 27, 2020 at 1:50 PM Vladimir Oltean wrote: > > On Fri, Nov 27, 2020 at 12:47:41PM -0600, George McCollister wrote: > > On Fri, Nov 27, 2020, 12:35 PM Jakub Kicinski wrote: > > > > > On Fri, 27 Nov 2020 00:05:00 +0200 Vladimir Oltean wrote: > > >

Re: [PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-26 Thread George McCollister
On Thu, Nov 26, 2020 at 11:56 AM Vladimir Oltean wrote: > > On Thu, Nov 26, 2020 at 03:24:18PM +0200, Vladimir Oltean wrote: > > On Wed, Nov 25, 2020 at 08:25:11PM -0600, George McCollister wrote: > > > > > + {XRS_RX_UNDERSIZE_L, "rx_undersize"}

Re: [PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-25 Thread George McCollister
On Wed, Nov 25, 2020 at 7:42 PM Jakub Kicinski wrote: > > On Wed, 25 Nov 2020 13:37:39 -0600 George McCollister wrote: > > Add a driver with initial support for the Arrow SpeedChips XRS7000 > > series of gigabit Ethernet switch chips which are typically used in >

[PATCH net-next v2 0/3] Arrow SpeedChips XRS700x DSA Driver

2020-11-25 Thread George McCollister
drew Lunn) * Add inbound policy to allow BPDUs. (Andrew Lunn) * Move files into their own subdir. (Vladimir Oltean) * Automate regmap field allocation. (Vladimir Oltean) * Move setting link speed to .mac_link_up. (Vladimir Oltean) * Use different compatible strings for e/f variants. George Mc

[PATCH net-next v2 3/3] dt-bindings: net: dsa: add bindings for xrs700x switches

2020-11-25 Thread George McCollister
Add documentation and an example for Arrow SpeedChips XRS7000 Series single chip Ethernet switches. Signed-off-by: George McCollister --- .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 74 ++ 1 file changed, 74 insertions(+) create mode 100644 Documentation/devicetree

[PATCH net-next v2 1/3] dsa: add support for Arrow XRS700x tag trailer

2020-11-25 Thread George McCollister
Add support for Arrow SpeedChips XRS700x single byte tag trailer. This is modeled on tag_trailer.c which works in a similar way. Signed-off-by: George McCollister --- include/net/dsa.h | 2 ++ net/dsa/Kconfig | 6 + net/dsa/Makefile | 1 + net/dsa/tag_xrs700x.c | 67

[PATCH net-next v2 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-25 Thread George McCollister
. Support for advanced features such as PTP and HSR/PRP (IEC 62439-3 Clause 5 & 4) is not included in this patch and may be added at a later date. Signed-off-by: George McCollister --- drivers/net/dsa/Kconfig| 2 + drivers/net/dsa/Makefile | 1 + drivers/net

Re: [PATCH net-next 1/3] dsa: add support for Arrow XRS700x tag trailer

2020-11-24 Thread George McCollister
On Mon, Nov 23, 2020 at 4:18 PM Florian Fainelli wrote: > > > > On 11/20/2020 10:16 AM, George McCollister wrote: > > Add support for Arrow SpeedChips XRS700x single byte tag trailer. This > > is modeled on tag_trailer.c which works in a similar way. > > > &g

Re: [PATCH net-next 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-23 Thread George McCollister
On Mon, Nov 23, 2020 at 4:09 PM Andrew Lunn wrote: > > > > > https://www.flexibilis.com/downloads/xrs/SpeedChip_XRS7000_3000_User_Manual.pdf > > Section 6.1.4 > > The forwarding decision is presented in Figure 19. Note that also > frames coming into a disabled port are received to the buffer memor

Re: [PATCH net-next 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-23 Thread George McCollister
On Sun, Nov 22, 2020 at 5:39 PM Vladimir Oltean wrote: > > Hi George, > > On Fri, Nov 20, 2020 at 12:16:26PM -0600, George McCollister wrote: > > Add a driver with initial support for the Arrow SpeedChips XRS7000 > > series of gigabit Ethernet switch chips which are typica

Re: [PATCH net-next 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-23 Thread George McCollister
On Fri, Nov 20, 2020 at 5:24 PM Andrew Lunn wrote: > > Hi George > > > > > +static void xrs700x_port_stp_state_set(struct dsa_switch *ds, int port, > > > > +u8 state) > > > > +{ > > > > + struct xrs700x *priv = ds->priv; > > > > + unsigned int val; > > >

Re: [PATCH net-next 2/3] net: dsa: add Arrow SpeedChips XRS700x driver

2020-11-20 Thread George McCollister
On Fri, Nov 20, 2020 at 1:33 PM Andrew Lunn wrote: > > > +static const struct xrs700x_mib xrs700x_mibs[] = { > > + {XRS_RX_GOOD_OCTETS_L(0), "rx_good_octets"}, > > + {XRS_RX_BAD_OCTETS_L(0), "rx_bad_octets"}, > > + {XRS_RX_UNICAST_L(0), "rx_unicast"}, > > + {XRS_RX_BROADCAST_L(0),

Re: [PATCH net-next 1/3] dsa: add support for Arrow XRS700x tag trailer

2020-11-20 Thread George McCollister
On Fri, Nov 20, 2020 at 12:58 PM Andrew Lunn wrote: > > On Fri, Nov 20, 2020 at 12:16:25PM -0600, George McCollister wrote: > > Add support for Arrow SpeedChips XRS700x single byte tag trailer. This > > is modeled on tag_trailer.c which works in a similar way. > > &

[PATCH net-next 0/3] Arrow SpeedChips XRS700x DSA Driver

2020-11-20 Thread George McCollister
439-3 Clause 5 & 4) offloading with integration into net/hsr. George McCollister (3): dsa: add support for Arrow XRS700x tag trailer net: dsa: add Arrow SpeedChips XRS700x driver dt-bindings: net: dsa: add bindings for xrs700x switches .../devicetree/bindings/net/dsa/arrow,xrs700x.y

[PATCH net-next 3/3] dt-bindings: net: dsa: add bindings for xrs700x switches

2020-11-20 Thread George McCollister
Add documentation and an example for Arrow SpeedChips XRS7000 Series single chip Ethernet switches. Signed-off-by: George McCollister --- .../devicetree/bindings/net/dsa/arrow,xrs700x.yaml | 72 ++ 1 file changed, 72 insertions(+) create mode 100644 Documentation/devicetree

[PATCH net-next 1/3] dsa: add support for Arrow XRS700x tag trailer

2020-11-20 Thread George McCollister
Add support for Arrow SpeedChips XRS700x single byte tag trailer. This is modeled on tag_trailer.c which works in a similar way. Signed-off-by: George McCollister --- include/net/dsa.h | 2 ++ net/dsa/Kconfig | 6 net/dsa/Makefile | 1 + net/dsa/tag_xrs700x.c | 91

  1   2   >