From mboxrd@z Thu Jan 1 00:00:00 1970 From: Randy Dunlap Subject: Re: [PATCH] net: vlan: fix compile breakage from 69ecca8 Date: Wed, 20 Jul 2011 14:06:46 -0700 Message-ID: <20110720140646.b66bd02b.rdunlap@xenotime.net> References: <0E5EAE44-204C-4F4D-926C-F4D973B0CDB1@qlogic.com> <1311191033-714631-1-git-send-email-equinox@diac24.net> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Stephen Rothwell , netdev , linux-next@vger.kernel.org, linux-kernel , linux-driver@qlogic.com, Anirban Chakraborty , "David S. Miller" To: David Lamparter Return-path: In-Reply-To: <1311191033-714631-1-git-send-email-equinox@diac24.net> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Wed, 20 Jul 2011 21:43:53 +0200 David Lamparter wrote: > well, there was too much wood to see the tree and I messed up the > configuration dependencies. let's make it work with unset > CONFIG_VLAN_8021Q and remove the second definition of vlan_find_dev. > > Signed-off-by: David Lamparter > Cc: linux-driver@qlogic.com > Cc: Anirban Chakraborty > Cc: David S. Miller > Cc: Randy Dunlap Reported-by: Randy Dunlap Acked-by: Randy Dunlap Thanks. > --- > sorry for the mess-up... compiled with CONFIG_VLAN_8021Q=n,m and y > this time; don't have qlcnic hw to test. > > drivers/net/qlcnic/qlcnic_main.c | 4 +++- > include/linux/if_vlan.h | 5 ----- > 2 files changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/net/qlcnic/qlcnic_main.c b/drivers/net/qlcnic/qlcnic_main.c > index 3579229..a2c39e9 100644 > --- a/drivers/net/qlcnic/qlcnic_main.c > +++ b/drivers/net/qlcnic/qlcnic_main.c > @@ -4198,13 +4198,15 @@ static void > qlcnic_restore_indev_addr(struct net_device *netdev, unsigned long event) > { > struct qlcnic_adapter *adapter = netdev_priv(netdev); > - struct vlan_group *grp; > + struct vlan_group *grp = NULL; > struct net_device *dev; > u16 vid; > > qlcnic_config_indev_addr(adapter, netdev, event); > > +#if defined(CONFIG_VLAN_8021Q) || defined(CONFIG_VLAN_8021Q_MODULE) > grp = rcu_dereference_rtnl(netdev->vlgrp); > +#endif > if (!grp) > return; > > diff --git a/include/linux/if_vlan.h b/include/linux/if_vlan.h > index bc03e40..dbe41dc 100644 > --- a/include/linux/if_vlan.h > +++ b/include/linux/if_vlan.h > @@ -135,11 +135,6 @@ vlan_gro_frags(struct napi_struct *napi, struct vlan_group *grp, > unsigned int vlan_tci); > > #else > -static inline struct net_device *vlan_find_dev(struct net_device *real_dev, > - u16 vlan_id) > -{ > - return NULL; > -} > > static inline struct net_device *vlan_dev_real_dev(const struct net_device *dev) > { > -- --- ~Randy *** Remember to use Documentation/SubmitChecklist when testing your code ***