[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <BLUPR03MB373807C6285A5A9D43CADDDF5830@BLUPR03MB373.namprd03.prod.outlook.com>
Date: Wed, 22 Jul 2015 06:58:10 +0000
From: Duan Andy <fugang.duan@...escale.com>
To: Peter Chen <Peter.Chen@...escale.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
CC: Andrew Lunn <andrew@...n.ch>,
"David S. Miller" <davem@...emloft.net>
Subject: RE: A Regression between v4.2-rc2 and v4.2-rc3
From: Peter Chen <peter.chen@...escale.com> Sent: Wednesday, July 22, 2015 1:45 PM
> To: netdev@...r.kernel.org
> Cc: Andrew Lunn; Duan Fugang-B38611; David S. Miller
> Subject: A Regression between v4.2-rc2 and v4.2-rc3
>
> Hi List,
>
> I run out a kernel oops [2] for nfsroot at several imx6 boards when
> rebase to v4.2-rc3, after revert below patch [1], it is ok.
> This patch is just adding runtime pm for ipg clock, I am wonder why it
> takes as a bug fix.
>
> [1]
> commit 6c3e921b18edca290099adfddde8a50236bf2d80
> Author: Andrew Lunn <andrew@...n.ch>
> Date: Mon Jul 6 20:34:55 2015 +0200
>
> net: fec: Ensure clocks are enabled while using mdio bus
>
> When a switch is attached to the mdio bus, the mdio bus can be used
> while the interface is not open. If the IPG clock is not enabled,
> MDIO
> reads/writes will simply time out.
>
> Add support for runtime PM to control this clock. Enable/disable this
> clock using runtime PM, with open()/close() and mdio read()/write()
> function triggering runtime PM operations. Since PM is optional, the
> IPG clock is enabled at probe and is no longer modified by
> fec_enet_clk_enable(), thus if PM is not enabled in the kernel, it is
> guaranteed the clock is running when MDIO operations are performed.
>
> Signed-off-by: Andrew Lunn <andrew@...n.ch>
> Acked-by: Fugang Duan <B38611@...escale.com>
> Signed-off-by: David S. Miller <davem@...emloft.net>
>
The patch was reverted in last week.
Regards,
Andy
--
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