[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [RFC PATCH 03/12] ohci: Allow sysbus version to be used a
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [RFC PATCH 03/12] ohci: Allow sysbus version to be used as a companion |
Date: |
Tue, 15 Aug 2017 14:13:12 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Sun, Aug 13, 2017 at 07:04:38PM +0200, BALATON Zoltan wrote:
Wants a commit message - why do you need this functionality.
Patch itself looks fine to me, but you should probably CC Gerd
<address@hidden> for review.
> Signed-off-by: BALATON Zoltan <address@hidden>
> ---
> hw/usb/hcd-ohci.c | 15 ++++++++++++---
> 1 file changed, 12 insertions(+), 3 deletions(-)
>
> diff --git a/hw/usb/hcd-ohci.c b/hw/usb/hcd-ohci.c
> index 267982e..17beedd 100644
> --- a/hw/usb/hcd-ohci.c
> +++ b/hw/usb/hcd-ohci.c
> @@ -1999,7 +1999,9 @@ typedef struct {
> /*< public >*/
>
> OHCIState ohci;
> + char *masterbus;
> uint32_t num_ports;
> + uint32_t firstport;
> dma_addr_t dma_offset;
> } OHCISysBusState;
>
> @@ -2007,10 +2009,15 @@ static void ohci_realize_pxa(DeviceState *dev, Error
> **errp)
> {
> OHCISysBusState *s = SYSBUS_OHCI(dev);
> SysBusDevice *sbd = SYS_BUS_DEVICE(dev);
> + Error *err = NULL;
>
> - /* Cannot fail as we pass NULL for masterbus */
> - usb_ohci_init(&s->ohci, dev, s->num_ports, s->dma_offset, NULL, 0,
> - &address_space_memory, &error_abort);
> + usb_ohci_init(&s->ohci, dev, s->num_ports, s->dma_offset,
> + s->masterbus, s->firstport,
> + &address_space_memory, &err);
> + if (err) {
> + error_propagate(errp, err);
> + return;
> + }
> sysbus_init_irq(sbd, &s->ohci.irq);
> sysbus_init_mmio(sbd, &s->ohci.mem);
> }
> @@ -2142,7 +2149,9 @@ static const TypeInfo ohci_pci_info = {
> };
>
> static Property ohci_sysbus_properties[] = {
> + DEFINE_PROP_STRING("masterbus", OHCISysBusState, masterbus),
> DEFINE_PROP_UINT32("num-ports", OHCISysBusState, num_ports, 3),
> + DEFINE_PROP_UINT32("firstport", OHCISysBusState, firstport, 0),
> DEFINE_PROP_DMAADDR("dma-offset", OHCISysBusState, dma_offset, 0),
> DEFINE_PROP_END_OF_LIST(),
> };
--
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
[Qemu-ppc] [RFC PATCH 04/12] ehci: Add ppc4xx-ehci for the USB 2.0 controller in embedded PPC SoCs, BALATON Zoltan, 2017/08/13
[Qemu-ppc] [RFC PATCH 02/12] ppc4xx: Make MAL emulation more generic, BALATON Zoltan, 2017/08/13
[Qemu-ppc] [RFC PATCH 10/12] ppc: Add 460EX embedded CPU, BALATON Zoltan, 2017/08/13