Re: [PATCH v4] wl1271: Change wl12xx Files Names

[Luciano Coelho <luciano.coelho@nokia.com>]

On Thu, 2010-11-04 at 17:37 +0100, ext Grazvydas Ignotas wrote:
> On Thu, Nov 4, 2010 at 5:07 PM, Shahar Levi <shahar_levi@ti.com> wrote:
> > All files name prefix removed due to the fact that wl12xx driver supports
> > wl1271 and wl1273.
> > Also the definition in Kconfig and header files changed respectively.
> >
> > Signed-off-by: Shahar Levi <shahar_levi@ti.com>
> 
> Hmh the name 'wl12xx' kind of implies it covers 1251, while in fact
> wl1251 is completely separate driver.. At least the Kconfig help text
> should mention it doesn't cover wl1251 and users should use respective
> driver for wl1251.

That's a good point.  We have discussed this before and this was the
best solution we found.  The wl12xx driver includes support for wl1271
and wl1273, so we considered calling it wl127x, but in the near future,
we'll have support for wl1281 and wl1283 as well.  So we have to use
wl12xx.

At least in my opinion, it is kind of clear that if there is a separate
driver for wl1251, the user would use that one.  In any case, A note in
the Kconfig help text would definitely not hurt.

Shahar, could you please change the text in the Kconfig so that it reads
something like this?

"This module adds support for wireless adapters based on TI wl1271 and
TI wl1273 chipsets.  This module does *not* include support for wl1251.
For wl1251 support, use the separate homonymous driver instead."


-- 
Cheers,
Luca.