[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20150217.103758.934788051050893871.davem@davemloft.net>
Date: Tue, 17 Feb 2015 10:37:58 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: linux@...ck-us.net
Cc: netdev@...r.kernel.org, f.fainelli@...il.com, andrew@...n.ch
Subject: Re: [PATCH] net: dsa: Set valid phy interface type
From: Guenter Roeck <linux@...ck-us.net>
Date: Mon, 16 Feb 2015 21:23:51 -0800
> If the phy interface mode is not found in devicetree, or if devicetree
> is not configured, of_get_phy_mode returns -ENODEV. The current code
> sets the phy interface mode to the return value from of_get_phy_mode
> without checking if it is valid.
>
> This invalid phy interface mode is passed as parameter to of_phy_connect
> or to phy_connect_direct. This sets the phy interface mode to the invalid
> value, which in turn causes problems for any code using phydev->interface.
>
> Fixes: b31f65fb4383 ("net: dsa: slave: Fix autoneg for phys on switch MDIO bus")
> Fixes: 0d8bcdd383b8 ("net: dsa: allow for more complex PHY setups")
> Cc: Florian Fainelli <f.fainelli@...il.com>
> Cc: Andrew Lunn <andrew@...n.ch>
> Signed-off-by: Guenter Roeck <linux@...ck-us.net>
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists