From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH CFT] net: phy: micrel: remove unnecessary duplicate interrupt enable Date: Thu, 14 Dec 2017 13:21:01 -0500 (EST) Message-ID: <20171214.132101.814656901567768370.davem@davemloft.net> References: Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: andrew@lunn.ch, f.fainelli@gmail.com, netdev@vger.kernel.org To: rmk+kernel@armlinux.org.uk Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:34066 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753583AbdLNSVF (ORCPT ); Thu, 14 Dec 2017 13:21:05 -0500 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Russell King Date: Tue, 12 Dec 2017 11:02:12 +0000 > Now that we call phy_resume() before enabling interrupts, the workaround > in micrel is no longer required. > > Signed-off-by: Russell King > --- > This depends on the previous "net: phy: fix resume handling" patch, but > is not necessary to be a part of it. Since kszphy_resume() is used for > PHYs which I don't have, I can't test this, so I decided to keep it > separate. Can we get it tested on KSZ8081, KSZ8091 or KSZ9031 PHYs? Russell, please repost this once someone does some testing. Yes, I realize it can be difficult to obtain this, but please do your best.