From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH net-next 01/12] net: phy: ste10Xp: Remove wrong SUPPORTED_Pause Date: Tue, 4 Sep 2018 03:20:36 +0200 Message-ID: <20180904012036.GA18913@lunn.ch> References: <1535908001-18593-1-git-send-email-andrew@lunn.ch> <1535908001-18593-2-git-send-email-andrew@lunn.ch> <5481c281-98ec-cfc1-2972-646c3191f6c8@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Miller , netdev , maxime.chevallier@bootlin.com To: Florian Fainelli Return-path: Received: from vps0.lunn.ch ([185.16.172.187]:51172 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725992AbeIDFnY (ORCPT ); Tue, 4 Sep 2018 01:43:24 -0400 Content-Disposition: inline In-Reply-To: <5481c281-98ec-cfc1-2972-646c3191f6c8@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: 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