Return-path: Received: from smtp.nokia.com ([192.100.122.233]:59625 "EHLO mgw-mx06.nokia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753124Ab0KDVqE (ORCPT ); Thu, 4 Nov 2010 17:46:04 -0400 Subject: Re: [PATCH v4] wl1271: Change wl12xx Files Names From: Luciano Coelho To: ext Grazvydas Ignotas Cc: Shahar Levi , "linux-wireless@vger.kernel.org" , Kalle Valo In-Reply-To: References: <1288883257-30060-1-git-send-email-shahar_levi@ti.com> Content-Type: text/plain; charset="UTF-8" Date: Thu, 04 Nov 2010 19:19:55 +0200 Message-ID: <1288891196.17643.6.camel@powerslave> Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Thu, 2010-11-04 at 17:37 +0100, ext Grazvydas Ignotas wrote: > On Thu, Nov 4, 2010 at 5:07 PM, Shahar Levi 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 > > 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.