mbox series

[net,v8,0/3] Handle new Microchip KSZ 9897 Errata

Message ID 20240625160520.358945-1-enguerrand.de-ribaucourt@savoirfairelinux.com (mailing list archive)
Headers show
Series Handle new Microchip KSZ 9897 Errata | expand

Message

Enguerrand de Ribaucourt June 25, 2024, 4:05 p.m. UTC
These patches implement some suggested workarounds from the Microchip KSZ 9897
Errata [1].

[1] https://ww1.microchip.com/downloads/aemDocuments/documents/UNG/ProductDocuments/Errata/KSZ9897R-Errata-DS80000758.pdf

---
v8:
 - split monitoring function in two to fit into 80 columns
v7: https://lore.kernel.org/netdev/20240621144322.545908-1-enguerrand.de-ribaucourt@savoirfairelinux.com/
 - use dev_crit_once instead of dev_crit_ratelimited
 - add a comment to help users understand the consequences of half-duplex errors
v6: https://lore.kernel.org/netdev/20240614094642.122464-1-enguerrand.de-ribaucourt@savoirfairelinux.com/
 - remove KSZ9897 phy_id workaround (was a configuration issue)
 - use macros for checking link down in monitoring function
 - check if VLAN is enabled before monitoring resources
v5: https://lore.kernel.org/all/20240604092304.314636-1-enguerrand.de-ribaucourt@savoirfairelinux.com/
 - use macros for bitfields
 - rewrap comments
 - check ksz_pread* return values
 - fix spelling mistakes
 - remove KSZ9477 suspend/resume deletion patch
v4: https://lore.kernel.org/all/20240531142430.678198-1-enguerrand.de-ribaucourt@savoirfairelinux.com/
 - Rebase on net/main
 - Add Fixes: tags to the patches
 - reverse x-mas tree order
 - use pseudo phy_id instead of match_phy_device
v3: https://lore.kernel.org/all/20240530102436.226189-1-enguerrand.de-ribaucourt@savoirfairelinux.com/

Comments

Jakub Kicinski June 26, 2024, 1:47 a.m. UTC | #1
On Tue, 25 Jun 2024 16:05:17 +0000 Enguerrand de Ribaucourt wrote:
> v8:
>  - split monitoring function in two to fit into 80 columns
> v7: https://lore.kernel.org/netdev/20240621144322.545908-1-enguerrand.de-ribaucourt@savoirfairelinux.com/

too late, v7 has been applied:

https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/commit/?id=4ae2c67840a0a3c88cd71fc3013f958d60f7e50c

:(
Enguerrand de Ribaucourt June 26, 2024, 7:28 a.m. UTC | #2
Ok, I can do an extra commit then?

On 26/06/2024 03:47, Jakub Kicinski wrote:
> On Tue, 25 Jun 2024 16:05:17 +0000 Enguerrand de Ribaucourt wrote:
>> v8:
>>   - split monitoring function in two to fit into 80 columns
>> v7: https://lore.kernel.org/netdev/20240621144322.545908-1-enguerrand.de-ribaucourt@savoirfairelinux.com/
> 
> too late, v7 has been applied:
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/commit/?id=4ae2c67840a0a3c88cd71fc3013f958d60f7e50c
> 
> :(
Jakub Kicinski June 26, 2024, 1:25 p.m. UTC | #3
On Wed, 26 Jun 2024 09:28:16 +0200 Enguerrand de Ribaucourt wrote:
> Ok, I can do an extra commit then?

Sure, but if it's cosmetic (no functional changes) please wait until
Friday and target net-next.