On Mon, Sep 03, 2018 at 10:53:58AM -0700, Florian Fainelli wrote: > > > On 9/2/2018 10:06 AM, Andrew Lunn wrote: > >The PHY driver should not indicate that Pause is supported. It is upto > >the MAC drive enable it, if it supports Pause frames. So remove it > >from the ste10Xp driver. > > This came up before when Timur was cleaning up the Pause|ASym_Pause > advertisment bits, and we agreed that a driver that cannot have the > Asym_Pause bit writable, e.g: bcm63xx, would have to specifically leave > SUPPORTED_Pause as a way to tell PHYLIB about that situation. See: > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/net/phy?id=529ed12752635ba8a35dc78ec70ed6f42570b4ca > > Can you check the datasheet if available?
Hi Florian Datasheet is at: https://www.st.com/resource/en/datasheet/ste100p.pdf It indicates that bit 10 of the Auto-Neg advertisement for Pause is R/W and defaults to 1. There is no support for Asym Pause, bit 11 is reserved. So we don't have the case of the bcm63xx. It should be that this PHY supports plain symmetric Pause. Andrew
