* FEC driver hangs hardware on i.MX6SX @ 2014-07-20 2:57 ` Shawn Guo 0 siblings, 0 replies; 10+ messages in thread From: Shawn Guo @ 2014-07-20 2:57 UTC (permalink / raw) To: linux-arm-kernel Hi Fugang, Your commit e8fcfcd5684a (net: fec: optimize the clock management to save power) landed on mainline with v3.16-rc1. It causes a hardware hang on i.MX6SX, if the rootfs is not on NFS but something else like MMC. I think the cause is that PTP is still accessing registers after FEC driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. Can you please try to provide a fix for this regression soon? Shawn ^ permalink raw reply [flat|nested] 10+ messages in thread
* FEC driver hangs hardware on i.MX6SX @ 2014-07-20 2:57 ` Shawn Guo 0 siblings, 0 replies; 10+ messages in thread From: Shawn Guo @ 2014-07-20 2:57 UTC (permalink / raw) To: Fugang Duan Cc: Frank Li, David S. Miller, Shawn Guo, netdev, linux-arm-kernel Hi Fugang, Your commit e8fcfcd5684a (net: fec: optimize the clock management to save power) landed on mainline with v3.16-rc1. It causes a hardware hang on i.MX6SX, if the rootfs is not on NFS but something else like MMC. I think the cause is that PTP is still accessing registers after FEC driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. Can you please try to provide a fix for this regression soon? Shawn ^ permalink raw reply [flat|nested] 10+ messages in thread
* FEC driver hangs hardware on i.MX6SX 2014-07-20 2:57 ` Shawn Guo @ 2014-07-20 6:20 ` Richard Cochran -1 siblings, 0 replies; 10+ messages in thread From: Richard Cochran @ 2014-07-20 6:20 UTC (permalink / raw) To: linux-arm-kernel On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: > > I think the cause is that PTP is still accessing registers after FEC > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. > Can you please try to provide a fix for this regression soon? What do you mean by, "PTP is still accessing registers"? The only access to any register is through the driver, and the driver can and should make sure all register accesses are safe. Thanks, Richard ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: FEC driver hangs hardware on i.MX6SX @ 2014-07-20 6:20 ` Richard Cochran 0 siblings, 0 replies; 10+ messages in thread From: Richard Cochran @ 2014-07-20 6:20 UTC (permalink / raw) To: Shawn Guo Cc: Fugang Duan, linux-arm-kernel, netdev, Frank Li, David S. Miller On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: > > I think the cause is that PTP is still accessing registers after FEC > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. > Can you please try to provide a fix for this regression soon? What do you mean by, "PTP is still accessing registers"? The only access to any register is through the driver, and the driver can and should make sure all register accesses are safe. Thanks, Richard ^ permalink raw reply [flat|nested] 10+ messages in thread
* FEC driver hangs hardware on i.MX6SX 2014-07-20 6:20 ` Richard Cochran @ 2014-07-21 1:59 ` Shawn Guo -1 siblings, 0 replies; 10+ messages in thread From: Shawn Guo @ 2014-07-21 1:59 UTC (permalink / raw) To: linux-arm-kernel On Sun, Jul 20, 2014 at 08:20:49AM +0200, Richard Cochran wrote: > On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: > > > > I think the cause is that PTP is still accessing registers after FEC > > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. > > Can you please try to provide a fix for this regression soon? > > What do you mean by, "PTP is still accessing registers"? > > The only access to any register is through the driver, and the driver > can and should make sure all register accesses are safe. Sorry for being vague, Richard. I meant PTP driver is still accessing registers. With Fugang's clock management patch, FEC clocks will be disabled to save power after FEC gets probed. But at that point, fec_ptp driver already launches a timer, which will trigger the read on FEC ATIME register once per second. Shawn ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: FEC driver hangs hardware on i.MX6SX @ 2014-07-21 1:59 ` Shawn Guo 0 siblings, 0 replies; 10+ messages in thread From: Shawn Guo @ 2014-07-21 1:59 UTC (permalink / raw) To: Richard Cochran Cc: Fugang Duan, linux-arm-kernel, netdev, Frank Li, David S. Miller On Sun, Jul 20, 2014 at 08:20:49AM +0200, Richard Cochran wrote: > On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: > > > > I think the cause is that PTP is still accessing registers after FEC > > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. > > Can you please try to provide a fix for this regression soon? > > What do you mean by, "PTP is still accessing registers"? > > The only access to any register is through the driver, and the driver > can and should make sure all register accesses are safe. Sorry for being vague, Richard. I meant PTP driver is still accessing registers. With Fugang's clock management patch, FEC clocks will be disabled to save power after FEC gets probed. But at that point, fec_ptp driver already launches a timer, which will trigger the read on FEC ATIME register once per second. Shawn ^ permalink raw reply [flat|nested] 10+ messages in thread
* FEC driver hangs hardware on i.MX6SX 2014-07-21 1:59 ` Shawn Guo @ 2014-07-21 2:13 ` fugang.duan -1 siblings, 0 replies; 10+ messages in thread From: fugang.duan at freescale.com @ 2014-07-21 2:13 UTC (permalink / raw) To: linux-arm-kernel From: Shawn Guo <shawn.guo@freescale.com> Data: Monday, July 21, 2014 9:59 AM >To: Richard Cochran >Cc: Duan Fugang-B38611; linux-arm-kernel at lists.infradead.org; >netdev at vger.kernel.org; Li Frank-B20596; David S. Miller >Subject: Re: FEC driver hangs hardware on i.MX6SX > >On Sun, Jul 20, 2014 at 08:20:49AM +0200, Richard Cochran wrote: >> On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: >> > >> > I think the cause is that PTP is still accessing registers after FEC >> > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. >> > Can you please try to provide a fix for this regression soon? >> >> What do you mean by, "PTP is still accessing registers"? >> >> The only access to any register is through the driver, and the driver >> can and should make sure all register accesses are safe. > >Sorry for being vague, Richard. I meant PTP driver is still accessing >registers. > >With Fugang's clock management patch, FEC clocks will be disabled to save >power after FEC gets probed. But at that point, fec_ptp driver already >launches a timer, which will trigger the read on FEC ATIME register once >per second. > >Shawn Yes, why does it cause at imx6sx platform ? Because imx5x, imx6q/dl/sl enet register access don't cause system hang when clock is disab led, just return one error value that is zero. But for imx6sx, if there disable enet ipg clock, and access register, which cause system hang. For clock management patch itself, there have no any problem. Just ptp driver has one issue which think clock is always on. I submit one patch to fix the ptp issue. Thanks, Andy ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: FEC driver hangs hardware on i.MX6SX @ 2014-07-21 2:13 ` fugang.duan 0 siblings, 0 replies; 10+ messages in thread From: fugang.duan @ 2014-07-21 2:13 UTC (permalink / raw) To: Shawn Guo, Richard Cochran Cc: linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org, Frank.Li@freescale.com, David S. Miller From: Shawn Guo <shawn.guo@freescale.com> Data: Monday, July 21, 2014 9:59 AM >To: Richard Cochran >Cc: Duan Fugang-B38611; linux-arm-kernel@lists.infradead.org; >netdev@vger.kernel.org; Li Frank-B20596; David S. Miller >Subject: Re: FEC driver hangs hardware on i.MX6SX > >On Sun, Jul 20, 2014 at 08:20:49AM +0200, Richard Cochran wrote: >> On Sun, Jul 20, 2014 at 10:57:33AM +0800, Shawn Guo wrote: >> > >> > I think the cause is that PTP is still accessing registers after FEC >> > driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. >> > Can you please try to provide a fix for this regression soon? >> >> What do you mean by, "PTP is still accessing registers"? >> >> The only access to any register is through the driver, and the driver >> can and should make sure all register accesses are safe. > >Sorry for being vague, Richard. I meant PTP driver is still accessing >registers. > >With Fugang's clock management patch, FEC clocks will be disabled to save >power after FEC gets probed. But at that point, fec_ptp driver already >launches a timer, which will trigger the read on FEC ATIME register once >per second. > >Shawn Yes, why does it cause at imx6sx platform ? Because imx5x, imx6q/dl/sl enet register access don't cause system hang when clock is disab led, just return one error value that is zero. But for imx6sx, if there disable enet ipg clock, and access register, which cause system hang. For clock management patch itself, there have no any problem. Just ptp driver has one issue which think clock is always on. I submit one patch to fix the ptp issue. Thanks, Andy ^ permalink raw reply [flat|nested] 10+ messages in thread
* FEC driver hangs hardware on i.MX6SX 2014-07-20 2:57 ` Shawn Guo @ 2014-07-20 8:06 ` fugang.duan -1 siblings, 0 replies; 10+ messages in thread From: fugang.duan at freescale.com @ 2014-07-20 8:06 UTC (permalink / raw) To: linux-arm-kernel Hi, Shawn, >-----Original Message----- >From: Shawn Guo [mailto:shawn.guo at freescale.com] >Sent: Sunday, July 20, 2014 10:58 AM >To: Duan Fugang-B38611 >Cc: Li Frank-B20596; David S. Miller; Guo Shawn-R65073; >netdev at vger.kernel.org; linux-arm-kernel at lists.infradead.org >Subject: FEC driver hangs hardware on i.MX6SX > >Hi Fugang, > >Your commit e8fcfcd5684a (net: fec: optimize the clock management to save >power) landed on mainline with v3.16-rc1. It causes a hardware hang on >i.MX6SX, if the rootfs is not on NFS but something else like MMC. > >I think the cause is that PTP is still accessing registers after FEC >driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. >Can you please try to provide a fix for this regression soon? > Yes, you analyze is right. We had the related patch on internal trees since imx6sx bringup. I will try to move the patch to upstream. Thanks, Andy ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: FEC driver hangs hardware on i.MX6SX @ 2014-07-20 8:06 ` fugang.duan 0 siblings, 0 replies; 10+ messages in thread From: fugang.duan @ 2014-07-20 8:06 UTC (permalink / raw) To: Shawn Guo Cc: Frank.Li@freescale.com, David S. Miller, Shawn Guo, netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org Hi, Shawn, >-----Original Message----- >From: Shawn Guo [mailto:shawn.guo@freescale.com] >Sent: Sunday, July 20, 2014 10:58 AM >To: Duan Fugang-B38611 >Cc: Li Frank-B20596; David S. Miller; Guo Shawn-R65073; >netdev@vger.kernel.org; linux-arm-kernel@lists.infradead.org >Subject: FEC driver hangs hardware on i.MX6SX > >Hi Fugang, > >Your commit e8fcfcd5684a (net: fec: optimize the clock management to save >power) landed on mainline with v3.16-rc1. It causes a hardware hang on >i.MX6SX, if the rootfs is not on NFS but something else like MMC. > >I think the cause is that PTP is still accessing registers after FEC >driver calls fec_enet_clk_enable(ndev, false) to disable FEC clocks. >Can you please try to provide a fix for this regression soon? > Yes, you analyze is right. We had the related patch on internal trees since imx6sx bringup. I will try to move the patch to upstream. Thanks, Andy ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-07-21 2:13 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-07-20 2:57 FEC driver hangs hardware on i.MX6SX Shawn Guo 2014-07-20 2:57 ` Shawn Guo 2014-07-20 6:20 ` Richard Cochran 2014-07-20 6:20 ` Richard Cochran 2014-07-21 1:59 ` Shawn Guo 2014-07-21 1:59 ` Shawn Guo 2014-07-21 2:13 ` fugang.duan at freescale.com 2014-07-21 2:13 ` fugang.duan 2014-07-20 8:06 ` fugang.duan at freescale.com 2014-07-20 8:06 ` fugang.duan
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.