[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 07/15] pseries: Cleanup duplications of ics_valid_ir
From: |
David Gibson |
Subject: |
[Qemu-devel] [PATCH 07/15] pseries: Cleanup duplications of ics_valid_irq() code |
Date: |
Thu, 18 Oct 2012 16:50:29 +1100 |
A couple of places in xics.c open-coded the same logic as is already
implemented in ics_valid_irq(). This patch fixes the code duplication.
Signed-off-by: David Gibson <address@hidden>
---
hw/xics.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/hw/xics.c b/hw/xics.c
index db01fe3..6b08430 100644
--- a/hw/xics.c
+++ b/hw/xics.c
@@ -326,8 +326,7 @@ static void ics_eoi(struct ics_state *ics, int nr)
qemu_irq xics_get_qirq(struct icp_state *icp, int irq)
{
- if ((irq < icp->ics->offset)
- || (irq >= (icp->ics->offset + icp->ics->nr_irqs))) {
+ if (!ics_valid_irq(icp->ics, irq)) {
return NULL;
}
@@ -336,8 +335,7 @@ qemu_irq xics_get_qirq(struct icp_state *icp, int irq)
void xics_set_irq_type(struct icp_state *icp, int irq, bool lsi)
{
- assert((irq >= icp->ics->offset)
- && (irq < (icp->ics->offset + icp->ics->nr_irqs)));
+ assert(ics_valid_irq(icp->ics, irq));
icp->ics->irqs[irq - icp->ics->offset].lsi = lsi;
}
--
1.7.10.4
- [Qemu-devel] [0/15] pseries patch queue, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 05/15] pseries: Clean up inconsistent variable name in xics.c, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 01/15] pseries: Don't allow duplicate registration of hcalls or RTAS calls, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 09/15] pseries: Return the token when we register an RTAS call, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 10/15] pseries: Allow RTAS tokens without a qemu handler, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 06/15] pseries: Use #define for XICS base irq number, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 02/15] target-ppc: Rework storage of VPA registration state, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 07/15] pseries: Cleanup duplications of ics_valid_irq() code,
David Gibson <=
- [Qemu-devel] [PATCH 12/15] pseries: Split xics irq configuration from state information, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 11/15] pseries: Add tracepoints to the XICS interrupt controller, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 04/15] target-ppc: Extend FPU state for newer POWER CPUs, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 08/15] pseries: Move XICS initialization before cpu initialization, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 13/15] target-pcc: Convert ppcemb_tlb_t to use fixed 64-bit RPN, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 03/15] pseries: Implement qemu initiated shutdowns using EPOW events, David Gibson, 2012/10/18
- [Qemu-devel] [PATCH 14/15] pseries: Implement PAPR NVRAM, David Gibson, 2012/10/18