Às 8:07 PM de 12/27/2016, Kweh, Hock Leong escreveu: > From: "Kweh, Hock Leong" <hock.leong.k...@intel.com> > > Fixing the gmac4 mdio write access to use MII_GMAC4_WRITE only instead of > OR together with MII_WRITE. > > Signed-off-by: Kweh, Hock Leong <hock.leong.k...@intel.com> > --- > drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c > b/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c > index fda01f7..b0344c2 100644 > --- a/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c > +++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_mdio.c > @@ -116,7 +116,7 @@ static int stmmac_mdio_write(struct mii_bus *bus, int > phyaddr, int phyreg, > unsigned int mii_address = priv->hw->mii.addr; > unsigned int mii_data = priv->hw->mii.data; > > - u32 value = MII_WRITE | MII_BUSY; > + u32 value = MII_BUSY; > > value |= (phyaddr << priv->hw->mii.addr_shift) > & priv->hw->mii.addr_mask; > @@ -126,6 +126,8 @@ static int stmmac_mdio_write(struct mii_bus *bus, int > phyaddr, int phyreg, > & priv->hw->mii.clk_csr_mask; > if (priv->plat->has_gmac4) > value |= MII_GMAC4_WRITE; > + else > + value |= MII_WRITE; > > /* Wait until any existing MII operation is complete */ > if (stmmac_mdio_busy_wait(priv->ioaddr, mii_address)) >
Acked-By: Joao Pinto <jpi...@synopsys.com>