>> +static void ppc_powernv_reset(void) >> +{ >> + MachineState *machine = MACHINE(qdev_get_machine()); >> + PnvMachineState *pnv = POWERNV_MACHINE(machine); >> + void *fdt; >> + >> + pnv->fdt_addr = FDT_ADDR; > > Not sure there's any point to pnv->fdt_addr, since it will always have > the value FDT_ADDR. Only worth changing if you're doing a respin for > other reasons though.
Yes, you are probably right. I will take a look. I have a new version with some basic core and xscom support for P9, booting up to the interrupt setting in skiboot. So I was already planning for v3. Thanks, C.