[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 17/26] fwcfg: use realize for fwcfg
From: |
Hu Tao |
Subject: |
Re: [Qemu-devel] [PATCH v2 17/26] fwcfg: use realize for fwcfg |
Date: |
Mon, 22 Jul 2013 16:37:53 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, Jul 21, 2013 at 11:35:55AM +0200, Andreas Färber wrote:
> Am 01.07.2013 12:18, schrieb Hu Tao:
> > Signed-off-by: Hu Tao <address@hidden>
> > ---
> > hw/nvram/fw_cfg.c | 26 ++++++++++++++++----------
> > 1 file changed, 16 insertions(+), 10 deletions(-)
> >
> > diff --git a/hw/nvram/fw_cfg.c b/hw/nvram/fw_cfg.c
> > index c8722c2..656ffcc 100644
> > --- a/hw/nvram/fw_cfg.c
> > +++ b/hw/nvram/fw_cfg.c
> > @@ -523,31 +523,37 @@ FWCfgState *fw_cfg_init(uint32_t ctl_port, uint32_t
> > data_port,
> > return s;
> > }
> >
> > -static int fw_cfg_init1(SysBusDevice *dev)
> > +static void fw_cfg_initfn(Object *obj)
> > {
> > - FWCfgState *s = FW_CFG(dev);
> > + FWCfgState *s = FW_CFG(obj);
> >
> > memory_region_init_io(&s->ctl_iomem, &fw_cfg_ctl_mem_ops, s,
> > "fwcfg.ctl", FW_CFG_SIZE);
> > - sysbus_init_mmio(dev, &s->ctl_iomem);
> > memory_region_init_io(&s->data_iomem, &fw_cfg_data_mem_ops, s,
> > "fwcfg.data", FW_CFG_DATA_SIZE);
> > - sysbus_init_mmio(dev, &s->data_iomem);
> > /* In case ctl and data overlap: */
> > memory_region_init_io(&s->comb_iomem, &fw_cfg_comb_mem_ops, s,
> > "fwcfg", FW_CFG_SIZE);
> > +}
>
> There's no reason not to have sysbus_init_mmio() in instance_init,
> changed that.
Thanks.
- [Qemu-devel] [PATCH v2 13/26] pflash_cfi02: use realize for pflash_cfi02, (continued)
- [Qemu-devel] [PATCH v2 13/26] pflash_cfi02: use realize for pflash_cfi02, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 15/26] ahci: use realize for ahci, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 16/26] fwcfg: QOM'ify some more, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 18/26] scsi esp: QOM'ify some more, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 19/26] scsi esp: use realize for scsi esp, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 17/26] fwcfg: use realize for fwcfg, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 21/26] hpet: use realize for hpet, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 23/26] kvmclock: use realize for kvmclock, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 20/26] hpet: QOM'ify some more, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 22/26] kvmclock: QOM'ify some more, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 26/26] isa bus: remove isabus_bridge_init since it does nothing, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 25/26] ioapic: use realize for ioapic, Hu Tao, 2013/07/01
- [Qemu-devel] [PATCH v2 24/26] kvmvapic realize, Hu Tao, 2013/07/01
- Re: [Qemu-devel] [PATCH v2 00/26] use realizefn for SysBusDevice, part 1, Andreas Färber, 2013/07/21