On Tue, Apr 06, 2010 at 01:44:08PM +0200, Stefan Weil wrote: > Signed-off-by: Stefan Weil <w...@mail.berlios.de> > --- > hw/eepro100.c | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/hw/eepro100.c b/hw/eepro100.c > index f0acdbc..2401888 100644 > --- a/hw/eepro100.c > +++ b/hw/eepro100.c > @@ -1622,8 +1622,9 @@ static void pci_mmio_map(PCIDevice * pci_dev, int > region_num, > "size=0x%08"FMT_PCIBUS", type=%d\n", > region_num, addr, size, type)); > > - if (region_num == 0) { > - /* Map control / status registers. */ > + assert(region_num == 0 || region_num == 2); > + if (region_num == 0 || region_num == 2) {
Looks a bit strange ... Why do we need the if here? > + /* Map control / status registers and flash. */ > cpu_register_physical_memory(addr, size, s->mmio_index); > s->region[region_num] = addr; > } > -- > 1.7.0