[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v3 3/8] prep_pci: Update I/O to MemoryRegion ops
From: |
Avi Kivity |
Subject: |
Re: [Qemu-ppc] [PATCH v3 3/8] prep_pci: Update I/O to MemoryRegion ops |
Date: |
Sun, 15 Jan 2012 11:19:22 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111222 Thunderbird/9.0 |
On 01/13/2012 05:09 AM, Andreas Färber wrote:
> Convert to new-style read/write callbacks.
>
>
> -static uint32_t PPC_PCIIO_readl (void *opaque, target_phys_addr_t addr)
> +static uint64_t ppc_pci_io_read(void *opaque, target_phys_addr_t addr,
> + unsigned int size)
> {
> PREPPCIState *s = opaque;
> - uint32_t val;
> - val = pci_data_read(s->bus, PPC_PCIIO_config(addr), 4);
> - return val;
> + switch (size) {
> + case 1:
> + case 2:
> + case 4:
> + return pci_data_read(s->bus, PPC_PCIIO_config(addr), size);
> + default:
> + abort();
> + }
> }
Huh? just call pci_data_read() unconditionally.
--
error compiling committee.c: too many arguments to function
- Re: [Qemu-ppc] [PATCH v2 2/3] prep: Add Raven PCI host SysBus device, (continued)
- [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Andreas Färber, 2012/01/12
- Re: [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Jan Kiszka, 2012/01/13
- Re: [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Andreas Färber, 2012/01/13
- Re: [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Jan Kiszka, 2012/01/13
- Re: [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Alexander Graf, 2012/01/13
- Re: [Qemu-ppc] [PATCH v3 8/8] prep: Use i82378 PCI->ISA bridge for 'prep' machine, Jan Kiszka, 2012/01/13
[Qemu-ppc] [PATCH v3 4/8] prep: qdev'ify Raven host bridge (SysBus), Andreas Färber, 2012/01/12
[Qemu-ppc] [PATCH v3 1/8] prep: qdev'ify Raven host bridge (PCIDevice), Andreas Färber, 2012/01/12