public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 08/18] sandbox: Use the reset driver to handle reset
Date: Mon, 10 Aug 2015 21:35:50 -0600	[thread overview]
Message-ID: <55C96D96.8060409@wwwdotorg.org> (raw)
In-Reply-To: <CAPnjgZ1gw8nOJ7xVw2d5Cx-Dx9-15ZU1M41dY=x8NtxL5pz64w@mail.gmail.com>

On 07/17/2015 05:58 PM, Simon Glass wrote:
> On 6 July 2015 at 12:54, Simon Glass <sjg@chromium.org> wrote:
>> Move sandbox over to use the reset uclass for reset, instead of a direct
>> call to do_reset(). This allows us to add tests.
>>
>> Signed-off-by: Simon Glass <sjg@chromium.org>
>> ---
>>
>>  arch/sandbox/cpu/cpu.c                    | 9 +--------
>>  arch/sandbox/dts/test.dts                 | 8 ++++++++
>>  arch/sandbox/include/asm/u-boot-sandbox.h | 3 +++
>>  configs/sandbox_defconfig                 | 1 +
>>  4 files changed, 13 insertions(+), 8 deletions(-)
> 
> Applied to u-boot-dm.

This patch causes the reset command to stop working in sandbox. It now
prints:

=> reset
Reset not supported on this platform
### ERROR ### Please RESET the board ###

Among other things, this causes ./test/fs/fs-test.sh to hang without any
particular indication why. (In that test, running under expect/pyexpect
might be nicer, so the user could see progress; the error above doesn't
even show up in the test log files).

  reply	other threads:[~2015-08-11  3:35 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-06 18:54 [U-Boot] [PATCH 00/18] dm: test: Add test code for new uclasses Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 01/18] dm: Add platform data advice and admonishment Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 02/18] dm: test: Allow test names to leave out the dm_test_ prefix Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 03/18] dm: test: Add tests for the clk uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 04/18] dm: test: Add tests for the pinctrl uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 05/18] sandbox: Support multiple reset types Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 06/18] dm: reset: Allow reset_walk() to return Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 07/18] sandbox: Add a warm and cold reset driver Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 08/18] sandbox: Use the reset driver to handle reset Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-08-11  3:35     ` Stephen Warren [this message]
2015-08-11  3:44       ` Simon Glass
2015-09-25  5:13         ` Stephen Warren
2015-09-25  5:32           ` Stephen Warren
2015-10-03 17:21             ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 09/18] dm: test: Add a test for the reset uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 10/18] dm: test: Add a test for the ram uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 11/18] dm: test: Add a test for the mmc uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 12/18] led: Return -ENODEV if the LED device cannot be found Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 13/18] dm: test: Add a test for the LED uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 14/18] dm: test: Add a test for the system controller uclass Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 15/18] dm: test: Add a size to each reg property Simon Glass
2015-07-17 23:58   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 16/18] test: Add a macro to check that a value is not an error pointer Simon Glass
2015-07-17 23:59   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 17/18] dm: core: Add device checking to syscon_get_regmap() Simon Glass
2015-07-17 23:59   ` Simon Glass
2015-07-06 18:54 ` [U-Boot] [PATCH 18/18] test: Add a test for regmap Simon Glass
2015-07-17 23:59   ` Simon Glass

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=55C96D96.8060409@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --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