From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nikolay Dimitrov Date: Thu, 16 Apr 2015 12:57:51 +0300 Subject: [U-Boot] [PATCH] mx6: Add initial SPL support for HummingBoard-i2eX In-Reply-To: References: <1429131475-12805-1-git-send-email-festevam@gmail.com> <552EF1DF.7030505@mail.bg> <552F0F32.5020209@mail.bg> Message-ID: <552F879F.9040108@mail.bg> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Fabio, On 04/16/2015 04:27 AM, Fabio Estevam wrote: > Hi Nikolay, > > On Wed, Apr 15, 2015 at 10:24 PM, Nikolay Dimitrov wrote: > >> imx6 supports up to 528 MHz DDR3 clock as per datasheet, which makes >> 1058 MT/s data rate. Unfortunately such comments (like above) in the >> code doesn't help to clarify the differences. Which is a nice way to >> say the comments are wrong :). >> >> Here's what happens - chip specs say that the chip supports 1600 MT/s >> (800 MHz DDR3 clock). The imx6 ddr code looks up in the switch/case and >> calculates timings for 800 MHz, while in reality the imx6 MMDC >> officially supports up to 528 MHz. What should happen - mem_speed >> should be clamped to imx6 model-specific speed limit, so the MMDC code >> can calculate valid timings. > > Care to submit a patch with your proposal? Yes. > This should be independent of this hummingboard support patch. OK. > Thanks, > > Fabio Estevam Regards, Nikolay