DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 243F62086C
DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 08753601D4
From:   Kalle Valo <kvalo@codeaurora.org>
To:     <Adham.Abozaeid@microchip.com>
Cc:     <Ajay.Kathat@microchip.com>, <linux-wireless@vger.kernel.org>,
        <gregkh@linuxfoundation.org>, <Ganesh.Krishna@microchip.com>,
        <Aditya.Shankar@microchip.com>, <Venkateswara.Kaja@microchip.com>,
        <Claudiu.Beznea@microchip.com>
Subject: Re: [RFC 00/19] wilc: added driver for wilc module
References: <1537957525-11467-1-git-send-email-ajay.kathat@microchip.com>
        <87zhvrdz0q.fsf@kamboji.qca.qualcomm.com>
        <20181008104757.5e27eacb@ajaysk-VirtualBox>
        <87o9c4zy53.fsf@codeaurora.org> <5BBBA348.3070503@microchip.com>
Date:   Thu, 15 Nov 2018 16:11:52 +0200
In-Reply-To: <5BBBA348.3070503@microchip.com> (Adham Abozaeid's message of
        "Mon, 8 Oct 2018 18:34:51 +0000")
Message-ID: <87d0r6bfbr.fsf@codeaurora.org>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
MIME-Version: 1.0
Content-Type: text/plain
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

<Adham.Abozaeid@microchip.com> writes:

> On 10/08/2018 12:38 AM, Kalle Valo wrote:
>> Ajay Singh <ajay.kathat@microchip.com> writes:
>> 
>>> On Sat, 6 Oct 2018 15:45:41 +0300
>>> Kalle Valo <kvalo@codeaurora.org> wrote:
>>>
>>>> Ajay Singh <ajay.kathat@microchip.com> writes:
>>>>
>>>>> This patch set contains the driver files from
>>>>> 'driver/staging/wilc1000'. Renamed the driver from 'wilc1000' to
>>>>> 'wilc' to have generic name, as the same driver will be used by
>>>>> other wilc family members.  
>>>>
>>>> I'm worried that the name 'wilc' is just too generic, I liked the
>>>> original name wilc1000 much more. Quite often when we have a new
>>>> generation of wireless devices there's also a new driver, so in the
>>>> long run I'm worried that a generic name like 'wilc' could be a
>>>> source of confusion. I think it's much smaller problem if
>>>> 'wilc1000' (the driver) also supports wilc3000 (the device), people
>>>> are already used to that. 
>
>> If I'm understanding correctly you are worried that 'wilc1000-spi.ko'
>> also supports wilc3000 devices, but I don't see that as a problem. I
>> think it's very common (not just in wireless) that the marketing names
>> don't always match with driver names.
>> 
>
> It's highly unlikely that microchip will have a new generation of wilc
> devices other than wilc1000 and wilc3000, since a new family is
> already in development.
> And in case a new generation was developed, it will be best to support
> it in the current driver because of the similarities between wilc
> devices.
>
> I'm afraid that it might be more confusing for users to use wilc1000
> naming while they are using wilc3000 hardware. It's not only that the
> name that is different from the marketing name, but it also refers to
> another existing product.

Well, I see it very differently. For example, if I google 'wilc1000' I
get directly to the product page but with 'wilc' I get nothing useful.
And I have been dealing with marketing for so long that "never say
never" about what they will decide ;)

So I'm still not convinced that renaming is a good idea. But actually my
opinion doesn't matter here, as the rename should happen in the staging
tree (when the driver is moved from staging to drivers/net/wireless it
should be a simple rename, no other changes). So I'll leave this for
Greg to decide if the rename is worthwhile or not. My vote would be a
clear "no" :)

-- 
Kalle Valo