diff mbox series

net: dsa: bcm_sf2: use 2 Gbps IMP port link on BCM4908

Message ID 20210310125159.28533-1-zajec5@gmail.com (mailing list archive)
State Accepted
Commit 8373a0fe9c7160a55482effa8a3f725efd3f8434
Delegated to: Netdev Maintainers
Headers show
Series net: dsa: bcm_sf2: use 2 Gbps IMP port link on BCM4908 | expand

Checks

Context Check Description
netdev/cover_letter success Link
netdev/fixes_present success Link
netdev/patch_count success Link
netdev/tree_selection success Guessed tree name to be net-next
netdev/subject_prefix warning Target tree name not specified in the subject
netdev/cc_maintainers success CCed 7 of 7 maintainers
netdev/source_inline success Was 0 now: 0
netdev/verify_signedoff success Link
netdev/module_param success Was 0 now: 0
netdev/build_32bit success Errors and warnings before: 0 this patch: 0
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/verify_fixes success Link
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 11 lines checked
netdev/build_allmodconfig_warn success Errors and warnings before: 0 this patch: 0
netdev/header_inline success Link

Commit Message

Rafał Miłecki March 10, 2021, 12:51 p.m. UTC 
From: Rafał Miłecki <rafal@milecki.pl>

BCM4908 uses 2 Gbps link between switch and the Ethernet interface.
Without this BCM4908 devices were able to achieve only 2 x ~895 Mb/s.
This allows handling e.g. NAT traffic with 940 Mb/s.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 drivers/net/dsa/bcm_sf2.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Florian Fainelli March 10, 2021, 5 p.m. UTC | #1 
On 3/10/21 4:51 AM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
> 
> BCM4908 uses 2 Gbps link between switch and the Ethernet interface.
> Without this BCM4908 devices were able to achieve only 2 x ~895 Mb/s.
> This allows handling e.g. NAT traffic with 940 Mb/s.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>

Acked-by: Florian Fainelli <f.fainelli@gmail.com>
patchwork-bot+netdevbpf@kernel.org March 10, 2021, 11:40 p.m. UTC | #2 
Hello:

This patch was applied to netdev/net.git (refs/heads/master):

On Wed, 10 Mar 2021 13:51:59 +0100 you wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
> 
> BCM4908 uses 2 Gbps link between switch and the Ethernet interface.
> Without this BCM4908 devices were able to achieve only 2 x ~895 Mb/s.
> This allows handling e.g. NAT traffic with 940 Mb/s.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> 
> [...]

Here is the summary with links:
  - net: dsa: bcm_sf2: use 2 Gbps IMP port link on BCM4908
    https://git.kernel.org/netdev/net/c/8373a0fe9c71

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
diff mbox series

Patch

diff --git a/drivers/net/dsa/bcm_sf2.c b/drivers/net/dsa/bcm_sf2.c
index 7583fc12a9d7..d183206c4bb4 100644
--- a/drivers/net/dsa/bcm_sf2.c
+++ b/drivers/net/dsa/bcm_sf2.c
@@ -114,7 +114,10 @@  static void bcm_sf2_imp_setup(struct dsa_switch *ds, int port)
 		/* Force link status for IMP port */
 		reg = core_readl(priv, offset);
 		reg |= (MII_SW_OR | LINK_STS);
-		reg &= ~GMII_SPEED_UP_2G;
+		if (priv->type == BCM4908_DEVICE_ID)
+			reg |= GMII_SPEED_UP_2G;
+		else
+			reg &= ~GMII_SPEED_UP_2G;
 		core_writel(priv, reg, offset);
 
 		/* Enable Broadcast, Multicast, Unicast forwarding to IMP port */