From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King - ARM Linux Subject: Re: [PATCH 1/4] Clean up the long arch list for the UID16 config option Date: Thu, 4 Oct 2012 20:46:48 +0100 Message-ID: <20121004194648.GB4625@n2100.arm.linux.org.uk> References: <1349282666-32100-1-git-send-email-catalin.marinas@arm.com> <1349282666-32100-2-git-send-email-catalin.marinas@arm.com> <20121003151505.8d59b646.akpm@linux-foundation.org> <20121004194131.GA4625@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Geert Uytterhoeven Cc: Andrew Morton , Catalin Marinas , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, Mike Frysinger , Mikael Starvik , Jesper Nilsson , David Howells , Yoshinori Sato , Martin Schwidefsky , Heiko Carstens , Paul Mundt , "David S. Miller" , Jeff Dike , Richard Weinberger , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org List-Id: linux-arch.vger.kernel.org On Thu, Oct 04, 2012 at 09:45:35PM +0200, Geert Uytterhoeven wrote: > On Thu, Oct 4, 2012 at 9:41 PM, Russell King - ARM Linux > wrote: > > On Wed, Oct 03, 2012 at 03:15:05PM -0700, Andrew Morton wrote: > >> Guys, don't do this. Either put the new item into a randomly-chosen > >> position or, probably better, alphanumerically sort the list. > > > > I've been telling people to alphanumerically sort Kconfig select > > options for some time, but as normal, no one's listening. > > Anyone who cares to sort the existing ones "at the end of the merge window", > and teaches checkpatch to enforce it for new ones? Well, I'll certainly do that for ARM... but how long it'll stay like that will be very hit and miss... From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from caramon.arm.linux.org.uk ([78.32.30.218]:46270 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756828Ab2JDTrG (ORCPT ); Thu, 4 Oct 2012 15:47:06 -0400 Date: Thu, 4 Oct 2012 20:46:48 +0100 From: Russell King - ARM Linux Subject: Re: [PATCH 1/4] Clean up the long arch list for the UID16 config option Message-ID: <20121004194648.GB4625@n2100.arm.linux.org.uk> References: <1349282666-32100-1-git-send-email-catalin.marinas@arm.com> <1349282666-32100-2-git-send-email-catalin.marinas@arm.com> <20121003151505.8d59b646.akpm@linux-foundation.org> <20121004194131.GA4625@n2100.arm.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-arch-owner@vger.kernel.org List-ID: To: Geert Uytterhoeven Cc: Andrew Morton , Catalin Marinas , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, Mike Frysinger , Mikael Starvik , Jesper Nilsson , David Howells , Yoshinori Sato , Martin Schwidefsky , Heiko Carstens , Paul Mundt , "David S. Miller" , Jeff Dike , Richard Weinberger , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org Message-ID: <20121004194648.pkmdoUPacyti5lPsy8xegNJQTTMsWMnqdTanbkPxUPs@z> On Thu, Oct 04, 2012 at 09:45:35PM +0200, Geert Uytterhoeven wrote: > On Thu, Oct 4, 2012 at 9:41 PM, Russell King - ARM Linux > wrote: > > On Wed, Oct 03, 2012 at 03:15:05PM -0700, Andrew Morton wrote: > >> Guys, don't do this. Either put the new item into a randomly-chosen > >> position or, probably better, alphanumerically sort the list. > > > > I've been telling people to alphanumerically sort Kconfig select > > options for some time, but as normal, no one's listening. > > Anyone who cares to sort the existing ones "at the end of the merge window", > and teaches checkpatch to enforce it for new ones? Well, I'll certainly do that for ARM... but how long it'll stay like that will be very hit and miss...