linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: Richard Farina <sidhayn@gmail.com>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
	"dwmw2@infradead.org" <dwmw2@infradead.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] atheros: add ar9170.fw 1-stage firmware
Date: Tue, 25 Aug 2009 12:29:44 -0700	[thread overview]
Message-ID: <20090825192944.GB7357@mosca> (raw)
In-Reply-To: <4A943AFE.1000500@gmail.com>

On Tue, Aug 25, 2009 at 12:26:54PM -0700, Richard Farina wrote:
> Luis R. Rodriguez wrote:
> > On Thu, Aug 20, 2009 at 01:07:57PM -0700, Richard Farina wrote:
> >
> >> Luis R. Rodriguez wrote:
> >>
> >>> This adds support for ar9170 for using a 1-stage firmware.
> >>> This means only 1 firmware file is required. AVM Fritz
> >>> devices require this.
> >>>
> >>> Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
> >>> ---
> >>>
> >>> Here is a resend, noticed today this wasn't yet in the tree.
> >>>
> >>>
> >>>
> >> Last I tested this firmware it didn't seem to function nearly as well as
> >> the 2 stage firmware, and if present, the 1 stage overrides the 2
> >> stage.
> >>
> >
> > Can you be more specific than that?
> >
> >
> >> Is this an improved version?
> >>
> >
> > No, it is the same firmware.
> >
> >
> >> Does this work properly in your testing?
> >>
> >
> > Works for me, and is also required for usage of AVM Fritz devices.
> >
> >
> I apologize for my confusion.  Since we spoke on irc I am now clear that
> this specific ar9170 firmware is slightly different than the open source
> one available in git and seems a bit more stable than the 2 stage I was
> previously using.  I have tested monitor mode and packet injection which
> are the only things that interest me about this device at the moment and
> both appear slightly more stable using the 1 stage firmware. Please do
> add the firmware located at
> http://www.kernel.org/pub/linux/kernel/people/mcgrof/firmware/ar9170/ar9170.fw
> to the linux firmware git repo.  I believe that it would be a benefit to
> many.
> 
> ACKED BY: Rick Farina
> TESTED BY: Rick Farina
> 
> On a similar note, the open firmware has had issues reported by many
> with speed and stability so I wouldn't recommend using it yet, but this
> is mostly based on the complaints I've heard rather than my own personal
> testing so remember a grain of salt.

Thanks for the clarification, yeah I do believe the open fw needs more work,
the code base is the same though so it remains unclear what could be the
issue. Anyway I'll push this through and if hopefully more interested people 
can work on the open fw. I'll respin with the WHENCE changed.

  Luis

      reply	other threads:[~2009-08-25 19:29 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-20 18:40 [PATCH] atheros: add ar9170.fw 1-stage firmware Luis R. Rodriguez
2009-08-20 19:26 ` David Woodhouse
2009-08-20 21:04   ` Luis R. Rodriguez
2009-08-20 20:07 ` Richard Farina
2009-08-20 20:28   ` Luis R. Rodriguez
2009-08-25 19:26     ` Richard Farina
2009-08-25 19:29       ` Luis R. Rodriguez [this message]

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=20090825192944.GB7357@mosca \
    --to=lrodriguez@atheros.com \
    --cc=Luis.Rodriguez@Atheros.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=sidhayn@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;
as well as URLs for NNTP newsgroup(s).