From mboxrd@z Thu Jan 1 00:00:00 1970 From: richardcochran@gmail.com (Richard Cochran) Date: Sun, 20 Jul 2014 08:20:49 +0200 Subject: FEC driver hangs hardware on i.MX6SX In-Reply-To: <20140720025732.GF8537@dragon> References: <20140720025732.GF8537@dragon> Message-ID: <20140720062048.GB5686@netboy> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Cochran Subject: Re: FEC driver hangs hardware on i.MX6SX Date: Sun, 20 Jul 2014 08:20:49 +0200 Message-ID: <20140720062048.GB5686@netboy> References: <20140720025732.GF8537@dragon> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Fugang Duan , linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org, Frank Li , "David S. Miller" To: Shawn Guo Return-path: Received: from mail-we0-f177.google.com ([74.125.82.177]:44502 "EHLO mail-we0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752039AbaGTGVB (ORCPT ); Sun, 20 Jul 2014 02:21:01 -0400 Received: by mail-we0-f177.google.com with SMTP id w62so6176793wes.36 for ; Sat, 19 Jul 2014 23:20:59 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140720025732.GF8537@dragon> Sender: netdev-owner@vger.kernel.org List-ID: 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