[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v3] e1000: add the ability to select among sever

From: Romain Dolbeau
Subject: Re: [Qemu-devel] [PATCH v3] e1000: add the ability to select among several specific types of e1000[e]; 82566DM emulation ; some pointers to documentations and details.
Date: Thu, 6 Mar 2014 10:01:18 +0100

2014-03-05 17:55 GMT+01:00 Andreas Färber <address@hidden>:
>> +        DBGOUT(UNKNOWN, "Flash unknown write 
>> addr=0x%08x,val=0x%08"PRIx64"\n",
>> +               index<<2, val);
> Spaces

There's a handful of place like that where I just duplicated lines - this
particular warning is a duplicate from a similar one about MMIO access.

Do I add the spaces to match the rules, or do I left them out to
match everything else in the file ?

>> -    pci_register_bar(pci_dev, 1, PCI_BASE_ADDRESS_SPACE_IO, &d->io);
>> +    pci_register_bar(pci_dev, 1, PCI_BASE_ADDRESS_SPACE_MEMORY, &d->flash);
>> +
>> +    pci_register_bar(pci_dev, 2, PCI_BASE_ADDRESS_SPACE_IO, &d->io);
> I wonder if this change is migration-compatible?

I must confess I didn't think about migration. I don't have much of a choice;
the e1000e driver requires the second mapping to be the flash registers,
if I understand the code correctly.

> Spaces around << at least please, not sure about file's surrounding
> style - Stefan's call.

As before - copy/pasting from a few lines above, so the style matches
the file.


Romain Dolbeau

reply via email to

[Prev in Thread] Current Thread [Next in Thread]