From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Wed, 5 Aug 2015 21:46:03 +0200 Subject: [U-Boot] [PATCH 00/28] socfpga: sdram.c cleanups In-Reply-To: <55C2392A.1040108@opensource.altera.com> References: <1438464897-8051-1-git-send-email-marex@denx.de> <55C2392A.1040108@opensource.altera.com> Message-ID: <201508052146.03365.marex@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 Wednesday, August 05, 2015 at 06:26:18 PM, Dinh Nguyen wrote: > On 8/1/15 4:34 PM, Marek Vasut wrote: > > This entire series focuses solely on cleaning up the > > drivers/ddr/altera/sdram.c file. After this series, this one file is > > totally checkpatch clean and does not pull in any weird qts-generated > > macros ; all that is wrapped in the board file. > > > > This micro-series applies on top of my previous mega-series [1]. > > This series is available via git at [2]. > > My only comment for this series is that there seems to be alot of > redundant debug() in the file drivers/ddr/altera/sdram.c in the function > sdr_load_regs(). No big deal and can probably be cleaned up further > with a separate patch. Would you like to work on that please ? > Otherwise, for the whole series: > > Acked-by: Dinh Nguyen Excellent, thank you for keeping up with this blast of patches! :-) Best regards, Marek Vasut