netdev - Re: [PATCH net v3 0/3] Fix broken link with TH1520 GMAC when linkspeed changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: 
 <175500191323.2536290.8876313312572303826.git-patchwork-notify@kernel.org>
Date: Tue, 12 Aug 2025 12:31:53 +0000
From: patchwork-bot+netdevbpf@...nel.org
To: Yao Zi <ziyao@...root.org>
Cc: fustini@...nel.org, guoren@...nel.org, wefu@...hat.com,
 andrew+netdev@...n.ch, davem@...emloft.net, edumazet@...gle.com,
 kuba@...nel.org, pabeni@...hat.com, robh@...nel.org, krzk+dt@...nel.org,
 conor+dt@...nel.org, paul.walmsley@...ive.com, palmer@...belt.com,
 aou@...s.berkeley.edu, alex@...ti.fr, emil.renner.berthing@...onical.com,
 jszhang@...nel.org, linux-riscv@...ts.infradead.org, netdev@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net v3 0/3] Fix broken link with TH1520 GMAC when
 linkspeed
 changes

Hello:

This series was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@...hat.com>:

On Fri,  8 Aug 2025 09:36:53 +0000 you wrote:
> It's noted that on TH1520 SoC, the GMAC's link becomes broken after
> the link speed is changed (for example, running ethtool -s eth0 speed
> 100 on the peer when negotiated to 1Gbps), but the GMAC could function
> normally if the speed is brought back to the initial.
> 
> Just like many other SoCs utilizing STMMAC IP, we need to adjust the TX
> clock supplying TH1520's GMAC through some SoC-specific glue registers
> when linkspeed changes. But it's found that after the full kernel
> startup, reading from them results in garbage and writing to them makes
> no effect, which is the cause of broken link.
> 
> [...]

Here is the summary with links:
  - [net,v3,1/3] dt-bindings: net: thead,th1520-gmac: Describe APB interface clock
    https://git.kernel.org/netdev/net/c/c8a9a619c072
  - [net,v3,2/3] net: stmmac: thead: Get and enable APB clock on initialization
    https://git.kernel.org/netdev/net/c/4cc339ce482b
  - [net,v3,3/3] riscv: dts: thead: Add APB clocks for TH1520 GMACs
    https://git.kernel.org/netdev/net/c/a7f75e2883c4

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