qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: agraf@suse.de
Cc: qemu-devel@nongnu.org, David Gibson <david@gibson.dropbear.id.au>
Subject: [Qemu-devel] [PATCH 1/3] target-ppc: Disable PReP testcases with !CONFIG_PREP
Date: Tue, 26 Aug 2014 14:30:16 +1000	[thread overview]
Message-ID: <1409027418-4453-2-git-send-email-david@gibson.dropbear.id.au> (raw)
In-Reply-To: <1409027418-4453-1-git-send-email-david@gibson.dropbear.id.au>

The test suites contain several tests which require the "prep" machine
type, which will fail if that is configured out.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
---
 tests/boot-order-test.c | 4 ++++
 tests/endianness-test.c | 2 ++
 2 files changed, 6 insertions(+)

diff --git a/tests/boot-order-test.c b/tests/boot-order-test.c
index 360a691..bff16b5 100644
--- a/tests/boot-order-test.c
+++ b/tests/boot-order-test.c
@@ -112,6 +112,7 @@ static void test_pc_boot_order(void)
     test_boot_orders(NULL, read_boot_order_pc, test_cases_pc);
 }
 
+#ifdef CONFIG_PREP
 static uint8_t read_m48t59(uint64_t addr, uint16_t reg)
 {
     writeb(addr, reg & 0xff);
@@ -135,6 +136,7 @@ static void test_prep_boot_order(void)
 {
     test_boot_orders("prep", read_boot_order_prep, test_cases_prep);
 }
+#endif /* CONFIG_PREP */
 
 static uint64_t read_boot_order_pmac(void)
 {
@@ -194,7 +196,9 @@ int main(int argc, char *argv[])
     if (strcmp(arch, "i386") == 0 || strcmp(arch, "x86_64") == 0) {
         qtest_add_func("boot-order/pc", test_pc_boot_order);
     } else if (strcmp(arch, "ppc") == 0 || strcmp(arch, "ppc64") == 0) {
+#ifdef CONFIG_PREP
         qtest_add_func("boot-order/prep", test_prep_boot_order);
+#endif /* CONFIG_PREP */
         qtest_add_func("boot-order/pmac_oldworld",
                        test_pmac_oldworld_boot_order);
         qtest_add_func("boot-order/pmac_newworld",
diff --git a/tests/endianness-test.c b/tests/endianness-test.c
index 92e17d2..0857784 100644
--- a/tests/endianness-test.c
+++ b/tests/endianness-test.c
@@ -41,7 +41,9 @@ static const TestCase test_cases[] = {
     { "mips64", "malta", 0x10000000, .bswap = true },
     { "mips64el", "fulong2e", 0x1fd00000 },
     { "ppc", "g3beige", 0xfe000000, .bswap = true, .superio = "i82378" },
+#ifdef CONFIG_PREP
     { "ppc", "prep", 0x80000000, .bswap = true },
+#endif /* CONFIG_PREP */
     { "ppc", "bamboo", 0xe8000000, .bswap = true, .superio = "i82378" },
     { "ppc64", "mac99", 0xf2000000, .bswap = true, .superio = "i82378" },
     { "ppc64", "pseries", 0x10080000000ULL,
-- 
1.9.3

  reply	other threads:[~2014-08-26  4:29 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-26  4:30 [Qemu-devel] [0/3] target-ppc Fixes for some missing config dependencies David Gibson
2014-08-26  4:30 ` David Gibson [this message]
2014-08-26  4:30 ` [Qemu-devel] [PATCH 2/3] target-ppc: Disable ppc Mac tests when !CONFIG_MAC David Gibson
2014-08-26  4:30 ` [Qemu-devel] [PATCH 3/3] target-ppc: virtex-ml507 machine type should depend on CONFIG_XILINX David Gibson
2014-08-26  6:12   ` Peter Crosthwaite
2014-09-08 11:10   ` Alexander Graf
2014-08-26  8:34 ` [Qemu-devel] [0/3] target-ppc Fixes for some missing config dependencies Paolo Bonzini
2014-08-27  8:22   ` David Gibson
2014-08-27 10:08     ` Paolo Bonzini

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=1409027418-4453-2-git-send-email-david@gibson.dropbear.id.au \
    --to=david@gibson.dropbear.id.au \
    --cc=agraf@suse.de \
    --cc=qemu-devel@nongnu.org \
    /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;
as well as URLs for NNTP newsgroup(s).