From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vasily Khoruzhick Date: Sat, 21 Apr 2012 10:35:12 +0300 Subject: [U-Boot] [PATCH v3] Add support for MINI2440 (s3c2440). Documentation about the product can be found on: http://www.friendlyarm.net/products/mini2440 In-Reply-To: <201204210107.55926.marex@denx.de> References: <20120416174633.GA7463@debian> <201204202203.33873.marex@denx.de> <201204210107.55926.marex@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 2012/4/21 Marek Vasut : >> ?I agree with Vasily Khoruzhick, it looks like a hack > > Please, bother to read the CPU manual and also the ARM Architecture reference > manual before you start with such a wild accusations. Let me explain my point of view. Doing SDRAM init very early (even in assembly code) is obvious and simple. Just few simple writes into RAM controller registers. But to use cache as RAM you need to program and enable MMU (again assembly, right?). And then one needs to program SDRAM controller (ok, this part is in C code, but I don't see any advantage). It's not as simple as previous solution, isn't it? And looks overengineered to me. I prefer to keep such low-level things simple. Regards Vasily