public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCHv4 0/3] drivers/ddr/altera: Add the DDR controller driver for SoCFPGA
Date: Fri, 24 Jul 2015 05:57:35 +0200	[thread overview]
Message-ID: <201507240557.35884.marex@denx.de> (raw)
In-Reply-To: <CADhT+wcca2jRqF5E8o+vz_FzHS=swanrx841HOZ3tDZov8H-Zg@mail.gmail.com>

On Thursday, July 23, 2015 at 08:29:42 PM, Dinh Nguyen wrote:

[...]

> >> BTW, I tested out our your branch, but I think I'm missing a step as
> >> where to put the SPL DTB?
> >> 
> >> 
> >> U-Boot SPL 2015.07-rc3-00100-ga2e2da6 (Jul 21 2015 - 17:15:54)
> >> drivers/ddr/altera/sequencer.c: Preparing to start memory calibration
> >> drivers/ddr/altera/sequencer.c: CALIBRATION PASSED
> >> drivers/ddr/altera/sequencer.c: Calibration complete
> >> Missing DTB
> >> ### ERROR ### Please RESET the board ###
> > 
> > Do:
> > 
> > make u-boot-with-spl-dtb.sfp
> 
> Does your updated branch still work for you on the SoCKit? I'm hitting
> the dreaded "CALIBRATION FAILED" on all my boards here, the devkit, SoCKit
> and the new Atlas SoCKit.
> 
> U-Boot SPL 2015.07-00293-ga6f4805 (Jul 23 2015 - 11:18:51)
> drivers/ddr/altera/sequencer.c: Preparing to start memory calibration
> drivers/ddr/altera/sequencer.c: CALIBRATION FAILED
> drivers/ddr/altera/sequencer.c: Calibration complete
> SDRAM calibration failed.
> ### ERROR ### Please RESET the board ###

Hi!

You probably need the use different DRAM config for the sockit (I do use
different ones, generated from Quartus). I pushed my SoCkit hacks to [1]
(u-boot-socfpga / vip/sockit). This is tested last time today, so it should
work for you.

Also, notice the topmost patch [2]. You can ignore (and drop) the delay in
spl.c , that's a hack I use to make it wait until my USB catches up after
cold reset of the sockit. The change to timer.c is something that's baffling
me -- without this change I also get calibration failure, but I didn't find
out why yet.

To make it ever weirder, if I use gcc 5.0.x , I do NOT get the calibration 
failure even WITHOUT this patch ; if I use gcc 4.8.4 (ELDK 5.6.1rc), I do
get calibration failure failure ; if I use gcc 4.7.x (ELDK 5.5.x) and the
gcc from ELDK 5.4.x (I don't remember the version), I do get calibration
failure failure. I am even starting to suspect compiler bugs here, but I
don't want to draw hasty conclusions yet.

btw. where can I get the atlas kit ? it seems out of stock everywhere :(

[1] http://git.denx.de/?p=u-boot/u-boot-
socfpga.git;a=shortlog;h=refs/heads/vip/sockit
[2] http://git.denx.de/?p=u-boot/u-boot-
socfpga.git;a=commit;h=a8fd1c256c24eea79aad0079dbdf4db0593070e0

Best regards,
Marek Vasut

  reply	other threads:[~2015-07-24  3:57 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-03  3:52 [U-Boot] [PATCHv4 0/3] drivers/ddr/altera: Add the DDR controller driver for SoCFPGA dinguyen at opensource.altera.com
2015-06-03  3:52 ` [U-Boot] [PATCHv4 1/3] driver/ddr/altera: Add DDR driver for Altera's SDRAM controller dinguyen at opensource.altera.com
2015-06-09 11:55   ` Pavel Machek
2015-06-09 12:58     ` Wolfgang Denk
2015-06-09 15:51     ` Dinh Nguyen
2015-06-22  9:38       ` Chin Liang See
2015-06-22 10:56         ` Pavel Machek
2015-06-03  3:52 ` [U-Boot] [PATCHv4 2/3] driver/ddr/altera: Add the sdram calibration portion dinguyen at opensource.altera.com
2015-06-09 12:21   ` Pavel Machek
2015-06-03  3:52 ` [U-Boot] [PATCHv4 3/3] arm: socfpga: enable the Altera SDRAM controller driver dinguyen at opensource.altera.com
2015-06-09 12:25   ` Pavel Machek
2015-06-26 16:43 ` [U-Boot] [PATCHv4 0/3] drivers/ddr/altera: Add the DDR controller driver for SoCFPGA Marek Vasut
2015-06-26 20:01   ` Marek Vasut
2015-07-12 19:50     ` Marek Vasut
2015-07-17 19:58       ` Dinh Nguyen
2015-07-17 20:22         ` Marek Vasut
2015-07-18 23:51           ` Marek Vasut
2015-07-20 13:40           ` Dinh Nguyen
2015-07-20 18:36             ` Marek Vasut
2015-07-20 19:31               ` Dinh Nguyen
2015-07-20 19:40                 ` Marek Vasut
2015-07-21 22:46                   ` Dinh Nguyen
2015-07-22  3:24                     ` Marek Vasut
2015-07-23 18:29                       ` Dinh Nguyen
2015-07-24  3:57                         ` Marek Vasut [this message]
2015-07-22  8:27                   ` Dinh Nguyen
2015-07-22  9:00                     ` Marek Vasut
2015-07-22 12:57                       ` Dinh Nguyen
2015-07-22 13:01                         ` Marek Vasut
2015-07-23  4:03                           ` Dinh Nguyen

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=201507240557.35884.marex@denx.de \
    --to=marex@denx.de \
    --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