From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Wed, 21 Jan 2015 09:03:46 +0100 Subject: [U-Boot] [PATCHv1 01/22] arm: socfpga: spl: Add main sdram code In-Reply-To: <54BEE9EE.4020003@opensource.altera.com> References: <1421253662-27222-1-git-send-email-dinguyen@opensource.altera.com> <1421253662-27222-2-git-send-email-dinguyen@opensource.altera.com> <201501150034.16486.marex@denx.de> <54BEE9EE.4020003@opensource.altera.com> Message-ID: <54BF5D62.1040203@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 21.01.2015 00:51, Dinh Nguyen wrote: > On 01/14/2015 05:34 PM, Marek Vasut wrote: >> On Wednesday, January 14, 2015 at 05:40:41 PM, dinguyen at opensource.altera.com >> wrote: >>> From: Dinh Nguyen >> >> Hi! >> >>> This adds the code to configure the SDRAM controller that is found in the >>> SoCFGPA Cyclone5 and Arria5 platforms. >>> >>> Signed-off-by: Dinh Nguyen >>> --- >>> arch/arm/cpu/armv7/socfpga/config.mk | 3 +- >>> board/altera/socfpga/Makefile | 1 + >>> board/altera/socfpga/sdram/Makefile | 12 + >>> board/altera/socfpga/sdram/sdram_config.h | 100 + >>> board/altera/socfpga/sdram/sdram_io.h | 44 + >>> board/altera/socfpga/sdram/sequencer.c | 7993 >> >> A general remark: SDRAM stuff should most likely reside in drivers/ddr/ , don't >> you think so ? > > Yes, I suppose. Move it to drivers/ddr/altera? Yes, this looks reasonable. Thanks, Stefan