From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chin Liang See Date: Mon, 7 Dec 2015 22:29:16 +0800 Subject: [U-Boot] [PATCH 5/6] arm: socfpga: Introduce common board code In-Reply-To: <201512071523.34567.marex@denx.de> References: <1449348262-18488-1-git-send-email-marex@denx.de> <201512071423.26624.marex@denx.de> <1449495124.2213.11.camel@altera.com> <201512071523.34567.marex@denx.de> Message-ID: <1449498556.2213.14.camel@altera.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, 2015-12-07 at 15:23 +0100, Marek Vasut wrote: > On Monday, December 07, 2015 at 02:32:04 PM, Chin Liang See wrote: > > On Mon, 2015-12-07 at 14:23 +0100, Marek Vasut wrote: > > > On Monday, December 07, 2015 at 01:53:48 PM, Chin Liang See > > > wrote: > > > > > > [...] > > > > > > > > diff --git a/arch/arm/mach-socfpga/board.c b/arch/arm/mach > > > > > -socfpga/board.c > > > > > new file mode 100644 > > > > > index 0000000..a41d089 > > > > > --- /dev/null > > > > > +++ b/arch/arm/mach-socfpga/board.c > > > > > > > > I am thinking this can go into misc.c instead new file. > > > > > > misc.c is bloated with random crap already, I would like to keep > > > things > > > orderly a little. Let's keep the board code and platform code > > > separated. > > > > Initial thinking misc.c should be as lean as possible but that > > seems > > not today. Can consider split them out per features such as usb.c > > ethernet.c. > > There is no ethernet/usb... anything in misc.c and there shouldn't > be. > All of this stuff is (or should be) probed from OF :-) Looking at misc.c, yup, they are specific to socfpga such as system manager emac registers. Till then, good from me Acked-by: Chin Liang See Thanks Chin Liang > > Best regards, > Marek Vasut