From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38185) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z2EEO-00017e-3s for qemu-devel@nongnu.org; Tue, 09 Jun 2015 03:45:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z2EEK-0007ut-JZ for qemu-devel@nongnu.org; Tue, 09 Jun 2015 03:45:55 -0400 Received: from cantor2.suse.de ([195.135.220.15]:35682 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z2EEK-0007uP-Dp for qemu-devel@nongnu.org; Tue, 09 Jun 2015 03:45:52 -0400 Message-ID: <557699AE.5080601@suse.de> Date: Tue, 09 Jun 2015 09:45:50 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1433805832-31863-1-git-send-email-i.am.cormac.obrien@gmail.com> In-Reply-To: <1433805832-31863-1-git-send-email-i.am.cormac.obrien@gmail.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] macio: remove nonexistent interrupt on pin 1 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cormac O'Brien , mark.cave-ayland@ilande.co.uk, qemu-devel@nongnu.org On 09.06.15 01:23, Cormac O'Brien wrote: > The current macio device implementation declares an interrupt that doesn't > seem to exist in the actual hardware or any other emulators. Mac OS 9 counts > the presence of this interrupt as a fatal error and progresses further without > it. This is only half of the problem unfortunately :). The reason the existence of an interrupt pin is a problem is that OpenBIOS automatically generates an interrupt property in the device tree for the macio device. Please mention this in the patch description as well. Also, please make sure to CC qemu-ppc@nongnu.org on v2 :). Thanks! Alex > > Signed-off-by: Cormac O'Brien > > --- > hw/misc/macio/macio.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/hw/misc/macio/macio.c b/hw/misc/macio/macio.c > index 063ad80..306c95e 100644 > --- a/hw/misc/macio/macio.c > +++ b/hw/misc/macio/macio.c > @@ -132,8 +132,6 @@ static int macio_common_initfn(PCIDevice *d) > SysBusDevice *sysbus_dev; > int ret; > > - d->config[0x3d] = 0x01; // interrupt on pin 1 > - > ret = qdev_init(DEVICE(&s->cuda)); > if (ret < 0) { > return ret; >