RE: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-24 Thread Bryan.Whitehead
Hi Florian, see below. > > /* bus->mdio_lock should be locked when using this function */ > > +/* Page should already be set to MSCC_PHY_PAGE_EXTENDED_GPIO */ > > +static int vsc8574_wait_for_micro_complete(struct phy_device *phydev) > > +{ > > + u16 timeout = 500; > > + u16 reg18g = 0; >

RE: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-24 Thread Bryan.Whitehead
et-next] mscc: Add LCPLL Reset to VSC8574 Family of phy > drivers > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the > content is safe > > On 7/23/20 1:09 PM, Bryan Whitehead wrote: > > The LCPLL Reset sequence is added to the initializati

RE: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-24 Thread Bryan.Whitehead
Thanks Andrew, I will apply your suggestions. > -Original Message- > From: Andrew Lunn > Sent: Friday, July 24, 2020 9:19 AM > To: Bryan Whitehead - C21958 > Cc: da...@davemloft.net; netdev@vger.kernel.org; UNGLinuxDriver > > Subject: Re: [PATCH net-next] msc

Re: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-24 Thread Florian Fainelli
On 7/24/20 9:29 AM, bryan.whiteh...@microchip.com wrote: > Hi Florian, see below. > >>> /* bus->mdio_lock should be locked when using this function */ >>> +/* Page should already be set to MSCC_PHY_PAGE_EXTENDED_GPIO */ >>> +static int vsc8574_wait_for_micro_complete(struct phy_device *phydev) >>

Re: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-24 Thread Andrew Lunn
On Thu, Jul 23, 2020 at 04:09:57PM -0400, Bryan Whitehead wrote: > The LCPLL Reset sequence is added to the initialization path > of the VSC8574 Family of phy drivers. > > The LCPLL Reset sequence is known to reduce hardware inter-op > issues when using the QSGMII MAC interface. > > This patch is

Re: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-23 Thread kernel test robot
Hi Bryan, I love your patch! Perhaps something to improve: [auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Bryan-Whitehead/mscc-Add-LCPLL-Reset-to-VSC8574-Family-of-phy-drivers/20200724-043103 base: https://git.kernel.org/pub/scm/linux/kernel/git/

[PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-23 Thread Bryan Whitehead
The LCPLL Reset sequence is added to the initialization path of the VSC8574 Family of phy drivers. The LCPLL Reset sequence is known to reduce hardware inter-op issues when using the QSGMII MAC interface. This patch is submitted to net-next to avoid merging conflicts that may arise if submitted t

Re: [PATCH net-next] mscc: Add LCPLL Reset to VSC8574 Family of phy drivers

2020-07-23 Thread Florian Fainelli
On 7/23/20 1:09 PM, Bryan Whitehead wrote: > The LCPLL Reset sequence is added to the initialization path > of the VSC8574 Family of phy drivers. > > The LCPLL Reset sequence is known to reduce hardware inter-op > issues when using the QSGMII MAC interface. > > This patch is submitted to net-next