virtio-pci: minor refactoring

!legacy && !modern is shorter than !(legacy || modern).
I also perfer this (less ()s) as a matter of taste.

Cc: Greg Kurz <gkurz@linux.vnet.ibm.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
This commit is contained in:
Michael S. Tsirkin 2016-09-09 20:56:27 +03:00
parent 96a3d98d2c
commit 71d19fc513

View File

@ -1776,7 +1776,7 @@ static void virtio_pci_realize(PCIDevice *pci_dev, Error **errp)
proxy->disable_legacy = pcie_port ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF; proxy->disable_legacy = pcie_port ? ON_OFF_AUTO_ON : ON_OFF_AUTO_OFF;
} }
if (!(virtio_pci_modern(proxy) || virtio_pci_legacy(proxy))) { if (!virtio_pci_modern(proxy) && !virtio_pci_legacy(proxy)) {
error_setg(errp, "device cannot work as neither modern nor legacy mode" error_setg(errp, "device cannot work as neither modern nor legacy mode"
" is enabled"); " is enabled");
error_append_hint(errp, "Set either disable-modern or disable-legacy" error_append_hint(errp, "Set either disable-modern or disable-legacy"