From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: [PATCH net-next resubmit 2/2] net: phy: core: don't disable device interrupts in phy_change Date: Mon, 04 Dec 2017 10:24:39 -0500 (EST) Message-ID: <20171204.102439.2288357008626128786.davem@davemloft.net> References: Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: f.fainelli@gmail.com, andrew@lunn.ch, netdev@vger.kernel.org, ard.biesheuvel@linaro.org To: hkallweit1@gmail.com Return-path: Received: from shards.monkeyblade.net ([184.105.139.130]:48448 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751463AbdLDPYl (ORCPT ); Mon, 4 Dec 2017 10:24:41 -0500 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: From: Heiner Kallweit Date: Thu, 30 Nov 2017 23:57:00 +0100 > If state is not PHY_HALTED I see no need to temporarily disable > interrupts on the device. As long as the current interrupt isn't acked > on the device no new interrupt can happen anyway. > > In addition remove a unneeded enabling of interrupts in the state > machine when handling state PHY_CHANGELINK. > > Tested on a Odroid-C2 with RTL8211F phy in interrupt mode. > > Signed-off-by: Heiner Kallweit Also applied, and again if this regresses things I'm reverting.