From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 0/2] OMAP: convert boards that use SMSC911x to use gpmc-smsc911x Date: Tue, 19 Apr 2011 14:45:47 +0300 Message-ID: <20110419114547.GC15620@atomide.com> References: <20110418112153.GK12272@atomide.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mho-01-ewr.mailhop.org ([204.13.248.71]:39850 "EHLO mho-01-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752028Ab1DSLpw (ORCPT ); Tue, 19 Apr 2011 07:45:52 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Mike Rapoport Cc: Mike Rapoport , linux-omap@vger.kernel.org * Mike Rapoport [110419 14:32]: > Hi Tony, > > On Mon, Apr 18, 2011 at 2:21 PM, Tony Lindgren wrote: > > * Mike Rapoport [110417 01:21]: > > Nice job Mike :) > > There's a _lot_ of very similar code in different board files. If > you'd like to I can try to create something like > common-board-devices.c and move most of that code there. It's not the > real consolidation Linus wanted, but at least all the copy-pasted > device initialization code would be removed. Yes please do if you can, that helps a lot. Also most beagle related boards could probably have a common board-beagle.c file. Eventually the board configuration will come from device tree. But until that's available, we can consolidate the code within arch/arm/*omap*/ as the common platform init functions will still be needed with configuration coming from DT. Regards, Tony