[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Tue, 3 Mar 2020 15:20:16 +0000
From: Russell King - ARM Linux admin <linux@...linux.org.uk>
To: Antoine Tenart <antoine.tenart@...tlin.com>
Cc: Andrew Lunn <andrew@...n.ch>,
Florian Fainelli <f.fainelli@...il.com>,
Heiner Kallweit <hkallweit1@...il.com>,
"David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org
Subject: Re: [PATCH net-next 1/3] net: phy: marvell10g: add mdix control
On Tue, Mar 03, 2020 at 04:09:58PM +0100, Antoine Tenart wrote:
> Hello Russell,
>
> On Tue, Mar 03, 2020 at 02:43:57PM +0000, Russell King wrote:
> >
> > +static int mv3310_config_mdix(struct phy_device *phydev)
> > +{
> > + u16 val;
> > + int err;
> > +
> > + switch (phydev->mdix_ctrl) {
> > + case ETH_TP_MDI_AUTO:
> > + val = MV_PCS_CSCR1_MDIX_AUTO;
> > + break;
> > + case ETH_TP_MDI_X:
> > + val = MV_PCS_CSCR1_MDIX_MDIX;
> > + break;
> > + case ETH_TP_MDI:
> > + val = MV_PCS_CSCR1_MDIX_MDI;
> > + break;
> > + default:
> > + return -EINVAL;
> > + }
> > +
> > + err = phy_modify_mmd_changed(phydev, MDIO_MMD_PCS, MV_PCS_CSCR1,
> > + MV_PCS_CSCR1_MDIX_MASK, val);
> > + if (err < 0)
> > + return err;
> > +
> > + return mv3310_maybe_reset(phydev, MV_PCS_BASE_T, err > 0);
>
> This helper is only introduced in patch 2.
Hmm, must have happened when I reordered the patches, and I hadn't
noticed. Thanks, v2 coming soon.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 12.1Mbps down 622kbps up
According to speedtest.net: 11.9Mbps down 500kbps up
Powered by blists - more mailing lists