[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] Allow ISA bus to be configured out
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] Allow ISA bus to be configured out |
Date: |
Tue, 30 Jun 2015 13:44:10 +1000 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Mon, Jun 29, 2015 at 12:26:50PM +0200, Andreas Färber wrote:
> Am 29.06.2015 um 07:36 schrieb David Gibson:
> > Currently, the code to handle the legacy ISA bus is always included in
> > qemu. However there are lots of platforms that don't include ISA legacy
> > devies, and quite a few that have never used ISA legacy devices at all.
> >
> > This patch allows the ISA bus code to be disabled in the configuration for
> > platforms where it doesn't make sense. For now, the default configs are
> > adjusted to include ISA on all platforms including PCI (since
> > CONFIG_IDE_CORE which is in pci.mak requires ISA support) and also several
> > others which include ISA devices. We may want to pare this down in future.
>
> PCI also allows to have a PCI-ISA bridge such as the i82378, which means
> that all PCI platforms can potentially obtain an actual ISA bus. So at
> least the commit message could use a makeover to avoid someone touching
> IDE to blindly disable the dependency.
Sorry, I'm not quite following what you're getting at here.
> You'd also need to think about the qtests then:
>
> $ git grep i82378 -- tests/
> tests/endianness-test.c: { "ppc", "g3beige", 0xfe000000, .bswap =
> true, .superio = "i82378" },
> tests/endianness-test.c: { "ppc", "bamboo", 0xe8000000, .bswap =
> true, .superio = "i82378" },
> tests/endianness-test.c: { "ppc64", "mac99", 0xf2000000, .bswap =
> true, .superio = "i82378" },
> tests/endianness-test.c: .bswap = true, .superio = "i82378" },
> tests/endianness-test.c: { "sh4", "r2d", 0xfe240000, .superio =
> "i82378" },
> tests/endianness-test.c: { "sh4eb", "r2d", 0xfe240000, .bswap = true,
> .superio = "i82378" },
Ah, good point. I'll need to rework for that,
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
pgpnZ8AdmXRX_.pgp
Description: PGP signature
[Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, David Gibson, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Laurent Vivier, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Thomas Huth, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Laurent Vivier, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Laurent Vivier, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Andreas Färber, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Laurent Vivier, 2015/06/29
- Re: [Qemu-devel] [PATCH 4/4] Disable "info irq" and "info pic" for target-ppc, Andreas Färber, 2015/06/29