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

[Ninad Palsule <ninad@linux.ibm.com>]

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