pull request: wireless-2.6 2008-10-29 - Kernel

This is a discussion on pull request: wireless-2.6 2008-10-29 - Kernel ; Dave, Another quick round of fixes intended for 2.6.28... This includes fixes for bugs 11749 and 11845, fixes a reset-related regression in ath5k, fixes a Kconfig-related error for rt2x00, and fixes some sparse warnings and a possible buffer overrun in ...

+ Reply to Thread
Results 1 to 8 of 8

Thread: pull request: wireless-2.6 2008-10-29

  1. pull request: wireless-2.6 2008-10-29

    Dave,

    Another quick round of fixes intended for 2.6.28...

    This includes fixes for bugs 11749 and 11845, fixes a reset-related
    regression in ath5k, fixes a Kconfig-related error for rt2x00, and
    fixes some sparse warnings and a possible buffer overrun in libertas.
    Also, it corrects a spelling error and marks Bob Copeland as an
    ath5k maintainer.

    Please let me know if there are problems!

    Thanks,

    John

    ---

    Individual patches are available here:

    http://www.kernel.org/pub/linux/kern.../wireless-2.6/

    ---

    The following changes since commit 882716604ecf388b2ff09bc76eb2a752aa9cc76b:
    Jesse Brandeburg (1):
    pktgen: fix multiple queue warning

    are available in the git repository at:

    git://git.kernel.org/pub/scm/linux/kernel/git/linville/wireless-2.6.git master

    Bob Copeland (3):
    ath5k: correct misspelling in debug help
    ath5k: honor FIF_BCN_PRBRESP_PROMISC in STA mode
    ath5k: add self to MAINTAINERS

    Elias Oltmanns (1):
    ath5k: Fix reset sequence for AR5212 in general and RF5111 in particular

    Holger Schurig (1):
    libertas: remove two libertas sparse warning

    Ivo van Doorn (1):
    rt2x00: Fix build error when mac80211=M rt2x00=Y

    Johannes Berg (1):
    libertas: fix buffer overrun

    Tomas Winkler (1):
    iwlwifi: fix suspend to RAM in iwlwifi

    MAINTAINERS | 2 ++
    drivers/net/wireless/ath5k/base.c | 4 +---
    drivers/net/wireless/ath5k/debug.c | 2 +-
    drivers/net/wireless/ath5k/initvals.c | 2 ++
    drivers/net/wireless/ath5k/reset.c | 22 +++++++---------------
    drivers/net/wireless/iwlwifi/iwl-agn.c | 2 +-
    drivers/net/wireless/libertas/cmd.c | 4 ++--
    drivers/net/wireless/libertas/scan.c | 4 ++--
    drivers/net/wireless/rt2x00/Kconfig | 2 +-
    9 files changed, 19 insertions(+), 25 deletions(-)

    diff --git a/MAINTAINERS b/MAINTAINERS
    index 16202c8..185b84a 100644
    --- a/MAINTAINERS
    +++ b/MAINTAINERS
    @@ -738,6 +738,8 @@ P: Nick Kossifidis
    M: mickflemm@gmail.com
    P: Luis R. Rodriguez
    M: mcgrof@gmail.com
    +P: Bob Copeland
    +M: me@bobcopeland.com
    L: linux-wireless@vger.kernel.org
    L: ath5k-devel@lists.ath5k.org
    S: Maintained
    diff --git a/drivers/net/wireless/ath5k/base.c b/drivers/net/wireless/ath5k/base.c
    index cfd4d05..9e47d72 100644
    --- a/drivers/net/wireless/ath5k/base.c
    +++ b/drivers/net/wireless/ath5k/base.c
    @@ -2942,10 +2942,8 @@ static void ath5k_configure_filter(struct ieee80211_hw *hw,
    sc->opmode != NL80211_IFTYPE_MESH_POINT &&
    test_bit(ATH_STAT_PROMISC, sc->status))
    rfilt |= AR5K_RX_FILTER_PROM;
    - if (sc->opmode == NL80211_IFTYPE_STATION ||
    - sc->opmode == NL80211_IFTYPE_ADHOC) {
    + if (sc->opmode == NL80211_IFTYPE_ADHOC)
    rfilt |= AR5K_RX_FILTER_BEACON;
    - }

    /* Set filters */
    ath5k_hw_set_rx_filter(ah,rfilt);
    diff --git a/drivers/net/wireless/ath5k/debug.c b/drivers/net/wireless/ath5k/debug.c
    index 8f92d67..19980cb 100644
    --- a/drivers/net/wireless/ath5k/debug.c
    +++ b/drivers/net/wireless/ath5k/debug.c
    @@ -339,7 +339,7 @@ static struct {
    { ATH5K_DEBUG_BEACON, "beacon", "beacon handling" },
    { ATH5K_DEBUG_CALIBRATE, "calib", "periodic calibration" },
    { ATH5K_DEBUG_TXPOWER, "txpower", "transmit power setting" },
    - { ATH5K_DEBUG_LED, "led", "LED mamagement" },
    + { ATH5K_DEBUG_LED, "led", "LED management" },
    { ATH5K_DEBUG_DUMP_RX, "dumprx", "print received skb content" },
    { ATH5K_DEBUG_DUMP_TX, "dumptx", "print transmit skb content" },
    { ATH5K_DEBUG_DUMPBANDS, "dumpbands", "dump bands" },
    diff --git a/drivers/net/wireless/ath5k/initvals.c b/drivers/net/wireless/ath5k/initvals.c
    index ea2e1a2..ceaa6c4 100644
    --- a/drivers/net/wireless/ath5k/initvals.c
    +++ b/drivers/net/wireless/ath5k/initvals.c
    @@ -806,6 +806,8 @@ static const struct ath5k_ini_mode ar5212_rf5111_ini_mode_end[] = {
    { 0x00000000, 0x00000000, 0x00000000, 0x00000000, 0x00000000 } },
    { AR5K_PHY(642),
    { 0xd03e6788, 0xd03e6788, 0xd03e6788, 0xd03e6788, 0xd03e6788 } },
    + { 0xa228,
    + { 0x000001b5, 0x000001b5, 0x000001b5, 0x000001b5, 0x000001b5 } },
    { 0xa23c,
    { 0x13c889af, 0x13c889af, 0x13c889af, 0x13c889af, 0x13c889af } },
    };
    diff --git a/drivers/net/wireless/ath5k/reset.c b/drivers/net/wireless/ath5k/reset.c
    index 8f18868..1b6d45b 100644
    --- a/drivers/net/wireless/ath5k/reset.c
    +++ b/drivers/net/wireless/ath5k/reset.c
    @@ -537,9 +537,10 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum nl80211_iftype op_mode,
    mdelay(1);

    /*
    - * Write some more initial register settings
    + * Write some more initial register settings for revised chips
    */
    - if (ah->ah_version == AR5K_AR5212) {
    + if (ah->ah_version == AR5K_AR5212 &&
    + ah->ah_phy_revision > 0x41) {
    ath5k_hw_reg_write(ah, 0x0002a002, 0x982c);

    if (channel->hw_value == CHANNEL_G)
    @@ -558,19 +559,10 @@ int ath5k_hw_reset(struct ath5k_hw *ah, enum nl80211_iftype op_mode,
    else
    ath5k_hw_reg_write(ah, 0x00000000, 0x994c);

    - /* Some bits are disabled here, we know nothing about
    - * register 0xa228 yet, most of the times this ends up
    - * with a value 0x9b5 -haven't seen any dump with
    - * a different value- */
    - /* Got this from decompiling binary HAL */
    - data = ath5k_hw_reg_read(ah, 0xa228);
    - data &= 0xfffffdff;
    - ath5k_hw_reg_write(ah, data, 0xa228);
    -
    - data = ath5k_hw_reg_read(ah, 0xa228);
    - data &= 0xfffe03ff;
    - ath5k_hw_reg_write(ah, data, 0xa228);
    - data = 0;
    + /* Got this from legacy-hal */
    + AR5K_REG_DISABLE_BITS(ah, 0xa228, 0x200);
    +
    + AR5K_REG_MASKED_BITS(ah, 0xa228, 0x800, 0xfffe03ff);

    /* Just write 0x9b5 ? */
    /* ath5k_hw_reg_write(ah, 0x000009b5, 0xa228); */
    diff --git a/drivers/net/wireless/iwlwifi/iwl-agn.c b/drivers/net/wireless/iwlwifi/iwl-agn.c
    index 24a1aeb..321dbc8 100644
    --- a/drivers/net/wireless/iwlwifi/iwl-agn.c
    +++ b/drivers/net/wireless/iwlwifi/iwl-agn.c
    @@ -2090,7 +2090,6 @@ static void iwl_alive_start(struct iwl_priv *priv)
    iwl4965_error_recovery(priv);

    iwl_power_update_mode(priv, 1);
    - ieee80211_notify_mac(priv->hw, IEEE80211_NOTIFY_RE_ASSOC);

    if (test_and_clear_bit(STATUS_MODE_PENDING, &priv->status))
    iwl4965_set_mode(priv, priv->iw_mode);
    @@ -2342,6 +2341,7 @@ static void iwl_bg_alive_start(struct work_struct *data)
    mutex_lock(&priv->mutex);
    iwl_alive_start(priv);
    mutex_unlock(&priv->mutex);
    + ieee80211_notify_mac(priv->hw, IEEE80211_NOTIFY_RE_ASSOC);
    }

    static void iwl4965_bg_rf_kill(struct work_struct *work)
    diff --git a/drivers/net/wireless/libertas/cmd.c b/drivers/net/wireless/libertas/cmd.c
    index 297696d..8265c7d 100644
    --- a/drivers/net/wireless/libertas/cmd.c
    +++ b/drivers/net/wireless/libertas/cmd.c
    @@ -605,9 +605,9 @@ int lbs_get_tx_power(struct lbs_private *priv, s16 *curlevel, s16 *minlevel,
    if (ret == 0) {
    *curlevel = le16_to_cpu(cmd.curlevel);
    if (minlevel)
    - *minlevel = le16_to_cpu(cmd.minlevel);
    + *minlevel = cmd.minlevel;
    if (maxlevel)
    - *maxlevel = le16_to_cpu(cmd.maxlevel);
    + *maxlevel = cmd.maxlevel;
    }

    lbs_deb_leave(LBS_DEB_CMD);
    diff --git a/drivers/net/wireless/libertas/scan.c b/drivers/net/wireless/libertas/scan.c
    index 8f66903..22c4c61 100644
    --- a/drivers/net/wireless/libertas/scan.c
    +++ b/drivers/net/wireless/libertas/scan.c
    @@ -598,8 +598,8 @@ static int lbs_process_bss(struct bss_descriptor *bss,

    switch (elem->id) {
    case MFIE_TYPE_SSID:
    - bss->ssid_len = elem->len;
    - memcpy(bss->ssid, elem->data, elem->len);
    + bss->ssid_len = min_t(int, 32, elem->len);
    + memcpy(bss->ssid, elem->data, bss->ssid_len);
    lbs_deb_scan("got SSID IE: '%s', len %u\n",
    escape_essid(bss->ssid, bss->ssid_len),
    bss->ssid_len);
    diff --git a/drivers/net/wireless/rt2x00/Kconfig b/drivers/net/wireless/rt2x00/Kconfig
    index f839ce0..95511ac 100644
    --- a/drivers/net/wireless/rt2x00/Kconfig
    +++ b/drivers/net/wireless/rt2x00/Kconfig
    @@ -1,5 +1,5 @@
    menuconfig RT2X00
    - bool "Ralink driver support"
    + tristate "Ralink driver support"
    depends on MAC80211 && WLAN_80211 && EXPERIMENTAL
    ---help---
    This will enable the experimental support for the Ralink drivers,
    --
    John W. Linville Linux should be at the core
    linville@tuxdriver.com of your literate lifestyle.
    --
    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/

  2. Re: pull request: wireless-2.6 2008-10-29

    On Wed, Oct 29, 2008 at 04:23:24PM -0700, Arjan van de Ven wrote:
    > On Wed, 29 Oct 2008 19:18:55 -0400
    > "John W. Linville" wrote:
    >
    > > Dave,
    > >
    > > Another quick round of fixes intended for 2.6.28...
    > >
    > > This includes fixes for bugs 11749 and 11845, fixes a reset-related
    > > regression in ath5k, fixes a Kconfig-related error for rt2x00, and
    > > fixes some sparse warnings and a possible buffer overrun in libertas.
    > > Also, it corrects a spelling error and marks Bob Copeland as an
    > > ath5k maintainer.

    >
    > speaking of ath5k:
    >
    > http://www.kerneloops.org/version.ph...&version=28-rc
    >
    > it seems to have added a few WARN_ON()s which now end up at the top of
    > the list .. are these known or should I spend some time writing reports
    > for netdev ?


    I wasn't aware of them -- thanks... :-)

    --
    John W. Linville Linux should be at the core
    linville@tuxdriver.com of your literate lifestyle.
    --
    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/

  3. Re: pull request: wireless-2.6 2008-10-29

    "John W. Linville" wrote:
    > Dave,
    >
    > Another quick round of fixes intended for 2.6.28...
    >
    > This includes fixes for bugs 11749 and 11845, fixes a reset-related
    > regression in ath5k, fixes a Kconfig-related error for rt2x00, and
    > fixes some sparse warnings and a possible buffer overrun in libertas.
    > Also, it corrects a spelling error and marks Bob Copeland as an
    > ath5k maintainer.
    >
    > Please let me know if there are problems!
    >
    > Thanks,
    >
    > John
    >
    > ---
    >
    > Individual patches are available here:
    >
    > http://www.kernel.org/pub/linux/kern.../wireless-2.6/
    >
    > ---
    >
    > The following changes since commit 882716604ecf388b2ff09bc76eb2a752aa9cc76b:

    [...]
    > Elias Oltmanns (1):
    > ath5k: Fix reset sequence for AR5212 in general and RF5111 in particular


    Pity, I just posted a *nicer* version of that patch for 2.6.28-rc2.
    Well, it doesn't really matter all that much, I suppose.

    Regards,

    Elias
    --
    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/

  4. Re: pull request: wireless-2.6 2008-10-29

    On Wed, Oct 29, 2008 at 7:33 PM, John W. Linville
    wrote:
    > On Wed, Oct 29, 2008 at 04:23:24PM -0700, Arjan van de Ven wrote:
    >> On Wed, 29 Oct 2008 19:18:55 -0400
    >> "John W. Linville" wrote:
    >> it seems to have added a few WARN_ON()s which now end up at the top of
    >> the list .. are these known or should I spend some time writing reports
    >> for netdev ?


    Pavel Roskin posted one such WARN_ON to ath5k-devel a few days ago.
    Seems related to setup of the rate tables, but I haven't reproduced it
    here yet. I'll see if he has more info...

    --
    Bob Copeland %% www.bobcopeland.com
    --
    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/

  5. Re: pull request: wireless-2.6 2008-10-29

    On Wed, 29 Oct 2008 21:19:01 -0400
    "Bob Copeland" wrote:

    > On Wed, Oct 29, 2008 at 7:33 PM, John W. Linville
    > wrote:
    > > On Wed, Oct 29, 2008 at 04:23:24PM -0700, Arjan van de Ven wrote:
    > >> On Wed, 29 Oct 2008 19:18:55 -0400
    > >> "John W. Linville" wrote:
    > >> it seems to have added a few WARN_ON()s which now end up at the
    > >> top of the list .. are these known or should I spend some time
    > >> writing reports for netdev ?

    >
    > Pavel Roskin posted one such WARN_ON to ath5k-devel a few days ago.
    > Seems related to setup of the rate tables, but I haven't reproduced it
    > here yet. I'll see if he has more info...


    if there's things that would help you diagnose such issue let me know
    (always trying to make kerneloops.org more useful for developers ;-)

    >



    --
    Arjan van de Ven Intel Open Source Technology Centre
    For development, discussion and tips for power savings,
    visit http://www.lesswatts.org
    --
    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/

  6. Re: pull request: wireless-2.6 2008-10-29

    John W. Linville schrieb:
    > On Wed, Oct 29, 2008 at 04:23:24PM -0700, Arjan van de Ven wrote:
    >
    >> On Wed, 29 Oct 2008 19:18:55 -0400
    >> "John W. Linville" wrote:
    >>
    >>
    >>> Dave,
    >>>
    >>> Another quick round of fixes intended for 2.6.28...
    >>>
    >>> This includes fixes for bugs 11749 and 11845, fixes a reset-related
    >>> regression in ath5k, fixes a Kconfig-related error for rt2x00, and
    >>> fixes some sparse warnings and a possible buffer overrun in libertas.
    >>> Also, it corrects a spelling error and marks Bob Copeland as an
    >>> ath5k maintainer.
    >>>

    >> speaking of ath5k:
    >>
    >> http://www.kerneloops.org/version.ph...&version=28-rc
    >>
    >> it seems to have added a few WARN_ON()s which now end up at the top of
    >> the list .. are these known or should I spend some time writing reports
    >> for netdev ?
    >>

    >
    > I wasn't aware of them -- thanks... :-)
    >
    >

    I think some of the reports on kerneloops are from my machine, I'm
    running wireless-testing#master in the hope to get my card fully
    working. It's a AR2425 chip (MAC: 0xe2, PHY: 0x70), in a Lenovo Notebook.

    The wireless-connection is nearly unusable. I can provide access to this
    machine or test patches if anyone can help.

    I get many warnings like

    WARNING: at /srv/devel/kernel/net/mac80211/rx.c:2203
    ath5k_tasklet_rx+0x318/0x5c0()
    Modules linked in:
    Pid: 0, comm: swapper Not tainted 2.6.28-rc1-wl #46
    Call Trace:
    [] warn_on_slowpath+0x5f/0xa0
    [] enqueue_task_fair+0xb9/0x100
    [] enqueue_task_fair+0xb9/0x100
    [] default_spin_lock_flags+0x5/0x10
    [] _spin_lock_irqsave+0x2b/0x40
    [] lock_timer_base+0x27/0x60
    [] __mod_timer+0x90/0xa0
    [] rexmit_timer+0x36a/0x3d0
    [] ath5k_tasklet_rx+0x318/0x5c0
    [] tasklet_action+0x70/0x100
    [] __do_softirq+0x97/0x160
    [] __do_softirq+0x0/0x160
    [] handle_fasteoi_irq+0x0/0xe0
    [] irq_exit+0x5d/0x80
    [] do_IRQ+0xcc/0x100
    [] common_interrupt+0x23/0x28
    [] acpi_idle_enter_bm+0x315/0x39e
    [] cpuidle_idle_call+0x74/0xc0
    [] cpu_idle+0x6d/0xd0
    ---[ end trace 8ac847f454371229 ]---

    --
    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/

  7. Re: pull request: wireless-2.6 2008-10-29

    On Thu, Oct 30, 2008 at 10:48 AM, Stefan Hellermann
    wrote:
    > I think some of the reports on kerneloops are from my machine, I'm
    > running wireless-testing#master in the hope to get my card fully
    > working. It's a AR2425 chip (MAC: 0xe2, PHY: 0x70), in a Lenovo Notebook.


    Okay, I just added a bugzilla for this:

    http://bugzilla.kernel.org/show_bug.cgi?id=11901

    I can reproduce it here after all...
    --
    Bob Copeland %% www.bobcopeland.com
    --
    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/

  8. Re: pull request: wireless-2.6 2008-10-29

    From: "John W. Linville"
    Date: Wed, 29 Oct 2008 19:18:55 -0400

    > Another quick round of fixes intended for 2.6.28...
    >
    > This includes fixes for bugs 11749 and 11845, fixes a reset-related
    > regression in ath5k, fixes a Kconfig-related error for rt2x00, and
    > fixes some sparse warnings and a possible buffer overrun in libertas.
    > Also, it corrects a spelling error and marks Bob Copeland as an
    > ath5k maintainer.
    >
    > Please let me know if there are problems!


    Pulled, thanks a lot John.
    --
    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/

+ Reply to Thread