[PATCH v3 0/3] Make hcd-xhci independent of pci hooks

Sai Pavan Boddu posted 3 patches 3 years, 8 months ago
Test checkpatch passed
Test docker-mingw@fedora passed
Test FreeBSD passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1594140271-16462-1-git-send-email-sai.pavan.boddu@xilinx.com
Maintainers: Gerd Hoffmann <kraxel@redhat.com>
There is a newer version of this series
hw/usb/Kconfig        |   6 ++
hw/usb/Makefile.objs  |   1 +
hw/usb/hcd-xhci-nec.c |  18 ++--
hw/usb/hcd-xhci-pci.c | 242 ++++++++++++++++++++++++++++++++++++++++++++++
hw/usb/hcd-xhci-pci.h |  45 +++++++++
hw/usb/hcd-xhci.c     | 259 ++++++++++----------------------------------------
hw/usb/hcd-xhci.h     |  20 ++--
7 files changed, 367 insertions(+), 224 deletions(-)
create mode 100644 hw/usb/hcd-xhci-pci.c
create mode 100644 hw/usb/hcd-xhci-pci.h
[PATCH v3 0/3] Make hcd-xhci independent of pci hooks
Posted by Sai Pavan Boddu 3 years, 8 months ago
Hi

This patch series attempts to make 'hcd-xhci' an independent model so
it can be used by both pci and system-bus interface.

Changes for V2:
    Make XHCIState non-qom
    Use container_of functions for retriving pci device instance
    Initialize the AddressSpace pointer in PATCH 1/3 itself
Changes for V3:
    Convert XHCIState to TYPE_DEVICE and register as a child of XHCIPciState.


Sai Pavan Boddu (3):
  usb/hcd-xhci: Make dma read/writes hooks pci free
  usb/hcd-xhci: Move qemu-xhci device to hcd-xhci-pci.c
  usb/hcd-xhci: Split pci wrapper for xhci base model

 hw/usb/Kconfig        |   6 ++
 hw/usb/Makefile.objs  |   1 +
 hw/usb/hcd-xhci-nec.c |  18 ++--
 hw/usb/hcd-xhci-pci.c | 242 ++++++++++++++++++++++++++++++++++++++++++++++
 hw/usb/hcd-xhci-pci.h |  45 +++++++++
 hw/usb/hcd-xhci.c     | 259 ++++++++++----------------------------------------
 hw/usb/hcd-xhci.h     |  20 ++--
 7 files changed, 367 insertions(+), 224 deletions(-)
 create mode 100644 hw/usb/hcd-xhci-pci.c
 create mode 100644 hw/usb/hcd-xhci-pci.h

-- 
2.7.4


RE: [PATCH v3 0/3] Make hcd-xhci independent of pci hooks
Posted by Sai Pavan Boddu 3 years, 8 months ago
Hi Gerd,

Any review on V3. So that I can work on.

Regards,
Sai Pavan

> -----Original Message-----
> From: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com>
> Sent: Tuesday, July 7, 2020 10:14 PM
> To: Gerd Hoffmann <kraxel@redhat.com>; Peter Maydell
> <peter.maydell@linaro.org>; Markus Armbruster <armbru@redhat.com>
> Cc: Thomas Huth <thuth@redhat.com>; Eduardo Habkost
> <ehabkost@redhat.com>; qemu-devel@nongnu.org; Alistair Francis
> <alistair.francis@wdc.com>; 'Marc-André Lureau'
> <marcandre.lureau@redhat.com>; Ying Fang <fangying1@huawei.com>;
> Paolo Bonzini <pbonzini@redhat.com>; 'Philippe Mathieu-Daudé'
> <philmd@redhat.com>; Vikram Garhwal <fnuv@xilinx.com>; Paul
> Zimmerman <pauldzim@gmail.com>; Sai Pavan Boddu <saipava@xilinx.com>
> Subject: [PATCH v3 0/3] Make hcd-xhci independent of pci hooks
> 
> Hi
> 
> This patch series attempts to make 'hcd-xhci' an independent model so it can
> be used by both pci and system-bus interface.
> 
> Changes for V2:
>     Make XHCIState non-qom
>     Use container_of functions for retriving pci device instance
>     Initialize the AddressSpace pointer in PATCH 1/3 itself Changes for V3:
>     Convert XHCIState to TYPE_DEVICE and register as a child of XHCIPciState.
> 
> 
> Sai Pavan Boddu (3):
>   usb/hcd-xhci: Make dma read/writes hooks pci free
>   usb/hcd-xhci: Move qemu-xhci device to hcd-xhci-pci.c
>   usb/hcd-xhci: Split pci wrapper for xhci base model
> 
>  hw/usb/Kconfig        |   6 ++
>  hw/usb/Makefile.objs  |   1 +
>  hw/usb/hcd-xhci-nec.c |  18 ++--
>  hw/usb/hcd-xhci-pci.c | 242
> ++++++++++++++++++++++++++++++++++++++++++++++
>  hw/usb/hcd-xhci-pci.h |  45 +++++++++
>  hw/usb/hcd-xhci.c     | 259 ++++++++++----------------------------------------
>  hw/usb/hcd-xhci.h     |  20 ++--
>  7 files changed, 367 insertions(+), 224 deletions(-)  create mode 100644
> hw/usb/hcd-xhci-pci.c  create mode 100644 hw/usb/hcd-xhci-pci.h
> 
> --
> 2.7.4


Re: [PATCH v3 0/3] Make hcd-xhci independent of pci hooks
Posted by Gerd Hoffmann 3 years, 8 months ago
On Tue, Jul 14, 2020 at 01:59:12PM +0000, Sai Pavan Boddu wrote:
> Hi Gerd,
> 
> Any review on V3. So that I can work on.

Busy due to freeze, sorry for the delay.
Looks good to me overall.  Not tested yet.

Wasn't the plan to include the sysbus bits too, so it's easier to see
how code code separation works out on both pci and sysbus?

take care,
  Gerd


RE: [PATCH v3 0/3] Make hcd-xhci independent of pci hooks
Posted by Sai Pavan Boddu 3 years, 8 months ago
Hi Gred,

> -----Original Message-----
> From: Gerd Hoffmann <kraxel@redhat.com>
> Sent: Wednesday, July 15, 2020 4:21 PM
> To: Sai Pavan Boddu <saipava@xilinx.com>
> Cc: Peter Maydell <peter.maydell@linaro.org>; Markus Armbruster
> <armbru@redhat.com>; Thomas Huth <thuth@redhat.com>; Eduardo
> Habkost <ehabkost@redhat.com>; qemu-devel@nongnu.org; Alistair Francis
> <alistair.francis@wdc.com>; 'Marc-André Lureau'
> <marcandre.lureau@redhat.com>; Ying Fang <fangying1@huawei.com>;
> Paolo Bonzini <pbonzini@redhat.com>; 'Philippe Mathieu-Daudé'
> <philmd@redhat.com>; Vikram Garhwal <fnuv@xilinx.com>; Paul
> Zimmerman <pauldzim@gmail.com>
> Subject: Re: [PATCH v3 0/3] Make hcd-xhci independent of pci hooks
> 
> On Tue, Jul 14, 2020 at 01:59:12PM +0000, Sai Pavan Boddu wrote:
> > Hi Gerd,
> >
> > Any review on V3. So that I can work on.
> 
> Busy due to freeze, sorry for the delay.
> Looks good to me overall.  Not tested yet.
> 
> Wasn't the plan to include the sysbus bits too, so it's easier to see how code
> code separation works out on both pci and sysbus?
[Sai Pavan Boddu] Ok, I will include that in V4.

Thanks,
Sai Pavan
> 
> take care,
>   Gerd