[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v14 2/4] usb: Add DWC3 model
From: |
Peter Maydell |
Subject: |
Re: [PATCH v14 2/4] usb: Add DWC3 model |
Date: |
Tue, 1 Dec 2020 11:25:46 +0000 |
On Tue, 1 Dec 2020 at 08:34, Sai Pavan Boddu <sai.pavan.boddu@xilinx.com> wrote:
>
> From: Vikram Garhwal <fnu.vikram@xilinx.com>
>
> This patch adds skeleton model of dwc3 usb controller attached to
> xhci-sysbus device. It defines global register space of DWC3 controller,
> global registers control the AXI/AHB interfaces properties, external FIFO
> support and event count support. All of which are unimplemented at
> present,we are only supporting core reset and read of ID register.
>
> Signed-off-by: Vikram Garhwal <fnu.vikram@xilinx.com>
> Signed-off-by: Sai Pavan Boddu <sai.pavan.boddu@xilinx.com>
> Reviewed-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
> ---
> +typedef struct USBDWC3 {
> + SysBusDevice parent_obj;
> + MemoryRegion iomem;
> + MemoryRegion fifos;
> + XHCISysbusState sysbus_xhci;
> +
> + uint32_t regs[USB_DWC3_R_MAX];
> + RegisterInfo regs_info[USB_DWC3_R_MAX];
This device has state but is missing the VMState struct
support for migration. Otherwise it looks OK.
thanks
-- PMM