From nobody Wed Mar 5 18:04:44 2025 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=pass(p=none dis=none) header.from=nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1740995851; cv=none; d=zohomail.com; s=zohoarc; b=Mpz26sJpXZGFmnxK7YPuqnyyKw4HhWWJ7zJeavy/FtFjVHklmtAu5IFKgkXmqGc+QELRiJ/lNXixG9iS+P1ZQrrmiE8nFsutVc+y1UGBaM0WW6bbMdFyUFPQr7DZRgqIkc0+6pzBtEA3ghlAqxEwst0b6VAT7GVxergHCc21hU8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1740995851; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Reply-To:References:Sender:Subject:Subject:To:To:Message-Id; bh=q02Ur6EAtb02w28DiyKVQHctXGFq9c77bgLSDyKt2GA=; b=T4+IRBJebY/A+4NocOrpNgwcPg0VLPYQ69xiRlSsOvQtDhjdUmHfztgvucZG3AsV2dEsJF8eRux9mkQTzFSU2ylw5RYmBsOtSOw9nl1u2BClmXEykmPsN1Xq63iebl23KZ9UoCqMBtTi28QLxyr/17nQ1kBkPc5TwGJVR7yiyeo= ARC-Authentication-Results: i=1; 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=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1740995851262162.346240100121; Mon, 3 Mar 2025 01:57:31 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tp2XS-0000b9-TJ; Mon, 03 Mar 2025 04:56:42 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tp2Wb-000819-Mw; Mon, 03 Mar 2025 04:55:51 -0500 Received: from mail.aspeedtech.com ([211.20.114.72] helo=TWMBX01.aspeed.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tp2WW-0001gH-Hd; Mon, 03 Mar 2025 04:55:49 -0500 Received: from TWMBX01.aspeed.com (192.168.0.62) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1258.12; Mon, 3 Mar 2025 17:55:02 +0800 Received: from mail.aspeedtech.com (192.168.10.10) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server id 15.2.1258.12 via Frontend Transport; Mon, 3 Mar 2025 17:55:02 +0800 To: =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= , Peter Maydell , Steven Lee , Troy Lee , Andrew Jeffery , "Joel Stanley" , "open list:All patches CC here" , "open list:ASPEED BMCs" CC: , , =?UTF-8?q?C=C3=A9dric=20Le=20Goater?= Subject: [PATCH v4 11/23] hw/intc/aspeed: Refactor INTC to support separate input and output pin indices Date: Mon, 3 Mar 2025 17:54:39 +0800 Message-ID: <20250303095457.2337631-12-jamin_lin@aspeedtech.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250303095457.2337631-1-jamin_lin@aspeedtech.com> References: <20250303095457.2337631-1-jamin_lin@aspeedtech.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable 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=211.20.114.72; envelope-from=jamin_lin@aspeedtech.com; helo=TWMBX01.aspeed.com X-Spam_score_int: -18 X-Spam_score: -1.9 X-Spam_bar: - X-Spam_report: (-1.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_FAIL=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.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-to: Jamin Lin From: Jamin Lin via Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZM-MESSAGEID: 1740995852844019100 Refactors the INTC to distinguish between input and output pin indices, improving interrupt handling clarity and accuracy. Updated the functions to handle both input and output pin indices. Added detailed logging for input and output pin indices in trace events. These changes ensure that the INTC controller can handle multiple input and output pins, improving support for the AST2700 A1. Signed-off-by: Jamin Lin Reviewed-by: C=C3=A9dric Le Goater --- hw/intc/aspeed_intc.c | 97 +++++++++++++++++++++++++++---------------- hw/intc/trace-events | 12 +++--- 2 files changed, 67 insertions(+), 42 deletions(-) diff --git a/hw/intc/aspeed_intc.c b/hw/intc/aspeed_intc.c index d558b3824c..9bc3e089d8 100644 --- a/hw/intc/aspeed_intc.c +++ b/hw/intc/aspeed_intc.c @@ -42,20 +42,32 @@ REG32(GICINT136_STATUS, 0x804) =20 #define GICINT_STATUS_BASE R_GICINT128_STATUS =20 -static void aspeed_intc_update(AspeedINTCState *s, int irq, int level) +/* + * Update the state of an interrupt controller pin by setting + * the specified output pin to the given level. + * The input pin index should be between 0 and the number of input pins. + * The output pin index should be between 0 and the number of output pins. + */ +static void aspeed_intc_update(AspeedINTCState *s, int inpin_idx, + int outpin_idx, int level) { AspeedINTCClass *aic =3D ASPEED_INTC_GET_CLASS(s); const char *name =3D object_get_typename(OBJECT(s)); =20 - if (irq >=3D aic->num_inpins) { - qemu_log_mask(LOG_GUEST_ERROR, - "%s: Invalid input pin index: %d\n", - __func__, irq); + if (inpin_idx >=3D aic->num_inpins) { + qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid input pin index: %d\n", + __func__, inpin_idx); return; } =20 - trace_aspeed_intc_update_irq(name, irq, level); - qemu_set_irq(s->output_pins[irq], level); + if (outpin_idx >=3D aic->num_outpins) { + qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid output pin index: %d\n= ", + __func__, outpin_idx); + return; + } + + trace_aspeed_intc_update_irq(name, inpin_idx, outpin_idx, level); + qemu_set_irq(s->output_pins[outpin_idx], level); } =20 /* @@ -73,6 +85,11 @@ static void aspeed_intc_set_irq(void *opaque, int irq, i= nt level) uint32_t select =3D 0; uint32_t enable; int i; + int inpin_idx; + int outpin_idx; + + inpin_idx =3D irq; + outpin_idx =3D irq; =20 if (irq >=3D aic->num_inpins) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid input pin index: %d\n", @@ -80,15 +97,15 @@ static void aspeed_intc_set_irq(void *opaque, int irq, = int level) return; } =20 - trace_aspeed_intc_set_irq(name, irq, level); - enable =3D s->enable[irq]; + trace_aspeed_intc_set_irq(name, inpin_idx, level); + enable =3D s->enable[inpin_idx]; =20 if (!level) { return; } =20 for (i =3D 0; i < aic->num_lines; i++) { - if (s->orgates[irq].levels[i]) { + if (s->orgates[inpin_idx].levels[i]) { if (enable & BIT(i)) { select |=3D BIT(i); } @@ -101,7 +118,7 @@ static void aspeed_intc_set_irq(void *opaque, int irq, = int level) =20 trace_aspeed_intc_select(name, select); =20 - if (s->mask[irq] || s->regs[status_addr]) { + if (s->mask[inpin_idx] || s->regs[status_addr]) { /* * a. mask is not 0 means in ISR mode * sources interrupt routine are executing. @@ -110,16 +127,17 @@ static void aspeed_intc_set_irq(void *opaque, int irq= , int level) * * save source interrupt to pending variable. */ - s->pending[irq] |=3D select; - trace_aspeed_intc_pending_irq(name, irq, s->pending[irq]); + s->pending[inpin_idx] |=3D select; + trace_aspeed_intc_pending_irq(name, inpin_idx, s->pending[inpin_id= x]); } else { /* * notify firmware which source interrupt are coming * by setting status register */ s->regs[status_addr] =3D select; - trace_aspeed_intc_trigger_irq(name, irq, s->regs[status_addr]); - aspeed_intc_update(s, irq, 1); + trace_aspeed_intc_trigger_irq(name, inpin_idx, outpin_idx, + s->regs[status_addr]); + aspeed_intc_update(s, inpin_idx, outpin_idx, 1); } } =20 @@ -132,13 +150,15 @@ static void aspeed_intc_enable_handler(AspeedINTCStat= e *s, hwaddr offset, uint32_t old_enable; uint32_t change; uint32_t irq; + int inpin_idx; =20 irq =3D (offset & 0x0f00) >> 8; + inpin_idx =3D irq; =20 - if (irq >=3D aic->num_inpins) { + if (inpin_idx >=3D aic->num_inpins) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid input pin index: %d\n", - __func__, irq); + __func__, inpin_idx); return; } =20 @@ -149,17 +169,17 @@ static void aspeed_intc_enable_handler(AspeedINTCStat= e *s, hwaddr offset, */ =20 /* disable all source interrupt */ - if (!data && !s->enable[irq]) { + if (!data && !s->enable[inpin_idx]) { s->regs[addr] =3D data; return; } =20 - old_enable =3D s->enable[irq]; - s->enable[irq] |=3D data; + old_enable =3D s->enable[inpin_idx]; + s->enable[inpin_idx] |=3D data; =20 /* enable new source interrupt */ - if (old_enable !=3D s->enable[irq]) { - trace_aspeed_intc_enable(name, s->enable[irq]); + if (old_enable !=3D s->enable[inpin_idx]) { + trace_aspeed_intc_enable(name, s->enable[inpin_idx]); s->regs[addr] =3D data; return; } @@ -167,11 +187,11 @@ static void aspeed_intc_enable_handler(AspeedINTCStat= e *s, hwaddr offset, /* mask and unmask source interrupt */ change =3D s->regs[addr] ^ data; if (change & data) { - s->mask[irq] &=3D ~change; - trace_aspeed_intc_unmask(name, change, s->mask[irq]); + s->mask[inpin_idx] &=3D ~change; + trace_aspeed_intc_unmask(name, change, s->mask[inpin_idx]); } else { - s->mask[irq] |=3D change; - trace_aspeed_intc_mask(name, change, s->mask[irq]); + s->mask[inpin_idx] |=3D change; + trace_aspeed_intc_mask(name, change, s->mask[inpin_idx]); } =20 s->regs[addr] =3D data; @@ -184,6 +204,8 @@ static void aspeed_intc_status_handler(AspeedINTCState = *s, hwaddr offset, const char *name =3D object_get_typename(OBJECT(s)); uint32_t addr =3D offset >> 2; uint32_t irq; + int inpin_idx; + int outpin_idx; =20 if (!data) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid data 0\n", __func__); @@ -191,11 +213,13 @@ static void aspeed_intc_status_handler(AspeedINTCStat= e *s, hwaddr offset, } =20 irq =3D (offset & 0x0f00) >> 8; + inpin_idx =3D irq; + outpin_idx =3D irq; =20 - if (irq >=3D aic->num_inpins) { + if (inpin_idx >=3D aic->num_inpins) { qemu_log_mask(LOG_GUEST_ERROR, "%s: Invalid input pin index: %d\n", - __func__, irq); + __func__, inpin_idx); return; } =20 @@ -214,21 +238,22 @@ static void aspeed_intc_status_handler(AspeedINTCStat= e *s, hwaddr offset, =20 /* All source ISR execution are done */ if (!s->regs[addr]) { - trace_aspeed_intc_all_isr_done(name, irq); - if (s->pending[irq]) { + trace_aspeed_intc_all_isr_done(name, inpin_idx); + if (s->pending[inpin_idx]) { /* * handle pending source interrupt * notify firmware which source interrupt are pending * by setting status register */ - s->regs[addr] =3D s->pending[irq]; - s->pending[irq] =3D 0; - trace_aspeed_intc_trigger_irq(name, irq, s->regs[addr]); - aspeed_intc_update(s, irq, 1); + s->regs[addr] =3D s->pending[inpin_idx]; + s->pending[inpin_idx] =3D 0; + trace_aspeed_intc_trigger_irq(name, inpin_idx, outpin_idx, + s->regs[addr]); + aspeed_intc_update(s, inpin_idx, outpin_idx, 1); } else { /* clear irq */ - trace_aspeed_intc_clear_irq(name, irq, 0); - aspeed_intc_update(s, irq, 0); + trace_aspeed_intc_clear_irq(name, inpin_idx, outpin_idx, 0); + aspeed_intc_update(s, inpin_idx, outpin_idx, 0); } } } diff --git a/hw/intc/trace-events b/hw/intc/trace-events index e9ca34755e..e97eea820b 100644 --- a/hw/intc/trace-events +++ b/hw/intc/trace-events @@ -82,12 +82,12 @@ aspeed_vic_write(uint64_t offset, unsigned size, uint32= _t data) "To 0x%" PRIx64 # aspeed_intc.c aspeed_intc_read(const char *s, uint64_t offset, unsigned size, uint32_t v= alue) "%s: From 0x%" PRIx64 " of size %u: 0x%" PRIx32 aspeed_intc_write(const char *s, uint64_t offset, unsigned size, uint32_t = data) "%s: To 0x%" PRIx64 " of size %u: 0x%" PRIx32 -aspeed_intc_set_irq(const char *s, int irq, int level) "%s: Set IRQ %d: %d" -aspeed_intc_clear_irq(const char *s, int irq, int level) "%s: Clear IRQ %d= : %d" -aspeed_intc_update_irq(const char *s, int irq, int level) "%s: Update IRQ:= %d: %d" -aspeed_intc_pending_irq(const char *s, int irq, uint32_t value) "%s: Pendi= ng IRQ: %d: 0x%x" -aspeed_intc_trigger_irq(const char *s, int irq, uint32_t value) "%s: Trigg= er IRQ: %d: 0x%x" -aspeed_intc_all_isr_done(const char *s, int irq) "%s: All source ISR execu= tion are done: %d" +aspeed_intc_set_irq(const char *s, int inpin_idx, int level) "%s: Set IRQ = %d: %d" +aspeed_intc_clear_irq(const char *s, int inpin_idx, int outpin_idx, int le= vel) "%s: Clear IRQ %d-%d: %d" +aspeed_intc_update_irq(const char *s, int inpin_idx, int outpin_idx, int l= evel) "%s: Update IRQ: %d-%d: %d" +aspeed_intc_pending_irq(const char *s, int inpin_idx, uint32_t value) "%s:= Pending IRQ: %d: 0x%x" +aspeed_intc_trigger_irq(const char *s, int inpin_idx, int outpin_idx, uint= 32_t value) "%s: Trigger IRQ: %d-%d: 0x%x" +aspeed_intc_all_isr_done(const char *s, int inpin_idx) "%s: All source ISR= execution are done: %d" aspeed_intc_enable(const char *s, uint32_t value) "%s: Enable: 0x%x" aspeed_intc_select(const char *s, uint32_t value) "%s: Select: 0x%x" aspeed_intc_mask(const char *s, uint32_t change, uint32_t value) "%s: Mask= : 0x%x: 0x%x" --=20 2.34.1