[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v2] ppc/pnv: change powernv_ prefix to pnv_ for ov
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH v2] ppc/pnv: change powernv_ prefix to pnv_ for overall naming consistency |
Date: |
Mon, 18 Dec 2017 15:24:02 +1100 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Dec 15, 2017 at 02:56:01PM +0100, Cédric Le Goater wrote:
> The 'pnv' prefix is now used for all and the routines populating the
> device tree start with 'pnv_dt'. The handler of the PnvXScomInterface
> is also renamed to 'dt_xscom' which should reflect that it is
> populating the device tree under the 'xscom@' node of the chip.
>
> Signed-off-by: Cédric Le Goater <address@hidden>
Applied to ppc-for-2.12.
> ---
>
> Changes since v2:
>
> - fixed code alignment
> - renamed device tree routines also.
>
> hw/ppc/pnv.c | 94
> ++++++++++++++++++++++------------------------
> hw/ppc/pnv_bmc.c | 2 +-
> hw/ppc/pnv_core.c | 8 ++--
> hw/ppc/pnv_lpc.c | 6 +--
> hw/ppc/pnv_psi.c | 4 +-
> hw/ppc/pnv_xscom.c | 10 ++---
> include/hw/ppc/pnv.h | 10 ++---
> include/hw/ppc/pnv_xscom.h | 4 +-
> 8 files changed, 67 insertions(+), 71 deletions(-)
>
> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c
> index c35c439d816b..6ed72ad335d1 100644
> --- a/hw/ppc/pnv.c
> +++ b/hw/ppc/pnv.c
> @@ -77,8 +77,7 @@ static const char *pnv_chip_core_typename(const PnvChip *o)
> * that has a different "affinity". In practice, it means one range
> * per chip.
> */
> -static void powernv_populate_memory_node(void *fdt, int chip_id, hwaddr
> start,
> - hwaddr size)
> +static void pnv_dt_memory(void *fdt, int chip_id, hwaddr start, hwaddr size)
> {
> char *mem_name;
> uint64_t mem_reg_property[2];
> @@ -119,7 +118,7 @@ static int get_cpus_node(void *fdt)
> * device tree, used in XSCOM to address cores and in interrupt
> * servers.
> */
> -static void powernv_create_core_node(PnvChip *chip, PnvCore *pc, void *fdt)
> +static void pnv_dt_core(PnvChip *chip, PnvCore *pc, void *fdt)
> {
> CPUState *cs = CPU(DEVICE(pc->threads));
> DeviceClass *dc = DEVICE_GET_CLASS(cs);
> @@ -228,8 +227,8 @@ static void powernv_create_core_node(PnvChip *chip,
> PnvCore *pc, void *fdt)
> servers_prop, sizeof(servers_prop))));
> }
>
> -static void powernv_populate_icp(PnvChip *chip, void *fdt, uint32_t pir,
> - uint32_t nr_threads)
> +static void pnv_dt_icp(PnvChip *chip, void *fdt, uint32_t pir,
> + uint32_t nr_threads)
> {
> uint64_t addr = PNV_ICP_BASE(chip) | (pir << 12);
> char *name;
> @@ -277,13 +276,13 @@ static int pnv_chip_lpc_offset(PnvChip *chip, void *fdt)
> return offset;
> }
>
> -static void powernv_populate_chip(PnvChip *chip, void *fdt)
> +static void pnv_dt_chip(PnvChip *chip, void *fdt)
> {
> const char *typename = pnv_chip_core_typename(chip);
> size_t typesize = object_type_get_instance_size(typename);
> int i;
>
> - pnv_xscom_populate(chip, fdt, 0);
> + pnv_dt_xscom(chip, fdt, 0);
>
> /* The default LPC bus of a multichip system is on chip 0. It's
> * recognized by the firmware (skiboot) using a "primary"
> @@ -298,20 +297,18 @@ static void powernv_populate_chip(PnvChip *chip, void
> *fdt)
> for (i = 0; i < chip->nr_cores; i++) {
> PnvCore *pnv_core = PNV_CORE(chip->cores + i * typesize);
>
> - powernv_create_core_node(chip, pnv_core, fdt);
> + pnv_dt_core(chip, pnv_core, fdt);
>
> /* Interrupt Control Presenters (ICP). One per core. */
> - powernv_populate_icp(chip, fdt, pnv_core->pir,
> - CPU_CORE(pnv_core)->nr_threads);
> + pnv_dt_icp(chip, fdt, pnv_core->pir, CPU_CORE(pnv_core)->nr_threads);
> }
>
> if (chip->ram_size) {
> - powernv_populate_memory_node(fdt, chip->chip_id, chip->ram_start,
> - chip->ram_size);
> + pnv_dt_memory(fdt, chip->chip_id, chip->ram_start, chip->ram_size);
> }
> }
>
> -static void powernv_populate_rtc(ISADevice *d, void *fdt, int lpc_off)
> +static void pnv_dt_rtc(ISADevice *d, void *fdt, int lpc_off)
> {
> uint32_t io_base = d->ioport_id;
> uint32_t io_regs[] = {
> @@ -331,7 +328,7 @@ static void powernv_populate_rtc(ISADevice *d, void *fdt,
> int lpc_off)
> _FDT((fdt_setprop_string(fdt, node, "compatible", "pnpPNP,b00")));
> }
>
> -static void powernv_populate_serial(ISADevice *d, void *fdt, int lpc_off)
> +static void pnv_dt_serial(ISADevice *d, void *fdt, int lpc_off)
> {
> const char compatible[] = "ns16550\0pnpPNP,501";
> uint32_t io_base = d->ioport_id;
> @@ -362,7 +359,7 @@ static void powernv_populate_serial(ISADevice *d, void
> *fdt, int lpc_off)
> _FDT((fdt_setprop_string(fdt, node, "device_type", "serial")));
> }
>
> -static void powernv_populate_ipmi_bt(ISADevice *d, void *fdt, int lpc_off)
> +static void pnv_dt_ipmi_bt(ISADevice *d, void *fdt, int lpc_off)
> {
> const char compatible[] = "bt\0ipmi-bt";
> uint32_t io_base;
> @@ -401,17 +398,17 @@ typedef struct ForeachPopulateArgs {
> int offset;
> } ForeachPopulateArgs;
>
> -static int powernv_populate_isa_device(DeviceState *dev, void *opaque)
> +static int pnv_dt_isa_device(DeviceState *dev, void *opaque)
> {
> ForeachPopulateArgs *args = opaque;
> ISADevice *d = ISA_DEVICE(dev);
>
> if (object_dynamic_cast(OBJECT(dev), TYPE_MC146818_RTC)) {
> - powernv_populate_rtc(d, args->fdt, args->offset);
> + pnv_dt_rtc(d, args->fdt, args->offset);
> } else if (object_dynamic_cast(OBJECT(dev), TYPE_ISA_SERIAL)) {
> - powernv_populate_serial(d, args->fdt, args->offset);
> + pnv_dt_serial(d, args->fdt, args->offset);
> } else if (object_dynamic_cast(OBJECT(dev), "isa-ipmi-bt")) {
> - powernv_populate_ipmi_bt(d, args->fdt, args->offset);
> + pnv_dt_ipmi_bt(d, args->fdt, args->offset);
> } else {
> error_report("unknown isa device address@hidden", qdev_fw_name(dev),
> d->ioport_id);
> @@ -420,7 +417,7 @@ static int powernv_populate_isa_device(DeviceState *dev,
> void *opaque)
> return 0;
> }
>
> -static void powernv_populate_isa(ISABus *bus, void *fdt, int lpc_offset)
> +static void pnv_dt_isa(ISABus *bus, void *fdt, int lpc_offset)
> {
> ForeachPopulateArgs args = {
> .fdt = fdt,
> @@ -429,14 +426,13 @@ static void powernv_populate_isa(ISABus *bus, void
> *fdt, int lpc_offset)
>
> /* ISA devices are not necessarily parented to the ISA bus so we
> * can not use object_child_foreach() */
> - qbus_walk_children(BUS(bus), powernv_populate_isa_device,
> - NULL, NULL, NULL, &args);
> + qbus_walk_children(BUS(bus), pnv_dt_isa_device, NULL, NULL, NULL, &args);
> }
>
> -static void *powernv_create_fdt(MachineState *machine)
> +static void *pnv_dt_create(MachineState *machine)
> {
> const char plat_compat[] = "qemu,powernv\0ibm,powernv";
> - PnvMachineState *pnv = POWERNV_MACHINE(machine);
> + PnvMachineState *pnv = PNV_MACHINE(machine);
> void *fdt;
> char *buf;
> int off;
> @@ -479,15 +475,15 @@ static void *powernv_create_fdt(MachineState *machine)
>
> /* Populate device tree for each chip */
> for (i = 0; i < pnv->num_chips; i++) {
> - powernv_populate_chip(pnv->chips[i], fdt);
> + pnv_dt_chip(pnv->chips[i], fdt);
> }
>
> /* Populate ISA devices on chip 0 */
> lpc_offset = pnv_chip_lpc_offset(pnv->chips[0], fdt);
> - powernv_populate_isa(pnv->isa_bus, fdt, lpc_offset);
> + pnv_dt_isa(pnv->isa_bus, fdt, lpc_offset);
>
> if (pnv->bmc) {
> - pnv_bmc_populate_sensors(pnv->bmc, fdt);
> + pnv_dt_bmc_sensors(pnv->bmc, fdt);
> }
>
> return fdt;
> @@ -495,17 +491,17 @@ static void *powernv_create_fdt(MachineState *machine)
>
> static void pnv_powerdown_notify(Notifier *n, void *opaque)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(qdev_get_machine());
> + PnvMachineState *pnv = PNV_MACHINE(qdev_get_machine());
>
> if (pnv->bmc) {
> pnv_bmc_powerdown(pnv->bmc);
> }
> }
>
> -static void ppc_powernv_reset(void)
> +static void pnv_reset(void)
> {
> MachineState *machine = MACHINE(qdev_get_machine());
> - PnvMachineState *pnv = POWERNV_MACHINE(machine);
> + PnvMachineState *pnv = PNV_MACHINE(machine);
> void *fdt;
> Object *obj;
>
> @@ -524,7 +520,7 @@ static void ppc_powernv_reset(void)
> pnv->bmc = IPMI_BMC(obj);
> }
>
> - fdt = powernv_create_fdt(machine);
> + fdt = pnv_dt_create(machine);
>
> /* Pack resulting tree */
> _FDT((fdt_pack(fdt)));
> @@ -552,9 +548,9 @@ static ISABus *pnv_isa_create(PnvChip *chip)
> return isa_bus;
> }
>
> -static void ppc_powernv_init(MachineState *machine)
> +static void pnv_init(MachineState *machine)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(machine);
> + PnvMachineState *pnv = PNV_MACHINE(machine);
> MemoryRegion *ram;
> char *fw_filename;
> long fw_size;
> @@ -567,7 +563,7 @@ static void ppc_powernv_init(MachineState *machine)
> }
>
> ram = g_new(MemoryRegion, 1);
> - memory_region_allocate_system_memory(ram, NULL, "ppc_powernv.ram",
> + memory_region_allocate_system_memory(ram, NULL, "pnv.ram",
> machine->ram_size);
> memory_region_add_subregion(get_system_memory(), 0, ram);
>
> @@ -974,7 +970,7 @@ static void pnv_chip_class_init(ObjectClass *klass, void
> *data)
>
> static ICSState *pnv_ics_get(XICSFabric *xi, int irq)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(xi);
> + PnvMachineState *pnv = PNV_MACHINE(xi);
> int i;
>
> for (i = 0; i < pnv->num_chips; i++) {
> @@ -987,7 +983,7 @@ static ICSState *pnv_ics_get(XICSFabric *xi, int irq)
>
> static void pnv_ics_resend(XICSFabric *xi)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(xi);
> + PnvMachineState *pnv = PNV_MACHINE(xi);
> int i;
>
> for (i = 0; i < pnv->num_chips; i++) {
> @@ -1021,7 +1017,7 @@ static ICPState *pnv_icp_get(XICSFabric *xi, int pir)
> static void pnv_pic_print_info(InterruptStatsProvider *obj,
> Monitor *mon)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(obj);
> + PnvMachineState *pnv = PNV_MACHINE(obj);
> int i;
> CPUState *cs;
>
> @@ -1039,13 +1035,13 @@ static void pnv_pic_print_info(InterruptStatsProvider
> *obj,
> static void pnv_get_num_chips(Object *obj, Visitor *v, const char *name,
> void *opaque, Error **errp)
> {
> - visit_type_uint32(v, name, &POWERNV_MACHINE(obj)->num_chips, errp);
> + visit_type_uint32(v, name, &PNV_MACHINE(obj)->num_chips, errp);
> }
>
> static void pnv_set_num_chips(Object *obj, Visitor *v, const char *name,
> void *opaque, Error **errp)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(obj);
> + PnvMachineState *pnv = PNV_MACHINE(obj);
> uint32_t num_chips;
> Error *local_err = NULL;
>
> @@ -1067,13 +1063,13 @@ static void pnv_set_num_chips(Object *obj, Visitor
> *v, const char *name,
> pnv->num_chips = num_chips;
> }
>
> -static void powernv_machine_initfn(Object *obj)
> +static void pnv_machine_initfn(Object *obj)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(obj);
> + PnvMachineState *pnv = PNV_MACHINE(obj);
> pnv->num_chips = 1;
> }
>
> -static void powernv_machine_class_props_init(ObjectClass *oc)
> +static void pnv_machine_class_props_init(ObjectClass *oc)
> {
> object_class_property_add(oc, "num-chips", "uint32",
> pnv_get_num_chips, pnv_set_num_chips,
> @@ -1083,15 +1079,15 @@ static void
> powernv_machine_class_props_init(ObjectClass *oc)
> NULL);
> }
>
> -static void powernv_machine_class_init(ObjectClass *oc, void *data)
> +static void pnv_machine_class_init(ObjectClass *oc, void *data)
> {
> MachineClass *mc = MACHINE_CLASS(oc);
> XICSFabricClass *xic = XICS_FABRIC_CLASS(oc);
> InterruptStatsProviderClass *ispc = INTERRUPT_STATS_PROVIDER_CLASS(oc);
>
> mc->desc = "IBM PowerNV (Non-Virtualized)";
> - mc->init = ppc_powernv_init;
> - mc->reset = ppc_powernv_reset;
> + mc->init = pnv_init;
> + mc->reset = pnv_reset;
> mc->max_cpus = MAX_CPUS;
> mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0");
> mc->block_default_type = IF_IDE; /* Pnv provides a AHCI device for
> @@ -1104,7 +1100,7 @@ static void powernv_machine_class_init(ObjectClass *oc,
> void *data)
> xic->ics_resend = pnv_ics_resend;
> ispc->print_info = pnv_pic_print_info;
>
> - powernv_machine_class_props_init(oc);
> + pnv_machine_class_props_init(oc);
> }
>
> #define DEFINE_PNV_CHIP_TYPE(type, class_initfn) \
> @@ -1116,11 +1112,11 @@ static void powernv_machine_class_init(ObjectClass
> *oc, void *data)
>
> static const TypeInfo types[] = {
> {
> - .name = TYPE_POWERNV_MACHINE,
> + .name = TYPE_PNV_MACHINE,
> .parent = TYPE_MACHINE,
> .instance_size = sizeof(PnvMachineState),
> - .instance_init = powernv_machine_initfn,
> - .class_init = powernv_machine_class_init,
> + .instance_init = pnv_machine_initfn,
> + .class_init = pnv_machine_class_init,
> .interfaces = (InterfaceInfo[]) {
> { TYPE_XICS_FABRIC },
> { TYPE_INTERRUPT_STATS_PROVIDER },
> diff --git a/hw/ppc/pnv_bmc.c b/hw/ppc/pnv_bmc.c
> index 7b60b4c36071..b2cf441ee7b2 100644
> --- a/hw/ppc/pnv_bmc.c
> +++ b/hw/ppc/pnv_bmc.c
> @@ -73,7 +73,7 @@ void pnv_bmc_powerdown(IPMIBmc *bmc)
> pnv_gen_oem_sel(bmc, SOFT_OFF);
> }
>
> -void pnv_bmc_populate_sensors(IPMIBmc *bmc, void *fdt)
> +void pnv_dt_bmc_sensors(IPMIBmc *bmc, void *fdt)
> {
> int offset;
> int i;
> diff --git a/hw/ppc/pnv_core.c b/hw/ppc/pnv_core.c
> index 82ff440b3334..78eeadbbb6a4 100644
> --- a/hw/ppc/pnv_core.c
> +++ b/hw/ppc/pnv_core.c
> @@ -37,7 +37,7 @@ static const char *pnv_core_cpu_typename(PnvCore *pc)
> return cpu_type;
> }
>
> -static void powernv_cpu_reset(void *opaque)
> +static void pnv_cpu_reset(void *opaque)
> {
> PowerPCCPU *cpu = opaque;
> CPUState *cs = CPU(cpu);
> @@ -54,7 +54,7 @@ static void powernv_cpu_reset(void *opaque)
> env->msr |= MSR_HVB; /* Hypervisor mode */
> }
>
> -static void powernv_cpu_init(PowerPCCPU *cpu, Error **errp)
> +static void pnv_cpu_init(PowerPCCPU *cpu, Error **errp)
> {
> CPUPPCState *env = &cpu->env;
> int core_pir;
> @@ -73,7 +73,7 @@ static void powernv_cpu_init(PowerPCCPU *cpu, Error **errp)
> /* Set time-base frequency to 512 MHz */
> cpu_ppc_tb_init(env, PNV_TIMEBASE_FREQ);
>
> - qemu_register_reset(powernv_cpu_reset, cpu);
> + qemu_register_reset(pnv_cpu_reset, cpu);
> }
>
> /*
> @@ -146,7 +146,7 @@ static void pnv_core_realize_child(Object *child,
> XICSFabric *xi, Error **errp)
> return;
> }
>
> - powernv_cpu_init(cpu, &local_err);
> + pnv_cpu_init(cpu, &local_err);
> if (local_err) {
> object_unparent(obj);
> error_propagate(errp, local_err);
> diff --git a/hw/ppc/pnv_lpc.c b/hw/ppc/pnv_lpc.c
> index f03a80a29bf6..b777b78e1837 100644
> --- a/hw/ppc/pnv_lpc.c
> +++ b/hw/ppc/pnv_lpc.c
> @@ -92,7 +92,7 @@ enum {
> #define LPC_HC_REGS_OPB_SIZE 0x00001000
>
>
> -static int pnv_lpc_populate(PnvXScomInterface *dev, void *fdt, int
> xscom_offset)
> +static int pnv_lpc_dt_xscom(PnvXScomInterface *dev, void *fdt, int
> xscom_offset)
> {
> const char compat[] = "ibm,power8-lpc\0ibm,lpc";
> char *name;
> @@ -482,7 +482,7 @@ static void pnv_lpc_class_init(ObjectClass *klass, void
> *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
> PnvXScomInterfaceClass *xdc = PNV_XSCOM_INTERFACE_CLASS(klass);
>
> - xdc->populate = pnv_lpc_populate;
> + xdc->dt_xscom = pnv_lpc_dt_xscom;
>
> dc->realize = pnv_lpc_realize;
> }
> @@ -515,7 +515,7 @@ type_init(pnv_lpc_register_types)
> */
> static void pnv_lpc_isa_irq_handler_cpld(void *opaque, int n, int level)
> {
> - PnvMachineState *pnv = POWERNV_MACHINE(qdev_get_machine());
> + PnvMachineState *pnv = PNV_MACHINE(qdev_get_machine());
> uint32_t old_state = pnv->cpld_irqstate;
> PnvLpcController *lpc = PNV_LPC(opaque);
>
> diff --git a/hw/ppc/pnv_psi.c b/hw/ppc/pnv_psi.c
> index 9876c266223d..5b969127c303 100644
> --- a/hw/ppc/pnv_psi.c
> +++ b/hw/ppc/pnv_psi.c
> @@ -510,7 +510,7 @@ static void pnv_psi_realize(DeviceState *dev, Error
> **errp)
> }
> }
>
> -static int pnv_psi_populate(PnvXScomInterface *dev, void *fdt, int
> xscom_offset)
> +static int pnv_psi_dt_xscom(PnvXScomInterface *dev, void *fdt, int
> xscom_offset)
> {
> const char compat[] = "ibm,power8-psihb-x\0ibm,psihb-x";
> char *name;
> @@ -546,7 +546,7 @@ static void pnv_psi_class_init(ObjectClass *klass, void
> *data)
> DeviceClass *dc = DEVICE_CLASS(klass);
> PnvXScomInterfaceClass *xdc = PNV_XSCOM_INTERFACE_CLASS(klass);
>
> - xdc->populate = pnv_psi_populate;
> + xdc->dt_xscom = pnv_psi_dt_xscom;
>
> dc->realize = pnv_psi_realize;
> dc->props = pnv_psi_properties;
> diff --git a/hw/ppc/pnv_xscom.c b/hw/ppc/pnv_xscom.c
> index 38bc85f117f6..e51d634f4062 100644
> --- a/hw/ppc/pnv_xscom.c
> +++ b/hw/ppc/pnv_xscom.c
> @@ -207,15 +207,15 @@ typedef struct ForeachPopulateArgs {
> int xscom_offset;
> } ForeachPopulateArgs;
>
> -static int xscom_populate_child(Object *child, void *opaque)
> +static int xscom_dt_child(Object *child, void *opaque)
> {
> if (object_dynamic_cast(child, TYPE_PNV_XSCOM_INTERFACE)) {
> ForeachPopulateArgs *args = opaque;
> PnvXScomInterface *xd = PNV_XSCOM_INTERFACE(child);
> PnvXScomInterfaceClass *xc = PNV_XSCOM_INTERFACE_GET_CLASS(xd);
>
> - if (xc->populate) {
> - _FDT((xc->populate(xd, args->fdt, args->xscom_offset)));
> + if (xc->dt_xscom) {
> + _FDT((xc->dt_xscom(xd, args->fdt, args->xscom_offset)));
> }
> }
> return 0;
> @@ -224,7 +224,7 @@ static int xscom_populate_child(Object *child, void
> *opaque)
> static const char compat_p8[] = "ibm,power8-xscom\0ibm,xscom";
> static const char compat_p9[] = "ibm,power9-xscom\0ibm,xscom";
>
> -int pnv_xscom_populate(PnvChip *chip, void *fdt, int root_offset)
> +int pnv_dt_xscom(PnvChip *chip, void *fdt, int root_offset)
> {
> uint64_t reg[] = { cpu_to_be64(PNV_XSCOM_BASE(chip)),
> cpu_to_be64(PNV_XSCOM_SIZE) };
> @@ -255,7 +255,7 @@ int pnv_xscom_populate(PnvChip *chip, void *fdt, int
> root_offset)
> args.fdt = fdt;
> args.xscom_offset = xscom_offset;
>
> - object_child_foreach(OBJECT(chip), xscom_populate_child, &args);
> + object_child_foreach(OBJECT(chip), xscom_dt_child, &args);
> return 0;
> }
>
> diff --git a/include/hw/ppc/pnv.h b/include/hw/ppc/pnv.h
> index 59524cd42b61..61896f9fd767 100644
> --- a/include/hw/ppc/pnv.h
> +++ b/include/hw/ppc/pnv.h
> @@ -26,7 +26,7 @@
> #include "hw/ppc/pnv_psi.h"
> #include "hw/ppc/pnv_occ.h"
>
> -#define TYPE_PNV_CHIP "powernv-chip"
> +#define TYPE_PNV_CHIP "pnv-chip"
> #define PNV_CHIP(obj) OBJECT_CHECK(PnvChip, (obj), TYPE_PNV_CHIP)
> #define PNV_CHIP_CLASS(klass) \
> OBJECT_CLASS_CHECK(PnvChipClass, (klass), TYPE_PNV_CHIP)
> @@ -117,9 +117,9 @@ typedef struct PnvChipClass {
> #define PNV_CHIP_INDEX(chip) \
> (((chip)->chip_id >> 2) * 2 + ((chip)->chip_id & 0x3))
>
> -#define TYPE_POWERNV_MACHINE MACHINE_TYPE_NAME("powernv")
> -#define POWERNV_MACHINE(obj) \
> - OBJECT_CHECK(PnvMachineState, (obj), TYPE_POWERNV_MACHINE)
> +#define TYPE_PNV_MACHINE MACHINE_TYPE_NAME("powernv")
> +#define PNV_MACHINE(obj) \
> + OBJECT_CHECK(PnvMachineState, (obj), TYPE_PNV_MACHINE)
>
> typedef struct PnvMachineState {
> /*< private >*/
> @@ -144,7 +144,7 @@ typedef struct PnvMachineState {
> /*
> * BMC helpers
> */
> -void pnv_bmc_populate_sensors(IPMIBmc *bmc, void *fdt);
> +void pnv_dt_bmc_sensors(IPMIBmc *bmc, void *fdt);
> void pnv_bmc_powerdown(IPMIBmc *bmc);
>
> /*
> diff --git a/include/hw/ppc/pnv_xscom.h b/include/hw/ppc/pnv_xscom.h
> index 38077b479699..7252e219e249 100644
> --- a/include/hw/ppc/pnv_xscom.h
> +++ b/include/hw/ppc/pnv_xscom.h
> @@ -36,7 +36,7 @@ typedef struct PnvXScomInterface {
>
> typedef struct PnvXScomInterfaceClass {
> InterfaceClass parent;
> - int (*populate)(PnvXScomInterface *dev, void *fdt, int offset);
> + int (*dt_xscom)(PnvXScomInterface *dev, void *fdt, int offset);
> } PnvXScomInterfaceClass;
>
> /*
> @@ -67,7 +67,7 @@ typedef struct PnvXScomInterfaceClass {
> #define PNV_XSCOM_OCC_SIZE 0x6000
>
> extern void pnv_xscom_realize(PnvChip *chip, Error **errp);
> -extern int pnv_xscom_populate(PnvChip *chip, void *fdt, int offset);
> +extern int pnv_dt_xscom(PnvChip *chip, void *fdt, int offset);
>
> extern void pnv_xscom_add_subregion(PnvChip *chip, hwaddr offset,
> MemoryRegion *mr);
--
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