Re: [PATCH v11 10/11] hw/fsi: Added FSI documentation

[Ninad Palsule]

Hello Cedric,

> > +QEMU files related to FSI interface:
> > + - ``hw/fsi/aspeed-apb2opb.c``
> > + - ``include/hw/fsi/aspeed-apb2opb.h``
> > + - ``hw/fsi/opb.c``
> > + - ``include/hw/fsi/opb.h``
> > + - ``hw/fsi/fsi.c``
> > + - ``include/hw/fsi/fsi.h``
> > + - ``hw/fsi/fsi-master.c``
> > + - ``include/hw/fsi/fsi-master.h``
> > + - ``hw/fsi/fsi-slave.c``
> > + - ``include/hw/fsi/fsi-slave.h``
> > + - ``hw/fsi/cfam.c``
> > + - ``include/hw/fsi/cfam.h``
> > + - ``hw/fsi/engine-scratchpad.c``
> > + - ``include/hw/fsi/engine-scratchpad.h``
> > + - ``include/hw/fsi/lbus.h``
>
> Please remove the file list above.
Removed files and listed only directories so that people can easily find 
the code.
> > +The following commands start the rainier machine with built-in FSI 
> > model.
>
> rainier -> ``rainier-bmc``
Replaced it.
> > +There are no model specific arguments.
> I would also add a reference to the docs/system/arm/aspeed.rst page.

Added the reference.

Thanks for the review.

Regards,

Ninad