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