From: Sven Schnelle <svens@stackframe.org>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: Helge Deller <deller@gmx.de>,
qemu-devel@nongnu.org, Richard Henderson <rth@twiddle.net>
Subject: Re: [PATCH 3/7] hppa: remove ISA region
Date: Mon, 21 Oct 2019 22:58:57 +0200 [thread overview]
Message-ID: <20191021205857.GB1439@t470p.stackframe.org> (raw)
In-Reply-To: <284a4bcc-a076-2b94-86e0-0bffbe3e61a0@linaro.org>
Hi Richard,
On Mon, Oct 21, 2019 at 11:17:24AM -0700, Richard Henderson wrote:
> On 10/20/19 1:47 PM, Sven Schnelle wrote:
> > B160L doesn't have an ISA bus, and we no longer need it to
> > workaround missing hardware, so remove it.
> >
> > Signed-off-by: Sven Schnelle <svens@stackframe.org>
> > ---
> > hw/hppa/hppa_hardware.h | 1 -
> > hw/hppa/machine.c | 32 --------------------------------
> > 2 files changed, 33 deletions(-)
>
> This should remove ISA_BUS from hw/hppa/Kconfig as well.
Right. I thought to do so is easy, but that turned out to reveal a lot
of other issues:
- hppa is using the I8259 code, which relies on ISA
- hw/hppa/pci.c seems to only contain unused stuff
- LASI use hw/char/parallel.c, which relies on ISA. parallel.c could
propably be split up into a ISA dependent part and a generic part
So it is more work to remove ISA completely, therefore i'll drop that
patch for now and submit it later again.
Thanks
Sven
next prev parent reply other threads:[~2019-10-21 20:59 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-20 20:47 [PATCH 0/7] HPPA: i82596, PS/2 and graphics emulation Sven Schnelle
2019-10-20 20:47 ` [PATCH 1/7] hw/hppa/dino.c: Improve emulation of Dino PCI chip Sven Schnelle
2019-10-21 18:26 ` Richard Henderson
2019-10-20 20:47 ` [PATCH 2/7] hppa: Add support for LASI chip with i82596 NIC Sven Schnelle
2019-10-20 20:47 ` [PATCH 3/7] hppa: remove ISA region Sven Schnelle
2019-10-21 18:17 ` Richard Henderson
2019-10-21 20:58 ` Sven Schnelle [this message]
2019-10-20 20:47 ` [PATCH 4/7] ps2: accept 'Set Key Make and Break' commands Sven Schnelle
2019-10-20 20:47 ` [PATCH 5/7] hppa: add emulation of LASI PS2 controllers Sven Schnelle
2019-10-20 20:47 ` [PATCH 6/7] hppa: Add emulation of Artist graphics Sven Schnelle
[not found] ` <20191020204724.31537-8-svens@stackframe.org>
2019-10-21 11:24 ` [PATCH 7/7] seabios-hppa: update to latest version Philippe Mathieu-Daudé
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=20191021205857.GB1439@t470p.stackframe.org \
--to=svens@stackframe.org \
--cc=deller@gmx.de \
--cc=qemu-devel@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=rth@twiddle.net \
/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).