From: Luciano Coelho <luciano.coelho@nokia.com>
To: ext Sergey Matyukevich <geomatsi@gmail.com>
Cc: Ohad Ben-Cohen <ohad@wizery.com>, linux-wireless@vger.kernel.org
Subject: Re: wl1271: how to use without runtime pm
Date: Fri, 31 Dec 2010 10:00:16 +0200 [thread overview]
Message-ID: <1293782416.15791.10.camel@powerslave> (raw)
In-Reply-To: <20101231010847.79a55fda@lair>
On Fri, 2010-12-31 at 01:08 +0300, ext Sergey Matyukevich wrote:
> On Wed, 29 Dec 2010 23:22:05 +0200
> Ohad Ben-Cohen <ohad@wizery.com> wrote:
>
> > On Wed, Dec 29, 2010 at 11:07 PM, Sergey Matyukevich
> > <geomatsi@gmail.com> wrote:
> > > So the question is: what is the proper use of wl1271 driver in the
> > > case when wl1271 card is powered all the time ?
> >
> > The wl1271 driver assumes it can control the power of the card.
> >
> > If your card's power is always on, you will not be able to toggle the
> > interface down and up (as you have experienced).
> >
> > It might be possible to change the driver around this "limitation",
> > but I'm not sure anyone has ever really tried.
>
> A simple solution which comes to mind would be to perform some kind of
> 'complete soft reset procedure' for wl1271 card in the beginning of the
> function 'wl1271_op_add_interface'. But I guess such a feature is not
> available for this card. Qtherwise it would be already implemented by
> Luciano and his colleagues.
I don't know about any reset functionality in the wl1271. Maybe the
hardware recovery mechanism could be used, but I'm not sure.
In any case, the reason why we power the card on only when the interface
is brought up is because of power consumption. If the interface is
down, there's no reason to keep the chip powered on and consume
unnecessary power.
If the case of cards that really cannot power off the wl1271 chip is
really unavoidable, we could think about a solution to solve the
problem.
PS: Note that this is the last day when this email address
(luciano.coelho@nokia.com) is valid, so please keep the linux-wireless
mailing list in CC so I can still read the messages. ;)
--
Cheers,
Luca.
next prev parent reply other threads:[~2010-12-31 8:00 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-29 21:07 wl1271: how to use without runtime pm Sergey Matyukevich
2010-12-29 21:22 ` Ohad Ben-Cohen
2010-12-30 22:08 ` Sergey Matyukevich
2010-12-31 8:00 ` Luciano Coelho [this message]
2012-02-15 9:57 ` Santosh Shukla
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=1293782416.15791.10.camel@powerslave \
--to=luciano.coelho@nokia.com \
--cc=geomatsi@gmail.com \
--cc=linux-wireless@vger.kernel.org \
--cc=ohad@wizery.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