From: Marc Gonzalez <marc_gonza...@sigmadesigns.com> Date: Tue, 25 Jul 2017 11:31:46 +0200
> In the current code, old and new PHY states are always logged. > From now on, log only PHY state transitions. > > Signed-off-by: Marc Gonzalez <marc_gonza...@sigmadesigns.com> > --- > drivers/net/phy/phy.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c > index d0626bf5c540..6bb764e716fc 100644 > --- a/drivers/net/phy/phy.c > +++ b/drivers/net/phy/phy.c > @@ -1226,7 +1226,8 @@ void phy_state_machine(struct work_struct *work) > if (err < 0) > phy_error(phydev); > > - phydev_dbg(phydev, "PHY state change %s -> %s\n", > + if (old_state != phydev->state) > + phydev_dbg(phydev, "PHY state change %s -> %s\n", > phy_state_to_str(old_state), > phy_state_to_str(phydev->state)); Something is not kosher with this indentation at all.
