From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52946) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VIz1p-0002Wd-W9 for qemu-devel@nongnu.org; Mon, 09 Sep 2013 06:49:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VIz1k-0006oS-1M for qemu-devel@nongnu.org; Mon, 09 Sep 2013 06:49:09 -0400 Message-ID: <522DA790.6050105@redhat.com> Date: Mon, 09 Sep 2013 12:48:48 +0200 From: Hans de Goede MIME-Version: 1.0 References: <1378720637-27473-1-git-send-email-kraxel@redhat.com> In-Reply-To: <1378720637-27473-1-git-send-email-kraxel@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] ehci: save device pointer in EHCIState List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gerd Hoffmann Cc: qemu-stable@nongnu.org, "qemu-devel@nongnu.org" Hi, On 09/09/2013 11:57 AM, Gerd Hoffmann wrote: > We'll need a pointer to the actual pci/sysbus device, > stick a pointer to it into the EHCIState struct. > > https://bugzilla.redhat.com/show_bug.cgi?id=1005495 Looks good, note you've forgotten to add qemu-stable, I've done so now. Acked-by: Hans de Goede Regards, Hans > > Signed-off-by: Gerd Hoffmann > --- > hw/usb/hcd-ehci.c | 7 +++---- > hw/usb/hcd-ehci.h | 1 + > 2 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c > index 137e200..22bdbf4 100644 > --- a/hw/usb/hcd-ehci.c > +++ b/hw/usb/hcd-ehci.c > @@ -1241,13 +1241,11 @@ static int ehci_init_transfer(EHCIPacket *p) > { > uint32_t cpage, offset, bytes, plen; > dma_addr_t page; > - USBBus *bus = &p->queue->ehci->bus; > - BusState *qbus = BUS(bus); > > cpage = get_field(p->qtd.token, QTD_TOKEN_CPAGE); > bytes = get_field(p->qtd.token, QTD_TOKEN_TBYTES); > offset = p->qtd.bufptr[0] & ~QTD_BUFPTR_MASK; > - qemu_sglist_init(&p->sgl, qbus->parent, 5, p->queue->ehci->as); > + qemu_sglist_init(&p->sgl, p->queue->ehci->device, 5, p->queue->ehci->as); > > while (bytes > 0) { > if (cpage > 4) { > @@ -1486,7 +1484,7 @@ static int ehci_process_itd(EHCIState *ehci, > return -1; > } > > - qemu_sglist_init(&ehci->isgl, DEVICE(ehci), 2, ehci->as); > + qemu_sglist_init(&ehci->isgl, ehci->device, 2, ehci->as); > if (off + len > 4096) { > /* transfer crosses page border */ > uint32_t len2 = off + len - 4096; > @@ -2529,6 +2527,7 @@ void usb_ehci_realize(EHCIState *s, DeviceState *dev, Error **errp) > > s->frame_timer = timer_new_ns(QEMU_CLOCK_VIRTUAL, ehci_frame_timer, s); > s->async_bh = qemu_bh_new(ehci_frame_timer, s); > + s->device = dev; > > qemu_register_reset(ehci_reset, s); > qemu_add_vm_change_state_handler(usb_ehci_vm_state_change, s); > diff --git a/hw/usb/hcd-ehci.h b/hw/usb/hcd-ehci.h > index 15a28e8..065c9fa 100644 > --- a/hw/usb/hcd-ehci.h > +++ b/hw/usb/hcd-ehci.h > @@ -255,6 +255,7 @@ typedef QTAILQ_HEAD(EHCIQueueHead, EHCIQueue) EHCIQueueHead; > > struct EHCIState { > USBBus bus; > + DeviceState *device; > qemu_irq irq; > MemoryRegion mem; > AddressSpace *as; >