Thread (8 messages) flat view 8 messages, 3 authors, 2021-02-16

Re: [EXT] Re: Phylink flow control support on ports with MLO_AN_FIXED auto negotiation

From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
Date: 2021-02-15 16:27:08

Possibly related (same subject, not in this thread)

On Mon, Feb 15, 2021 at 04:19:19PM +0000, Stefan Chulski wrote:
quoted
quoted
quoted
I discussed it with Andrew earlier last year, and his response was:

 DT configuration of pause for fixed link probably is sufficient. I
don't  remember it ever been really discussed for DSA. It was a
Melanox  discussion about limiting pause for the CPU. So I think it
is safe to  not implement ethtool -A, at least until somebody has a
real use case  for it.

So I chose not to support it - no point supporting features that
people aren't using. If you have a "real use case" then it can be added.
This patch may be sufficient - I haven't fully considered all the
implications of changing this though.
Did you try this patch? What's the outcome?
For me patch worked as expected.
Great, thanks. Andrew, do you have any further opinions on this
subject, or shall I sent a proper patch for net-next?

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help