>> +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.