On Mon, 14 Aug 2017, David Gibson wrote:
On Sun, Aug 13, 2017 at 07:04:38PM +0200, BALATON Zoltan wrote:
These devices are found in some other ppc4xx cores too.
Thanks for the quick review, hope more will follow for other patches
too maybe also from others.
Elsewhere in the series you moved things that were used elsewhere out
of 405_uc, why not do that here as well?
I've moved those because those devices needed to be modified extensively
while these two in this patch are used basically unchanged (they may not
even be completely correct for 460EX but seem to work well enough) so I've
left them here. I could add this explanation to the commit message or
maybe move them to ppc4xx_devs to make it clearer that they are not only
used by 405 emulation.
Also add some more PLB registers for 460EX.
Separate patch for that please, it's logically unrelated. Plus the
commit message should say what these registers are, where they appear,
why do we need them?
OK. I don't really know what these registers are. I guess we need them
because U-Boot or other guests poke them but they are only added to avoid
the crash, not really emulated (just read zero, ignore write) for now.
Regards,
BALATON Zoltan
