All of lore.kernel.org
 help / color / mirror / Atom feed
From: York Sun <yorksun@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/7] powerpc/mpc8xxx: Enable entering DDR debugging by key press
Date: Tue, 8 Jan 2013 11:24:10 -0800	[thread overview]
Message-ID: <50EC725A.4010505@freescale.com> (raw)
In-Reply-To: <20130108063532.117F6200732@gemini.denx.de>

On 01/07/2013 10:35 PM, Wolfgang Denk wrote:
> Dear York Sun,
> 
> In message <1357323245-12455-1-git-send-email-yorksun@freescale.com> you wrote:
>> ...
>> CONFIG_FSL_DDR_INTERACTIVE needs to be defined in header file. To enter the
>> debug mode by key press, press key 'd' shortly after reset, like one would
>> do to abort auto booting. It is fixed to lower case 'd' at this moment.
> ...
>> --- a/doc/README.fsl-ddr
>> +++ b/doc/README.fsl-ddr
>> @@ -268,6 +268,13 @@ be activated by saving an environment variable "ddr_interactive". The value
>>  doesn't matter. Once activated, U-boot prompts "FSL DDR>" before enabling DDR
>>  controller. The available commands can be seen by typing "help".
>>  
>> +Another way to enter debug mode without using environment variable is to send
>> +a key press during boot, like one would do to abort auto boot. To save booting
>> +time, no additioal delay is added so the window to send the key press is very
>> +short. For example, user can send the key press using reset command followed by
>> +hitting enter key twice. In case of power on reset, user can keep hitting any
>> +key while applying the power.
> 
> The documentation here does not mention the 'd' key at all.  Guess it
> should?
> 
>
It should. And James found my error and fixed in the patch 5/7 in this
series.

York

      parent reply	other threads:[~2013-01-08 19:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-04 18:13 [U-Boot] [PATCH 1/7] powerpc/mpc8xxx: Enable entering DDR debugging by key press York Sun
2013-01-04 18:14 ` [U-Boot] [PATCH 2/7] Move DDR command parsing to separate function York Sun
2013-01-04 18:14 ` [U-Boot] [PATCH 3/7] Fix data stage name matching issue York Sun
2013-01-04 18:14 ` [U-Boot] [PATCH 4/7] Add copy command to FSL DDR interactive York Sun
2013-01-04 18:14 ` [U-Boot] [PATCH 5/7] README.fsl-ddr typos and update to reflect hotkey York Sun
2013-01-04 18:14 ` [U-Boot] [PATCH 6/7] getenv_f() env variable exist w/o needing a buffer York Sun
2013-01-04 22:06   ` Wolfgang Denk
2013-01-04 23:08     ` James Yang
2013-01-05  6:23       ` Wolfgang Denk
2013-01-07 17:46   ` [U-Boot] [u-boot-release] " Timur Tabi
2013-01-04 18:14 ` [U-Boot] [PATCH 7/7] powerpc/mpc8xxx: FSL DDR debugger auto run of stored commands York Sun
2013-01-08  6:35 ` [U-Boot] [PATCH 1/7] powerpc/mpc8xxx: Enable entering DDR debugging by key press Wolfgang Denk
2013-01-08  6:39   ` sun york-R58495
2013-01-08 19:24   ` York Sun [this message]

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=50EC725A.4010505@freescale.com \
    --to=yorksun@freescale.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.