From: Luciano Coelho <luciano.coelho@nokia.com>
To: "ext Gabay, Benzy" <benzyg@ti.com>
Cc: "ext Gábor Stefanik" <netrolller.3d@gmail.com>,
"ext Grazvydas Ignotas" <notasas@gmail.com>,
"Levi, Shahar" <shahar_levi@ti.com>,
"Kalle Valo" <kvalo@adurom.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: RE: [PATCH v4] wl1271: Change wl12xx Files Names
Date: Wed, 17 Nov 2010 20:48:32 +0200 [thread overview]
Message-ID: <1290019712.28892.5.camel@powerslave> (raw)
In-Reply-To: <676FD6AA0F002F49B89E70F0C5EFD22C136DEA0A14@dlee04.ent.ti.com>
On Wed, 2010-11-17 at 12:08 -0600, ext Gabay, Benzy wrote:
> Hi All,
Hi Benzy,
> > > What about wl1251 vs. wl12agn (or wl12bgn if no .11a support), like
> > iwlwifi?
> >
> > Hmmm... wl1251 supports bg. wl12xx supports abgn. I don't really see
> > the logic in this? What difference does it make comparing to calling it
> > wl12xx? From wl12agn the user still doesn't know whether wl1251 is
> > supported or not.
>
> I have another idea for naming convention here.
> But first to align everyone on the TI chip conventions:
> 12x1 == support for 2.4Ghz
> 12x3 == supports both 2.4Ghz and 5.0Ghz.
>
> The TI project naming convention per chip is as follow:
>
> wilink4 == 1251 and 1253
> wilink6 == 1271 and 1273
> wilink7 == 1281 and 1281
>
> So I propose to use the project name instead of using the chip name.
> e.g.:
> for wl125x we can use: wl4 or wilink4. If we want to imply the band support: wl4_abg and wl4_bg
> for wl127x we can use: wl6 or wilink6. If we want to imply the band and capability support: wl6_abgn and wl4_bgn
>
> etc'
>
> What say?
Doesn't help much. The problem is that the driver which is now called
wl12xx already support wl1271 and wl1273. In the near future, it will
also support wl1281 and wl1283, so the wl6 or wilink6 scheme would break
already.
The wl1251 driver, at least at the moment, only supports wl1251, it
doesn't have the 11a implementation. If in the future someone adds
support for wl1253, it could be renamed to wl125x.
In any case, I think the name of the driver is not that important, as
long as we're clear about which chips each one of them supports in the
Kconfig description.
Renaming the driver all the time (we already did it twice), ends up
confusing more than clarifying things.
--
Cheers,
Luca.
next prev parent reply other threads:[~2010-11-17 18:48 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-04 15:07 [PATCH v4] wl1271: Change wl12xx Files Names Shahar Levi
2010-11-04 16:37 ` Grazvydas Ignotas
2010-11-04 17:19 ` Luciano Coelho
2010-11-04 21:48 ` Gábor Stefanik
2010-11-04 21:52 ` Luciano Coelho
2010-11-07 10:07 ` Kalle Valo
2010-11-08 9:16 ` Luciano Coelho
2010-11-17 18:08 ` Gabay, Benzy
2010-11-17 18:48 ` Luciano Coelho [this message]
2010-11-18 2:13 ` Gabay, Benzy
2010-11-18 5:18 ` Krakowski, Oz
2010-11-18 6:21 ` Luciano Coelho
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=1290019712.28892.5.camel@powerslave \
--to=luciano.coelho@nokia.com \
--cc=benzyg@ti.com \
--cc=kvalo@adurom.com \
--cc=linux-wireless@vger.kernel.org \
--cc=netrolller.3d@gmail.com \
--cc=notasas@gmail.com \
--cc=shahar_levi@ti.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.