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

Reply via email to