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 <picmas...@mail.bg> 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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to