Arnaud Patard 写道:
Thanks for your reviewing this patch and your advice. I will split them
into smaller pieces.
yanhua <email@example.com> writes:
Mini fuloong, yeeloong are all Loongson2F based systems. Loongson2F have
builtin DDR2 and PCIX controller. The PCIX controller have a similar
programming interface with FPGA northbridge used in Loongson2E.
First, please read Documentation/SubmittingPatches first. There's no
signed-off-by and this patch is too big. So big patches are making
review a nightmare, please split it into smaller pieces :(
This is just for historical reasons. Before, we have loongson2e machines
named as fulong(from loogson2f, changeed to fuloong) merged into main
Also, I'd like to see a different directories layout. You're doing :
so I keep that to make 2e machines unchanged.
Maybe it's better to break the historical doing to get a more clear
This is quite annoying because:
- I'll prefer seeing loongson instead of Lemote. I've some ST machines
here they do share a lot of code with the 2e/2f so I'd like to avoid
- There's some code very similar between 2e, 2f-yeelong and
2f-fuloong and other machines. Why not putting them in a common
directory instead of duplicating again some code ?
To sum up, imho, it would be better to have something like :
2f/ (or something similar)
OK, I will rearange the patch. Thanks again.