From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Mon, 18 Jun 2012 17:16:19 +0200 Subject: [U-Boot] [PATCH] i.MX28: Add function to adjust memory parameters In-Reply-To: <201206181623.34836.marex@denx.de> References: <1338356593-6897-1-git-send-email-marex@denx.de> <201205312045.24308.marex@denx.de> <201206181623.34836.marex@denx.de> Message-ID: <4FDF4643.7000100@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 18/06/2012 16:23, Marek Vasut wrote: > Dear Fabio Estevam, > >> Hi Marek, >> >> On Thu, May 31, 2012 at 3:45 PM, Marek Vasut wrote: >>> Call this or similar in your spl_boot.c to adjust the DRAM configuration. >>> Actually I think even the following value should fix your problem: >>> >>> void mx28_adjust_memory_params(uint32_t *dram_vals) >>> { >>> dram_vals[0x74 >> 2] = 0x0f02010a; >>> } >> >> I finally had a chance to try this and my mx28evk can boot again with >> your patch + the code above. >> >> How do we handle this? Would you send a v2 with this additional code >> or should I do it myself after your original patch in this thread gets >> applied? > > Stefano, can you please apply this? So Fabio can apply his on top of it? I do it. > >> Also, please explain to the mortal folks what 'dram_vals[0x74 >> 2] = >> 0x0f02010a;' means. > > Well, check that particular register (memory configuration register at +0x74 > offset) in the datasheet. It's basically adjusting the number of enabled address > row and columns. > >> It is not something very trivial to figure out :-) > > It actually is, see above ;-) I see, but I will never apply a patch programmed in hexadecimal instead of plain C ;-) Stefano -- ===================================================================== DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================