linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: "Stefanik Gábor" <netrolller.3d@gmail.com>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
	Andrey Yurovsky <andrey@cozybit.com>,
	Dan Williams <dcbw@redhat.com>,
	David Woodhouse <dwmw2@infradead.org>,
	linux-wireless <linux-wireless@vger.kernel.org>,
	Luis Carlos Cobo <luisca@cozybit.com>
Subject: Re: libertas_tf should replace libertas_usb on compat-wireless?
Date: Wed, 12 Nov 2008 15:39:56 -0800	[thread overview]
Message-ID: <20081112233956.GC15803@tesla> (raw)
In-Reply-To: <69e28c910811121523n49f79c8ye490a29ee165508b@mail.gmail.com>

On Wed, Nov 12, 2008 at 03:23:05PM -0800, Stefanik G=E1bor wrote:
> On Wed, Nov 12, 2008 at 11:32 PM, Luis R. Rodriguez
> <lrodriguez@atheros.com> wrote:
> > On Wed, Nov 12, 2008 at 2:29 PM, Stefanik G=E1bor <netrolller.3d@gm=
ail.com> wrote:
> >> On Wed, Nov 12, 2008 at 11:27 PM, Luis R. Rodriguez
> >> <lrodriguez@atheros.com> wrote:
> >>> On Wed, Nov 12, 2008 at 01:16:36PM -0800, Stefanik G=E1bor wrote:
> >>>> On Wed, Nov 12, 2008 at 10:07 PM, Andrey Yurovsky <andrey@cozybi=
t.com> wrote:
> >>>> > libertas_tf is a softmac driver developed mainly so that XO's =
can run
> >>>> > hostapd and the like.  libertas is the general (fullmac) drive=
r and,
> >>>> > in my opinion, should be the one users get unless they specifi=
cally
> >>>> > wanted libertas_tf because they're experimenting with AP mode =
or other
> >>>> > mac80211 features.  So I guess I'd vote for libertas over libe=
rtas_tf
> >>>> > at this time, for what it's worth.
> >>>> >
> >>>> >  -Andrey
> >>>>
> >>>> I think libertas_usb should be a default (as it's HW accelerated=
), but
> >>>> libertas_tf should be included as an option (disabled by default=
 in
> >>>> config.mk, or auto-added to /etc/modprobe.d/blacklist upon make
> >>>> install).
> >>>
> >>> OK so I've done this on config.mk:
> >>>
> >>> ifneq ($(CONFIG_USB),)
> >>> ifneq ($(CONFIG_LIBERTAS_THINFIRM_USB),m)
> >>> CONFIG_LIBERTAS_USB=3Dm
> >>> NEED_LIBERTAS=3Dy
> >>> endif
> >>> endif
> >>>
> >>> This means you'll get CONFIG_LIBERTAS_USB unless your kernel
> >>> already was built with CONFIG_LIBERTAS_THINFIRM_USB for all those
> >>> building with it. If your kernel was built with both then its you=
r
> >>> problem.
> >>>
> >>>  Luis
> >>>
> >>
> >> If the kernel has both, compat-wireless should also build both.
> >
> > The kernel has both, compat-wireless has both but we respect your
> > kernel .config unless we know better (we picked b43 over bcm43xx
> > before). In this case we don't want to prefer one over the other so=
 we
> > simply stick to the decision of whoever built your kernel.
> >
> >  Luis
> >
>=20
> I mean if both are enabled, and both were built.

OK yeah agreed, but if the user has nothing then we should pick one
for them too, in this case it'll be LIBERTAS_USB.

 Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2008-11-12 23:39 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-12  0:32 libertas_tf should replace libertas_usb on compat-wireless? Luis R. Rodriguez
2008-11-12 15:53 ` Dan Williams
2008-11-12 17:57   ` Dan Williams
2008-11-12 19:55     ` Luis R. Rodriguez
2008-11-12 21:07       ` Andrey Yurovsky
2008-11-12 21:16         ` Stefanik Gábor
2008-11-12 22:27           ` Luis R. Rodriguez
2008-11-12 22:29             ` Stefanik Gábor
2008-11-12 22:32               ` Luis R. Rodriguez
2008-11-12 23:23                 ` Stefanik Gábor
2008-11-12 23:39                   ` 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=20081112233956.GC15803@tesla \
    --to=lrodriguez@atheros.com \
    --cc=Luis.Rodriguez@Atheros.com \
    --cc=andrey@cozybit.com \
    --cc=dcbw@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=luisca@cozybit.com \
    --cc=netrolller.3d@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).