Hi Gred,

> -----Original Message-----
> From: Gerd Hoffmann <[email protected]>
> Sent: Monday, August 31, 2020 12:33 PM
> To: Sai Pavan Boddu <[email protected]>
> Cc: Peter Maydell <[email protected]>; Markus Armbruster
> <[email protected]>; 'Marc-André Lureau'
> <[email protected]>; Paolo Bonzini <[email protected]>;
> Edgar Iglesias <[email protected]>; Francisco Eduardo Iglesias
> <[email protected]>; [email protected]; Alistair Francis
> <[email protected]>; Eduardo Habkost <[email protected]>;
> Ying Fang <[email protected]>; 'Philippe Mathieu-Daudé'
> <[email protected]>; Vikram Garhwal <[email protected]>; Paul
> Zimmerman <[email protected]>
> 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