[OpenBIOS] Patch:Add 7447a support.
Segher Boessenkool
segher at kernel.crashing.org
Sun Jan 21 13:36:44 CET 2018
On Sun, Jan 21, 2018 at 09:29:18AM +0000, Mark Cave-Ayland wrote:
> On 20/01/18 15:51, Segher Boessenkool wrote:
> >On Sat, Jan 20, 2018 at 09:38:21AM -0500, Jd Lyons via OpenBIOS wrote:
> >>--- a/arch/ppc/qemu/init.c
> >>+++ b/arch/ppc/qemu/init.c
> >>@@ -550,6 +550,18 @@ static const struct cpudef ppc_defs[] = {
> >> .tlb_size = 0x80,
> >> .initfn = cpu_g4_init,
> >> },
> >>+ .iu_version = 0x80030000,
> >>+ .name = "PowerPC,G4",
> >>+ .icache_size = 0x8000,
> >>+ .dcache_size = 0x8000,
> >>+ .icache_sets = 0x80,
> >>+ .dcache_sets = 0x80,
> >>+ .icache_block_size = 0x20,
> >>+ .dcache_block_size = 0x20,
> >>+ .tlb_sets = 0x40,
> >>+ .tlb_size = 0x80,
> >>+ .initfn = cpu_g4_init,
> >>+},
> >
> >These numbers are all correct (I checked against the manual).
> >
> >Should there be something for L2 as well?
>
> Good question - I suspect you're the best person to know here ;) Could
> not having the L2 properties cause strange behaviour, or will the guest
> OS carry on fine without them present? (I presume this isn't a NACK?)
This isn't a NAK; the other CPUs are like that as well in the openbios
code I guess, and some of those have L2.
L2 is described in a node _under_ the "cpu" node in the device tree.
Linux does not care if it is absent; I don't know about the other OSes
though. If everything works it is not a pressing concern ;-)
Segher
More information about the OpenBIOS
mailing list