[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH] ppc: Make uninorth interrupt swizzling identical
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH] ppc: Make uninorth interrupt swizzling identical to Grackle |
Date: |
Tue, 22 Nov 2016 10:05:29 +1100 |
User-agent: |
Mutt/1.7.1 (2016-10-04) |
On Mon, Nov 21, 2016 at 12:12:31AM +1000, BALATON Zoltan wrote:
> From: Benjamin Herrenschmidt <address@hidden>
>
> It's currently broken as it uses an incorrect shift, it tries
> to use the slot number but uses the top bits of the bus number
> instead.
>
> Note: Neither implementation matches what OpenBIOS ends up putting
> in the device-tree either, which will have to be fixed separately.
>
> This is not quite correct for modelling a real Mac since Apple
> tend to tie all 4 interrupt lines of a slot together and have
> separate interrupts for every slot and every motherboard devices
> going straight to the PIC but we'll sort that out later.
>
> Signed-off-by: Benjamin Herrenschmidt <address@hidden>
> ---
> hw/pci-host/uninorth.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> This needs a corresponding fix in OpenBIOS but this has to be
> committed first for that. As this is already broken making this change
> should not make things worse as they are now. Could we get this in now
> as a bugfix commit?
Applied to ppc-for-2.8.
>
> diff --git a/hw/pci-host/uninorth.c b/hw/pci-host/uninorth.c
> index 7aac4d6..df342ac 100644
> --- a/hw/pci-host/uninorth.c
> +++ b/hw/pci-host/uninorth.c
> @@ -62,9 +62,7 @@ typedef struct UNINState {
>
> static int pci_unin_map_irq(PCIDevice *pci_dev, int irq_num)
> {
> - int devfn = pci_dev->devfn & 0x00FFFFFF;
> -
> - return (((devfn >> 11) & 0x1F) + irq_num) & 3;
> + return (irq_num + (pci_dev->devfn >> 3)) & 3;
> }
>
> static void pci_unin_set_irq(void *opaque, int irq_num, int level)
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature