From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1TgtK3-0001VM-Iq for barebox@lists.infradead.org; Fri, 07 Dec 2012 08:30:16 +0000 Date: Fri, 7 Dec 2012 09:30:12 +0100 From: Sascha Hauer Message-ID: <20121207083012.GB10369@pengutronix.de> References: <1354792603-15915-1-git-send-email-fabio.porcedda@gmail.com> <20121207065249.GK8327@game.jcrosoft.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20121207065249.GK8327@game.jcrosoft.org> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH v2] at91: reorder BOARDINFO alphabetically To: Jean-Christophe PLAGNIOL-VILLARD Cc: barebox@lists.infradead.org On Fri, Dec 07, 2012 at 07:52:49AM +0100, Jean-Christophe PLAGNIOL-VILLARD wrote: > On 12:16 Thu 06 Dec , Fabio Porcedda wrote: > > This is to have a nicely ordered list and to decrease the > > likelihood of a conflict when adding a new board. > > > > Signed-off-by: Fabio Porcedda > > Cc: Jean-Christophe PLAGNIOL-VILLARD > > --- > > v2: > NAK > > I order them by Kconfig keepo it that way I'd say it makes more sense to keep them in the order the user actually sees them, it makes it easier to find the entries. What's the value in ordering them by Kconfig name? If you order them by Kconfig name, the Calao boards end up in three different places in the user visible view. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox