Return-path: Received: from charlotte.tuxdriver.com ([70.61.120.58]:54042 "EHLO smtp.tuxdriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754121AbZAWTAz (ORCPT ); Fri, 23 Jan 2009 14:00:55 -0500 Date: Fri, 23 Jan 2009 13:50:53 -0500 From: "John W. Linville" To: Sedat Dilek Cc: Samuel Ortiz , linux-wireless@vger.kernel.org, Alina Friedrichsen , Kalle Valo , Larry Finger , Reinette Chatre Subject: Re: [PATCH] mac80211: ieee80211_sta_set_bssid should call config_interface() Message-ID: <20090123185052.GB30486@tuxdriver.com> (sfid-20090123_200058_359630_3E30BE37) References: <20090123013000.GA5604@sortiz.org> <2d0a357f0901221844h1849f4fanc577dcc2c5037101@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <2d0a357f0901221844h1849f4fanc577dcc2c5037101@mail.gmail.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: I'm going to drop this patch too. Alina, please account for this change when reposting the original patch. Thanks, John On Fri, Jan 23, 2009 at 03:44:53AM +0100, Sedat Dilek wrote: > Just to clarify: > The revert-patch of Alina's fix (causing regression) is no more needed? > > -SD > > On Fri, Jan 23, 2009 at 2:30 AM, Samuel Ortiz wrote: > > From: Samuel Ortiz > > > > With commit e32b1b9a33759e8a83ac566c4c43f23ed5d6343b, we end up not calling > > config_interface() when we're in STA, but not Ad-Hoc. This breaks several > > drivers, and the fix is to get ieee80211_sta_set_bssid() to call > > ieee80211_if_config() again. > > > > Signed-off-by: Samuel Ortiz > > Signed-off-by: Reinette Chatre > > --- > > net/mac80211/mlme.c | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > > > Index: wireless-testing/net/mac80211/mlme.c > > =================================================================== > > --- wireless-testing.orig/net/mac80211/mlme.c 2009-01-23 01:33:54.000000000 +0100 > > +++ wireless-testing/net/mac80211/mlme.c 2009-01-23 01:36:51.000000000 +0100 > > @@ -2650,6 +2650,18 @@ int ieee80211_sta_set_bssid(struct ieee8 > > ifsta->flags &= ~IEEE80211_STA_BSSID_SET; > > } > > > > + if (netif_running(sdata->dev)) { > > + int ret; > > + > > + ret = ieee80211_if_config(sdata, IEEE80211_IFCC_BSSID); > > + > > + if (ret) { > > + printk(KERN_DEBUG "%s: Failed to config new BSSID to " > > + "the low-level driver\n", sdata->dev->name); > > + return ret; > > + } > > + } > > + > > return ieee80211_sta_set_ssid(sdata, ifsta->ssid, ifsta->ssid_len); > > } > > > > -- > > Intel Open Source Technology Centre > > http://oss.intel.com/ > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > -- John W. Linville Someday the world will need a hero, and you linville@tuxdriver.com might be all we have. Be ready.