From nobody Sun Feb 8 12:58:07 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=eik.bme.hu Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1633277867379117.42440888185024; Sun, 3 Oct 2021 09:17:47 -0700 (PDT) Received: from localhost ([::1]:34728 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mX4BL-0003Cz-7O for importer@patchew.org; Sun, 03 Oct 2021 12:17:43 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:41752) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mX4AI-0002Fu-Az for qemu-devel@nongnu.org; Sun, 03 Oct 2021 12:16:39 -0400 Received: from zero.eik.bme.hu ([152.66.115.2]:17768) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mX4AF-0002rs-7B for qemu-devel@nongnu.org; Sun, 03 Oct 2021 12:16:37 -0400 Received: from zero.eik.bme.hu (blah.eik.bme.hu [152.66.115.182]) by localhost (Postfix) with SMTP id 8497C7462D3; Sun, 3 Oct 2021 18:16:29 +0200 (CEST) Received: by zero.eik.bme.hu (Postfix, from userid 432) id 614BA745953; Sun, 3 Oct 2021 18:16:29 +0200 (CEST) From: BALATON Zoltan Subject: [PATCH] hw/usb/vt82c686-uhci-pci: Use ISA instead of PCI interrupts Date: Sun, 03 Oct 2021 18:12:25 +0200 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: qemu-devel@nongnu.org Message-Id: <20211003161629.614BA745953@zero.eik.bme.hu> X-Spam-Probability: 8% Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=152.66.115.2; envelope-from=balaton@eik.bme.hu; helo=zero.eik.bme.hu X-Spam_score_int: -41 X-Spam_score: -4.2 X-Spam_bar: ---- X-Spam_report: (-4.2 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Huacai Chen , Gerd Hoffmann , Philippe M-D Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZM-MESSAGEID: 1633277868731100001 Content-Type: text/plain; charset="utf-8" This device is part of a sperio/ISA bridge chip and IRQs from it are routed to an ISA interrupt set by the Interrupt Line PCI config register. Change uhci_update_irq() to allow this and use it from vt82c686-uhci-pci. Signed-off-by: BALATON Zoltan Reviewed-by: Jiaxun Yang --- Maybe bit of a hack but fixes USB interrupts on pegasos2 until somebody has a better idea. I'm not sure about fuloong2e but the VT82C686B docs say it also does the same routing IRQs to ISA PIC like VT8231 used on pegasos2. hw/usb/hcd-uhci.c | 12 +++++++++--- hw/usb/hcd-uhci.h | 1 + hw/usb/vt82c686-uhci-pci.c | 1 + 3 files changed, 11 insertions(+), 3 deletions(-) diff --git a/hw/usb/hcd-uhci.c b/hw/usb/hcd-uhci.c index 0cb02a6432..8f28241f70 100644 --- a/hw/usb/hcd-uhci.c +++ b/hw/usb/hcd-uhci.c @@ -31,6 +31,7 @@ #include "hw/usb/uhci-regs.h" #include "migration/vmstate.h" #include "hw/pci/pci.h" +#include "hw/irq.h" #include "hw/qdev-properties.h" #include "qapi/error.h" #include "qemu/timer.h" @@ -290,7 +291,7 @@ static UHCIAsync *uhci_async_find_td(UHCIState *s, uint= 32_t td_addr) =20 static void uhci_update_irq(UHCIState *s) { - int level; + int level =3D 0; if (((s->status2 & 1) && (s->intr & (1 << 2))) || ((s->status2 & 2) && (s->intr & (1 << 3))) || ((s->status & UHCI_STS_USBERR) && (s->intr & (1 << 0))) || @@ -298,10 +299,15 @@ static void uhci_update_irq(UHCIState *s) (s->status & UHCI_STS_HSERR) || (s->status & UHCI_STS_HCPERR)) { level =3D 1; + } + if (s->isa_irqs) { + uint8_t irq =3D pci_get_byte(s->dev.config + PCI_INTERRUPT_LINE); + if (irq < ISA_NUM_IRQS) { + qemu_set_irq(isa_get_irq(NULL, irq), level); + } } else { - level =3D 0; + pci_set_irq(&s->dev, level); } - pci_set_irq(&s->dev, level); } =20 static void uhci_reset(DeviceState *dev) diff --git a/hw/usb/hcd-uhci.h b/hw/usb/hcd-uhci.h index e61d8fcb19..c91805a65e 100644 --- a/hw/usb/hcd-uhci.h +++ b/hw/usb/hcd-uhci.h @@ -59,6 +59,7 @@ typedef struct UHCIState { uint32_t frame_bytes; uint32_t frame_bandwidth; bool completions_only; + bool isa_irqs; UHCIPort ports[NB_PORTS]; =20 /* Interrupts that should be raised at the end of the current frame. = */ diff --git a/hw/usb/vt82c686-uhci-pci.c b/hw/usb/vt82c686-uhci-pci.c index b109c21603..5c79e293ef 100644 --- a/hw/usb/vt82c686-uhci-pci.c +++ b/hw/usb/vt82c686-uhci-pci.c @@ -13,6 +13,7 @@ static void usb_uhci_vt82c686b_realize(PCIDevice *dev, Er= ror **errp) /* USB legacy support */ pci_set_long(pci_conf + 0xc0, 0x00002000); =20 + s->isa_irqs =3D true; usb_uhci_common_realize(dev, errp); } =20 --=20 2.21.4