From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King Subject: Re: [PATCH net-next 2/5] net: phy: Export gen10g_* functions Date: Wed, 28 Feb 2018 19:43:23 +0000 Message-ID: <20180228194323.GA30733@flint.armlinux.org.uk> References: <20180228193612.29409-1-f.fainelli@gmail.com> <20180228193612.29409-3-f.fainelli@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: netdev@vger.kernel.org, Andrew Lunn , open list To: Florian Fainelli Return-path: Content-Disposition: inline In-Reply-To: <20180228193612.29409-3-f.fainelli@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Wed, Feb 28, 2018 at 11:36:09AM -0800, Florian Fainelli wrote: > In order to remove a fair amount of duplication in the different 10G PHY > drivers, export all gen10g_* functions to be able to make use of those. The gen10g functions tend to be barely functional - for example, gen10g_soft_reset() doesn't do the soft-reset thing, which could be argued to be a bug, as there's a generic way to do that. However, doing that for the Marvell 10G driver would be a big mistake. The PHY crashes if you try to set the reset bit in any of the control register 1s. -- Russell King