From: Srinivas Kandagatla Date: Mon, 2 Apr 2012 00:02:09 +0000 (+0000) Subject: phy:icplus:fix Auto Power Saving in ip101a_config_init. X-Git-Tag: MMI-PSA29.97-13-9~17049^2~35 X-Git-Url: https://git.stricted.de/?a=commitdiff_plain;h=b3300146aa8efc5d3937fd33f3cfdc580a3843bc;p=GitHub%2FMotorolaMobilityLLC%2Fkernel-slsi.git phy:icplus:fix Auto Power Saving in ip101a_config_init. This patch fixes Auto Power Saving configuration in ip101a_config_init which was broken as there is no phy register write followed after setting IP101A_APS_ON flag. This patch also fixes the return value of ip101a_config_init. Without this patch ip101a_config_init returns 2 which is not an error accroding to IS_ERR and the mac driver will continue accessing 2 as valid pointer to phy_dev resulting in memory fault. Signed-off-by: Srinivas Kandagatla Signed-off-by: David S. Miller --- diff --git a/drivers/net/phy/icplus.c b/drivers/net/phy/icplus.c index 0856e1b7a849..f08c85acf761 100644 --- a/drivers/net/phy/icplus.c +++ b/drivers/net/phy/icplus.c @@ -162,7 +162,8 @@ static int ip101a_g_config_init(struct phy_device *phydev) /* Enable Auto Power Saving mode */ c = phy_read(phydev, IP10XX_SPEC_CTRL_STATUS); c |= IP101A_G_APS_ON; - return c; + + return phy_write(phydev, IP10XX_SPEC_CTRL_STATUS, c); } static int ip175c_read_status(struct phy_device *phydev)