public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Siarhei Siamashka <siarhei.siamashka@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 1/5] sun6i: Make dram clk and zq value Kconfig options
Date: Fri, 12 Dec 2014 22:29:16 +0200	[thread overview]
Message-ID: <20141212222916.0b4747f0@t61> (raw)
In-Reply-To: <1416750195-25318-2-git-send-email-hdegoede@redhat.com>

On Sun, 23 Nov 2014 14:43:11 +0100
Hans de Goede <hdegoede@redhat.com> wrote:

> It turns out that there is a too large spread between boards to handle this
> with a default value, turn this into Kconfig options, and set the values
> the factory images are using for the Colombus and Mele_M9 boards.
> 
> Note this changes the ZQ default when not overriden through defconfig from
> 120 to 123, as that is what most boards seem to actually use.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>

0x7b (or 123 in the decimal) form is a default reset value in the
hardware register (at least on A10/A13/A20 and also on TI Keystone2
hardware). So it indeed makes sense to have it as the default value for
u-boot. And also very likely shows that whoever provided the DRAM
configurations with this value, did not in fact bother to really
configure ZQ to something meaningful :-)

In general, if we read the JEDEC booklets, then ZQ calibration is the
feature, which allows to improve reliability and increase DRAM clock
speeds. The DRAM clock speeds, which are well beyond what the
Allwinner A31 boards seem to be using at the moment.

I hope that 0x78 value for ZQ on Mele M9 is really there for a reason,
and not something like actually 0x7B that got misread from the printed
documentation or from screen :-) As the DRAM clock speeds on A31
hardware seem to be really very low, I would suspect that almost any
ZQ value would be probably good enough in practice.

For A10/A13/A20 hardware we actually have easy tools to measure DRAM
reliability. These tools can help to pick good ZQ settings. Something
similar may be potentially implemented for A31 too.

-- 
Best regards,
Siarhei Siamashka

  parent reply	other threads:[~2014-12-12 20:29 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-23 13:43 [U-Boot] [PATCH v2 0/5] sun6i: A31s / CSQ_CS908 board support Hans de Goede
2014-11-23 13:43 ` [U-Boot] [PATCH v2 1/5] sun6i: Make dram clk and zq value Kconfig options Hans de Goede
2014-11-25  8:55   ` Ian Campbell
2014-11-25  9:08     ` Hans de Goede
2014-12-12 20:29   ` Siarhei Siamashka [this message]
2014-11-23 13:43 ` [U-Boot] [PATCH v2 2/5] sun6i: Add sunxi_get_ss_bonding_id() function Hans de Goede
2014-11-23 13:43 ` [U-Boot] [PATCH v2 3/5] sun6i: dram: Do not try to initialize a second dram chan on A31s Hans de Goede
2014-11-25  8:56   ` Ian Campbell
2014-12-12 20:25   ` Siarhei Siamashka
2014-12-13 11:00     ` Hans de Goede
2014-12-19 10:02       ` Siarhei Siamashka
2014-11-23 13:43 ` [U-Boot] [PATCH v2 4/5] sun6i: Drop some "unknown magic" from dram init Hans de Goede
2014-12-12 20:24   ` Siarhei Siamashka
2014-12-13 10:57     ` Hans de Goede
2014-11-23 13:43 ` [U-Boot] [PATCH v2 5/5] sun6i: Add new CSQ_CS908 board Hans de Goede
2014-12-14 14:57   ` Ian Campbell
2014-12-18 10:39     ` Hans de Goede
2014-12-18 18:56       ` Ian Campbell
  -- strict thread matches above, loose matches on Subject: below --
2014-11-23 13:39 [U-Boot] [PATCH v2 1/5] sun6i: Make dram clk and zq value Kconfig options Hans de Goede

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141212222916.0b4747f0@t61 \
    --to=siarhei.siamashka@gmail.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox