From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 4 Jan 2017 15:26:38 -0500 Subject: [U-Boot] [PATCH 00/12] arm: aspeed: Basic support for Aspeed AST2500 part and eval board In-Reply-To: <20170104194656.124368-1-maxims@google.com> References: <20170104194656.124368-1-maxims@google.com> Message-ID: <20170104202638.GU24312@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Jan 04, 2017 at 11:46:44AM -0800, Maxim Sloyko wrote: > This series adds minimal support for AST2500 part and eval board, > enough to boot EVB into prompt. It contains WDT, Timer, Sysreset, > Clock (very basic) and SDRAM MC drivers, all written from scratch, > using AST2500 datasheet. Aspeed's SDK was used only for reference. > Given very limited documentation provided by Aspeed, some parts of SDRAM > init sequence were basically rewritten to do the same thing that is done > in Aspeed SDK, without real understanding of what is going on. > > The file layout closely follows the example of rk3288 chip and firefly-rk3288 > board. > > For the first round of reviews I'm mostly looking for a nod to add > mach-aspeed and arch-aspeed directories, as well as for feedback > on naming, file locations and overall approach. Thanks for doing this. I'll review the individual patches for other changes but in general: > Maxim Sloyko (12): > aspeed: Add mach-aspeed directory and basic Kconfig > aspeed: Add support for Watchdot Timer > aspeed: Add Timer Support > aspeed: Add sysreset driver > aspeed/ast2500: Device Tree and bindings for some of the clocks > aspeed/ast2500: Add Clock Driver > aspeed/ast2500: Helper function to get access to SCU > aspeed/ast2500: Add SDRAM MC driver > aspeed/ast2500: Common board init functions for ast2500 based boards This can really be one patch. Or maybe 2-3 (wdt, timer, sysreset, everything else). The stuff that one could lump together and say "this must be here for the SoC to build and have minimal functionality" is one patch. The changes that adds useful functionality is a patch each. > aspeed: Common configuration parameters for aspeed boards > aspeed: Device Tree for ast2500 Eval Board > aspeed: Configuration for ast2500 eval board Adding the ast2500 EVB would be another single patch. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: