You could be more precise on the commit-subject (not blaming you) next time. Worth reading on this topic: "On Commit Messages" [1]. - Sedat - [1] http://who-t.blogspot.com/2009/12/on-commit-messages.html On Wed, Feb 17, 2010 at 8:04 PM, okias <d.okias@xxxxxxxxx> wrote: > ok, sorry I tested it on x86_64. Here is correct patch: > > From c4dd3d87e090dd0b44dc030840f2ae1fd0bff729 Mon Sep 17 00:00:00 2001 > From: David Heidelberger <d.okias@xxxxxxxxx> > Date: Wed, 17 Feb 2010 19:41:15 +0100 > Subject: [PATCH] rtl8187: micro cleanup v3 > > --- > drivers/net/wireless/rtl818x/rtl8187_dev.c | 4 +--- > 1 files changed, 1 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/wireless/rtl818x/rtl8187_dev.c > b/drivers/net/wireless/rtl818x/rtl8187_dev.c > index 7ba3052..2cc6881 100644 > --- a/drivers/net/wireless/rtl818x/rtl8187_dev.c > +++ b/drivers/net/wireless/rtl818x/rtl8187_dev.c > @@ -1165,7 +1165,7 @@ static void rtl8187_bss_info_changed(struct > ieee80211_hw *dev, > { > struct rtl8187_priv *priv = dev->priv; > int i; > - u8 reg; > + u8 reg = 0; > > if (changed & BSS_CHANGED_BSSID) { > mutex_lock(&priv->conf_mutex); > @@ -1175,8 +1175,6 @@ static void rtl8187_bss_info_changed(struct > ieee80211_hw *dev, > > if (priv->is_rtl8187b) > reg = RTL818X_MSR_ENEDCA; > - else > - reg = 0; > > if (is_valid_ether_addr(info->bssid)) { > reg |= RTL818X_MSR_INFRA; > -- > 1.7.0 > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html