From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 3/4 V2] i.MX28: Add delay after CPU bypass is cleared
Date: Fri, 4 May 2012 13:13:23 +0200 [thread overview]
Message-ID: <201205041313.23932.marex@denx.de> (raw)
In-Reply-To: <m2zk9o4817.fsf@lamuella.denx.de>
Dear Detlev Zundel,
> Hi Marek,
>
> > This solves issues when larger amount of DRAM is used. Behave the
> > same in case of CPU bypass as we do in case of EMI bypass, wait
> > 15 ms. We need to wait until the clock domain stabilizes.
>
> Sorry to be somewhat persistent here, but can you please include the
> information what "larger amount of DRAM" is that this delay works for?
> Also is this guessed, measured or calculated?
Pure guesswork. I have no scientific background for this.
> The reason why I am so persistent is that this is _available_
> information now and it will be very valuable information for the next
> person reading the code while pondering the question "ok, this worked in
> the past, but maybe it is a problem on my brand new hardware".
There is zero information available, I'll need to dig deeper ... Fabio, can you
comment on this? Is this needed or is it some other problem?
> Thanks
> Detlev
Best regards,
Marek Vasut
next prev parent reply other threads:[~2012-05-04 11:13 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-03 15:47 [U-Boot] [PATCH 1/4 V2] Revert "i.MX28: Enable additional DRAM address bits" Marek Vasut
2012-05-03 15:47 ` [U-Boot] [PATCH 2/4 RESEND] M28: Scan only first 512 MB of DRAM to avoid memory wraparound Marek Vasut
2012-05-03 15:47 ` [U-Boot] [PATCH 3/4 V2] i.MX28: Add delay after CPU bypass is cleared Marek Vasut
2012-05-04 9:20 ` Detlev Zundel
2012-05-04 11:13 ` Marek Vasut [this message]
2012-05-04 11:32 ` [U-Boot] [PATCH] " Marek Vasut
2012-05-06 16:38 ` Stefano Babic
2012-05-06 16:39 ` Marek Vasut
2012-05-07 9:49 ` Detlev Zundel
2012-05-06 16:28 ` [U-Boot] [PATCH 3/4 V2] " Stefano Babic
2012-05-06 16:30 ` Marek Vasut
2012-05-03 15:47 ` [U-Boot] [PATCH 4/4] M28: Enable FDT support Marek Vasut
2012-05-06 15:53 ` Stefano Babic
2012-05-06 16:12 ` Marek Vasut
2012-05-06 16:18 ` Stefano Babic
2012-05-06 16:41 ` Marek Vasut
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=201205041313.23932.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