linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kalle Valo <kvalo@qca.qualcomm.com>
To: Larry Finger <Larry.Finger@lwfinger.net>
Cc: Julien Massot <jmassot@aldebaran-robotics.com>,
	Mohammed Shafi Shajakhan <mohammed@qca.qualcomm.com>,
	<linux-wireless@vger.kernel.org>, <ath6kl-devel@qca.qualcomm.com>
Subject: Re: ath6kl: AR6004 failed to start on computer boot
Date: Fri, 22 Feb 2013 17:06:06 +0200	[thread overview]
Message-ID: <87mwuwqtgh.fsf@kamboji.qca.qualcomm.com> (raw)
In-Reply-To: <51265413.9080609@lwfinger.net> (Larry Finger's message of "Thu, 21 Feb 2013 11:06:27 -0600")

Larry Finger <Larry.Finger@lwfinger.net> writes:

> On 02/21/2013 08:03 AM, Julien Massot wrote:
>> Hi,
>> I just find a quick and dirty patch to workaround this issue.
>> I just reset the device on initialization failure.
>>
>> I hope this helps to understand the real issue.
>>

[...]

> I am certain that the real problem here is that ath6kl is requesting
> firmware with a synchronous call using request_firmware() rather than
> with the asynchronous request_firmware_nowait(). That used to work,
> but updates to udev caused the firmware read operation to time out. It
> works after a reset because the file reading routines are now running;
> however, the correct fix is to rewrite the firmware reading section.

I haven't looked at all the details yet, but AFAIK this is about cold vs
warm booting the chip. When rebooting the host while maintaining the
power to ar6004 ("warm boot") ath6kl probe fails as the chip is in odd
state and needs to be reset. It's not about dowloading the firmware
image from user space, it's about starting the firmware inside the chip.

And besides, didn't udev finally fix that (after some "feedback" from
Linus) so that we don't need to change all the drivers?

-- 
Kalle Valo

  reply	other threads:[~2013-02-22 15:06 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-10 14:39 ath6kl: AR6004 failed to start on computer boot Julien Massot
2012-12-13 17:46 ` Kalle Valo
2013-02-15 10:09 ` Kalle Valo
2013-02-15 14:10   ` Julien Massot
2013-02-15 14:22     ` Mohammed Shafi Shajakhan
2013-02-21 14:03       ` Julien Massot
2013-02-21 14:18         ` Mohammed Shafi Shajakhan
2013-03-08 10:28           ` Kalle Valo
2013-03-12 16:15             ` Mohammed Shafi Shajakhan
2013-02-21 17:06         ` Larry Finger
2013-02-22 15:06           ` Kalle Valo [this message]
2013-02-25 17:19             ` Yvan
2013-02-25 19:39               ` Larry Finger
     [not found]                 ` <1448005898.305768.1361866657564.open-xchange@ox-webdesk.1and1.fr>
2013-02-26 14:09                   ` Larry Finger

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=87mwuwqtgh.fsf@kamboji.qca.qualcomm.com \
    --to=kvalo@qca.qualcomm.com \
    --cc=Larry.Finger@lwfinger.net \
    --cc=ath6kl-devel@qca.qualcomm.com \
    --cc=jmassot@aldebaran-robotics.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=mohammed@qca.qualcomm.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;
as well as URLs for NNTP newsgroup(s).