Re: [PATCH v2] carl9170: Clarify kconfig text

From: Christian Lamparter
Date: Sat Apr 16 2016 - 11:33:01 EST


On Saturday, April 16, 2016 05:18:56 PM Lauri Kasanen wrote:
> The previous text was confusing, leading readers to think this
> driver was a duplicate, and so didn't need to be enabled.
>
> After the removal of the older staging driver, this is the only
> driver in mainline for these devices.
>
> Signed-off-by: Lauri Kasanen <cand@xxxxxxx>
Acked-by: Christian Lamparter <chunkeey@xxxxxxxxxxxxxx>

> ---
> v2: Remove the mention of the previous driver, suggested by Christian.

Thanks!

> drivers/net/wireless/ath/carl9170/Kconfig | 8 +++-----
> 1 file changed, 3 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/wireless/ath/carl9170/Kconfig b/drivers/net/wireless/ath/carl9170/Kconfig
> index 1a796e5..2e34bae 100644
> --- a/drivers/net/wireless/ath/carl9170/Kconfig
> +++ b/drivers/net/wireless/ath/carl9170/Kconfig
> @@ -5,12 +5,10 @@ config CARL9170
> select FW_LOADER
> select CRC32
> help
> - This is another driver for the Atheros "otus" 802.11n USB devices.
> + This is the mainline driver for the Atheros "otus" 802.11n USB devices.
>
> - This driver provides more features than the original,
> - but it needs a special firmware (carl9170-1.fw) to do that.
> -
> - The firmware can be downloaded from our wiki here:
> + It needs a special firmware (carl9170-1.fw), which can be downloaded
> + from our wiki here:
> <http://wireless.kernel.org/en/users/Drivers/carl9170>
>
> If you choose to build a module, it'll be called carl9170.
>