From mboxrd@z Thu Jan 1 00:00:00 1970 From: Larry Finger Subject: Re: linux-next: build failure after merge of the staging tree Date: Thu, 25 Aug 2011 00:15:56 -0500 Message-ID: <4E55DA8C.10102@lwfinger.net> References: <20110825150209.5d159c549777290de4644bcf@canb.auug.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-yi0-f46.google.com ([209.85.218.46]:60708 "EHLO mail-yi0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750858Ab1HYFQA (ORCPT ); Thu, 25 Aug 2011 01:16:00 -0400 In-Reply-To: <20110825150209.5d159c549777290de4644bcf@canb.auug.org.au> Sender: linux-next-owner@vger.kernel.org List-ID: To: Stephen Rothwell Cc: Greg KH , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, wlanfae , Jiri Pirko , David Miller , netdev@vger.kernel.org On 08/25/2011 12:02 AM, Stephen Rothwell wrote: > Hi Greg, > > After merging the staging tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > drivers/staging/rtl8192e/rtl_core.c:2917:2: error: unknown field 'ndo_set_multicast_list' specified in initializer > > Caused by commit 94a799425eee ("From: wlanfae" - > really "[PATCH 1/8] rtl8192e: Import new version of driver from realtek" > Larry, that patch was badly imported ...) interacting with commit > b81693d9149c ("net: remove ndo_set_multicast_list callback") from the net > tree. > > I applied the following patch (which seems to be what was done to the > other drivers in the net tree - there is probably more required): > > From: Stephen Rothwell > Date: Thu, 25 Aug 2011 14:57:55 +1000 > Subject: [PATCH] rtl8192e: update for ndo_set_multicast_list removal. > > Signed-off-by: Stephen Rothwell > --- > drivers/staging/rtl8192e/rtl_core.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/drivers/staging/rtl8192e/rtl_core.c b/drivers/staging/rtl8192e/rtl_core.c > index f8a13d9..b38f626 100644 > --- a/drivers/staging/rtl8192e/rtl_core.c > +++ b/drivers/staging/rtl8192e/rtl_core.c > @@ -2914,7 +2914,7 @@ static const struct net_device_ops rtl8192_netdev_ops = { > .ndo_stop = rtl8192_close, > .ndo_tx_timeout = rtl8192_tx_timeout, > .ndo_do_ioctl = rtl8192_ioctl, > - .ndo_set_multicast_list = r8192_set_multicast, > + .ndo_set_rx_mode = r8192_set_multicast, > .ndo_set_mac_address = r8192_set_mac_adr, > .ndo_validate_addr = eth_validate_addr, > .ndo_change_mtu = eth_change_mtu, Stephan, Thanks for the notice. It seems that commit b81693d9149c had not made it into my copy of staging. I'll look into the issue. Larry