Hi, Peter,

On 06/29/2021 07:52 PM, Peter Maydell wrote:
> On Mon, 28 Jun 2021 at 13:04, Song Gao <gaos...@loongson.cn> wrote:
>>
>> This patch give an introduction to the LoongArch target.
>>
>> Signed-off-by: Song Gao <gaos...@loongson.cn>
>> ---
> 
>> +There are several code directories
>> +
>> +    1 qemu/target/loongarch
>> +        This is the instruction emulation directory
>> +    2 qemu/linux-user/loongarch64
>> +        Linux-user simulation
>> +    3 qemu/hw/loongarch
>> +        System simulation
>> +    4 qemu/tests/tcg/loongarch
>> +        Linux-user test
> 
> This isn't particularly specific to loongarch -- it's how
> all QEMU targets are structured. So I'm not sure you need
> to document it in an loongarch-specific README.
> 
> thanks
> -- PMM
> 

This part is redundant, I'll change it as Alex suggests.

> 
> This part, perhaps slightly expanded is probably better aimed at
> docs/system/target-loongarch.rst so it can be included in the generated
> manual. Bonus points for something along the lines of
> docs/system/arm/versatile.rst where you give a quick breakdown of the
> kernel config and an example command line.
> 

thanks


Reply via email to