From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH net] net: fec: Fix unbalanced PM runtime calls Date: Sun, 18 Mar 2018 14:42:22 -0700 Message-ID: References: <20180318194952.31411-1-f.fainelli@gmail.com> <20180318.163501.2001276876693998811.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Cc: netdev@vger.kernel.org, andrew@lunn.ch, l.stach@pengutronix.de, fugang.duan@nxp.com, linux-kernel@vger.kernel.org To: David Miller Return-path: In-Reply-To: <20180318.163501.2001276876693998811.davem@davemloft.net> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 03/18/2018 01:35 PM, David Miller wrote: > From: Florian Fainelli > Date: Sun, 18 Mar 2018 12:49:51 -0700 > >> When unbinding/removing the driver, we will run into the following warnings: >> >> [ 259.655198] fec 400d1000.ethernet: 400d1000.ethernet supply phy not found, using dummy regulator >> [ 259.665065] fec 400d1000.ethernet: Unbalanced pm_runtime_enable! >> [ 259.672770] fec 400d1000.ethernet (unnamed net_device) (uninitialized): Invalid MAC address: 00:00:00:00:00:00 >> [ 259.683062] fec 400d1000.ethernet (unnamed net_device) (uninitialized): Using random MAC address: f2:3e:93:b7:29:c1 >> [ 259.696239] libphy: fec_enet_mii_bus: probed >> >> Avoid these warnings by balancing the runtime PM calls during fec_drv_remove(). >> >> Signed-off-by: Florian Fainelli > > Applied, thank you. > > Queue this up for -stable? I would be inclined to say yes, it was not exactly easy to track down a set of commits, since this was changed quite a bit. -- Florian