Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Thu, 26 Jul 2018 13:39:20 -0700 (PDT)
Message-Id: <20180726.133920.1400134352576859104.davem@davemloft.net>
To:     jdmason@kudzu.us
Cc:     helgaas@kernel.org, f.fainelli@gmail.com, rasesh.mody@cavium.com,
        sudarsana.kalluru@cavium.com, Dept-GELinuxNICDev@cavium.com,
        netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] net: sort Kconfig menu items alphabetically
From:   David Miller <davem@davemloft.net>
In-Reply-To: <20180724154340.GA5871@kudzu.us>
References: <153237967407.172440.6754268418710104431.stgit@bhelgaas-glaptop.roam.corp.google.com>
        <20180724154340.GA5871@kudzu.us>
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

From: Jon Mason <jdmason@kudzu.us>
Date: Tue, 24 Jul 2018 11:43:41 -0400

> On Mon, Jul 23, 2018 at 04:01:14PM -0500, Bjorn Helgaas wrote:
>> From: Bjorn Helgaas <bhelgaas@google.com>
>> 
>> 6c541b4595a2 ("net: ethernet: Sort Kconfig sourcing alphabetically")
>> sorted Kconfig sourcing based on directory names, but in a couple cases,
>> the menu item text is quite different from the directory name and is not
>> sorted correctly:
>> 
>>   drivers/net/ethernet/neterion/Kconfig    => "Exar devices"
> 
> Stupid question, does it make more sense to clarify the names in the
> Kconfig to more naturally follow what people are expecting?  In the
> case of Neterion, it was standalone for several years, then aquired by
> Exar and shutdown in 11 months without ever making any Exar branded
> adatpers.  Users, if they still exist, would probably think of it as
> Neterion and not Exar (as there have been no follow-on adapters and I
> don't believe any plans to ever do so).
> 
> So, this might be better as a rename of the Kconfig to something like
> "Neterion (Exar) devices".  Of course, there is a fair amount of Exar
> names in the Kconfig.  If you agree, I can do a clean-up of this to
> match the expectations.
> 
> Thoughts?

I think this is a better way to handle this.

That way, a developer can read the Kconfig file and see if the entries
are sorted properly without having to check the text of every
sub-directory entry.