public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH RFC] arm: mx6: Add CCGR0 configuration to default DCD (spl_sd.cfg)
Date: Mon, 08 Dec 2014 15:32:26 +0100	[thread overview]
Message-ID: <5485B67A.8000800@denx.de> (raw)
In-Reply-To: <CAOMZO5BHrE8XiGnUBa6OD8bu11XqhDEDiyPaEYmz+8BcCFCn=Q@mail.gmail.com>

Hi Fabio,

On 08.12.2014 15:15, Fabio Estevam wrote:
> On Mon, Dec 8, 2014 at 11:31 AM, Stefan Roese <sr@denx.de> wrote:
>> While switching a custom i.MX6DL board port to the common mx6
>> infrastructure without any board specific DCD file (*.cfg), booting
>> from SD-card (mmc0) via the bmode command (bmode mmc0) did not work
>> any more. Adding this one line for the CCGR0 solves this issue.
>>
>> I have to admit that I'm not really sure why this is needed in
>> this case. So if somebody has an explanation for the need for
>> this CCGR0 register setup in the DCD, then please let me know.
>
> Could you please try to narrow down what exact field of the CGR0
> register makes your board boot?
>
> I suspect it could be the AIPS related clocks (CG0 / CG1).

I tested with these values:

0x00C03F30
0x00C03F33
0x00C03F3C

All do not boot. Only the original one does:

0x00C03F3F

So yes, AIPS related clocks are definitely needed!

I'm wondering about this, since the default value after reset should be 
0xffffffff for this register, right? All clocks enabled. But this does 
not seem to be the case. As SPL is not started correctly (no output at 
all on the console) when I don't add this line to spl_sd.cfg.

Its not easy for me to debug this issue since the board usually boots 
from NAND. I can't strap this board to boot via MMC0 right now. So I 
have to use the "bmode mmc0" command. And JTAG debugging via BDI3000 
doesn't seem to "survive" the bmode command.

> Please confirm. If this is the case, then we should turn on the AIPS
> clocks inside init_aips().

See above.

Could you test this on one of your board? If your board also fails to 
boot via the "bmode mmc0" command if the CCGR0 register value is missing?

Thanks,
Stefan

  reply	other threads:[~2014-12-08 14:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-08 13:31 [U-Boot] [PATCH RFC] arm: mx6: Add CCGR0 configuration to default DCD (spl_sd.cfg) Stefan Roese
2014-12-08 14:15 ` Fabio Estevam
2014-12-08 14:32   ` Stefan Roese [this message]
2014-12-08 14:51     ` Fabio Estevam
2014-12-08 15:19       ` Stefan Roese
2014-12-08 15:36         ` Fabio Estevam
2014-12-08 15:42           ` Stefan Roese
2014-12-08 16:13             ` Fabio Estevam
2014-12-08 16:27               ` Stefan Roese
2014-12-18 13:27                 ` Fabio Estevam

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=5485B67A.8000800@denx.de \
    --to=sr@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