Linux wireless drivers development
 help / color / mirror / Atom feed
From: Denis 'GNUtoo' Carikli <GNUtoo@no-log.org>
To: Bob Copeland <me@bobcopeland.com>
Cc: Kalle Valo <kvalo@adurom.com>,
	Grazvydas Ignotas <notasas@gmail.com>,
	"John W.Linville" <linville@tuxdriver.com>,
	linux-wireless@vger.kernel.org
Subject: Re: [PATCH 2/2] wl1251: fix ELP_CTRL register reads
Date: Mon, 21 Jun 2010 20:45:01 +0200	[thread overview]
Message-ID: <1277145901.28895.29.camel@gnutoo-laptop> (raw)
In-Reply-To: <AANLkTimLKDqDn_97BbOYr1cdxbFdPvyTz_lxJ59LVJ1b@mail.gmail.com>

On Mon, 2010-06-21 at 13:54 -0400, Bob Copeland wrote:
> On Mon, Jun 21, 2010 at 8:46 AM, Denis 'GNUtoo' Carikli
> <GNUtoo@no-log.org> wrote:
> 
> > Hi, me and some other people are trying to run standard GNU/Linux (not
> > android ) on the htcdream.
> 
> > [ 1556.425354] ADDRCONF(NETDEV_CHANGE): wlan0: link becomes ready
> > [ 1556.734130] mmc0: Command timeout
> 
> In this case it was only up for 1/3 second before it quit
> responding to reads.  With power saving disabled it works?
In this case I think only some command timeout(If I remember well it
worked for a bit and then stopped working)
But I'm sure that you can ping with the command timeout and have some
transfers etc...

With power saving disabled it works fine,until the battery goes out.
Disabling power saving also stop the flow of timeout messages...

> Probably not relevant to power saving, but:
> 
> Are you using the dedicated irq line or sdio interrupt?
> It makes a big difference in overall throughput to use
> the former.
I don't know. what should I grep for?

> > +       /*
> > +        * FIXME: scanning while associated causes lockups,
> > +        * so we don't allow that
> > +        */
> > +       if (wl->associated)
> > +               return -EBUSY;
> 
> Any idea why?  Could you turn on lockdep?
No I just used the patches that weren't mine
I took them from openpandora wireless repository,looked if they were
already included,and include the one that weren't.

Denis.



  reply	other threads:[~2010-06-21 18:45 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-04 23:25 [PATCH 1/2] wl1251: fix a memory leak in probe Grazvydas Ignotas
2010-06-04 23:25 ` [PATCH 2/2] wl1251: fix ELP_CTRL register reads Grazvydas Ignotas
2010-06-05  7:04   ` Kalle Valo
2010-06-21 12:46     ` Denis 'GNUtoo' Carikli
2010-06-21 17:54       ` Bob Copeland
2010-06-21 18:45         ` Denis 'GNUtoo' Carikli [this message]
2010-06-21 22:48           ` Grazvydas Ignotas
2010-06-22 23:33             ` Denis 'GNUtoo' Carikli
2010-06-24 23:09               ` Denis 'GNUtoo' Carikli
2010-06-24 23:34                 ` Grazvydas Ignotas
2010-06-25 16:01                   ` Denis 'GNUtoo' Carikli
2010-06-25 18:06                     ` Denis 'GNUtoo' Carikli
2010-06-05  7:00 ` [PATCH 1/2] wl1251: fix a memory leak in probe Kalle Valo

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1277145901.28895.29.camel@gnutoo-laptop \
    --to=gnutoo@no-log.org \
    --cc=kvalo@adurom.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=me@bobcopeland.com \
    --cc=notasas@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox