[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [RFC PATCH 11/12] ppc4xx: Export ECB and PLB emulation
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [RFC PATCH 11/12] ppc4xx: Export ECB and PLB emulation |
Date: |
Fri, 18 Aug 2017 16:11:49 +1000 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
On Mon, Aug 14, 2017 at 01:06:42PM +0200, BALATON Zoltan wrote:
> 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.
I think move them. Probably to their own file - I think smaller files
are usually going to be more readable than one big file with heaps of
4xx devices.
> > > 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.
Ok, so say that in the commit message.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
[Qemu-ppc] [RFC PATCH 04/12] ehci: Add ppc4xx-ehci for the USB 2.0 controller in embedded PPC SoCs, BALATON Zoltan, 2017/08/13
[Qemu-ppc] [RFC PATCH 02/12] ppc4xx: Make MAL emulation more generic, BALATON Zoltan, 2017/08/13