On Mon, Sep 1, 2014 at 5:18 PM, Siarhei Siamashka <[email protected]> wrote: > On Mon, 1 Sep 2014 12:40:06 +0800 > Chen-Yu Tsai <[email protected]> wrote: > >> Signed-off-by: Chen-Yu Tsai <[email protected]> >> --- >> >> I want to get some input before i push this. >> Is the dram_zq parameter suppose to be filled? >> or just the lower 8 bits? It seems all the other >> sun4/5/7i boards only have the lower 8 bits set. > > Appears that we need a separate documentation for the dram_para struct > in addition to the hardware registers. But the 'dram_zq' parameter is > basically the same as > http://linux-sunxi.org/A10_DRAM_Controller_Register_Guide#SDR_ZQCR0 > with just the order of ZPROG and ZDATA changed (ZPROG is configured in > the lower 8 bits of 'dram_zq'). > >> ChenYu >> >> --- >> sys_config/a13/hsg_h702.fex | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/sys_config/a13/hsg_h702.fex b/sys_config/a13/hsg_h702.fex >> index 0748b6b..b691df4 100644 >> --- a/sys_config/a13/hsg_h702.fex >> +++ b/sys_config/a13/hsg_h702.fex >> @@ -59,11 +59,11 @@ dram_baseaddr = 0x40000000 >> dram_clk = 432 >> dram_type = 3 >> dram_rank_num = 1 >> -dram_chip_density = 2048 >> -dram_io_width = 8 >> +dram_chip_density = 4096 >> +dram_io_width = 16 > > This appears to be a change from two ddr3 chips with density 2048 and 8 > data lines to a single ddr3 chip with density 4096 and 16 data lines. > > They are mostly equivalent. And I don't think that it is even possible > to distinguish these two configurations in any way from the software. > Except that higher density chips require more time for refresh. So > using the configuration with density 2048 is not quite a safe choice > if there is in fact only one chip with density 4096.
Opened the tablet to check. The DRAM chip is Micron's MT41J256M16HA with 256X16DDR3 labeled as well. According to manufacturer specs, it is indeed 16 data lines and density 4096. [1] http://www.micron.com/parts/dram/ddr3-sdram/mt41j256m16ha-125 >> dram_bus_width = 16 >> dram_cas = 9 >> -dram_zq = 0x7b >> +dram_zq = 0x56b9697b > > It looks like this sets ZDATA to 0x6b969 and also sets the ZDEN bit. > Which means that ZQ calibration is bypassed and the ZPROG value (0x7b) > is not used. > > And it additionally sets the 'NOICAL' bit: > > https://github.com/omegamoon/Rockchip-GPL-Kernel/blob/master/arch/arm/mach-rk29/ddr.c#L218 > Which still needs to be tested and documented in the wiki. > > Was this dram configuration originally set by boot0? This was captured using the Luc's cleaned up version of a10-meminfo, under stock Android firmware. So yes, this is set by boot0. But i did use the A13_MID target in u-boot-sunxi before and it did work. >> dram_odt_en = 0 > > Maybe I need to run more tests to be completely sure, but when I checked > this stuff, I could not observe any measurable effects of setting ZQ > configuration unless ODT is also enabled. > >> dram_size = 512 >> dram_tpr0 = 0x42d899b7 Cheers ChenYu -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/d/optout.
