diff options
author | Isaku Yamahata <yamahata@valinux.co.jp> | 2010-05-27 14:42:06 +0900 |
---|---|---|
committer | Michael S. Tsirkin <mst@redhat.com> | 2010-05-31 16:33:52 +0300 |
commit | 5e434f4e60f8e5a9dadf324b3ee31c3ce0e80165 (patch) | |
tree | f642f62e50321360c372b785169a83c8908a3516 /hw | |
parent | aa6f63fff62faf2fe9ffba5a789675d49293614d (diff) |
pci: clean up of pci_set_default_subsystem_id().
Use pci accessor function.
don't return value because it always return 0 and
the caller doesn't check the return value.
Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw')
-rw-r--r-- | hw/pci.c | 12 |
1 files changed, 5 insertions, 7 deletions
@@ -419,14 +419,12 @@ int pci_device_load(PCIDevice *s, QEMUFile *f) return ret; } -static int pci_set_default_subsystem_id(PCIDevice *pci_dev) +static void pci_set_default_subsystem_id(PCIDevice *pci_dev) { - uint16_t *id; - - id = (void*)(&pci_dev->config[PCI_SUBSYSTEM_VENDOR_ID]); - id[0] = cpu_to_le16(pci_default_sub_vendor_id); - id[1] = cpu_to_le16(pci_default_sub_device_id); - return 0; + pci_set_word(pci_dev->config + PCI_SUBSYSTEM_VENDOR_ID, + pci_default_sub_vendor_id); + pci_set_word(pci_dev->config + PCI_SUBSYSTEM_ID, + pci_default_sub_device_id); } /* |