[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-1.6? v2 19/21] shix: Don't require firmware
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH for-1.6? v2 19/21] shix: Don't require firmware presence for qtest |
Date: |
Wed, 7 Aug 2013 12:13:50 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Aug 05, 2013 at 03:27:25PM +0200, Andreas Färber wrote:
> Adopt error_report() while at it.
>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
> hw/block/tc58128.c | 10 ++++++----
> hw/sh4/shix.c | 9 +++++----
> 2 files changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/hw/block/tc58128.c b/hw/block/tc58128.c
> index a3929d4..728f1c3 100644
> --- a/hw/block/tc58128.c
> +++ b/hw/block/tc58128.c
> @@ -1,6 +1,8 @@
> #include "hw/hw.h"
> #include "hw/sh4/sh.h"
> #include "hw/loader.h"
> +#include "sysemu/qtest.h"
> +#include "qemu/error-report.h"
>
> #define CE1 0x0100
> #define CE2 0x0200
> @@ -36,10 +38,10 @@ static void init_dev(tc58128_dev * dev, const char
> *filename)
> /* Load flash image skipping the first block */
> ret = load_image(filename, dev->flash_contents + 528 * 32);
> if (ret < 0) {
> - fprintf(stderr, "ret=%d\n", ret);
> - fprintf(stderr, "qemu: could not load flash image %s\n",
> - filename);
> - exit(1);
> + if (!qtest_enabled()) {
> + error_report("Could not load flash image %s", filename);
> + exit(1);
> + }
> } else {
> /* Build first block with number of blocks */
> blocks = (ret + 528 * 32 - 1) / (528 * 32);
> diff --git a/hw/sh4/shix.c b/hw/sh4/shix.c
> index 66cbea4..5ee7af6 100644
> --- a/hw/sh4/shix.c
> +++ b/hw/sh4/shix.c
> @@ -30,9 +30,11 @@
> #include "hw/hw.h"
> #include "hw/sh4/sh.h"
> #include "sysemu/sysemu.h"
> +#include "sysemu/qtest.h"
> #include "hw/boards.h"
> #include "hw/loader.h"
> #include "exec/address-spaces.h"
> +#include "qemu/error-report.h"
>
> #define BIOS_FILENAME "shix_bios.bin"
> #define BIOS_ADDRESS 0xA0000000
> @@ -72,10 +74,9 @@ static void shix_init(QEMUMachineInitArgs *args)
> if (bios_name == NULL)
> bios_name = BIOS_FILENAME;
> ret = load_image_targphys(bios_name, 0, 0x4000);
> - if (ret < 0) { /* Check bios size */
> - fprintf(stderr, "qemu: could not load SHIX bios '%s'\n",
> - bios_name);
> - exit(1);
> + if (ret < 0 && !qtest_enabled()) {
> + error_report("Could not load SHIX bios '%s'", bios_name);
> + exit(1);
> }
>
> /* Register peripherals */
Reviewed-by: Aurelien Jarno <address@hidden>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net
- [Qemu-devel] [PATCH for-1.6? v2 07/21] palm: Don't enforce loading ROM or kernel for qtest, (continued)
- [Qemu-devel] [PATCH for-1.6? v2 07/21] palm: Don't enforce loading ROM or kernel for qtest, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 10/21] armv7m: Don't enforce use of kernel for qtest, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 16/21] ppc405_uc: Disable debug output, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 14/21] milkymist: Suppress -kernel/-bios/-drive error for qtest, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 17/21] ppc405_boards: Don't enforce presence of firmware for qtest, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 19/21] shix: Don't require firmware presence for qtest, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 18/21] shix: Drop debug output, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 21/21] qtest: Prepare QOM machine tests, Andreas Färber, 2013/08/05
- [Qemu-devel] [PATCH for-1.6? v2 20/21] leon3: Don't enforce use of -bios with qtest, Andreas Färber, 2013/08/05
- Re: [Qemu-devel] [PATCH for-1.6? v2 00/21] qtest: Test all targets, Markus Armbruster, 2013/08/06