Hi Gred,

> -----Original Message-----
> From: Gerd Hoffmann <kra...@redhat.com>
> Sent: Monday, August 31, 2020 12:33 PM
> To: Sai Pavan Boddu <saip...@xilinx.com>
> Cc: Peter Maydell <peter.mayd...@linaro.org>; Markus Armbruster
> <arm...@redhat.com>; 'Marc-André Lureau'
> <marcandre.lur...@redhat.com>; Paolo Bonzini <pbonz...@redhat.com>;
> Edgar Iglesias <edg...@xilinx.com>; Francisco Eduardo Iglesias
> <figle...@xilinx.com>; qemu-devel@nongnu.org; Alistair Francis
> <alistair.fran...@wdc.com>; Eduardo Habkost <ehabk...@redhat.com>;
> Ying Fang <fangyi...@huawei.com>; 'Philippe Mathieu-Daudé'
> <phi...@redhat.com>; Vikram Garhwal <f...@xilinx.com>; Paul
> Zimmerman <pauld...@gmail.com>
> Subject: Re: [PATCH v4 5/7] misc: Add versal-usb2-regs module
> 
> On Sat, Aug 29, 2020 at 12:49:38AM +0530, Sai Pavan Boddu wrote:
> > This is a dummy module to emulate control registers of versal usb2
> > controller.
> 
> --verbose please.  xhci supports all usb speeds, including usb2.
[Sai Pavan Boddu] This module is has soc specific control/status register to 
control the phy-reset, adjust frame length time  and coherency properties for 
the transactions.

> 
> So why this hardware is here, what does it on a real device and what is the
> emulation doing?  "dummy" sounds like it does just enough to make the
> guest driver happy, probably pretending nothing is connected?
Yes, this is added to make guest happy.

I will the commit message with more info

Regards,
Sai Pavan
> 
> thanks,
>   Gerd


Reply via email to