netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: "Marek Behún" <kabel@kernel.org>
Cc: Andrew Lunn <andrew@lunn.ch>, Vladimir Oltean <olteanv@gmail.com>,
	Vivien Didelot <vivien.didelot@gmail.com>,
	Tobias Waldekranz <tobias@waldekranz.com>,
	netdev@vger.kernel.org, Lev Olshvang <lev_o@rad.com>
Subject: Re: [PATCH net 00/13] Fixes for mv88e6xxx (mainly 6320 family)
Date: Thu, 13 Mar 2025 14:14:58 +0000	[thread overview]
Message-ID: <Z9LoYsOCWT-YUPXZ@shell.armlinux.org.uk> (raw)
In-Reply-To: <20250313134146.27087-1-kabel@kernel.org>

Overall, your patch descriptions talk about "forgot to". Realistically,
it's probably "don't know about this switch because we don't have
access to the documentation so best leave it alone"

On Thu, Mar 13, 2025 at 02:41:33PM +0100, Marek Behún wrote:
> Hello Andrew et al.,
> 
> This is a series of fixes for the mv88e6xxx driver:
> - one change removes an unused method
> - one fix for 6341 family
> - eleven fixes for 6320 family
> 
> Marek
> 
> Marek Behún (13):
>   net: dsa: mv88e6xxx: remove unused .port_max_speed_mode()
>   net: dsa: mv88e6xxx: fix VTU methods for 6320 family
>   net: dsa: mv88e6xxx: fix number of g1 interrupts for 6320 family
>   net: dsa: mv88e6xxx: allow SPEED_200 for 6320 family on supported
>     ports
>   net: dsa: mv88e6xxx: fix atu_move_port_mask for 6341 family
>   net: dsa: mv88e6xxx: enable PVT for 6321 switch
>   net: dsa: mv88e6xxx: define .pot_clear() for 6321
>   net: dsa: mv88e6xxx: enable .rmu_disable() for 6320 family
>   net: dsa: mv88e6xxx: enable .port_set_policy() for 6320 family
>   net: dsa: mv88e6xxx: enable devlink ATU hash param for 6320 family
>   net: dsa: mv88e6xxx: enable STU methods for 6320 family
>   net: dsa: mv88e6xxx: fix internal PHYs for 6320 family
>   net: dsa: mv88e6xxx: workaround RGMII transmit delay erratum for 6320
>     family
> 
>  drivers/net/dsa/mv88e6xxx/chip.c | 68 ++++++++++++++++++++++----------
>  drivers/net/dsa/mv88e6xxx/chip.h |  4 --
>  drivers/net/dsa/mv88e6xxx/port.c | 55 +++++++-------------------
>  drivers/net/dsa/mv88e6xxx/port.h | 11 +-----
>  4 files changed, 64 insertions(+), 74 deletions(-)
> 
> -- 
> 2.48.1
> 
> 

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!

      parent reply	other threads:[~2025-03-13 14:15 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-03-13 13:41 [PATCH net 00/13] Fixes for mv88e6xxx (mainly 6320 family) Marek Behún
2025-03-13 13:41 ` [PATCH net 01/13] net: dsa: mv88e6xxx: remove unused .port_max_speed_mode() Marek Behún
2025-03-13 14:25   ` Andrew Lunn
2025-03-13 13:41 ` [PATCH net 02/13] net: dsa: mv88e6xxx: fix VTU methods for 6320 family Marek Behún
2025-03-13 14:33   ` Andrew Lunn
2025-03-13 13:41 ` [PATCH net 03/13] net: dsa: mv88e6xxx: fix number of g1 interrupts " Marek Behún
2025-03-13 13:41 ` [PATCH net 04/13] net: dsa: mv88e6xxx: allow SPEED_200 for 6320 family on supported ports Marek Behún
2025-03-13 14:35   ` Andrew Lunn
2025-03-13 13:41 ` [PATCH net 05/13] net: dsa: mv88e6xxx: fix atu_move_port_mask for 6341 family Marek Behún
2025-03-13 13:41 ` [PATCH net 06/13] net: dsa: mv88e6xxx: enable PVT for 6321 switch Marek Behún
2025-03-13 13:41 ` [PATCH net 07/13] net: dsa: mv88e6xxx: define .pot_clear() for 6321 Marek Behún
2025-03-13 13:41 ` [PATCH net 08/13] net: dsa: mv88e6xxx: enable .rmu_disable() for 6320 family Marek Behún
2025-03-13 14:40   ` Andrew Lunn
2025-03-13 13:41 ` [PATCH net 09/13] net: dsa: mv88e6xxx: enable .port_set_policy() " Marek Behún
2025-03-13 13:41 ` [PATCH net 10/13] net: dsa: mv88e6xxx: enable devlink ATU hash param " Marek Behún
2025-03-13 13:41 ` [PATCH net 11/13] net: dsa: mv88e6xxx: enable STU methods " Marek Behún
2025-03-13 13:41 ` [PATCH net 12/13] net: dsa: mv88e6xxx: fix internal PHYs " Marek Behún
2025-03-13 13:41 ` [PATCH net 13/13] net: dsa: mv88e6xxx: workaround RGMII transmit delay erratum " Marek Behún
2025-03-13 14:14 ` Russell King (Oracle) [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z9LoYsOCWT-YUPXZ@shell.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=andrew@lunn.ch \
    --cc=kabel@kernel.org \
    --cc=lev_o@rad.com \
    --cc=netdev@vger.kernel.org \
    --cc=olteanv@gmail.com \
    --cc=tobias@waldekranz.com \
    --cc=vivien.didelot@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).