Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755045Ab0K2HZa (ORCPT ); Mon, 29 Nov 2010 02:25:30 -0500 Received: from eu1sys200aog114.obsmtp.com ([207.126.144.137]:32794 "EHLO eu1sys200aog114.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754780Ab0K2HZ2 convert rfc822-to-8bit (ORCPT ); Mon, 29 Nov 2010 02:25:28 -0500 From: Peppe CAVALLARO To: David Miller Cc: "sfr@canb.auug.org.au" , "netdev@vger.kernel.org" , "linux-next@vger.kernel.org" , "linux-kernel@vger.kernel.org" Date: Mon, 29 Nov 2010 08:25:05 +0100 Subject: Re: linux-next: build failure after merge of the net tree Thread-Topic: linux-next: build failure after merge of the net tree Thread-Index: AcuPlou/0BXe0VYuTSmlSNJe3QQ4Yw== Message-ID: <4CF35551.40201@st.com> References: <20101129110824.639b2659.sfr@canb.auug.org.au> <20101128.181134.104049260.davem@davemloft.net> In-Reply-To: <20101128.181134.104049260.davem@davemloft.net> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-enigmail-version: 1.1.1 user-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2.12) Gecko/20101027 Lightning/1.0b2 Thunderbird/3.1.6 openpgp: url=wwwkeys.bri.st.com acceptlanguage: en-US Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2285 Lines: 74 On 11/29/2010 3:11 AM, David Miller wrote: > > From: Stephen Rothwell > Date: Mon, 29 Nov 2010 11:08:24 +1100 > > > Hi all, > > > > After merging the net tree, today's linux-next build (x86_64 > allmodconfig) > > failed like this: > > > > drivers/net/stmmac/stmmac_main.c: In function 'stmmac_resume': > > drivers/net/stmmac/stmmac_main.c:1849: error: 'struct stmmac_priv' > has no member named 'shutdown' > > > > Caused by commit 874bd42d24c2a74f5dbd65e81e175982240fecd8 ("stmmac: > > convert to dev_pm_ops"). > > > Yes, this was my fault. > > I have used the net tree from next-20101126 for today. > > I have pushed the following fix which should cure this: > > -------------------- > stmmac: fix stmmac_resume removing not yet used shutdown flag > The patch, David already applied, fixes this problem. Peppe > The commit to convert to use the dev_pm_ops struct > introduces a bug. The shutdown flag is not yet used > because the hibernation on memory is done by using > the freeze callback. > Thanks to Vlad for having reported it. > > Reported-by: Vlad Lungu > Signed-off-by: Giuseppe Cavallaro > Signed-off-by: David S. Miller > --- > drivers/net/stmmac/stmmac_main.c | 7 ------- > 1 files changed, 0 insertions(+), 7 deletions(-) > > diff --git a/drivers/net/stmmac/stmmac_main.c > b/drivers/net/stmmac/stmmac_main.c > index f1dbc18..730a6fd 100644 > --- a/drivers/net/stmmac/stmmac_main.c > +++ b/drivers/net/stmmac/stmmac_main.c > @@ -1846,13 +1846,6 @@ static int stmmac_resume(struct device *dev) > if (!netif_running(ndev)) > return 0; > > - if (priv->shutdown) { > - /* Re-open the interface and re-init the MAC/DMA > - and the rings (i.e. on hibernation stage) */ > - stmmac_open(dev); > - return 0; > - } > - > spin_lock(&priv->lock); > > /* Power Down bit, into the PM register, is cleared > -- > 1.7.3.2 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/