From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Murphy Subject: Re: [PATCH 1/2] net: phy: DP83TC811: Add INT_STAT3 Date: Thu, 28 Jun 2018 06:54:23 -0500 Message-ID: <98e615dd-3fa7-78b2-3401-13cae4916c14@ti.com> References: <20180627181618.23463-1-dmurphy@ti.com> <20180628081612.GB16727@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Cc: , To: Andrew Lunn Return-path: Received: from fllv0016.ext.ti.com ([198.47.19.142]:44728 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933157AbeF1MEP (ORCPT ); Thu, 28 Jun 2018 08:04:15 -0400 In-Reply-To: <20180628081612.GB16727@lunn.ch> Content-Language: en-US Sender: netdev-owner@vger.kernel.org List-ID: Andrew On 06/28/2018 03:16 AM, Andrew Lunn wrote: >> err = phy_write(phydev, MII_DP83811_INT_STAT1, 0); >> if (err < 0) >> return err; >> >> - err = phy_write(phydev, MII_DP83811_INT_STAT1, 0); >> + err = phy_write(phydev, MII_DP83811_INT_STAT2, 0); >> + if (err < 0) >> + return err; >> + >> + err = phy_write(phydev, MII_DP83811_INT_STAT3, 0); >> } >> > > Hi Dan > > This seems like a bug fix, and so should be in a patch of its own, for > net, not net-next. > Yes I was debating whether to include this change in the patch or have it stand alone for stable back port. I will pull it out and cc stable > Andrew > -- ------------------ Dan Murphy