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 picmaster@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